On Sat, Jul 09, 2016 at 10:22:37AM +0200, Marek Vasut wrote: > On 07/09/2016 02:28 AM, Tom Rini wrote: > >On Fri, Jul 08, 2016 at 11:48:12PM +0200, Marek Vasut wrote: > >>On 07/08/2016 06:50 PM, Tom Rini wrote: > >>>On Fri, Jul 08, 2016 at 03:18:35PM +0100, Andre Przywara wrote: > >>> > >>>>As printf calls may be executed quite early, we should avoid using any > >>>>BSS stored variables, since some boards put BSS in DRAM, which may not > >>>>have been initialised yet. > >>>>Explicitly mark those "static global" variables as belonging to the > >>>>.data section, to keep tiny-printf clear of any BSS usage. > >>>> > >>>>Signed-off-by: Andre Przywara <andre.przyw...@arm.com> > >>> > >>>Applied to u-boot/master, thanks! > >>> > >>Mind you, __section(".data") would be enough, no need to spell the > >>whole attribute. > > > >True, but that's a helper that's not really utilized today. > > > That's something that should likely be changed, as it allows to > seamlessly deal with compiler quirks (if some were to ever pop up in > this area).
Patches welcome for the next release ;) -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot