dm: serial: Reset the watchdog while waiting in getc()
authorSimon Glass <sjg@chromium.org>
Thu, 23 Oct 2014 03:37:02 +0000 (21:37 -0600)
committerSimon Glass <sjg@chromium.org>
Fri, 24 Oct 2014 01:29:07 +0000 (19:29 -0600)
We have moved the busy-wait loop out of drivers and into the uclass. This
means that we must reset the watchdog when busy-waiting.

Note: some drivers may still have a busy-wait even with driver model, as
a transition mechanism. Driver model will tolerate this, and is can be
cleaned up when all users of the driver use driver model. An example is
ns16550.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@ti.com>
drivers/serial/serial-uclass.c

index 1a75950d19fa4c738f7f5c881da08561d416faf2..1983a3d55bde3850bcd7369e8a5a807260fcf4f5 100644 (file)
 #include <os.h>
 #include <serial.h>
 #include <stdio_dev.h>
+#include <watchdog.h>
 #include <dm/lists.h>
 #include <dm/device-internal.h>
 
+#include <ns16550.h>
+
 DECLARE_GLOBAL_DATA_PTR;
 
 /* The currently-selected console serial device */
@@ -119,6 +122,8 @@ static int serial_getc_dev(struct udevice *dev)
 
        do {
                err = ops->getc(dev);
+               if (err == -EAGAIN)
+                       WATCHDOG_RESET();
        } while (err == -EAGAIN);
 
        return err >= 0 ? err : 0;