public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 10/11] Add u-boot-pad.bin target to the Makefile
Date: Fri, 21 Sep 2012 13:33:15 -0500	[thread overview]
Message-ID: <1348252395.19917.6@snotra> (raw)
In-Reply-To: <20120921054348.E7801203200@gemini.denx.de> (from wd@denx.de on Fri Sep 21 00:43:48 2012)

On 09/21/2012 12:43:48 AM, Wolfgang Denk wrote:
> Dear Tom,
> 
> In message  
> <5FBF8E85CA34454794F0F7ECBA79798F379F6FD992@HQMAIL04.nvidia.com> you  
> wrote:
> >
> > If you flash u-boot-dtb-tegra.bin, you'll get a fully functioning
> > U-Boot. There's an intermediate file (u-boot-dtb.bin) that I assume
> > is u-boot.bin+dtb - I'm not sure why it's left around - Allen could
> > comment here.
> 
> I _dislike_ the idea of having image names which include architecture
> or even board parts.  I would really like to have generic names, that
> can be used in a consistent way across platforms, architectures and
> boards.
> 
> > So in my eyes, all you really need is u-boot-dtb-tegra.bin - an
> > unwieldy name, to be sure, but it seems to satisfy your request for  
> a
> > Soc identifier in the name. I voted for just having u-boot.bin be  
> the
> 
> Please reconsider.  I definitely do NOT want to have SoC names or that
> in any such images!
> 
> 
> IIRC, the original idea was to provide image names (common for all
> architectures, SoCs, boards) that only depend on where you install
> U-Boot to.  in this way, we would have:
> 
> - u-boot.bin	for the generic case (say, for installation into NOR
>                 flash, no SPL or similar needed).
> - u-boot-nand.bin
> 		for installation in NAND (with all needed headers,
> 		padding etc. included)
> - u-boot-onenand.bin
> 		for installation in OneNAND
> - u-boot.sd	for installation on a SDCard
> 		[actually we have an inconsistency in names here; this
> 		should have been "u-boot-sd.bin" or maybe even better
> 		"u-boot-sdcard.bin"]
> etc.
> 
> It is very important to me that we do NOT include any architectures,
> SoCs, or board specifc parts in the names because this will cause
> major PITA for all kind of automatic test suites etc.

The awkwardness with naming based on nand/onenand/sd is that we no  
longer have build infrastructure that is specific to the type of boot  
device -- and IIRC with some of the newer SPL targets, the same image  
works on multiple types of boot device.

Having u-boot.bin be the final output regardless of internal  
implementation details such as spl would avoid that problem, and be  
even nicer to automated testing than the nand/onenand/sd names.

-Scott

  parent reply	other threads:[~2012-09-21 18:33 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-19 11:25 [U-Boot] [PATCH v4 00/11] S3C24XX: Add support to MINI2416 board José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 01/11] Add configuration option to select printf() inclusion on SPL José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 02/11] S3C24XX: Add core support for Samsung's S3C24XX SoCs José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 03/11] serial: Add support to 4 ports in serial_s3c24x0 José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 04/11] serial: Use a more precise baud rate generation for serial_s3c24x0 José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 05/11] serial: Remove unnecessary delay in serial_s3c24x0 José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 06/11] rtc: Improve rtc_get() on s3c24x0_rtc José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 07/11] rtc: Fix rtc_reset() " José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 08/11] rtc: Don't allow setting unsuported years " José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 09/11] S3C24XX: Add NAND Flash driver José Miguel Gonçalves
2012-09-19 16:19   ` Scott Wood
2012-09-19 16:34     ` José Miguel Gonçalves
2012-09-19 11:25 ` [U-Boot] [PATCH v4 10/11] Add u-boot-pad.bin target to the Makefile José Miguel Gonçalves
2012-09-19 16:10   ` Scott Wood
2012-09-19 16:58     ` José Miguel Gonçalves
2012-09-19 17:08       ` Scott Wood
2012-09-19 17:40         ` José Miguel Gonçalves
2012-09-19 17:11       ` Stephen Warren
2012-09-19 18:19         ` Tom Rini
2012-09-19 18:36           ` Scott Wood
2012-09-19 20:17             ` Tom Rini
2012-09-19 18:44           ` José Miguel Gonçalves
2012-09-19 22:39   ` Scott Wood
2012-09-19 23:31     ` Tom Rini
2012-09-19 23:36       ` Scott Wood
2012-09-19 23:40         ` Tom Rini
2012-09-20  0:38           ` José Miguel Gonçalves
2012-09-20  1:29             ` Tom Rini
2012-09-20 16:01               ` Tom Warren
2012-09-20 16:23                 ` Tom Rini
2012-09-20 16:32                   ` Tom Warren
2012-09-20 18:09                 ` Scott Wood
2012-09-21  1:08                   ` José Miguel Gonçalves
2012-09-21  5:43                 ` Wolfgang Denk
2012-09-21  8:13                   ` José Miguel Gonçalves
2012-09-21 15:52                     ` Wolfgang Denk
2012-09-21 16:08                       ` Marek Vasut
2012-09-21 16:13                       ` Tom Rini
2012-09-21 16:26                         ` José Miguel Gonçalves
2012-09-21 16:38                           ` Tom Rini
2012-09-21 16:37                         ` Langer Thomas
2012-09-21 18:33                   ` Scott Wood [this message]
2012-09-21 18:43                     ` Marek Vasut
2012-09-21 19:03                       ` Scott Wood
2012-09-21 19:24                         ` Marek Vasut
2012-09-21 19:33                           ` Scott Wood
2012-09-23 16:25                           ` Wolfgang Denk
2012-09-19 11:25 ` [U-Boot] [PATCH v4 11/11] S3C24XX: Add support to MINI2416 board José Miguel Gonçalves
2012-09-19 19:18   ` Tom Rini
2012-09-19 20:34     ` José Miguel Gonçalves

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=1348252395.19917.6@snotra \
    --to=scottwood@freescale.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox