On Wed, Jun 21, 2023 at 10:11 PM Jit Loon Lim <[email protected]> wrote: > > From: Teik Heng Chong <[email protected]> > > The controller registers should not be accessed while the controller's > vcc_reset_n is asserted. > > Signed-off-by: Teik Heng Chong <[email protected]> > --- > drivers/usb/host/xhci-dwc3.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/host/xhci-dwc3.c b/drivers/usb/host/xhci-dwc3.c > index 1dbd65dfaa..3172956bd6 100644 > --- a/drivers/usb/host/xhci-dwc3.c > +++ b/drivers/usb/host/xhci-dwc3.c > @@ -227,6 +227,11 @@ static int xhci_dwc3_probe(struct udevice *dev) > static int xhci_dwc3_remove(struct udevice *dev) > { > struct xhci_dwc3_plat *plat = dev_get_plat(dev); > + int ret; > + > + ret = xhci_deregister(dev); > + if (ret) > + return ret; > > dwc3_shutdown_phy(dev, &plat->phys); > > @@ -234,7 +239,7 @@ static int xhci_dwc3_remove(struct udevice *dev) > > reset_release_bulk(&plat->resets); > > - return xhci_deregister(dev); > + return 0; > } > > static const struct udevice_id xhci_dwc3_ids[] = {
Please remove the HSD tag in the commit summary. Regards, Bin

