From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/4] common: imx: Implement generic u-boot.nand target
Date: Tue, 26 Feb 2013 08:19:42 +0100 [thread overview]
Message-ID: <201302260819.43138.marex@denx.de> (raw)
In-Reply-To: <954640255.109456.1361836253032.JavaMail.root@advansee.com>
Dear Beno?t Th?baudeau,
> Dear Scott Wood,
>
> On Tuesday, February 26, 2013 12:07:25 AM, Scott Wood wrote:
> > On 02/25/2013 05:03:30 PM, Marek Vasut wrote:
> > > Dear Scott Wood,
> > >
> > > > So maybe we need a more general (but optional) CONFIG_BUILD_TARGET.
> > >
> > > Can you elaborate?
> >
> > Same as CONFIG_SPL_TARGET, but not SPL-specific. Basically a way for a
> > board config file to add to $(ALL-y).
> >
> > > > So each one would set the appropriate CONFIG_BUILD_TARGET for
> > >
> > > whatever
> > >
> > > > needs to get built, and then something like CONFIG_NAND_IMAGE could
> > > > hold the image name that should be linked to produce a standard
> > > > u-boot-nand.bin output.
> > >
> > > Yea, sounds reasonable. But why call it CONFIG_ , it can't be stored
> > > in the
> > > board.h files, it has to be somewhere in the Makefile hierarchy.
> >
> > Why can't it go in the board.h files?
>
> We could do all that, but should we? As I said to Marek, I think that it's
> a big mistake to omit the SPL here. The only other solution to get a
> reliable boot would be the DBBT, but it's very hard to use in real life,
> away from a production line. The SPL is really easy to enable here, and
> it's only a matter of time before someone gets bitten by this lack of
> reliability, so why not just do things right? The boot time and footprint
> of an SPL would really be negligible, and it's not because other
> implementations omit both SPL and a valid DBBT that U-Boot should do the
> same.
I'm not against SPL, but then we're starting to drift away from the whole idea
of generating u-boot-nand.bin or similar image. Being able to generate u-boot-
nand.bin or u-boot-sd.bin etc ... on a per-CPU basis (since this is CPU
specific) is the ultimate goal here, whatever is embedded in the image.
Best regards,
Marek Vasut
next prev parent reply other threads:[~2013-02-26 7:19 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-25 18:19 [U-Boot] [PATCH 1/4] common: imx: Implement generic u-boot.nand target Marek Vasut
2013-02-25 18:19 ` [U-Boot] [PATCH 2/4] imx: Align the imximage header and payload to multiples of 4k Marek Vasut
2013-02-25 19:13 ` Benoît Thébaudeau
2013-02-25 21:07 ` Marek Vasut
2013-02-25 21:16 ` Benoît Thébaudeau
2013-02-25 18:19 ` [U-Boot] [PATCH 3/4] mx5: Add NAND clock handling Marek Vasut
2013-02-25 19:27 ` Benoît Thébaudeau
2013-02-25 18:24 ` [U-Boot] [PATCH 4/4] mx5: Add support for DENX M53EVK Marek Vasut
2013-02-25 19:33 ` Benoît Thébaudeau
2013-02-26 13:52 ` Fabio Estevam
2013-02-26 19:32 ` Fabio Estevam
2013-04-14 17:20 ` Marek Vasut
2013-04-14 19:07 ` Wolfgang Denk
2013-04-14 19:18 ` Marek Vasut
2013-02-25 18:51 ` [U-Boot] [PATCH 1/4] common: imx: Implement generic u-boot.nand target Benoît Thébaudeau
2013-02-25 21:09 ` Marek Vasut
2013-02-25 21:13 ` Benoît Thébaudeau
2013-02-26 1:02 ` [U-Boot] patch error wanxs
2013-02-25 20:10 ` [U-Boot] [PATCH 1/4] common: imx: Implement generic u-boot.nand target Scott Wood
2013-02-25 21:10 ` Marek Vasut
2013-02-25 22:15 ` Scott Wood
2013-02-25 23:03 ` Marek Vasut
2013-02-25 23:07 ` Scott Wood
2013-02-25 23:50 ` Benoît Thébaudeau
2013-02-26 7:19 ` Marek Vasut [this message]
2013-02-26 11:33 ` Benoît Thébaudeau
2013-02-27 17:33 ` Tom Rini
2013-02-28 18:50 ` Marek Vasut
2013-02-28 18:53 ` Tom Rini
2013-02-26 7:17 ` Marek Vasut
2013-02-26 23:10 ` Scott Wood
2013-02-27 22:18 ` Benoît Thébaudeau
2013-02-27 23:44 ` Tom Rini
2013-02-27 23:47 ` Benoît Thébaudeau
2013-02-28 12:18 ` Benoît Thébaudeau
2013-02-28 14:03 ` Tom Rini
2013-02-28 15:24 ` Benoît Thébaudeau
2013-02-28 16:06 ` Benoît Thébaudeau
2013-02-28 16:21 ` Tom Rini
2013-02-28 16:29 ` Benoît Thébaudeau
2013-02-28 16:42 ` Tom Rini
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=201302260819.43138.marex@denx.de \
--to=marex@denx.de \
--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.