arm: socfpga: reset: Repair bridge reset handling
authorMarek Vasut <marex@denx.de>
Thu, 9 Jul 2015 01:52:12 +0000 (03:52 +0200)
committerMarek Vasut <marex@denx.de>
Sat, 8 Aug 2015 12:14:06 +0000 (14:14 +0200)
The current bridge reset code, which de-asserted the bridge reset,
was activelly polling whether the FPGA is programmed and ready and
in case it was (!), the code called hang(). This makes no sense at
all. Repair it such that the code instead checks whether the FPGA
is programmed, but without any polling involved, and only if it is
programmed, it de-asserts the reset.

Signed-off-by: Marek Vasut <marex@denx.de>
arch/arm/mach-socfpga/reset_manager.c

index 18af25c..6a11c19 100644 (file)
@@ -85,10 +85,10 @@ void socfpga_bridges_reset(int enable)
                writel(0xffffffff, &reset_manager_base->brg_mod_reset);
        } else {
                /* Check signal from FPGA. */
-               if (fpgamgr_poll_fpga_ready()) {
-                       /* FPGA not ready. Wait for watchdog timeout. */
-                       printf("%s: fpga not ready, hanging.\n", __func__);
-                       hang();
+               if (!fpgamgr_test_fpga_ready()) {
+                       /* FPGA not ready, do nothing. */
+                       printf("%s: FPGA not ready, aborting.\n", __func__);
+                       return;
                }
 
                /* brdmodrst */