From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 0/17] sandbox: Generic board support and other improvements
Date: Wed, 1 May 2013 16:18:46 -0400 [thread overview]
Message-ID: <20130501201846.GX25397@bill-the-cat> (raw)
In-Reply-To: <CAPnjgZ0vTKaLacVWxn=xTpVN5WjbLumV8BTfpXYLucsfbyt3dQ@mail.gmail.com>
On Fri, Apr 26, 2013 at 06:10:15AM -0700, Simon Glass wrote:
> Hi Tom,
>
> On Mon, Apr 22, 2013 at 9:08 AM, Simon Glass <sjg@chromium.org> wrote:
> > Hi Tom,
> >
> > On Mon, Apr 22, 2013 at 8:18 AM, Tom Rini <trini@ti.com> wrote:
> >> On Sat, Apr 20, 2013 at 11:42:35AM -0700, Simon Glass wrote:
> >>
> >>> This series adds generic board support to sandbox and switches to use this
> >>> always.
> >>>
> >>> With sandbox it was noticed that turning CONFIG_SYS_GENERIC_BOARD off
> >>> can cause a build failure if a previous autoconf.mk exists which indicates
> >>> that generic board is not supported, so a patch is provided to fix this.
> >>>
> >>> It is useful to convert a pointer into an 'address' in the sandbox RAM
> >>> buffer - the opposite of map_sysmem(). This is added in this series and
> >>> used in several places.
> >>>
> >>> With sandbox it is easier to read a file from the host than to use the
> >>> CONFIG_OF_SEPARATE option, since this option requires knowledge of the
> >>> executable image structure which is not really appropriate on the host
> >>> system. A new CONFIG_OF_HOSTFILE provides this.
> >>>
> >>> A few related FDT changes are included in this series also.
> >>>
> >>> The -c option is enhanced to support passing entire scripts to sandbox.
> >>> This is useful when writing non-trivial test code.
> >>>
> >>> Most of these patches were previously submitted as part of the verified
> >>> boot effort. This series collects the independent sandbox-related patches
> >>> together to make it easier to review. THe whole series is marked as
> >>> version 3 for this reason.
> >>
> >> For the series,
> >> Reviewed-by: Tom Rini <trini@ti.com>
> >>
> >> And I'd say 3/4/5 should be squashed into one patch, but it's your arch
> >> so I'l defer if you think it adds bisect value or similar to do it in
> >> that manner.
> >
> > I did that so that it could be kind-of an example of how this can be
> > done for an arch, given that I am not planning to convert the rest. By
> > removing the dead code in a separate step it seemed a bit clearer to
> > me.
> >
> > But it's fine either way - I will squash it and resend.
>
>
> I have put this series in patchwork as:
>
> http://patchwork.ozlabs.org/bundle/sjg/sandbox/
This has now been applied to u-boot/master, thanks!
> and below is a pull request if you want to take that instead.
>
> I did not go through and add your Reviewed-by to each patch. Am I
> supposed to do that?
No, that pain is supposed to help spur us into improving patchwork or
the new tool we talked about back at LSM.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130501/992de569/attachment.pgp>
prev parent reply other threads:[~2013-05-01 20:18 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-20 18:42 [U-Boot] [PATCH v3 0/17] sandbox: Generic board support and other improvements Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 01/17] Trigger generic board error only when building Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 02/17] sandbox: Provide a way to map from host RAM to U-Boot RAM Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 03/17] sandbox: Add support for generic board Simon Glass
2013-04-22 14:03 ` Tom Rini
2013-04-22 14:11 ` Tom Rini
2013-04-26 12:54 ` Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 04/17] sandbox: Use generic board init Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 05/17] sandbox: Remove old board init code Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 06/17] sandbox: Add CONFIG_OF_HOSTFILE to read FDT from host file Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 07/17] fdt: Add a parameter to fdt_valid() Simon Glass
2013-05-06 23:41 ` Jerry Van Baren
2013-04-20 18:42 ` [U-Boot] [PATCH v3 08/17] Add getenv_hex() to return an environment variable as hex Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 09/17] fdt: Allow fdt command to check and update control FDT Simon Glass
2013-05-06 23:41 ` Jerry Van Baren
2013-04-20 18:42 ` [U-Boot] [PATCH v3 10/17] sandbox: fdt: Support fdt command for sandbox Simon Glass
2013-05-06 23:42 ` Jerry Van Baren
2013-04-20 18:42 ` [U-Boot] [PATCH v3 11/17] fdt: Skip checking FDT if the pointer is NULL Simon Glass
2013-05-06 23:43 ` Jerry Van Baren
2013-04-20 18:42 ` [U-Boot] [PATCH v3 12/17] Revert "fdt- Tell the FDT library where the device tree is" Simon Glass
2013-05-06 23:43 ` Jerry Van Baren
2013-04-20 18:42 ` [U-Boot] [PATCH v3 13/17] sandbox: Allow -c argument to provide a command list Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 14/17] sandbox: Support 'source' command Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 15/17] fs: Add support for saving data to filesystems Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 16/17] sandbox: fs: Add support for saving files to host filesystem Simon Glass
2013-04-20 18:42 ` [U-Boot] [PATCH v3 17/17] sandbox: config: Enable CONFIG_FIT and CONFIG_CMD_FIT Simon Glass
2013-04-22 15:18 ` [U-Boot] [PATCH v3 0/17] sandbox: Generic board support and other improvements Tom Rini
2013-04-22 16:08 ` Simon Glass
2013-04-26 13:10 ` Simon Glass
2013-05-01 20:18 ` Tom Rini [this message]
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=20130501201846.GX25397@bill-the-cat \
--to=trini@ti.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.