Commit 39b36668 authored Aug 25, 2020 by Jérôme Pouiller Committed by Greg Kroah-Hartman Aug 28, 2020

staging: wfx: add workaround for 'timeout while wake up chip'

The host and the device can be connected with a called Wake-Up GPIO.
When the host fall down this GPIO, it allows the device to enter in deep
sleep and no communication with the device is no more possible (the
device wakes up automatically on DTIM and fetch data if necessary).

So, before to communicate with the device, the driver have to raise the
Wake-up GPIO and then wait for an IRQ from the device.

Unfortunately, old firmwares have a race in sleep/wake-up process and
the device may never wake up. In this case, the IRQ is not sent and
driver complains with "timeout while wake up chip". Then, the driver
tries anyway to access the bus and an other error is raised by the bus.

Fortunately, when the bug occurs, it is possible to fall down the IRQ
and the device will eventually finish the sleep process. Then the driver
can wake it up normally.

The patch implements that workaround and add a retry limit in case
something goes very wrong.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200825085828.399505-12-Jerome.Pouiller@silabs.com

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 3b5583a9

drivers/staging/wfx/bh.c

+19 −4

Original line number	Diff line number	Diff line
		@@ -18,25 +18,40 @@

		static void device_wakeup(struct wfx_dev *wdev)
		{
		int max_retry = 3;

		if (!wdev->pdata.gpio_wakeup)
		return;
		if (gpiod_get_value_cansleep(wdev->pdata.gpio_wakeup))
		return;

		gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 1);
		if (wfx_api_older_than(wdev, 1, 4)) {
		gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 1);
		if (!completion_done(&wdev->hif.ctrl_ready))
		usleep_range(2000, 2500);
		} else {
		return;
		}
		for (;;) {
		gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 1);
		// completion.h does not provide any function to wait
		// completion without consume it (a kind of
		// wait_for_completion_done_timeout()). So we have to emulate
		// it.
		if (wait_for_completion_timeout(&wdev->hif.ctrl_ready,
		msecs_to_jiffies(2)))
		msecs_to_jiffies(2))) {
		complete(&wdev->hif.ctrl_ready);
		else
		return;
		} else if (max_retry-- > 0) {
		// Older firmwares have a race in sleep/wake-up process.
		// Redo the process is sufficient to unfreeze the
		// chip.
		dev_err(wdev->dev, "timeout while wake up chip\n");
		gpiod_set_value_cansleep(wdev->pdata.gpio_wakeup, 0);
		usleep_range(2000, 2500);
		} else {
		dev_err(wdev->dev, "max wake-up retries reached\n");
		return;
		}
		}
		}

Admin message