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

Marek Vasut
2014-01-07T10:59:30Z

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