[U-Boot] [PATCH v3 17/21] arm: Implement the 'fake' go command

Albert ARIBAUD albert.u.boot at aribaud.net
Thu Jun 20 11:55:20 CEST 2013


Hi Simon,

On Wed, 19 Jun 2013 21:15:10 -0700, Simon Glass <sjg at chromium.org>
wrote:

> Implement this feature on ARM for tracing.
> 
> It would be nice to have generic bootm support so that it is easily
> implemented on any arch.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> Changes in v3:
> - Revert to using 'images' instead of 'image' in boot_jump_linux()
> 
> Changes in v2: None

Small nitpick: I prefer whole series to be reposted even when there is
only a single patch changed. My rationale for this is: when a series
has been posted as V2 and a single patch appears as V3, as a reviewer I
do not know whether that patch was really the only one intended as V3,
or whether other patches were too but did not make it through for some
reason.

Also, as this V3 patch did not have any "In-Reply-To" header, it is not
threaded to the V2 series by my claws-mail. That makes it less easy to
try and get an overview of the current series.

[insert witty remark about how patman quite adequately addresses the two
points above :) ]

Here I will assume 17/21 is the only patch in the series and the rest
should come from v2 unchanged. As this is the only pure ARM related
patch in the series (1), I guess my contribution shall be...

Acked-by: Albert ARIBAUD <albert.u.boot at aribaud.net>

(1) patch 18/21 is Samsung and patch 19/21 touches on exynos5250-based
boards; my ack is less adequate than that of, respectively, Minkyu as
the Samsung custodian and the exynos5250-based boards' maintainers.

Note: who/what exactly is "u-boot-review at google.com"?

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list