Thread (11 messages) 11 messages, 5 authors, 2017-06-17

Re: [PATCH v4 3/4] MIPS: DTS: img: Don't attempt to build-in all .dtb files

From: Rahul Bedarkar <rahulbedarkar89@gmail.com>
Date: 2017-06-13 16:13:45

Hi Paul,

On Fri, Jun 2, 2017 at 11:50 PM, Paul Burton [off-list ref] wrote:
When building a FIT image we may want the kernel to build multiple .dtb
files, but we don't want to build them all into the kernel binary as
object files since they'll instead be included in the FIT image.

Commit daa10170da27 ("MIPS: DTS: img: add device tree for Marduk board")
however created arch/mips/boot/dts/img/Makefile with a line that builds
any enabled .dtb files into the kernel. Remove this & build the
pistachio object specifically, in preparation for adding .dtb targets
which we don't want to build into the kernel.

Signed-off-by: Paul Burton <redacted>
Cc: Rahul Bedarkar <redacted>
Cc: Ralf Baechle <redacted>
Cc: linux-mips@linux-mips.org

---

Changes in v4:
- New patch.

Changes in v3: None
Changes in v2: None

 arch/mips/boot/dts/img/Makefile | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
It looks good to me.

Reviewed-by: Rahul Bedarkar <rahulbedarkar89@gmail.com>
quoted hunk ↗ jump to hunk
diff --git a/arch/mips/boot/dts/img/Makefile b/arch/mips/boot/dts/img/Makefile
index 69a65f0f82d2..c178cf56f5b8 100644
--- a/arch/mips/boot/dts/img/Makefile
+++ b/arch/mips/boot/dts/img/Makefile
@@ -1,6 +1,5 @@
 dtb-$(CONFIG_MACH_PISTACHIO)   += pistachio_marduk.dtb
-
-obj-y                          += $(patsubst %.dtb, %.dtb.o, $(dtb-y))
It was probably copy/paste from other board Makefiles. But If I
understand it correctly, please correct me if I am wrong, linking of
object file of device tree to kernel image is useful if boot loader
doesn't support loading external device tree and in that case kernel
can use inbuilt device tree.

Thanks,
Rahul
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help