York, I don't know the file " arch/arm/include/asm/arch-fsl-layerscape/soc.h " is not included for ls1021a, maybe ls1021a is one special ARM SoC. Now I will add one macro " CONFIG_LS1012A " to resolve this issue.
Best Regards Jerry Huang -----Original Message----- From: york sun Sent: Thursday, December 01, 2016 3:10 AM To: Jerry Huang <jerry.hu...@nxp.com>; u-boot@lists.denx.de Subject: Re: [PATCH] fsl/usb: enable the errata-a005697 for ls1012a On 11/29/2016 06:39 PM, Jerry Huang wrote: > Any comment for this patch? > > Best Regards > Jerry Huang > > -----Original Message----- > From: Changming Huang [mailto:jerry.hu...@nxp.com] > Sent: Thursday, November 24, 2016 3:14 PM > To: york sun <york....@nxp.com>; u-boot@lists.denx.de > Cc: Jerry Huang <jerry.hu...@nxp.com> > Subject: [PATCH] fsl/usb: enable the errata-a005697 for ls1012a > > Enable the errata-a005697 for ls1012a > > Signed-off-by: Changming Huang <jerry.hu...@nxp.com> > --- > drivers/usb/common/fsl-errata.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/usb/common/fsl-errata.c > b/drivers/usb/common/fsl-errata.c index 0dc3dd8..59a69a1 100644 > --- a/drivers/usb/common/fsl-errata.c > +++ b/drivers/usb/common/fsl-errata.c > @@ -139,6 +139,10 @@ bool has_erratum_a005697(void) > case SVR_9132: > return IS_SVR_REV(svr, 1, 0) || IS_SVR_REV(svr, 1, 1); #endif > +#ifdef CONFIG_ARM > + case SVR_LS1012A: > + return IS_SVR_REV(svr, 1, 0); > +#endif > } Actually this patch has a problem with the SVR. SVR_LS1012A is defined in arch/arm/include/asm/arch-fsl-layerscape/soc.h. You will see compiling errors for LS1021A, eg ls1021atwr_nor. York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
