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
next prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox