[PATCH 1/2] OMAP4: fix writel data/address swap regression

Vicente Bergas vicencb at gmail.com
Sat Nov 23 14:58:23 EST 2013


---
 arch/arm/mach-omap/omap4_clock.c | 176 +++++++++++++++++++--------------------
 1 file changed, 88 insertions(+), 88 deletions(-)

diff --git a/arch/arm/mach-omap/omap4_clock.c b/arch/arm/mach-omap/omap4_clock.c
index 268f7c1..72f72a1 100644
--- a/arch/arm/mach-omap/omap4_clock.c
+++ b/arch/arm/mach-omap/omap4_clock.c
@@ -89,7 +89,7 @@ void omap4_configure_per_dpll(const struct dpll_param *dpll_param)
 void omap4_configure_abe_dpll(const struct dpll_param *dpll_param)
 {
 	/* Select sys_clk as ref clk for ABE dpll */
-	writel(CM_ABE_PLL_REF_CLKSEL, 0x0);
+	writel(0x0, CM_ABE_PLL_REF_CLKSEL);
 
 	/* Unlock the ABE dpll */
 	sr32(CM_CLKMODE_DPLL_ABE, 0, 3, PLL_MN_POWER_BYPASS);
@@ -102,11 +102,11 @@ void omap4_configure_abe_dpll(const struct dpll_param *dpll_param)
 	sr32(CM_CLKSEL_DPLL_ABE, 0, 6, dpll_param->n);
 
 	/* Force DPLL CLKOUTHIF to stay enabled */
-	writel(CM_DIV_M2_DPLL_ABE, 0x500);
+	writel(0x500, CM_DIV_M2_DPLL_ABE);
 	sr32(CM_DIV_M2_DPLL_ABE, 0, 5, dpll_param->m2);
 	sr32(CM_DIV_M2_DPLL_ABE, 8, 1, 0x1);
 	/* Force DPLL CLKOUTHIF to stay enabled */
-	writel(CM_DIV_M3_DPLL_ABE, 0x100);
+	writel(0x100, CM_DIV_M3_DPLL_ABE);
 	sr32(CM_DIV_M3_DPLL_ABE, 0, 5, dpll_param->m3);
 	sr32(CM_DIV_M3_DPLL_ABE, 8, 1, 0x1);
 
@@ -120,7 +120,7 @@ void omap4_configure_abe_dpll(const struct dpll_param *dpll_param)
 void omap4_configure_usb_dpll(const struct dpll_param *dpll_param)
 {
 	/* Select the 60Mhz clock 480/8 = 60*/
-	writel(CM_CLKSEL_USB_60MHz, 0x1);
+	writel(0x1, CM_CLKSEL_USB_60MHz);
 
 	/* Unlock the USB dpll */
 	sr32(CM_CLKMODE_DPLL_USB, 0, 3, PLL_MN_POWER_BYPASS);
@@ -133,7 +133,7 @@ void omap4_configure_usb_dpll(const struct dpll_param *dpll_param)
 	sr32(CM_CLKSEL_DPLL_USB, 0, 6, dpll_param->n);
 
 	/* Force DPLL CLKOUT to stay active */
-	writel(CM_DIV_M2_DPLL_USB, 0x100);
+	writel(0x100, CM_DIV_M2_DPLL_USB);
 	sr32(CM_DIV_M2_DPLL_USB, 0, 5, dpll_param->m2);
 	sr32(CM_DIV_M2_DPLL_USB, 8, 1, 0x1);
 	sr32(CM_CLKDCOLDO_DPLL_USB, 8, 1, 0x1);
@@ -143,7 +143,7 @@ void omap4_configure_usb_dpll(const struct dpll_param *dpll_param)
 	wait_on_value((1 << 0), 1, CM_IDLEST_DPLL_USB, LDELAY);
 
 	/* force enable the CLKDCOLDO clock */
-	writel(CM_CLKDCOLDO_DPLL_USB, 0x100);
+	writel(0x100, CM_CLKDCOLDO_DPLL_USB);
 
 	return;
 }
@@ -151,7 +151,7 @@ void omap4_configure_usb_dpll(const struct dpll_param *dpll_param)
 void omap4_configure_core_dpll_no_lock(const struct dpll_param *param)
 {
 	/* CORE_CLK=CORE_X2_CLK/2, L3_CLK=CORE_CLK/2, L4_CLK=L3_CLK/2 */
-	writel(CM_CLKSEL_CORE, 0x110);
+	writel(0x110, CM_CLKSEL_CORE);
 
 	/* Unlock the CORE dpll */
 	sr32(CM_CLKMODE_DPLL_CORE, 0, 3, PLL_MN_POWER_BYPASS);
@@ -205,107 +205,107 @@ void omap4_lock_core_dpll_shadow(const struct dpll_param *param)
 
 void omap4_enable_gpio_clocks(void)
 {
-	writel(CM_L4PER_GPIO2_CLKCTRL, 0x1);
+	writel(0x1, CM_L4PER_GPIO2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_GPIO2_CLKCTRL, LDELAY);
-	writel(CM_L4PER_GPIO3_CLKCTRL, 0x1);
+	writel(0x1, CM_L4PER_GPIO3_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_GPIO3_CLKCTRL, LDELAY);
-	writel(CM_L4PER_GPIO4_CLKCTRL, 0x1);
+	writel(0x1, CM_L4PER_GPIO4_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_GPIO4_CLKCTRL, LDELAY);
-	writel(CM_L4PER_GPIO5_CLKCTRL, 0x1);
+	writel(0x1, CM_L4PER_GPIO5_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_GPIO5_CLKCTRL, LDELAY);
-	writel(CM_L4PER_GPIO6_CLKCTRL, 0x1);
+	writel(0x1, CM_L4PER_GPIO6_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_GPIO6_CLKCTRL, LDELAY);
 }
 
 void omap4_enable_gpio1_wup_clocks(void)
 {
 	/* WKUP clocks */
-	writel(CM_WKUP_GPIO1_CLKCTRL, 0x1);
+	writel(0x1, CM_WKUP_GPIO1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_WKUP_GPIO1_CLKCTRL, LDELAY);
 }
 
 void omap4_enable_all_clocks(void)
 {
 	/* Enable Ducati clocks */
-	writel(CM_DUCATI_DUCATI_CLKCTRL, 0x1);
-	writel(CM_DUCATI_CLKSTCTRL, 0x2);
+	writel(0x1, CM_DUCATI_DUCATI_CLKCTRL);
+	writel(0x2, CM_DUCATI_CLKSTCTRL);
 
 	wait_on_value((1 << 8), (1 << 8), CM_DUCATI_CLKSTCTRL, LDELAY);
 
 	/* Enable ivahd and sl2 clocks */
-	writel(IVAHD_IVAHD_CLKCTRL, 0x1);
-	writel(IVAHD_SL2_CLKCTRL, 0x1);
-	writel(IVAHD_CLKSTCTRL, 0x2);
+	writel(0x1, IVAHD_IVAHD_CLKCTRL);
+	writel(0x1, IVAHD_SL2_CLKCTRL);
+	writel(0x2, IVAHD_CLKSTCTRL);
 
 	wait_on_value((1 << 8), (1 << 8), IVAHD_CLKSTCTRL, LDELAY);
 
 	/* Enable Tesla clocks */
-	writel(DSP_DSP_CLKCTRL, 0x1);
-	writel(DSP_CLKSTCTRL, 0x2);
+	writel(0x1, DSP_DSP_CLKCTRL);
+	writel(0x2, DSP_CLKSTCTRL);
 
 	wait_on_value((1 << 8), (1 << 8), DSP_CLKSTCTRL, LDELAY);
 
 	/* wait for tesla to become accessible */
 
 	/* ABE clocks */
-	writel(CM1_ABE_CLKSTCTRL, 0x3);
-	writel(CM1_ABE_AESS_CLKCTRL, 0x2);
-	writel(CM1_ABE_PDM_CLKCTRL, 0x2);
-	writel(CM1_ABE_DMIC_CLKCTRL, 0x2);
-	writel(CM1_ABE_MCASP_CLKCTRL, 0x2);
-	writel(CM1_ABE_MCBSP1_CLKCTRL, 0x2);
-	writel(CM1_ABE_MCBSP2_CLKCTRL, 0x2);
-	writel(CM1_ABE_MCBSP3_CLKCTRL, 0x2);
-	writel(CM1_ABE_SLIMBUS_CLKCTRL, 0xf02);
-	writel(CM1_ABE_TIMER5_CLKCTRL, 0x2);
-	writel(CM1_ABE_TIMER6_CLKCTRL, 0x2);
-	writel(CM1_ABE_TIMER7_CLKCTRL, 0x2);
-	writel(CM1_ABE_TIMER8_CLKCTRL, 0x2);
-	writel(CM1_ABE_WDT3_CLKCTRL, 0x2);
+	writel(0x3, CM1_ABE_CLKSTCTRL);
+	writel(0x2, CM1_ABE_AESS_CLKCTRL);
+	writel(0x2, CM1_ABE_PDM_CLKCTRL);
+	writel(0x2, CM1_ABE_DMIC_CLKCTRL);
+	writel(0x2, CM1_ABE_MCASP_CLKCTRL);
+	writel(0x2, CM1_ABE_MCBSP1_CLKCTRL);
+	writel(0x2, CM1_ABE_MCBSP2_CLKCTRL);
+	writel(0x2, CM1_ABE_MCBSP3_CLKCTRL);
+	writel(0xf02, CM1_ABE_SLIMBUS_CLKCTRL);
+	writel(0x2, CM1_ABE_TIMER5_CLKCTRL);
+	writel(0x2, CM1_ABE_TIMER6_CLKCTRL);
+	writel(0x2, CM1_ABE_TIMER7_CLKCTRL);
+	writel(0x2, CM1_ABE_TIMER8_CLKCTRL);
+	writel(0x2, CM1_ABE_WDT3_CLKCTRL);
 	/* Disable sleep transitions */
-	writel(CM1_ABE_CLKSTCTRL, 0x0);
+	writel(0x0, CM1_ABE_CLKSTCTRL);
 
 	/* L4PER clocks */
-	writel(CM_L4PER_CLKSTCTRL, 0x2);
-	writel(CM_L4PER_DMTIMER10_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_CLKSTCTRL);
+	writel(0x2, CM_L4PER_DMTIMER10_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_DMTIMER10_CLKCTRL, LDELAY);
-	writel(CM_L4PER_DMTIMER11_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_DMTIMER11_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_DMTIMER11_CLKCTRL, LDELAY);
-	writel(CM_L4PER_DMTIMER2_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_DMTIMER2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_DMTIMER2_CLKCTRL, LDELAY);
-	writel(CM_L4PER_DMTIMER3_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_DMTIMER3_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_DMTIMER3_CLKCTRL, LDELAY);
-	writel(CM_L4PER_DMTIMER4_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_DMTIMER4_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_DMTIMER4_CLKCTRL, LDELAY);
-	writel(CM_L4PER_DMTIMER9_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_DMTIMER9_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_DMTIMER9_CLKCTRL, LDELAY);
 
 	/* GPIO clocks */
 	omap4_enable_gpio_clocks();
 
-	writel(CM_L4PER_HDQ1W_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_HDQ1W_CLKCTRL);
 
 	/* I2C clocks */
-	writel(CM_L4PER_I2C1_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_I2C1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_I2C1_CLKCTRL, LDELAY);
-	writel(CM_L4PER_I2C2_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_I2C2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_I2C2_CLKCTRL, LDELAY);
-	writel(CM_L4PER_I2C3_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_I2C3_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_I2C3_CLKCTRL, LDELAY);
-	writel(CM_L4PER_I2C4_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_I2C4_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_I2C4_CLKCTRL, LDELAY);
 
-	writel(CM_L4PER_MCBSP4_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MCBSP4_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_MCBSP4_CLKCTRL, LDELAY);
 
 	/* MCSPI clocks */
-	writel(CM_L4PER_MCSPI1_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MCSPI1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_MCSPI1_CLKCTRL, LDELAY);
-	writel(CM_L4PER_MCSPI2_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MCSPI2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_MCSPI2_CLKCTRL, LDELAY);
-	writel(CM_L4PER_MCSPI3_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MCSPI3_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_MCSPI3_CLKCTRL, LDELAY);
-	writel(CM_L4PER_MCSPI4_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MCSPI4_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_MCSPI4_CLKCTRL, LDELAY);
 
 	/* MMC clocks */
@@ -313,86 +313,86 @@ void omap4_enable_all_clocks(void)
 	sr32(CM_L3INIT_HSMMC1_CLKCTRL, 24, 1, 0x1);
 	sr32(CM_L3INIT_HSMMC2_CLKCTRL, 0, 2, 0x2);
 	sr32(CM_L3INIT_HSMMC2_CLKCTRL, 24, 1, 0x1);
-	writel(CM_L4PER_MMCSD3_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MMCSD3_CLKCTRL);
 	wait_on_value((1 << 18)|(1 << 17)|(1 << 16), 0, CM_L4PER_MMCSD3_CLKCTRL, LDELAY);
-	writel(CM_L4PER_MMCSD4_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MMCSD4_CLKCTRL);
 	wait_on_value((1 << 18)|(1 << 17)|(1 << 16), 0, CM_L4PER_MMCSD4_CLKCTRL, LDELAY);
-	writel(CM_L4PER_MMCSD5_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_MMCSD5_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_MMCSD5_CLKCTRL, LDELAY);
 
 	/* UART clocks */
-	writel(CM_L4PER_UART1_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_UART1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_UART1_CLKCTRL, LDELAY);
-	writel(CM_L4PER_UART2_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_UART2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_UART2_CLKCTRL, LDELAY);
-	writel(CM_L4PER_UART3_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_UART3_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_UART3_CLKCTRL, LDELAY);
-	writel(CM_L4PER_UART4_CLKCTRL, 0x2);
+	writel(0x2, CM_L4PER_UART4_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L4PER_UART4_CLKCTRL, LDELAY);
 
 	/* WKUP clocks */
 	omap4_enable_gpio1_wup_clocks();
-	writel(CM_WKUP_TIMER1_CLKCTRL, 0x01000002);
+	writel(0x01000002, CM_WKUP_TIMER1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_WKUP_TIMER1_CLKCTRL, LDELAY);
 
-	writel(CM_WKUP_KEYBOARD_CLKCTRL, 0x2);
+	writel(0x2, CM_WKUP_KEYBOARD_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_WKUP_KEYBOARD_CLKCTRL, LDELAY);
 
-	writel(CM_SDMA_CLKSTCTRL, 0x0);
-	writel(CM_MEMIF_CLKSTCTRL, 0x3);
-	writel(CM_MEMIF_EMIF_1_CLKCTRL, 0x1);
+	writel(0x0, CM_SDMA_CLKSTCTRL);
+	writel(0x3, CM_MEMIF_CLKSTCTRL);
+	writel(0x1, CM_MEMIF_EMIF_1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_MEMIF_EMIF_1_CLKCTRL, LDELAY);
-	writel(CM_MEMIF_EMIF_2_CLKCTRL, 0x1);
+	writel(0x1, CM_MEMIF_EMIF_2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_MEMIF_EMIF_2_CLKCTRL, LDELAY);
-	writel(CM_D2D_CLKSTCTRL, 0x3);
-	writel(CM_L3_2_GPMC_CLKCTRL, 0x1);
+	writel(0x3, CM_D2D_CLKSTCTRL);
+	writel(0x1, CM_L3_2_GPMC_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L3_2_GPMC_CLKCTRL, LDELAY);
-	writel(CM_L3INSTR_L3_3_CLKCTRL, 0x1);
+	writel(0x1, CM_L3INSTR_L3_3_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L3INSTR_L3_3_CLKCTRL, LDELAY);
-	writel(CM_L3INSTR_L3_INSTR_CLKCTRL, 0x1);
+	writel(0x1, CM_L3INSTR_L3_INSTR_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L3INSTR_L3_INSTR_CLKCTRL, LDELAY);
-	writel(CM_L3INSTR_OCP_WP1_CLKCTRL, 0x1);
+	writel(0x1, CM_L3INSTR_OCP_WP1_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_L3INSTR_OCP_WP1_CLKCTRL, LDELAY);
 
 	/* WDT clocks */
-	writel(CM_WKUP_WDT2_CLKCTRL, 0x2);
+	writel(0x2, CM_WKUP_WDT2_CLKCTRL);
 	wait_on_value((1 << 17)|(1 << 16), 0, CM_WKUP_WDT2_CLKCTRL, LDELAY);
 
 	/* Enable Camera clocks */
-	writel(CM_CAM_CLKSTCTRL, 0x3);
-	writel(CM_CAM_ISS_CLKCTRL, 0x102);
-	writel(CM_CAM_FDIF_CLKCTRL, 0x2);
-	writel(CM_CAM_CLKSTCTRL, 0x0);
+	writel(0x3, CM_CAM_CLKSTCTRL);
+	writel(0x102, CM_CAM_ISS_CLKCTRL);
+	writel(0x2, CM_CAM_FDIF_CLKCTRL);
+	writel(0x0, CM_CAM_CLKSTCTRL);
 
 	/* Enable DSS clocks */
 	/* PM_DSS_PWRSTCTRL ON State and LogicState = 1 (Retention) */
 	__raw_writel(7, 0x4A307100); /* DSS_PRM */
 
-	writel(CM_DSS_CLKSTCTRL, 0x2);
-	writel(CM_DSS_DSS_CLKCTRL, 0xf02);
-	writel(CM_DSS_DEISS_CLKCTRL, 0x2);
+	writel(0x2, CM_DSS_CLKSTCTRL);
+	writel(0xf02, CM_DSS_DSS_CLKCTRL);
+	writel(0x2, CM_DSS_DEISS_CLKCTRL);
 
 	/* Check for DSS Clocks */
 	while ((__raw_readl(0x4A009100) & 0xF00) != 0xE00)
 		;
 	/* Set HW_AUTO transition mode */
-	writel(CM_DSS_CLKSTCTRL, 0x3);
+	writel(0x3, CM_DSS_CLKSTCTRL);
 
 	/* Enable SGX clocks */
-	writel(CM_SGX_CLKSTCTRL, 0x2);
-	writel(CM_SGX_SGX_CLKCTRL, 0x2);
+	writel(0x2, CM_SGX_CLKSTCTRL);
+	writel(0x2, CM_SGX_SGX_CLKCTRL);
 	/* Check for SGX FCLK and ICLK */
 	while (__raw_readl(0x4A009200) != 0x302)
 		;
 	/* Enable hsi/unipro/usb clocks */
-	writel(CM_L3INIT_HSI_CLKCTRL, 0x1);
-	writel(CM_L3INIT_UNIPRO1_CLKCTRL, 0x2);
-	writel(CM_L3INIT_HSUSBHOST_CLKCTRL, 0x2);
-	writel(CM_L3INIT_HSUSBOTG_CLKCTRL, 0x1);
-	writel(CM_L3INIT_HSUSBTLL_CLKCTRL, 0x1);
-	writel(CM_L3INIT_FSUSB_CLKCTRL, 0x2);
+	writel(0x1, CM_L3INIT_HSI_CLKCTRL);
+	writel(0x2, CM_L3INIT_UNIPRO1_CLKCTRL);
+	writel(0x2, CM_L3INIT_HSUSBHOST_CLKCTRL);
+	writel(0x1, CM_L3INIT_HSUSBOTG_CLKCTRL);
+	writel(0x1, CM_L3INIT_HSUSBTLL_CLKCTRL);
+	writel(0x2, CM_L3INIT_FSUSB_CLKCTRL);
 	/* enable the 32K, 48M optional clocks and enable the module */
-	writel(CM_L3INIT_USBPHY_CLKCTRL, 0x301);
+	writel(0x301, CM_L3INIT_USBPHY_CLKCTRL);
 }
 
 void omap4_do_scale_tps62361(u32 reg, u32 volt_mv)
-- 
1.8.4.2




More information about the barebox mailing list