On 2/11/20 11:27 AM, Tom Rini wrote:
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 <s...@google.com> wrote:

Hi Stephen,

On Thu, 6 Feb 2020 at 15:38, Simon Glass <s...@google.com> wrote:

Hi Stephen,

On Thu, 6 Feb 2020 at 15:32, Stephen Warren <swar...@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.

Now that this has been merged, the build break has come along with it. Is there any chance of picking up the fix from the x86 tree quickly?

Reply via email to