Hi Tom, On Sat, 8 Feb 2020 at 07:51, Simon Glass <[email protected]> wrote: > > Hi Stephen, > > On Thu, 6 Feb 2020 at 15:38, Simon Glass <[email protected]> wrote: > > > > Hi Stephen, > > > > On Thu, 6 Feb 2020 at 15:32, Stephen Warren <[email protected]> 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? Regards, Simon
