| Message ID | 1333463576-8365-1-git-send-email-sbabic@denx.de |
|---|---|
| State | Accepted |
| Commit | 38fcc71cc58b1c9a224b707b03eb6574cb2dc027 |
| Delegated to: | Stefano Babic |
| Headers | show |
On Tue, Apr 3, 2012 at 07:32, Stefano Babic wrote: > --- a/arch/arm/cpu/armv7/config.mk > +++ b/arch/arm/cpu/armv7/config.mk > > +ifneq ($(CONFIG_IMX_CONFIG),) > +ALL-y += $(obj)u-boot.imx > +endif i guess CONFIG_IMX_CONFIG is a free-form string and not just boolean, so: Acked-by: Mike Frysinger <vapier@gentoo.org> -mike
On 03.04.2012 16:32, Stefano Babic wrote: > Freescale SOCs require an header to u-boot.bin > > The patch adds u-boot.imx to the default targets > if the imx file is set (IMX_CONFIG). > > Signed-off-by: Stefano Babic <sbabic@denx.de> > Cc: Albert ARIBAUD <albert.u.boot@aribaud.net> > CC: Loïc Minier <loic.minier@linaro.org> > CC: Mike Frysinger <vapier@gentoo.org> > --- > arch/arm/cpu/armv7/config.mk | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/cpu/armv7/config.mk b/arch/arm/cpu/armv7/config.mk > index 83ddf10..f532d62 100644 > --- a/arch/arm/cpu/armv7/config.mk > +++ b/arch/arm/cpu/armv7/config.mk > @@ -31,3 +31,6 @@ PLATFORM_CPPFLAGS += -march=armv5 > # ========================================================================= > PF_RELFLAGS_SLB_AT := $(call cc-option,-mshort-load-bytes,$(call cc-option,-malignment-traps,)) > PLATFORM_RELFLAGS += $(PF_RELFLAGS_SLB_AT) > +ifneq ($(CONFIG_IMX_CONFIG),) > +ALL-y += $(obj)u-boot.imx > +endif Not tested yet, but I like the idea: Acked-by: Dirk Behme <dirk.behme@de.bosch.com> Any chance to still get this into 2012.04? Thanks! Dirk
On 04/04/2012 07:54, Dirk Behme wrote: > On 03.04.2012 16:32, Stefano Babic wrote: >> Freescale SOCs require an header to u-boot.bin >> >> The patch adds u-boot.imx to the default targets >> if the imx file is set (IMX_CONFIG). >> >> Signed-off-by: Stefano Babic <sbabic@denx.de> >> Cc: Albert ARIBAUD <albert.u.boot@aribaud.net> >> CC: Loïc Minier <loic.minier@linaro.org> >> CC: Mike Frysinger <vapier@gentoo.org> >> --- >> arch/arm/cpu/armv7/config.mk | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/cpu/armv7/config.mk b/arch/arm/cpu/armv7/config.mk >> index 83ddf10..f532d62 100644 >> --- a/arch/arm/cpu/armv7/config.mk >> +++ b/arch/arm/cpu/armv7/config.mk >> @@ -31,3 +31,6 @@ PLATFORM_CPPFLAGS += -march=armv5 >> # >> ========================================================================= >> PF_RELFLAGS_SLB_AT := $(call cc-option,-mshort-load-bytes,$(call >> cc-option,-malignment-traps,)) >> PLATFORM_RELFLAGS += $(PF_RELFLAGS_SLB_AT) >> +ifneq ($(CONFIG_IMX_CONFIG),) >> +ALL-y += $(obj)u-boot.imx >> +endif > > Not tested yet, but I like the idea: > > Acked-by: Dirk Behme <dirk.behme@de.bosch.com> > > Any chance to still get this into 2012.04? This was my intention. Albert, could you pick up this patch ? Thanks, Stefano
On 04.04.2012 07:54, Dirk Behme wrote: > On 03.04.2012 16:32, Stefano Babic wrote: >> Freescale SOCs require an header to u-boot.bin >> >> The patch adds u-boot.imx to the default targets >> if the imx file is set (IMX_CONFIG). >> >> Signed-off-by: Stefano Babic <sbabic@denx.de> >> Cc: Albert ARIBAUD <albert.u.boot@aribaud.net> >> CC: Loïc Minier <loic.minier@linaro.org> >> CC: Mike Frysinger <vapier@gentoo.org> >> --- >> arch/arm/cpu/armv7/config.mk | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/cpu/armv7/config.mk >> b/arch/arm/cpu/armv7/config.mk >> index 83ddf10..f532d62 100644 >> --- a/arch/arm/cpu/armv7/config.mk >> +++ b/arch/arm/cpu/armv7/config.mk >> @@ -31,3 +31,6 @@ PLATFORM_CPPFLAGS += -march=armv5 >> # >> ========================================================================= >> >> PF_RELFLAGS_SLB_AT := $(call cc-option,-mshort-load-bytes,$(call >> cc-option,-malignment-traps,)) >> PLATFORM_RELFLAGS += $(PF_RELFLAGS_SLB_AT) >> +ifneq ($(CONFIG_IMX_CONFIG),) >> +ALL-y += $(obj)u-boot.imx >> +endif > > Not tested yet, but I like the idea: > > Acked-by: Dirk Behme <dirk.behme@de.bosch.com> Tested-by: Dirk Behme <dirk.behme@googlemail.com> Dirk
diff --git a/arch/arm/cpu/armv7/config.mk b/arch/arm/cpu/armv7/config.mk index 83ddf10..f532d62 100644 --- a/arch/arm/cpu/armv7/config.mk +++ b/arch/arm/cpu/armv7/config.mk @@ -31,3 +31,6 @@ PLATFORM_CPPFLAGS += -march=armv5 # ========================================================================= PF_RELFLAGS_SLB_AT := $(call cc-option,-mshort-load-bytes,$(call cc-option,-malignment-traps,)) PLATFORM_RELFLAGS += $(PF_RELFLAGS_SLB_AT) +ifneq ($(CONFIG_IMX_CONFIG),) +ALL-y += $(obj)u-boot.imx +endif
Freescale SOCs require an header to u-boot.bin The patch adds u-boot.imx to the default targets if the imx file is set (IMX_CONFIG). Signed-off-by: Stefano Babic <sbabic@denx.de> Cc: Albert ARIBAUD <albert.u.boot@aribaud.net> CC: Loïc Minier <loic.minier@linaro.org> CC: Mike Frysinger <vapier@gentoo.org> --- arch/arm/cpu/armv7/config.mk | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)