imx: Remove boolean parameter from wdog powerdown function
authorFabio Estevam <fabio.estevam@nxp.com>
Thu, 23 Nov 2017 12:55:33 +0000 (10:55 -0200)
committerStefano Babic <sbabic@denx.de>
Mon, 27 Nov 2017 09:34:49 +0000 (10:34 +0100)
imx_set_wdog_powerdown() is always used to disable the power down
enable bit, so remove the boolean parameter of the function.

It is also a bit strange to write a boolean value into registers,
so this new version makes explicit that we are writing 0.

While at it, rename it to imx_wdog_disable_powerdown().

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
arch/arm/include/asm/mach-imx/sys_proto.h
arch/arm/mach-imx/init.c
arch/arm/mach-imx/mx6/soc.c
arch/arm/mach-imx/mx7/soc.c

index 703634334f163fdd363af3a9076c2aa4610c049a..a80a392a739d3c0467dfecc31e562a4bd8cb755e 100644 (file)
@@ -105,7 +105,7 @@ void set_chipselect_size(int const);
 
 void init_aips(void);
 void init_src(void);
-void imx_set_wdog_powerdown(bool enable);
+void imx_wdog_disable_powerdown(void);
 
 int board_mmc_get_env_dev(int devno);
 
index 2a05c596422acc50d5f3b4d38d98a6e574730f51..0aa82957513e3c83e17185360be1c534056f2ce0 100644 (file)
@@ -65,7 +65,7 @@ void init_aips(void)
        }
 }
 
-void imx_set_wdog_powerdown(bool enable)
+void imx_wdog_disable_powerdown(void)
 {
        struct wdog_regs *wdog1 = (struct wdog_regs *)WDOG1_BASE_ADDR;
        struct wdog_regs *wdog2 = (struct wdog_regs *)WDOG2_BASE_ADDR;
@@ -75,13 +75,13 @@ void imx_set_wdog_powerdown(bool enable)
 #endif
 
        /* Write to the PDE (Power Down Enable) bit */
-       writew(enable, &wdog1->wmcr);
-       writew(enable, &wdog2->wmcr);
+       writew(0, &wdog1->wmcr);
+       writew(0, &wdog2->wmcr);
 
        if (is_mx6sx() || is_mx6ul() || is_mx6ull() || is_mx7())
-               writew(enable, &wdog3->wmcr);
+               writew(0, &wdog3->wmcr);
 #ifdef CONFIG_MX7D
-       writew(enable, &wdog4->wmcr);
+       writew(0, &wdog4->wmcr);
 #endif
 }
 
index ad72c125117dd5cd7cade5a07d12934dfc5f8f48..43cb58106b92b694137a9b14b766d37f282d3d18 100644 (file)
@@ -436,7 +436,7 @@ int arch_cpu_init(void)
        if (is_mx6sl())
                setbits_le32(&ccm->cscmr1, MXC_CCM_CSCMR1_PER_CLK_SEL_MASK);
 
-       imx_set_wdog_powerdown(false); /* Disable PDE bit of WMCR register */
+       imx_wdog_disable_powerdown(); /* Disable PDE bit of WMCR register */
 
        if (is_mx6sx())
                setbits_le32(&ccm->cscdr1, MXC_CCM_CSCDR1_UART_CLK_SEL);
index 87bf105f385d4c12bee6d038c0ba50e2e588124c..d160e80146dc925b0ce4c4799c53c9970e06888c 100644 (file)
@@ -236,7 +236,7 @@ int arch_cpu_init(void)
 
        init_csu();
        /* Disable PDE bit of WMCR register */
-       imx_set_wdog_powerdown(false);
+       imx_wdog_disable_powerdown();
 
        imx_enet_mdio_fixup();