On Thu, Oct 4, 2018 at 11:17 AM Marek Vasut <ma...@denx.de> wrote: > > On 10/03/2018 10:21 PM, Chris Packham wrote: > > Workaround makes FS as default mode on all affected socs. > > > > Add support to check erratum-A005275 validity for an soc. This info is > > required to determine whether a given soc is affected by this erratum. > > Add quirk for this erratum "has_fsl_erratum_a005275" . This quirk is used > > to enable workaround for the errata > > > > Force FS mode as default by: > > - making EPS as FS > > - setting PFSC bit to disable HS chirping > > > > This workaround can be disabled by mentioning "no_erratum_a005275" in > > hwconfig string > > > > Signed-off-by: Chris Packham <judge.pack...@gmail.com> > [...] > > > diff --git a/drivers/usb/host/ehci-fsl.c b/drivers/usb/host/ehci-fsl.c > > index a04f6a31c8e4..71e4ded53100 100644 > > --- a/drivers/usb/host/ehci-fsl.c > > +++ b/drivers/usb/host/ehci-fsl.c > > @@ -93,6 +93,7 @@ static int ehci_fsl_probe(struct udevice *dev) > > struct usb_ehci *ehci = NULL; > > struct ehci_hccr *hccr; > > struct ehci_hcor *hcor; > > + struct ehci_ctrl *echi_ctrl = &priv->echi; > > Was this ever even compile-tested ? See echi_ctrl vs ehci_ctrl below ... >
I must confess no. The only targets that use this with DM_USB are ARMv8 and I didn't have a toolchain for that. I did test P2040RDB. I will fire the next version through travis-ci to catch dumb errors like this. > > /* > > * Get the base address for EHCI controller from the device node > > @@ -107,6 +108,8 @@ static int ehci_fsl_probe(struct udevice *dev) > > hcor = (struct ehci_hcor *) > > ((void *)hccr + HC_LENGTH(ehci_readl(&hccr->cr_capbase))); > > > > + ehci_ctrl->has_fsl_erratum_a005275 = has_erratum_a005275(); > > [...] > > -- > Best regards, > Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot