public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs
@ 2012-04-03 14:32 Stefano Babic
  2012-04-03 19:29 ` Mike Frysinger
  2012-04-04  5:54 ` Dirk Behme
  0 siblings, 2 replies; 5+ messages in thread
From: Stefano Babic @ 2012-04-03 14:32 UTC (permalink / raw)
  To: u-boot

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
-- 
1.7.5.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs
  2012-04-03 14:32 [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs Stefano Babic
@ 2012-04-03 19:29 ` Mike Frysinger
  2012-04-04  5:54 ` Dirk Behme
  1 sibling, 0 replies; 5+ messages in thread
From: Mike Frysinger @ 2012-04-03 19:29 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs
  2012-04-03 14:32 [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs Stefano Babic
  2012-04-03 19:29 ` Mike Frysinger
@ 2012-04-04  5:54 ` Dirk Behme
  2012-04-04  7:57   ` Stefano Babic
  2012-04-06  6:12   ` Dirk Behme
  1 sibling, 2 replies; 5+ messages in thread
From: Dirk Behme @ 2012-04-04  5:54 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs
  2012-04-04  5:54 ` Dirk Behme
@ 2012-04-04  7:57   ` Stefano Babic
  2012-04-06  6:12   ` Dirk Behme
  1 sibling, 0 replies; 5+ messages in thread
From: Stefano Babic @ 2012-04-04  7:57 UTC (permalink / raw)
  To: u-boot

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

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs
  2012-04-04  5:54 ` Dirk Behme
  2012-04-04  7:57   ` Stefano Babic
@ 2012-04-06  6:12   ` Dirk Behme
  1 sibling, 0 replies; 5+ messages in thread
From: Dirk Behme @ 2012-04-06  6:12 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-04-06  6:12 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-03 14:32 [U-Boot] [PATCH] ARM: add u-boot.imx as target for i.MX SOCs Stefano Babic
2012-04-03 19:29 ` Mike Frysinger
2012-04-04  5:54 ` Dirk Behme
2012-04-04  7:57   ` Stefano Babic
2012-04-06  6:12   ` Dirk Behme

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox