From: Radu Moisan <radu.moisan@intel.com>
To: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH][RFC] u-boot: Upgrade to upstream stable 2012.07
Date: Fri, 03 Aug 2012 17:45:13 +0300 [thread overview]
Message-ID: <501BE3F9.9050809@intel.com> (raw)
In-Reply-To: <20120803124658.GC30148@jama.jama.net>
On 08/03/2012 03:46 PM, Martin Jansa wrote:
> On Fri, Aug 03, 2012 at 03:38:42PM +0300, Radu Moisan wrote:
>> Building u-boot requires UBOOT_MACHINE. In the u-boot README file
>> building u-boot is achieved with "make NAME_config" and then
>> "make all". I assumend UBOOT_MACHINE to be the "NAME" part and thus,
>> the "_config" addition in u-boot.inc
>> NAME is the machine name, and choices area vailable in boards.conf,
>> I used for testing coreboot-x86.
> From this description I don't see why _config is now appended when
> probably all BSPs had it included in UBOOT_MACHINE variable before
>
> meta-nokia/recipes-bsp/u-boot/u-boot_git.bb:UBOOT_MACHINE_nokia900 = "nokia_rx51_config"
> meta-openmoko/conf/machine/om-gta04.conf:UBOOT_MACHINE = "omap3_gta04_config"
> meta-openmoko/recipes-bsp/u-boot/u-boot_git.bb:UBOOT_MACHINE_om-gta04 = "omap3_gta04_config"
> conf/machine/am180x-evm.conf:UBOOT_MACHINE = "da850evm_config"
> conf/machine/am335x-evm.conf:UBOOT_MACHINE = "am335x_evm_config"
> conf/machine/am3517-evm.conf:UBOOT_MACHINE = "am3517_evm_config"
> conf/machine/am37x-evm.conf:UBOOT_MACHINE = "omap3_evm_config"
> conf/machine/beagleboard.conf:UBOOT_MACHINE = "omap3_beagle_config"
> conf/machine/beaglebone.conf:UBOOT_MACHINE = "am335x_evm_config"
> conf/machine/hawkboard.conf:UBOOT_MACHINE = "da850_omapl138_evm_config"
> conf/machine/include/davinci.inc:UBOOT_MACHINE = "davinci_dvevm_config"
> conf/machine/include/ti814x.inc:UBOOT_MACHINE = "ti8148_evm_config_nand"
> conf/machine/include/ti816x.inc:UBOOT_MACHINE = "ti8168_evm_config"
> conf/machine/omap3-touchbook.conf:UBOOT_MACHINE = "omap3_beagle_config"
> conf/machine/omap3evm.conf:UBOOT_MACHINE = "omap3_evm_config"
> conf/machine/pandaboard.conf:UBOOT_MACHINE = "omap4_panda_config"
>
> And any sensible is to append it to *MACHINE* variable as default:
> recipes-bsp/u-boot/u-boot.inc:UBOOT_MACHINE ?= "${MACHINE}_config"
>
> Cheers,
Ok, missed that, I'll fix it.
radu
next prev parent reply other threads:[~2012-08-03 14:57 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-03 12:38 [PATCH][RFC] u-boot: Upgrade to upstream stable 2012.07 Radu Moisan
2012-08-03 12:46 ` Martin Jansa
2012-08-03 14:45 ` Radu Moisan [this message]
2012-08-03 16:19 ` Wolfgang Denk
2012-08-06 6:28 ` Radu Moisan
2012-08-06 7:45 ` Wolfgang Denk
2012-08-06 13:40 ` Radu Moisan
2012-08-06 13:46 ` Wolfgang Denk
2012-08-06 14:09 ` Radu Moisan
2012-08-06 14:13 ` Wolfgang Denk
2012-08-07 2:58 ` Khem Raj
2012-08-07 5:50 ` Radu Moisan
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=501BE3F9.9050809@intel.com \
--to=radu.moisan@intel.com \
--cc=openembedded-core@lists.openembedded.org \
/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