[PATCH 1/2] powerpc/boot/dts: Install dts from the right directory

Subsystems: linux for powerpc (32-bit and 64-bit), the rest

STALE5720d

3 messages, 2 authors, 2011-01-11 · open the first message on its own page

[PATCH 1/2] powerpc/boot/dts: Install dts from the right directory

From: Ben Hutchings <hidden>
Date: 2011-01-09 00:42:09

The dts-installed variable is initialised using a wildcard path that
will be expanded relative to the build directory.  Use the existing
variable dtstree to generate an absolute wildcard path that will work
when building in a separate directory.

Reported-by: Gerhard Pircher <redacted>
Signed-off-by: Ben Hutchings <redacted>
Tested-by: Gerhard Pircher <redacted> [against 2.6.32]
---
 arch/powerpc/boot/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile
index 96deec6..8917816 100644
--- a/arch/powerpc/boot/Makefile
+++ b/arch/powerpc/boot/Makefile
@@ -368,7 +368,7 @@ INSTALL :=3D install
 extra-installed		:=3D $(patsubst $(obj)/%, $(DESTDIR)$(WRAPPER_OBJDIR)/%, =
$(extra-y))
 hostprogs-installed	:=3D $(patsubst %, $(DESTDIR)$(WRAPPER_BINDIR)/%, $(ho=
stprogs-y))
 wrapper-installed	:=3D $(DESTDIR)$(WRAPPER_BINDIR)/wrapper
-dts-installed		:=3D $(patsubst $(obj)/dts/%, $(DESTDIR)$(WRAPPER_DTSDIR)/%=
, $(wildcard $(obj)/dts/*.dts))
+dts-installed		:=3D $(patsubst $(dtstree)/%, $(DESTDIR)$(WRAPPER_DTSDIR)/%=
, $(wildcard $(dtstree)/*.dts))
=20
 all-installed		:=3D $(extra-installed) $(hostprogs-installed) $(wrapper-in=
stalled) $(dts-installed)
=20
--=20
1.7.2.3

[PATCH 2/2] powerpc/boot/Makefile: Use $(src) and $(obj) as per makefiles.txt

From: Ben Hutchings <hidden>
Date: 2011-01-09 00:41:57

$(src) and $(obj) are normally the same, but are supposed to be used
for paths under $(srctree) and $(objtree) respectively.

Also use $(dtstree) and $(wrapper) as appropriate.

Signed-off-by: Ben Hutchings <redacted>
---
This is totally untested, so please review carefully!

Ben.

 arch/powerpc/boot/Makefile |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile
index 8917816..bd7abba 100644
--- a/arch/powerpc/boot/Makefile
+++ b/arch/powerpc/boot/Makefile
@@ -33,7 +33,7 @@ ifeq ($(call cc-option-yn, -fstack-protector),y)
 BOOTCFLAGS	+=3D -fno-stack-protector
 endif
=20
-BOOTCFLAGS	+=3D -I$(obj) -I$(srctree)/$(obj)
+BOOTCFLAGS	+=3D -I$(obj) -I$(srctree)/$(src)
=20
 DTC_FLAGS	?=3D -p 1024
=20
@@ -399,10 +399,10 @@ $(extra-installed)	: $(DESTDIR)$(WRAPPER_OBJDIR)/% : =
$(obj)/% | $(DESTDIR)$(WRAP
 $(hostprogs-installed)  : $(DESTDIR)$(WRAPPER_BINDIR)/% : $(obj)/% | $(DES=
TDIR)$(WRAPPER_BINDIR)
 	$(call cmd,install_exe)
=20
-$(dts-installed)	: $(DESTDIR)$(WRAPPER_DTSDIR)/% : $(srctree)/$(obj)/dts/%=
 | $(DESTDIR)$(WRAPPER_DTSDIR)
+$(dts-installed)	: $(DESTDIR)$(WRAPPER_DTSDIR)/% : $(dtstree)/% | $(DESTDI=
R)$(WRAPPER_DTSDIR)
 	$(call cmd,install_dts)
=20
-$(wrapper-installed): $(DESTDIR)$(WRAPPER_BINDIR) $(srctree)/$(obj)/wrappe=
r | $(DESTDIR)$(WRAPPER_BINDIR)
+$(wrapper-installed): $(DESTDIR)$(WRAPPER_BINDIR) $(wrapper) | $(DESTDIR)$=
(WRAPPER_BINDIR)
 	$(call cmd,install_wrapper)
=20
 $(obj)/bootwrapper_install: $(all-installed)
--=20
1.7.2.3

Re: [PATCH 1/2] powerpc/boot/dts: Install dts from the right directory

From: Gerhard Pircher <hidden>
Date: 2011-01-11 19:05:17

-------- Original-Nachricht --------
Datum: Sun, 09 Jan 2011 00:24:01 +0000
Von: Ben Hutchings [off-list ref]
An: Benjamin Herrenschmidt [off-list ref], Paul Mackerras [off-list ref]
CC: Gerhard Pircher <redacted>, linuxppc-dev@lists.ozlabs.org
Betreff: [PATCH 1/2] powerpc/boot/dts: Install dts from the right directory
The dts-installed variable is initialised using a wildcard path that
will be expanded relative to the build directory.  Use the existing
variable dtstree to generate an absolute wildcard path that will work
when building in a separate directory.
Wouldn't this patch be a good candidate for the stable kernel series?
quoted hunk
Reported-by: Gerhard Pircher <redacted>
Signed-off-by: Ben Hutchings <redacted>
Tested-by: Gerhard Pircher <redacted> [against 2.6.32]
---
 arch/powerpc/boot/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile
index 96deec6..8917816 100644
--- a/arch/powerpc/boot/Makefile
+++ b/arch/powerpc/boot/Makefile
@@ -368,7 +368,7 @@ INSTALL := install
 extra-installed		:= $(patsubst $(obj)/%, $(DESTDIR)$(WRAPPER_OBJDIR)/%,
$(extra-y))
 hostprogs-installed	:= $(patsubst %, $(DESTDIR)$(WRAPPER_BINDIR)/%,
$(hostprogs-y))
 wrapper-installed	:= $(DESTDIR)$(WRAPPER_BINDIR)/wrapper
-dts-installed		:= $(patsubst $(obj)/dts/%, $(DESTDIR)$(WRAPPER_DTSDIR)/%,
$(wildcard $(obj)/dts/*.dts))
+dts-installed		:= $(patsubst $(dtstree)/%, $(DESTDIR)$(WRAPPER_DTSDIR)/%,
$(wildcard $(dtstree)/*.dts))
 
 all-installed		:= $(extra-installed) $(hostprogs-installed)
$(wrapper-installed) $(dts-installed)
 
-- 
1.7.2.3
Gerhard

-- 
NEU: FreePhone - kostenlos mobil telefonieren und surfen!			
Jetzt informieren: http://www.gmx.net/de/go/freephone
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help