[U-Boot] [PATCH v2 07/10] usb: mxs: Disable USB Port 1 for i.MX23

[Marek Vasut]

Dear Otavio Salvador,

> On Wed, Feb 13, 2013 at 7:15 PM, Marek Vasut <marex at denx.de> wrote:
> > Dear Otavio Salvador,
> > 
> >> The i.MX23 just one USB port so disable the second controller probe
> >> when building for i.MX23.
> >> 
> >> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> >> ---
> >> Changes in v2:
> >> - Avoid wrong clock setting in MX23
> >> 
> >>  drivers/usb/host/ehci-mxs.c | 12 ++++++++++++
> >>  1 file changed, 12 insertions(+)
> >> 
> >> diff --git a/drivers/usb/host/ehci-mxs.c b/drivers/usb/host/ehci-mxs.c
> >> index 5062af5..b7bf856 100644
> >> --- a/drivers/usb/host/ehci-mxs.c
> >> +++ b/drivers/usb/host/ehci-mxs.c
> >> @@ -50,10 +50,12 @@ int mxs_ehci_get_port(struct ehci_mxs *mxs_usb, int
> >> port) usb_base = MXS_USBCTRL0_BASE;
> >> 
> >>               phy_base = MXS_USBPHY0_BASE;
> >>               break;
> >> 
> >> +#ifdef CONFIG_MX28
> >> 
> >>       case 1:
> >>               usb_base = MXS_USBCTRL1_BASE;
> >>               phy_base = MXS_USBPHY1_BASE;
> >>               break;
> >> 
> >> +#endif
> >> 
> >>       default:
> >>               printf("CONFIG_EHCI_MXS_PORT (port = %d)\n", port);
> >>               return -1;
> >> 
> >> @@ -67,7 +69,9 @@ int mxs_ehci_get_port(struct ehci_mxs *mxs_usb, int
> >> port)
> >> 
> >>  /* This DIGCTL register ungates clock to USB */
> >>  #define      HW_DIGCTL_CTRL                  0x8001c000
> >>  #define      HW_DIGCTL_CTRL_USB0_CLKGATE     (1 << 2)
> >> 
> >> +#ifdef CONFIG_MX28
> >> 
> >>  #define      HW_DIGCTL_CTRL_USB1_CLKGATE     (1 << 16)
> >> 
> >> +#endif
> >> 
> >>  int ehci_hcd_init(int index, struct ehci_hccr **hccr, struct ehci_hcor
> >> 
> >> **hcor) {
> >> @@ -95,8 +99,12 @@ int ehci_hcd_init(int index, struct ehci_hccr **hccr,
> >> struct ehci_hcor **hcor) writel(CLKCTRL_PLL1CTRL0_EN_USB_CLKS |
> >> CLKCTRL_PLL1CTRL0_POWER,
> >> 
> >>                       &clkctrl_regs->hw_clkctrl_pll1ctrl0_set);
> >> 
> >> +#if defined(CONFIG_MX23)
> >> +     writel(HW_DIGCTL_CTRL_USB0_CLKGATE,     &digctl_ctrl->reg_clr);
> > 
> > What is that large space before '&digctl_regs->...' ?
> 
> My fault. I can fix it.
> 
> >> +#elif defined(CONFIG_MX28)
> >> 
> >>       writel(HW_DIGCTL_CTRL_USB0_CLKGATE | HW_DIGCTL_CTRL_USB1_CLKGATE,
> >>       
> >>               &digctl_ctrl->reg_clr);
> >> 
> >> +#endif
> > 
> > I'm sure these can be wrapped in much more elegant way, yes?
> 
> I wanted to keep the changes at minimum; so seems the right route.
> What you'd like me to do? Use a temporary variable?

Either that or wrap it all in struct ehci_mxs and do these ungatings etc on a 
per-port basis.