[U-Boot] [PATCH] USB: ehci-mx6: Fix broken IO access

[U-Boot] [PATCH] USB: ehci-mx6: Fix broken IO access

[Wolfgang Grandegger]

To get USB working again on the i.MX6, this patch fixes a bug introduced
with commit 522b2a0 "Add proper IO accessors for mx6 usb registers.".
At that occasion, I also added the missing __iomem directive.

Cc: Marek Vasut <marex@denx.de>
CC: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Wolfgang Grandegger <wg@denx.de>
---
 drivers/usb/host/ehci-mx6.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
index 5dec673..42c77fe 100644
--- a/drivers/usb/host/ehci-mx6.c
+++ b/drivers/usb/host/ehci-mx6.c
@@ -73,7 +73,8 @@ static void usbh1_internal_phy_clock_gate(int on)
 
 static void usbh1_power_config(void)
 {
-	struct anatop_regs *anatop = (struct anatop_regs *)ANATOP_BASE_ADDR;
+	struct anatop_regs __iomem *anatop =
+		(struct anatop_regs __iomem *)ANATOP_BASE_ADDR;
 	/*
 	 * Some phy and power's special controls for host1
 	 * 1. The external charger detector needs to be disabled
@@ -87,7 +88,7 @@ static void usbh1_power_config(void)
 		     &anatop->usb2_chrg_detect);
 
 	__raw_writel(ANADIG_USB2_PLL_480_CTRL_BYPASS,
-		     &anatop->usb2_pll_480_ctrl);
+		     &anatop->usb2_pll_480_ctrl_clr);
 
 	__raw_writel(ANADIG_USB2_PLL_480_CTRL_ENABLE |
 		     ANADIG_USB2_PLL_480_CTRL_POWER |
-- 
1.7.7.6

[U-Boot] [PATCH] USB: ehci-mx6: Fix broken IO access

[Fabio Estevam]

On Wed, May 2, 2012 at 11:36 AM, Wolfgang Grandegger <wg@denx.de> wrote:
> To get USB working again on the i.MX6, this patch fixes a bug introduced
> with commit 522b2a0 "Add proper IO accessors for mx6 usb registers.".

Good catch on the usb register name change.

> At that occasion, I also added the missing __iomem directive.

Is this really needed?

[U-Boot] [PATCH] USB: ehci-mx6: Fix broken IO access

[Wolfgang Grandegger]

Hi Fabio,

On 05/02/2012 05:24 PM, Fabio Estevam wrote:
> On Wed, May 2, 2012 at 11:36 AM, Wolfgang Grandegger <wg@denx.de> wrote:
>> To get USB working again on the i.MX6, this patch fixes a bug introduced
>> with commit 522b2a0 "Add proper IO accessors for mx6 usb registers.".
> 
> Good catch on the usb register name change.
> 
>> At that occasion, I also added the missing __iomem directive.
> 
> Is this really needed?

As I see it, __iomem is a dummy function in U-Boot. I just added it for
consistency reasons. If we remove it, we should also remove all other
__iomem's from that file. Maybe somebody else could clarify if __iomem
should be used.

Wolfgang.

[U-Boot] [PATCH] USB: ehci-mx6: Fix broken IO access

[Stefano Babic]

On 02/05/2012 16:36, Wolfgang Grandegger wrote:
> To get USB working again on the i.MX6, this patch fixes a bug introduced
> with commit 522b2a0 "Add proper IO accessors for mx6 usb registers.".
> At that occasion, I also added the missing __iomem directive.
> 
> Cc: Marek Vasut <marex@denx.de>
> CC: Fabio Estevam <festevam@gmail.com>
> Signed-off-by: Wolfgang Grandegger <wg@denx.de>
> ---

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================