[PATCH] Revert "spl: Drop bd_info in the data section"

Tom Rini trini at konsulko.com
Mon Apr 12 20:38:00 CEST 2021


On Tue, Apr 13, 2021 at 06:26:08AM +1200, Simon Glass wrote:
> Hi Tom,
> 
> On Tue, 13 Apr 2021 at 06:18, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Tue, Apr 13, 2021 at 05:49:19AM +1200, Simon Glass wrote:
[snip]
> > > As to a weak function, what would the default behaviour be? If we can
> > > define that, then it would be better IMO.
> > >
> > > When we try to refactor things, weak functions and undefined (or
> > > arch-specific behaviour) can really make things tough.
> >
> > Well, what was the problem you were trying to solve here?  I assumed
> > there was a case where things actively broke, with the previous design,
> > and it's not just the 64byte savings in some cases.  But is it?
> 
> Yes the region of memory is not writable, so must be allocated at runtime.

Where, on x86?  Some ARM cases?  That's one of the other things to sort
out here.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210412/bce79a9b/attachment.sig>


More information about the U-Boot mailing list