[OE-core] [PATCH] u-boot: Update to v2020.01-rc5

Tom Rini trini at konsulko.com
Sun Dec 29 18:33:15 UTC 2019


On Sun, Dec 29, 2019 at 10:18:54AM -0800, Khem Raj wrote:
> On Sun, Dec 29, 2019 at 5:46 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Sun, Dec 29, 2019 at 09:23:47AM +0000, Richard Purdie wrote:
> > > On Tue, 2019-12-24 at 14:41 -0800, Alistair Francis wrote:
> > > > Signed-off-by: Alistair Francis <alistair.francis at wdc.com>
> > > > ---
> > > >  meta/recipes-bsp/u-boot/u-boot-common.inc | 4 ++--
> > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/meta/recipes-bsp/u-boot/u-boot-common.inc
> > > > b/meta/recipes-bsp/u-boot/u-boot-common.inc
> > > > index a0f9c41469..4799948bb1 100644
> > > > --- a/meta/recipes-bsp/u-boot/u-boot-common.inc
> > > > +++ b/meta/recipes-bsp/u-boot/u-boot-common.inc
> > > > @@ -11,11 +11,11 @@ LIC_FILES_CHKSUM = "
> > > > file://Licenses/README;md5=30503fd321432fc713238f582193b78e"
> > > >  PE = "1"
> > > >
> > > >  # Drop this line when updating to 2020.01 final
> > > > -PV = "2020.01~rc3"
> > > > +PV = "2020.01~rc5"
> > > >
> > > >  # We use the revision in order to avoid having to fetch it from the
> > > >  # repo during parse
> > > > -SRCREV = "d4a31e8ee5592072d8d5208b3e950cba2d89b6bd"
> > > > +SRCREV = "ef7c2af65966a57c98d3c47e6c2fe1ce2103b7f6"
> > > >
> > > >  SRC_URI = "git://git.denx.de/u-boot.git \
> > >
> > > Seems to break on musl:
> > >
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/64/builds/1392
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/45/builds/1396
> >
> > Looks like the Gentoo folks hit this as well:
> > http://patchwork.ozlabs.org/patch/1211367/
> >
> > Can someone review/ack that please if that's the best way (rather than
> > reworking include/u-boot/crc.h) to deal with the problem?  Thanks!
> >
> 
> this patch looks ok to me 

Would you mind following up with a Reviewed-by?

> but I do have one question
> Include should be "compiler.h" and not <compiler.h> which would be for
> system includes, is there a reason to not do so.

We're not consistent about "" vs <> for headers U-Boot provides for
system tools, a follow-up patch would be appreciated.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20191229/32ef05be/attachment.sig>


More information about the Openembedded-core mailing list