arm: socfpga: soc64: Add timeout waiting for NOC idle ACK
authorChee Hong Ang <chee.hong.ang@intel.com>
Mon, 10 Aug 2020 14:59:49 +0000 (22:59 +0800)
committerLey Foon Tan <ley.foon.tan@intel.com>
Fri, 9 Oct 2020 09:53:13 +0000 (17:53 +0800)
Add timeout waiting for NOC idle ACK during FPGA bridge
disable/enable.

Signed-off-by: Chee Hong Ang <chee.hong.ang@intel.com>
arch/arm/mach-socfpga/reset_manager_s10.c

index 9f16bf9..3746e6a 100644 (file)
@@ -9,6 +9,7 @@
 #include <asm/arch/reset_manager.h>
 #include <asm/arch/system_manager.h>
 #include <dt-bindings/reset/altr,rst-mgr-s10.h>
+#include <linux/iopoll.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -54,6 +55,8 @@ void socfpga_per_reset_all(void)
 
 void socfpga_bridges_reset(int enable)
 {
+       u32 reg;
+
        if (enable) {
                /* clear idle request to all bridges */
                setbits_le32(socfpga_get_sysmgr_addr() +
@@ -64,9 +67,9 @@ void socfpga_bridges_reset(int enable)
                             ~0);
 
                /* Poll until all idleack to 0 */
-               while (readl(socfpga_get_sysmgr_addr() +
-                            SYSMGR_SOC64_NOC_IDLEACK))
-                       ;
+               read_poll_timeout(readl, socfpga_get_sysmgr_addr() +
+                                 SYSMGR_SOC64_NOC_IDLEACK, reg, !reg, 1000,
+                                 300000);
        } else {
                /* set idle request to all bridges */
                writel(~0,
@@ -77,14 +80,18 @@ void socfpga_bridges_reset(int enable)
                writel(1, socfpga_get_sysmgr_addr() + SYSMGR_SOC64_NOC_TIMEOUT);
 
                /* Poll until all idleack to 1 */
-               while ((readl(socfpga_get_sysmgr_addr() + SYSMGR_SOC64_NOC_IDLEACK) ^
-                       (SYSMGR_NOC_H2F_MSK | SYSMGR_NOC_LWH2F_MSK)))
-                       ;
+               read_poll_timeout(readl, socfpga_get_sysmgr_addr() +
+                                 SYSMGR_SOC64_NOC_IDLEACK, reg,
+                                 reg == (SYSMGR_NOC_H2F_MSK |
+                                         SYSMGR_NOC_LWH2F_MSK),
+                                 1000, 300000);
 
                /* Poll until all idlestatus to 1 */
-               while ((readl(socfpga_get_sysmgr_addr() + SYSMGR_SOC64_NOC_IDLESTATUS) ^
-                       (SYSMGR_NOC_H2F_MSK | SYSMGR_NOC_LWH2F_MSK)))
-                       ;
+               read_poll_timeout(readl, socfpga_get_sysmgr_addr() +
+                                 SYSMGR_SOC64_NOC_IDLESTATUS, reg,
+                                 reg == (SYSMGR_NOC_H2F_MSK |
+                                         SYSMGR_NOC_LWH2F_MSK),
+                                 1000, 300000);
 
                /* Reset all bridges (except NOR DDR scheduler & F2S) */
                setbits_le32(socfpga_get_rstmgr_addr() + RSTMGR_SOC64_BRGMODRST,