[U-Boot] [PATCH] arc: Introduce a possibility to not relocate U-boot

Alexey Brodkin Alexey.Brodkin at synopsys.com
Tue Feb 20 17:54:21 UTC 2018


Hi Simon, Tom,

On Sun, 2018-02-04 at 06:40 -0700, Simon Glass wrote:
> On 25 January 2018 at 11:22, Alexey Brodkin <Alexey.Brodkin at synopsys.com> wrote:
> > From: Alexey Brodkin <Alexey.Brodkin at synopsys.com>
> > 
> > Disabling relocation might be useful on ARC for 2 reasons:
> >  a) For advanced debugging with Synopsys proprietary MetaWare debugger
> >     which is capable of accessing much more specific hardware resources
> >     compared to gdb. For example it may show contents of L1 and L2 caches,
> >     internal states of some hardware blocks etc.
> > 
> >     But on the downside MetaWare debugger still cannot work with PIE.
> >     Even though that limitation could be work-arounded with change of ELF's
> >     header and stripping down all debug info but with it we won't have
> >     debug info for source-level debugging which is quite inconvenient.
> > 
> >  b) Some platforms which might benefit from usage of U-Boot basically
> >     don't have enough RAM to accommodate relocation of U-Boot so we
> >     keep code in flash and use as much of RAM as possible for more
> >     interesting things.
> > 
> > Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
> > Cc: Simon Glass <sjg at chromium.org>
> > Cc: Bin Meng <bmeng.cn at gmail.com>
> > Cc: Heiko Schocher <hs at denx.de>
> > Cc: York Sun <york.sun at nxp.com>
> > Cc: Stefan Roese <sr at denx.de>
> > ---
> >  arch/arc/lib/relocate.c | 6 ++++++
> >  arch/arc/lib/start.S    | 8 +++++++-
> >  common/board_f.c        | 3 ++-
> >  3 files changed, 15 insertions(+), 2 deletions(-)
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>

Is it OK if I pull this into my "next" tree?

I'm asking because common/board_f.c is affected even though
that's just an addition of CONFIG_ARC in one #ifdef.

-Alexey


More information about the U-Boot mailing list