From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 17/21] arm: Implement the 'fake' go command
Date: Thu, 20 Jun 2013 11:55:20 +0200 [thread overview]
Message-ID: <20130620115520.6464e91f@lilith> (raw)
In-Reply-To: <1371701714-14849-18-git-send-email-sjg@chromium.org>
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 :) ]
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.
Note: who/what exactly is "u-boot-review at google.com"?
Amicalement,
--
Albert.
next prev parent reply other threads:[~2013-06-20 9:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-20 4:15 [U-Boot] [PATCH v3 17/21] arm: Implement the 'fake' go command Simon Glass
2013-06-20 9:55 ` Albert ARIBAUD [this message]
2013-06-20 20:40 ` Simon Glass
2013-08-21 21:59 ` vb at vsbe.com
2013-08-21 22:16 ` Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130620115520.6464e91f@lilith \
--to=albert.u.boot@aribaud.net \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox