[U-Boot] [PATCH 5/8] usb: xhci-exynos5: Add support for multiple USB 3.0 controllers

[Marek Vasut]

On Tuesday, January 07, 2014 at 10:15:20 AM, Vivek Gautam wrote:

[...]

> >> diff --git a/arch/arm/include/asm/arch-exynos/cpu.h
> >> b/arch/arm/include/asm/arch-exynos/cpu.h index 718940b..d93cba9 100644
> >> --- a/arch/arm/include/asm/arch-exynos/cpu.h
> >> +++ b/arch/arm/include/asm/arch-exynos/cpu.h

[...]

> >> @@ -153,6 +159,8 @@
> >> 
> >>  #define EXYNOS5420_USBPHY_BASE               0x12130000
> >>  #define EXYNOS5420_MMC_BASE          0x12200000
> >>  #define EXYNOS5420_SROMC_BASE                0x12250000
> >> 
> >> +#define EXYNOS5420_USB_HOST_XHCI_1_BASE      0x12400000
> >> +#define EXYNOS5420_USB3PHY_1_BASE    0x12500000
> > 
> > No. we don't have to add phy_1 and xhci_1.
> > It looks weird.
> > please access phy1 and xhci1 by offset.
> 
> Ok, so you mean i should add offsets definitions for phy1 and xhci1
> may be in the driver file or in
> "arch/arm/include/asm/arch-exynos/xhci-exynos.h" file, and add these
> offsets to xhci_base and
> phy_base addresses ?
> That should be fine, i will modify this accordingly.
> 
> >>  #define EXYNOS5420_UART_BASE         0x12C00000
> >>  #define EXYNOS5420_I2C_BASE          0x12C60000
> >>  #define EXYNOS5420_I2C_8910_BASE     0x12E00000
> >> 
[...]

Minor rant, please trim the email when replying so it contains only the relevant 
parts. Otherwise, the irrelevant noise might cause some of the relevant replies 
to get lost.

Best regards,
Marek Vasut