public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] build: Add targets for auto gen of asm-offsets.h and use it in imx35
@ 2011-06-30 15:24 Matthias Weisser
  2011-07-01  7:41 ` Stefano Babic
  2011-07-06  6:47 ` Stefano Babic
  0 siblings, 2 replies; 3+ messages in thread
From: Matthias Weisser @ 2011-06-30 15:24 UTC (permalink / raw)
  To: u-boot

asm-offsets.h should be auto generated. This patch adds two rules to rules.mk
which makes this possible and removes the rules on imx35.

Signed-off-by: Matthias Weisser <weisserm@arcor.de>
---
 arch/arm/cpu/arm1136/mx35/Makefile |   11 -----------
 rules.mk                           |   10 ++++++++++
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/arch/arm/cpu/arm1136/mx35/Makefile b/arch/arm/cpu/arm1136/mx35/Makefile
index 20f36e3..284cdc5 100644
--- a/arch/arm/cpu/arm1136/mx35/Makefile
+++ b/arch/arm/cpu/arm1136/mx35/Makefile
@@ -50,14 +50,3 @@ include $(SRCTREE)/rules.mk
 sinclude $(obj).depend
 
 #########################################################################
-
-$(TOPDIR)/include/asm/arch/asm-offsets.h:	$(TOPDIR)/include/autoconf.mk.dep \
-	./asm-offsets.s
-	@echo Generating $@
-	$(TOPDIR)/tools/scripts/make-asm-offsets ./asm-offsets.s $@
-
-asm-offsets.s:	$(TOPDIR)/include/autoconf.mk.dep \
-	./asm-offsets.c
-	$(CC) -DDO_DEPS_ONLY \
-		$(CFLAGS) $(CFLAGS_$(BCURDIR)/$(@F)) $(CFLAGS_$(BCURDIR)) \
-		-o $@ ./asm-offsets.c -c -S
diff --git a/rules.mk b/rules.mk
index c2860e5..5fd12a0 100644
--- a/rules.mk
+++ b/rules.mk
@@ -42,4 +42,14 @@ $(HOSTOBJS): $(obj)%.o: %.c
 $(NOPEDOBJS): $(obj)%.o: %.c
 	$(HOSTCC) $(HOSTCFLAGS_NOPED) $(HOSTCFLAGS_$(@F)) $(HOSTCFLAGS_$(BCURDIR)) -o $@ $< -c
 
+$(TOPDIR)/include/asm/arch/asm-offsets.h:	$(TOPDIR)/include/autoconf.mk.dep \
+	$(TOPDIR)/$(CPUDIR)/$(SOC)/asm-offsets.s
+	@echo Generating $@
+	$(TOPDIR)/tools/scripts/make-asm-offsets $(TOPDIR)/$(CPUDIR)/$(SOC)/asm-offsets.s $@
+
+$(TOPDIR)/$(CPUDIR)/$(SOC)/asm-offsets.s:	$(TOPDIR)/include/autoconf.mk.dep \
+	$(TOPDIR)/$(CPUDIR)/$(SOC)/asm-offsets.c
+	$(CC) -DDO_DEPS_ONLY \
+		$(CFLAGS) $(CFLAGS_$(BCURDIR)/$(@F)) $(CFLAGS_$(BCURDIR)) \
+		-o $@ $(TOPDIR)/$(CPUDIR)/$(SOC)/asm-offsets.c -c -S
 #########################################################################
-- 
1.7.0.4

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

* [U-Boot] [PATCH] build: Add targets for auto gen of asm-offsets.h and use it in imx35
  2011-06-30 15:24 [U-Boot] [PATCH] build: Add targets for auto gen of asm-offsets.h and use it in imx35 Matthias Weisser
@ 2011-07-01  7:41 ` Stefano Babic
  2011-07-06  6:47 ` Stefano Babic
  1 sibling, 0 replies; 3+ messages in thread
From: Stefano Babic @ 2011-07-01  7:41 UTC (permalink / raw)
  To: u-boot

On 06/30/2011 05:24 PM, Matthias Weisser wrote:
> asm-offsets.h should be auto generated. This patch adds two rules to rules.mk
> which makes this possible and removes the rules on imx35.
> 

Hi Matthias,

> diff --git a/rules.mk b/rules.mk
> index c2860e5..5fd12a0 100644
> --- a/rules.mk
> +++ b/rules.mk
> @@ -42,4 +42,14 @@ $(HOSTOBJS): $(obj)%.o: %.c

Agree. Moving the rules to a general place is the first step, as
suggested by Wolfgang.

Acked-by: Stefano Babic <sbabic@denx.de>

Best regards,
Stefano Babic

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

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

* [U-Boot] [PATCH] build: Add targets for auto gen of asm-offsets.h and use it in imx35
  2011-06-30 15:24 [U-Boot] [PATCH] build: Add targets for auto gen of asm-offsets.h and use it in imx35 Matthias Weisser
  2011-07-01  7:41 ` Stefano Babic
@ 2011-07-06  6:47 ` Stefano Babic
  1 sibling, 0 replies; 3+ messages in thread
From: Stefano Babic @ 2011-07-06  6:47 UTC (permalink / raw)
  To: u-boot

On 06/30/2011 05:24 PM, Matthias Weisser wrote:
> asm-offsets.h should be auto generated. This patch adds two rules to rules.mk
> which makes this possible and removes the rules on imx35.
> 
> Signed-off-by: Matthias Weisser <weisserm@arcor.de>
> ---
>  arch/arm/cpu/arm1136/mx35/Makefile |   11 -----------
>  rules.mk                           |   10 ++++++++++
>  2 files changed, 10 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm/cpu/arm1136/mx35/Makefile b/arch/arm/cpu/arm1136/mx35/Makefile
> index 20f36e3..284cdc5 100644
> --- a/arch/arm/cpu/arm1136/mx35/Makefile
> +++ b/arch/arm/cpu/arm1136/mx35/Makefile
> @@ -50,14 +50,3 @@ include $(SRCTREE)/rules.mk
>  sinclude $(obj).depend
>  

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic

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

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

end of thread, other threads:[~2011-07-06  6:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-30 15:24 [U-Boot] [PATCH] build: Add targets for auto gen of asm-offsets.h and use it in imx35 Matthias Weisser
2011-07-01  7:41 ` Stefano Babic
2011-07-06  6:47 ` Stefano Babic

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