Hi Heiko, On Mon, 04 Feb 2013 09:21:02 +0100, Heiko Schocher <h...@denx.de> wrote:
> Hello Albert, > > On 02.02.2013 18:02, Albert ARIBAUD wrote: > > Refactor linker-generated array code so that symbols > > which were previously linker-generated are now compiler- > > generated. This causes relocation records of type > > R_ARM_ABS32 to become R_ARM_RELATIVE, which makes > > code which uses LGA able to run before relocation as > > well as after. > > > > Note: this affects more than ARM targets, as linker- > > lists span possibly all target architectures, notably > > PowerPC. > > > > Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> > > --- > > for arm926ejs > > Tested-by: Heiko Schocher <h...@denx.de> Thanks! > Just one minor comment ... > > [...] > > diff --git a/common/command.c b/common/command.c > > index 50c8429..6ac59e4 100644 > > --- a/common/command.c > > +++ b/common/command.c > > @@ -507,6 +507,7 @@ static int cmd_call(cmd_tbl_t *cmdtp, int flag, int > > argc, char * const argv[]) > > int result; > > > > result = (cmdtp->cmd)(cmdtp, flag, argc, argv); > > + > > remove this Codingstyle change please. Sorry, should have triple-checked that -- this single blank line change in common/command.c stems from ad hoc code I'd added here for test purposes when the patch was only proof-of-concept. Will fix in V2. > bye, > Heiko Amicalement, -- Albert. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
