[PATCH v1] HSD #18028953892: usb: xhci-dwc3: Fix USB3.1 controller register access in reset state

[Chong, Teik Heng]

Hi

> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: Wednesday, 26 July, 2023 6:30 PM
> To: Chong, Teik Heng <teik.heng.chong at intel.com>; Lim, Jit Loon
> <jit.loon.lim at intel.com>; u-boot at lists.denx.de
> Cc: Jagan Teki <jagan at amarulasolutions.com>; Simon
> <simon.k.r.goldschmidt at gmail.com>; Chee, Tien Fong
> <tien.fong.chee at intel.com>; Hea, Kok Kiang <kok.kiang.hea at intel.com>;
> Maniyam, Dinesh <dinesh.maniyam at intel.com>; Ng, Boon Khai
> <boon.khai.ng at intel.com>; Yuslaimi, Alif Zakuan
> <alif.zakuan.yuslaimi at intel.com>; Zamri, Muhammad Hazim Izzat
> <muhammad.hazim.izzat.zamri at intel.com>; Tang, Sieu Mun
> <sieu.mun.tang at intel.com>; Bin Meng <bmeng.cn at gmail.com>; Michal
> Simek <monstr at monstr.eu>; Tom Rini <trini at konsulko.com>; Eugen Hristev
> <eugen.hristev at microchip.com>
> Subject: Re: [PATCH v1] HSD #18028953892: usb: xhci-dwc3: Fix USB3.1
> controller register access in reset state
> 
> On 7/26/23 06:04, Chong, Teik Heng wrote:
> [...]
> >>>>> Linux:  (__dwc3_of_simple_teardown)
> >>>>> https://elixir.bootlin.com/linux/latest/source/drivers/usb/dwc3/dw
> >>>>> c3
> >>>>> -o
> >>>>> f-simple.c#L98
> >>>>> U-Boot: (xhci_dwc3_remove)
> >>>>> https://elixir.bootlin.com/u-boot/latest/source/drivers/usb/host/x
> >>>>> hc
> >>>>> i-
> >>>>> dwc3.c#L227
> >>>>>
> >>>>> So we believed that we can't directly pickup all Linux kernel dwc3
> >>>>> patches
> >>>> and merge to U-Boot.
> >>>>
> >>>> If you were to sync the driver from Linux to U-Boot, then the same
> >>>> sequence as Linux uses would be automatically used too, right ?
> >>>>
> >>>> Sorry for the abysmal delay in my reply.
> >>>
> >>> Are we saying that we shall port/use Linux driver in U-Boot and
> >>> abandon
> >> the existing USB host driver in U-Boot?
> >>
> >> No, the existing driver in U-Boot is a port of the Linux driver, it
> >> is just outdated. I am saying, just pick the missing patches from
> >> Linux and add them to U-Boot, so the two drivers are in sync.
> >
> > We do not see a DWC3 host driver under usb host folder in Linux
> > https://elixir.bootlin.com/linux/latest/source/drivers/usb/host. Would
> > you mind to tell us which DWC3 host driver we should use in usb host
> > folder? Then, we can pick up the missing patches
> 
> drivers/usb/dwc3
> 
> Same as u-boot

Ok. We will use the driver from drivers/usb/dwc3. Thank you for the pointers