All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ulf Samuelsson <ulf.samuelsson@atmel.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] RFC: "make DESTDIR=xxx install" ?
Date: Thu, 13 Aug 2009 22:29:19 +0200	[thread overview]
Message-ID: <4A84779F.7010201@atmel.com> (raw)
In-Reply-To: <20090813180653.CFB3F833DBD2@gemini.denx.de>

Wolfgang Denk skrev:
> Dear Ulf Samuelsson,
> 
> In message <4A843392.4020800@atmel.com> you wrote:
>> Many packages support installing the resulting binary in another
>> location, but U-Boot does not.
>>
>> When you use buildsystems like buildroot and openembedded,
>> you want to collect the end result in a target directory,
>> and while you can use internal knowledge about u-boot
>> to do so, it seems cleaner to me, to do a "make DESTDIR install".
> 
> Did you consider using out-of-tree builds for that? 

This still means that the external buildsystem need to
know the internals of u-boot.

It must know that the binary is called u-boot.bin
and where it is located (topdir).

> 
>> Since you may want to put the binaries for several
>> boards in the same directory (like /tftpboot)
> 
There is nothing to stop me calling u-boot with

make DESTDIR=/tftpboot/$(MACHINE)
TARGET=$(MACHINE)-u-boot-$(U_BOOT_VERSION)-$(REVISION).bin install


> In my experience this is not exactly a lucky choice; if youu have to
> maintain more than just a few boards, you definitely want to have a
> subdirectory per board in /tftpboot.
> 
>> it is not always good to call the binary simply u-boot.bin.
> 
> ...which then is not problem any more.

If you maintain just a couple of boards, then it is inconvenient to
have a subdirectory.
If the tftp command in u-boot would fetch from that subdirectory
automatically, then it would be less of a pain.

> 
>> I guess "make DESTDIR=<destination> TARGET=<name> install" would work
>>
>> Alternatively, we collect the final binary from several variables,
> 
> One question remains: what is "the final binary"?  for  example,  for
> the  "kilauea"  board  it  may be "u-boot.bin" (when booting from NOR
> flash), but it might also be "u-boot-nand.bin" (when booting from NOR
> flash). Oh, and board "foo"  uses  not  the  binary  image,  but  the
> S-Record  file  in  their factory installer, so we use "u-boot.srec".
> Another  board  requires  am  Intel  hex   image,   so   they   build
> "u-boot.hex". For the Marvell processors, we will use a special image
> format,  so  it's "u-boot.kwb". BlackFin uses some similar mechanism,
> but a different name, IIRC.

I am happy with u-boot.bin as a default,
but there is nothing stopping you from adding more parameters
like FORMAT=srec or FORMAT=hex.

If we come to use Kconfig, there will likely be board specific things
which can be used.

ifeq ($(CONFIG_BOARD_KILAUEA),y)
ifeq ($(CONFIG_BOARD_KILAUEA_NAND),y)
INSTALL_TARGETS += u-boot-nand.bin
else
INSTALL_TARGETS += u-boot.bin
end
endif

I think there are plenty of different solutions to this.

> 
> And no, we definitlely do not always want to build (and install) all
> these images. That would be just a waste of resources.
> 

You build what you currently build, and the make install
will copy that to another place.
I see no harm in copying both u-boot.bin and u-boot.srec,
since it is easier to delete what you see you do not need
than to get what you are not even aware is existing.

With the current method,assuming u-boot.bin is what you always
wanted in the past, u-boot.kwb would certainly NOT have been copied.



> Best regards,
> 
> Wolfgang Denk
> 

BR
Ulf Samuelsson

  reply	other threads:[~2009-08-13 20:29 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-13 15:38 [U-Boot] RFC: "make DESTDIR=xxx install" ? Ulf Samuelsson
2009-08-13 18:06 ` Wolfgang Denk
2009-08-13 20:29   ` Ulf Samuelsson [this message]
2009-08-13 21:03     ` Wolfgang Denk
2009-08-15  5:01       ` Ulf Samuelsson
2009-08-15  6:32         ` Wolfgang Denk
2009-08-15  9:35           ` Ulf Samuelsson
2009-08-15 10:42             ` Wolfgang Denk
2009-08-13 18:06 ` ksi at koi8.net
2009-08-13 20:29   ` Ulf Samuelsson

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=4A84779F.7010201@atmel.com \
    --to=ulf.samuelsson@atmel.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.