[U-Boot] Pull request: u-boot-mips

Paul Burton paul.burton at imgtec.com
Thu Jul 13 17:58:47 UTC 2017


Hi Daniel & Tom,

On Thursday, 13 July 2017 03:51:00 PDT Daniel Schwierzeck wrote:
> Hi Paul,
> 
> 2017-07-13 2:33 GMT+02:00 Tom Rini <trini at konsulko.com>:
> > On Wed, Jul 12, 2017 at 04:57:42PM -0400, Tom Rini wrote:
> >> On Wed, Jul 12, 2017 at 10:32:29PM +0200, Daniel Schwierzeck wrote:
> >> > Hi Tom,
> >> > 
> >> > This supports dynamic relocation on MIPS without the need for building
> >> > a
> >> > position-independent executable. This notably reduces the code size for
> >> > all MIPS boards.
> >> > 
> >> > The following changes since commit 
d85ca029f257b53a96da6c2fb421e78a003a9943:
> >> >   Prepare v2017.07 (2017-07-10 13:07:38 -0400)
> >> > 
> >> > are available in the git repository at:
> >> >   git://git.denx.de/u-boot-mips.git master
> >> > 
> >> > for you to fetch changes up to 
f653dcd5720c4135607211f7304283d7a8ec3b8a:
> >> >   MIPS: bootm: Fix broken boot_env_legacy codepath (2017-07-12 22:10:42
> >> >   +0200)>> 
> >> I'm seeing:
> >>       mips:  +   tplink_wdr4300
> >> 
> >> +(tplink_wdr4300)    pfx##hdr32[idx].field = _val;   \
> >> +(tplink_wdr4300)                          ^
> >> +(tplink_wdr4300) ../tools/mips-relocs.c:51:11: note: ?_val? was declared
> >> here +(tplink_wdr4300)   uint64_t _val;      \
> >> +(tplink_wdr4300)            ^
> >> +(tplink_wdr4300) ../tools/mips-relocs.c:88:2: note: in expansion of
> >> macro ?set_hdr_field? +(tplink_wdr4300)   set_hdr_field(p, idx, field,
> >> val)
> >> +(tplink_wdr4300)   ^~~~~~~~~~~~~
> >> +(tplink_wdr4300) ../tools/mips-relocs.c:408:3: note: in expansion of
> >> macro ?set_phdr_field? +(tplink_wdr4300)    set_phdr_field(i, p_filesz,
> >> load_sz);
> >> +(tplink_wdr4300)    ^~~~~~~~~~~~~~
> >> w+(tplink_wdr4300) ../tools/mips-relocs.c: In function ?main?:
> >> w+(tplink_wdr4300) ../tools/mips-relocs.c:77:25: warning: ?_val? may be
> >> used uninitialized in this function [-Wmaybe-uninitialized]
> >> 
> >> for what I suspect is going to be all MIPS.  Host tools here are gcc-6.3.
> > 
> > Yeah, this is all MIPS boards.  Please fix, thanks!
> 
> Paul, could you send a follow-up patch to fix this? Thanks.

Sure. I'm on gcc 7.1.1 which doesn't show this issue. Is the following 
sufficient to fix this for you Tom? I can submit it as a proper patch if you 
like & it works out.

Thanks,
    Paul

diff --git a/tools/mips-relocs.c b/tools/mips-relocs.c
index b690fa53c4..75d532546b 100644
--- a/tools/mips-relocs.c
+++ b/tools/mips-relocs.c
@@ -69,6 +69,9 @@
        case 8:                                                 \
                _val = is_be ? htobe64(val) : htole64(val);     \
                break;                                          \
+       default:                                                \
+               __builtin_unreachable();                        \
+               break;                                          \
        }                                                       \
                                                                \
        if (is_64)                                              \
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170713/6e29d47e/attachment.sig>


More information about the U-Boot mailing list