FYI: Please pull u-boot-dm

Tom Rini trini at konsulko.com
Tue Feb 11 19:27:26 CET 2020


On Tue, Feb 11, 2020 at 11:20:36AM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Sat, 8 Feb 2020 at 07:51, Simon Glass <sjg at google.com> wrote:
> >
> > Hi Stephen,
> >
> > On Thu, 6 Feb 2020 at 15:38, Simon Glass <sjg at google.com> wrote:
> > >
> > > Hi Stephen,
> > >
> > > On Thu, 6 Feb 2020 at 15:32, Stephen Warren <swarren at wwwdotorg.org> wrote:
> > > >
> > > > On 2/6/20 2:55 PM, Simon Glass wrote:
> > > > > Hi Tom,
> > > > >
> > > > > This cannot be pulled yet since we need to update gitlab's docker
> > > > > image to include SDL2. But gitlab seems to be having various problems
> > > > > this week and today i won't work at all: ...
> > > >
> > > > I see the following build error in u-boot-dm/master via Jenkins:
> > > >
> > > > > drivers/misc/p2sb_emul.c: In function ‘sandbox_p2sb_emul_map_physmem’:
> > > > > drivers/misc/p2sb_emul.c:237:6: warning: ‘child’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> > > > >   ret = axi_read(child, offset, priv->regs, AXI_SIZE_32);
> > > > >       ^
> > > > >   CC      common
> > >
> > > Hmmm that's odd. I don't see that, but there are so many device-tree
> > > and libfdt warnings at present I may have missed it. Will take a look.
> >
> > That doesn't happen with my gcc 7.3 toolchain. I'll send a patch to
> > set child to NULL at the start, but a look at the code shows that it
> > is correct.
> >
> > Also this is in master and was not introduced by this series.
> 
> A fix for this is going in via the x86 tree.
> 
> Tom, is there anything else you need from me for this pull request?

Testing it now, thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200211/a129cf88/attachment.sig>


More information about the U-Boot mailing list