Hi Albert,
On Thu, Jun 20, 2013 at 2:55 AM, Albert ARIBAUD albert.u.boot@aribaud.netwrote:
Hi Simon,
On Wed, 19 Jun 2013 21:15:10 -0700, Simon Glass sjg@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@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 :) ]
:-) I did use patman but only let it send a single patch. I felt that my list spam quotient was high enough already, but perhaps not.
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@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.
Yes
Note: who/what exactly is "u-boot-review@google.com"?
It is a group of U-Boot people in Google, just so I don't have to copy heaps of people and break the cc limit of the mailing list.
Regards, Simon