[U-Boot] [PATCH v3 2/7] arm: usb: dra7xx: xHCI registers based on USB port index

[Tom Rini]

On Sun, Feb 19, 2017 at 04:13:02PM +0100, Marek Vasut wrote:
> On 02/19/2017 03:55 PM, Uri Mashiach wrote:
> > 
> > 
> > On 02/19/2017 04:27 PM, Marek Vasut wrote:
> >> On 02/19/2017 02:27 PM, Uri Mashiach wrote:
> >>> Modify the determination of the base address of xHCI registers of DRA7XX
> >>> targets.
> >>> Before the commit: by the target.
> >>> After the commit: by the USB port index.
> >>>
> >>> Cc: Lokesh Vutla <lokeshvutla at ti.com>
> >>> Cc: Marek Vasut <marex at denx.de>
> >>> Signed-off-by: Uri Mashiach <uri.mashiach at compulab.co.il>
> >>> ---
> >>> V1 -> V2: Replace the commit "fix XHCI registers base address".
> >>> V2 -> V3: Replace the commit "reintroduce the CONFIG_AM57XX symbol"
> >>>
> >>>  configs/dra7xx_evm_defconfig    |  1 +
> >>>  configs/dra7xx_hs_evm_defconfig |  1 +
> >>>  drivers/usb/host/Kconfig        | 16 ++++++++++++++++
> >>>  include/linux/usb/xhci-omap.h   |  6 ++++--
> >>>  4 files changed, 22 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/configs/dra7xx_evm_defconfig b/configs/dra7xx_evm_defconfig
> >>> index 26b26cc..1f47f92 100644
> > 
> > [...]
> > 
> >>> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> >>> index 5129a57..440fbcf 100644
> >>> --- a/drivers/usb/host/Kconfig
> >>> +++ b/drivers/usb/host/Kconfig
> >>> @@ -43,6 +43,22 @@ config USB_XHCI_ZYNQMP
> >>>      help
> >>>        Enables support for the on-chip xHCI controller on Xilinx
> >>> ZynqMP SoCs.
> >>>
> >>> +choice
> >>> +    prompt "DRA7XX xHCI USB index select"
> >>> +    depends on DRA7XX
> >>> +
> >>> +config USB_XHCI_DRA7XX_INDEX0
> >>> +    bool "USB0"
> >>> +    help
> >>> +      DRA7XX xHCI USB0.
> >>> +
> >>> +config USB_XHCI_DRA7XX_INDEX1
> >>> +    bool "USB1"
> >>> +    help
> >>> +      DRA7XX xHCI USB1.
> >>
> >> What is this all about ? Shouldn't this come from DT ? And what if I
> >> want to use both XHCI ? This looks totally bogus ...
> >>
> > 
> > The support for both XHCI is currently missing.
> > This could be a temporary solution until the DT solution.
> > The current situation is worse - selecting USB0 or USB1 based on the
> > target.
> 
> So we're replacing it with equally bad solution ? Hmmm , no.
> The MW will open mid-march, there's about a month to fix this,
> so please do.

Do note that the relevant driver here is not yet using DM_USB.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170219/60625185/attachment.sig>