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

Daniel Schwierzeck daniel.schwierzeck at gmail.com
Thu Jul 13 11:05:33 UTC 2017


Hi Tom,

2017-07-12 22:57 GMT+02:00 Tom Rini <trini at konsulko.com>:
> 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.
>

how about adding a separate build step in Travis CI for all host tools
and building that step with different compilers (recent gcc or clang)?
Or we always install a recent x86_64 toolchain and override the host
toolchain contained in the Ubuntu builder image. This toolchain is
then used for all build steps.

-- 
- Daniel


More information about the U-Boot mailing list