package/uboot-tools: migrate BR2_TARGET_UBOOT_BOOT_SCRIPT from U-Boot pkg
authorMatt Weber <matthew.weber@rockwellcollins.com>
Thu, 24 Sep 2020 19:29:11 +0000 (14:29 -0500)
committerArnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Tue, 6 Oct 2020 21:37:48 +0000 (23:37 +0200)
For consistancy and dependencies between uboot and uboot-tools,
this patch migrates the script creation over in a similar way as
the env image creation.

Cc: Arnout Vandecappelle <arnout@mind.be>
Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Config.in.legacy
boot/uboot/Config.in
boot/uboot/uboot.mk
package/uboot-tools/Config.in.host
package/uboot-tools/uboot-tools.mk

index d39f50a8008d2eb74afbbb316c31d2e3c87ef11d..95c2fb33b3da3c5257e909f3ff66ec3531117cd2 100644 (file)
@@ -146,6 +146,24 @@ endif
 
 comment "Legacy options removed in 2020.11"
 
+config BR2_TARGET_UBOOT_BOOT_SCRIPT
+       bool "u-boot script generation was moved"
+       select BR2_LEGACY
+       select BR2_PACKAGE_HOST_UBOOT_TOOLS
+       select BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT
+       help
+         Migrated U-Boot script generation to uboot-tools
+
+# Note: BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE is still referenced from
+# package/uboot-tools/Config.in
+config BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE
+       string "The uboot script source string has been renamed"
+       depends on BR2_TARGET_UBOOT_BOOT_SCRIPT
+       help
+         Migrated U-Boot script generation to uboot-tools.
+         New option is named
+         BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT_SOURCE
+
 config BR2_TARGET_UBOOT_ENVIMAGE
        bool "u-boot env generation was moved"
        select BR2_LEGACY
index 2bda0619a4237cc66fa84b17a39d131ce8e9e80d..9e44daffab7a32562e9a6b1c3b2bdb656b474e39 100644 (file)
@@ -533,22 +533,6 @@ config BR2_TARGET_UBOOT_ALTERA_SOCFPGA_IMAGE_CRC
          In either case the resulting file will be given a .crc
          extension.
 
-config BR2_TARGET_UBOOT_BOOT_SCRIPT
-       bool "Generate a U-Boot boot script"
-       help
-         Generate a U-Boot boot script, given a file listing U-Boot
-         commands to be executed at boot time. The generated boot
-         script will be called 'boot.scr'.
-
-if BR2_TARGET_UBOOT_BOOT_SCRIPT
-
-config BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE
-       string "U-Boot boot script source"
-       help
-         Source file to generate the U-Boot boot script.
-
-endif
-
 if BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG
 
 config BR2_TARGET_UBOOT_CUSTOM_DTS_PATH
index 9dbd06b64b3f7b8f130e55f653acb929c45e8db3..f27abbd5ba0e7d0b48bd7c3ce17cbe5964edc346 100644 (file)
@@ -323,10 +323,6 @@ define UBOOT_INSTALL_IMAGES_CMDS
                        cp -dpf $(@D)/$(f) $(BINARIES_DIR)/
                )
        )
-       $(if $(BR2_TARGET_UBOOT_BOOT_SCRIPT),
-               $(MKIMAGE) -C none -A $(MKIMAGE_ARCH) -T script \
-                       -d $(call qstrip,$(BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE)) \
-                       $(BINARIES_DIR)/boot.scr)
 endef
 
 ifeq ($(BR2_TARGET_UBOOT_ZYNQMP),y)
@@ -427,15 +423,6 @@ define UBOOT_KCONFIG_FIXUP_CMDS
        $(UBOOT_ZYNQMP_KCONFIG_PSU_INIT)
 endef
 
-ifeq ($(BR2_TARGET_UBOOT_BOOT_SCRIPT),y)
-ifeq ($(BR_BUILDING),y)
-ifeq ($(call qstrip,$(BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE)),)
-$(error Please define a source file for U-Boot boot script (BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE setting))
-endif
-endif
-UBOOT_DEPENDENCIES += host-uboot-tools
-endif
-
 ifeq ($(BR2_TARGET_UBOOT)$(BR_BUILDING),yy)
 
 #
index a10835c5894abb9c41fa4f489cf9281e988643b4..431cc8631d12e7dc625beea393c6780fefc94cf1 100644 (file)
@@ -91,4 +91,21 @@ config BR2_PACKAGE_HOST_UBOOT_TOOLS_ENVIMAGE_REDUNDANT
 
 endif # BR2_PACKAGE_HOST_UBOOT_TOOLS_ENVIMAGE
 
+config BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT
+       bool "Generate a U-Boot boot script"
+       help
+         Generate a U-Boot boot script, given a file listing U-Boot
+         commands to be executed at boot time. The generated boot
+         script will be called 'boot.scr'.
+
+if BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT
+
+config BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT_SOURCE
+       string "U-Boot boot script source"
+       default BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE if BR2_TARGET_UBOOT_BOOT_SCRIPT_SOURCE != "" # legacy
+       help
+         Source file to generate the U-Boot boot script.
+
+endif # BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT
+
 endif # BR2_PACKAGE_HOST_UBOOT_TOOLS
index f4480aa5857e0cd87c7d0cc5b1409b68af08ad66..6aa7cba2ddbcfbbe511b15892712bb7b6bd32d37 100644 (file)
@@ -148,11 +148,23 @@ endif #BR_BUILDING
 
 endif #BR2_PACKAGE_HOST_UBOOT_TOOLS_ENVIMAGE
 
+ifeq ($(BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT),y)
+ifeq ($(BR_BUILDING),y)
+ifeq ($(call qstrip,$(BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT_SOURCE)),)
+$(error Please define a source file for U-Boot boot script (BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT_SOURCE setting))
+endif
+endif #BR_BUILDING
+endif #BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT
+
 define HOST_UBOOT_TOOLS_INSTALL_CMDS
        $(INSTALL) -m 0755 -D $(@D)/tools/mkimage $(HOST_DIR)/bin/mkimage
        $(INSTALL) -m 0755 -D $(@D)/tools/mkenvimage $(HOST_DIR)/bin/mkenvimage
        $(INSTALL) -m 0755 -D $(@D)/tools/dumpimage $(HOST_DIR)/bin/dumpimage
        $(HOST_UBOOT_TOOLS_GENERATE_ENV_IMAGE)
+       $(if $(BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT),
+               $(MKIMAGE) -C none -A $(MKIMAGE_ARCH) -T script \
+                       -d $(call qstrip,$(BR2_PACKAGE_HOST_UBOOT_TOOLS_BOOT_SCRIPT_SOURCE)) \
+                       $(BINARIES_DIR)/boot.scr)
 endef
 
 $(eval $(generic-package))