On Fri, 12 Apr 2019, Simon Glass wrote: > Hi Robert, > > On Fri, 12 Apr 2019 at 06:31, Robert P. J. Day <rpj...@crashcourse.ca> wrote: > > > > > > was tracing the ARM-based boot sequence and ended up at the call to > > board_init_r(): > > > > void board_init_r(gd_t *new_gd, ulong dest_addr) > > { > > /* > > * Set up the new global data pointer. So far only x86 does this > > * here. > > * TODO(s...@chromium.org): Consider doing this for all archs, or > > * dropping the new_gd parameter. > > */ > > ... snip ... > > > > but as i read board_init_r(), it's not clear what that routine is > > doing with that second argument, "dest_addr".
... snip ... > I believe this is not used. The value is available in gd->dest_addr > and we now read it from there. > > So perhaps this argument could be dropped. massive sigh of relief here as i was totally prepared for a response like, "oh, come on, how can you not see how it's being used?" :-) anyway, i can take a shot at a patch unless someone else wants to do it. i'm just relieved that it wasn't that dumb a question after all. rday -- ======================================================================== Robert P. J. Day Ottawa, Ontario, CANADA http://crashcourse.ca Twitter: http://twitter.com/rpjday LinkedIn: http://ca.linkedin.com/in/rpjday ======================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot