[U-Boot] [PATCH v3 4/4] arm: factorize relocate_code routine

Albert ARIBAUD
2013-05-16T16:57:19Z

Hi Beno?t,

On Thu, 16 May 2013 17:56:29 +0200 (CEST), Beno?t Th?baudeau
<benoit.thebaudeau@advansee.com> wrote:

> Hi Albert,

> Keeping an empty line here would be good for code clarity.

> Or "&_start" in the comment above to be more consistent with the comments below.

> It's __rel_dyn_start and __rel_dyn_end with 2 leading underscores in the 4
> comments above.

Will fix all three, thanks.

> Argh! The "bx lr" issue is back. That was correct in v2.

As you say: argh! Will fix, and apologies.

> The rest of the v3 series is fine.

Thanks for your review!

> Best regards,
> Beno?t

Amicalement,
--
Albert.