Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2012-04-03 Thread Stefano Babic
On 02/04/2012 20:25, Mike Frysinger wrote: > > the suggested ALL-$(CONFIG_xxx) syntax should be usable in the > arch/soc config.mk files Right. I push a new version ;-) Stefano -- = DENX Software Engineering GmbH, MD: Wol

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2012-04-02 Thread Mike Frysinger
On Mon, Apr 2, 2012 at 13:03, Stefano Babic wrote: > On 11/11/2011 14:26, Loďc Minier wrote: >> On Fri, Nov 11, 2011, Daniel Schwierzeck wrote: >>> ALL-$(CONFIG_UBOOT_IMG) += $(obj)u-boot.img >>> ALL-$(CONFIG_UBOOT_IMX) += $(obj)u-boot.imx >>> ALL-$(CONFIG_UBOOT_KWB) += $(obj)u-boot.kwb >>> >>> the

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2012-04-02 Thread Stefano Babic
On 11/11/2011 14:26, Loïc Minier wrote: > On Fri, Nov 11, 2011, Daniel Schwierzeck wrote: >> ALL-$(CONFIG_UBOOT_IMG) += $(obj)u-boot.img >> ALL-$(CONFIG_UBOOT_IMX) += $(obj)u-boot.imx >> ALL-$(CONFIG_UBOOT_KWB) += $(obj)u-boot.kwb >> >> then you can define CONFIG_UBOOT_KWB or CONFIG_UBOOT_IMX in yo

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-16 Thread Wolfgang Denk
Dear =?iso-8859-1?Q?Lo=EFc?= Minier, In message <2011133514.gc2...@bee.dooz.org> you wrote: > > The CONFIG_IMX_CONFIG actually changes per-board, so the boards.cfg > approach to set it seems ok; however we could try defining the No. The ability to pass additional parameters in boards.cfg wa

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-16 Thread Stefano Babic
On 11/11/2011 02:35 PM, Loïc Minier wrote: > On Thu, Nov 10, 2011, Mike Frysinger wrote: >>> CONFIG_IMX_CONFIG is set in boards.cfg. >> >> ugly undocumented CONFIG's are awesome >> >> this really should be in a subdir rather than the top level. we want >> to keep arch/soc-specific cruft out of t

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-11 Thread Mike Frysinger
On Friday 11 November 2011 08:35:14 Loïc Minier wrote: > On Thu, Nov 10, 2011, Mike Frysinger wrote: > > > CONFIG_IMX_CONFIG is set in boards.cfg. > > > > ugly undocumented CONFIG's are awesome > > > > this really should be in a subdir rather than the top level. we want > > to keep arch/soc-spe

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-11 Thread Loïc Minier
On Thu, Nov 10, 2011, Mike Frysinger wrote: > > CONFIG_IMX_CONFIG is set in boards.cfg. > > ugly undocumented CONFIG's are awesome > > this really should be in a subdir rather than the top level. we want > to keep arch/soc-specific cruft out of the top level Makefile if > possible. isn't there

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-11 Thread Loïc Minier
On Fri, Nov 11, 2011, Aneesh V wrote: > arch/arm/cpu/armv7/omap4/config.mk has this: > > ifdef CONFIG_SPL_BUILD > ALL-y += $(OBJTREE)/MLO > else > ALL-y += $(obj)u-boot.img > endif > > However, this may have to be duplicated in many such config.mk files. Ah I had seen this snippet and wondered

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-11 Thread Loïc Minier
On Fri, Nov 11, 2011, Daniel Schwierzeck wrote: > ALL-$(CONFIG_UBOOT_IMG) += $(obj)u-boot.img > ALL-$(CONFIG_UBOOT_IMX) += $(obj)u-boot.imx > ALL-$(CONFIG_UBOOT_KWB) += $(obj)u-boot.kwb > > then you can define CONFIG_UBOOT_KWB or CONFIG_UBOOT_IMX in your > board config if needed That'd be a fine

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Aneesh V
On Friday 11 November 2011 07:21 AM, Mike Frysinger wrote: > On Thursday 10 November 2011 18:45:30 Loïc Minier wrote: >> On Thu, Nov 10, 2011, Mike Frysinger wrote: +ifneq ($(CONFIG_IMX_CONFIG),) +ALL-y += $(obj)u-boot.imx +endif >>> >>> why won't this work: >>> ALL-$(CONFIG_IMX_CONF

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Daniel Schwierzeck
On 11.11.2011 00:45, Loïc Minier wrote: > On Thu, Nov 10, 2011, Mike Frysinger wrote: >>> +ifneq ($(CONFIG_IMX_CONFIG),) >>> +ALL-y += $(obj)u-boot.imx >>> +endif >> why won't this work: >> ALL-$(CONFIG_IMX_CONFIG) += $(obj)u-boot.mix >> that was the point of naming it "ALL-y" in the first place ..

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Mike Frysinger
On Thursday 10 November 2011 18:45:30 Loïc Minier wrote: > On Thu, Nov 10, 2011, Mike Frysinger wrote: > > > +ifneq ($(CONFIG_IMX_CONFIG),) > > > +ALL-y += $(obj)u-boot.imx > > > +endif > > > > why won't this work: > > ALL-$(CONFIG_IMX_CONFIG) += $(obj)u-boot.mix > > that was the point of naming i

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Loïc Minier
On Thu, Nov 10, 2011, Mike Frysinger wrote: > > +ifneq ($(CONFIG_IMX_CONFIG),) > > +ALL-y += $(obj)u-boot.imx > > +endif > why won't this work: > ALL-$(CONFIG_IMX_CONFIG) += $(obj)u-boot.mix > that was the point of naming it "ALL-y" in the first place ... That's what I tried at first, but CONFIG_

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Mike Frysinger
On Thursday 10 November 2011 08:49:41 Loïc Minier wrote: > --- a/Makefile > +++ b/Makefile > > +ifneq ($(CONFIG_IMX_CONFIG),) > +ALL-y += $(obj)u-boot.imx > +endif why won't this work: ALL-$(CONFIG_IMX_CONFIG) += $(obj)u-boot.mix that was the point of naming it "ALL-y" in the first place ... -mi

Re: [U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Stefano Babic
On 11/10/2011 02:49 PM, Loïc Minier wrote: > Signed-off-by: Loïc Minier > --- > Makefile |4 > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/Makefile b/Makefile > index 294c762..0ccc3bf 100644 > --- a/Makefile > +++ b/Makefile > @@ -365,6 +365,10 @@ ALL-$(CONFIG_MMC_

[U-Boot] [PATCH 1/2] Build u-boot.imx by default when board uses it

2011-11-10 Thread Loïc Minier
Signed-off-by: Loïc Minier --- Makefile |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/Makefile b/Makefile index 294c762..0ccc3bf 100644 --- a/Makefile +++ b/Makefile @@ -365,6 +365,10 @@ ALL-$(CONFIG_MMC_U_BOOT) += $(obj)mmc_spl/u-boot-mmc-spl.bin ALL-$(CONFIG_SPL)