Skip to contentSkip to conversation

dtb: provide option to install dtb to boot directory

PATCH New Gary Coulbourne 3 participants 3 messages Dec 20, 2012 – Aug 1, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 dtb: provide option to install dtb to boot directory

---
 linux/Config.in |    6 ++++++
 linux/linux.mk  |    7 +++++++
 2 files changed, 13 insertions(+)
linux/Config.in +6 -0
index 9ebe0c9..8918297 100644
@@ -284,6 +284,12 @@ config BR2_LINUX_KERNEL_INSTALL_TARGET
 	  /boot in the target root filesystem, as is typically done on
 	  x86/x86_64 systems.
 
+config BR2_LINUX_DEVICE_TREE_INSTALL_TARGET
+	bool "Install Device Tree to /boot in target"
+	depends on BR2_LINUX_KERNEL_DTS_SUPPORT
+	help
+	  Select this option to have the device tree installed to
+	  /boot in the target root filesystem.
 
 # Linux extensions
 source "linux/Config.ext.in"
linux/linux.mk +7 -0
index 3321d80..6d8c752 100644
@@ -211,6 +211,12 @@ define LINUX_APPEND_DTB
 endef
 endif
 
+ifeq ($(BR2_LINUX_DEVICE_TREE_INSTALL_TARGET),y)
+define LINUX_INSTALL_DTB_TO_TARGET
+	install -m 0644 -D $(KERNEL_ARCH_PATH)/boot/$(KERNEL_DTS_NAME).dtb $(TARGET_DIR)/boot/$(KERNEL_DTS_NAME).dtb
+endef
+endif
+
 # Compilation. We make sure the kernel gets rebuilt when the
 # configuration has changed.
 define LINUX_BUILD_CMDS
@@ -247,6 +253,7 @@ endef
 define LINUX_INSTALL_TARGET_CMDS
 	$(LINUX_INSTALL_KERNEL_IMAGE_TO_TARGET)
 	$(LINUX_INSTALL_DTB)
+	$(LINUX_INSTALL_DTB_TO_TARGET)
 	# Install modules and remove symbolic links pointing to build
 	# directories, not relevant on the target
 	@if grep -q "CONFIG_MODULES=y" $(@D)/.config; then 	\
Signature · 1 line
1.7.10.4
#1 lore raw

PATCH 1/1 dtb: provide option to install dtb to boot directory

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Gary" == Gary Coulbourne <[email protected]> writes:

Gary> Signed-off-by: Gary Coulbourne <[email protected]> Gary> ---

 Gary>  linux/Config.in |    6 ++++++
 Gary>  linux/linux.mk  |    7 +++++++

Gary> 2 files changed, 13 insertions(+)

Gary> diff --git a/linux/Config.in b/linux/Config.in Gary> index 9ebe0c9..8918297 100644
Gary> --- a/linux/Config.in
Gary> +++ b/linux/Config.in
Gary> @@ -284,6 +284,12 @@ config BR2_LINUX_KERNEL_INSTALL_TARGET

 Gary>  	  /boot in the target root filesystem, as is typically done on
 Gary>  	  x86/x86_64 systems.

Gary> +config BR2_LINUX_DEVICE_TREE_INSTALL_TARGET
Gary> + bool "Install Device Tree to /boot in target"
Gary> + depends on BR2_LINUX_KERNEL_DTS_SUPPORT
Gary> + help
Gary> + Select this option to have the device tree installed to
Gary> + /boot in the target root filesystem.

Gary> # Linux extensions
Gary> source "linux/Config.ext.in"
Gary> diff --git a/linux/linux.mk b/linux/linux.mk Gary> index 3321d80..6d8c752 100644
Gary> --- a/linux/linux.mk
Gary> +++ b/linux/linux.mk
Gary> @@ -211,6 +211,12 @@ define LINUX_APPEND_DTB Gary> endef
Gary> endif

Gary> +ifeq ($(BR2_LINUX_DEVICE_TREE_INSTALL_TARGET),y) Gary> +define LINUX_INSTALL_DTB_TO_TARGET
Gary> + install -m 0644 -D $(KERNEL_ARCH_PATH)/boot/$(KERNEL_DTS_NAME).dtb $(TARGET_DIR)/boot/$(KERNEL_DTS_NAME).dtb

I've just pushed two changes to support the new dtb location since 3.8-rc1, and I committed Maxime's 'Support multiple device tree build' patch yesterday, care to rebase against that and update the logic so it can handle multiple .dtb files?

Thanks.

Signature · 1 line
Bye, Peter Korsgaard
#2 lore raw

PATCH 1/1 dtb: provide option to install dtb to boot directory

In reply to Gary Coulbourne

Dear Gary Coulbourne,

Quoted 6 lines · Gary Coulbourne
---
 linux/Config.in |    6 ++++++
 linux/linux.mk  |    7 +++++++
 2 files changed, 13 insertions(+)

This patch is in fact not needed, because the existing BR2_LINUX_KERNEL_INSTALL_TARGET option installs both the kernel image and the DTBs to /boot. I've updated the BR2_LINUX_KERNEL_INSTALL_TARGET help text to clarify this.

Thanks!

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com