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! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
