From b0d446dee427cfdb71c55c2a46472786de7b8ffc Mon Sep 17 00:00:00 2001 From: Thomas Petazzoni Date: Mon, 11 Jul 2011 22:46:10 +0200 Subject: [PATCH] Improve TARGETS handling for bootloaders and kernel As the kernel and bootloaders do not use the normal BR2_PACKAGE_* Kconfig options, their target name was not automatically added to the global TARGETS variable. Each bootloader .mk and the linux.mk had to add their own target manually to TARGETS, and the package infrastructure was making tests on non-existing Kconfig variables. This commit improves the package infrastructure so that it looks at BR2_PACKAGE_ for packages, BR2_TARGET_ for bootloaders and at the special BR2_LINUX_KERNEL for the linux package. This allows to simplify a little bit the bootloaders and linux .mk files. Signed-off-by: Thomas Petazzoni Signed-off-by: Peter Korsgaard --- boot/at91bootstrap/at91bootstrap.mk | 8 -------- boot/at91dataflashboot/at91dataflashboot.mk | 9 --------- boot/barebox/barebox.mk | 3 --- boot/grub/grub.mk | 9 --------- boot/syslinux/syslinux.mk | 9 --------- boot/uboot/uboot.mk | 8 -------- linux/linux.mk | 5 ----- package/Makefile.package.in | 16 +++++++++++++--- 8 files changed, 13 insertions(+), 54 deletions(-) diff --git a/boot/at91bootstrap/at91bootstrap.mk b/boot/at91bootstrap/at91bootstrap.mk index 37f8ac70c4..871b0ffb49 100644 --- a/boot/at91bootstrap/at91bootstrap.mk +++ b/boot/at91bootstrap/at91bootstrap.mk @@ -31,19 +31,11 @@ endef $(eval $(call GENTARGETS,boot,at91bootstrap)) -############################################################# -# -# Toplevel Makefile options -# -############################################################# ifeq ($(BR2_TARGET_AT91BOOTSTRAP),y) -TARGETS+=at91bootstrap - # we NEED a board name unless we're at make source ifeq ($(filter source,$(MAKECMDGOALS)),) ifeq ($(AT91BOOTSTRAP_BOARD),) $(error No AT91Bootstrap board name set. Check your BR2_TARGET_AT91BOOTSTRAP_BOARD setting) endif endif - endif diff --git a/boot/at91dataflashboot/at91dataflashboot.mk b/boot/at91dataflashboot/at91dataflashboot.mk index 9b6cf21166..14fb8c54d3 100644 --- a/boot/at91dataflashboot/at91dataflashboot.mk +++ b/boot/at91dataflashboot/at91dataflashboot.mk @@ -20,12 +20,3 @@ define AT91DATAFLASHBOOT_INSTALL_IMAGES_CMDS endef $(eval $(call GENTARGETS,boot,at91dataflashboot)) - -############################################################# -# -# Toplevel Makefile options -# -############################################################# -ifeq ($(BR2_TARGET_AT91DATAFLASHBOOT),y) -TARGETS+=at91dataflashboot -endif diff --git a/boot/barebox/barebox.mk b/boot/barebox/barebox.mk index 34ec5dadf2..1952c987f7 100644 --- a/boot/barebox/barebox.mk +++ b/boot/barebox/barebox.mk @@ -54,13 +54,10 @@ endif $(eval $(call GENTARGETS,boot,barebox)) ifeq ($(BR2_TARGET_BAREBOX),y) -TARGETS+=barebox - # we NEED a board defconfig file unless we're at make source ifeq ($(filter source,$(MAKECMDGOALS)),) ifeq ($(BAREBOX_BOARD_DEFCONFIG),) $(error No Barebox defconfig file. Check your BR2_TARGET_BAREBOX_BOARD_DEFCONFIG setting) endif endif - endif diff --git a/boot/grub/grub.mk b/boot/grub/grub.mk index 49384169f1..2ce8eabc90 100644 --- a/boot/grub/grub.mk +++ b/boot/grub/grub.mk @@ -83,12 +83,3 @@ define GRUB_UNINSTALL_TARGET_CMDS endef $(eval $(call AUTOTARGETS,boot,grub)) - -############################################################# -# -# Toplevel Makefile options -# -############################################################# -ifeq ($(BR2_TARGET_GRUB),y) -TARGETS+=grub -endif diff --git a/boot/syslinux/syslinux.mk b/boot/syslinux/syslinux.mk index e9c32b6409..686124fe59 100644 --- a/boot/syslinux/syslinux.mk +++ b/boot/syslinux/syslinux.mk @@ -27,12 +27,3 @@ define SYSLINUX_INSTALL_IMAGES_CMDS endef $(eval $(call GENTARGETS,boot,syslinux)) - -############################################################# -# -# Toplevel Makefile options -# -############################################################# -ifeq ($(BR2_TARGET_SYSLINUX),y) -TARGETS+=syslinux -endif diff --git a/boot/uboot/uboot.mk b/boot/uboot/uboot.mk index 3e945c7f25..dfa012a8fc 100644 --- a/boot/uboot/uboot.mk +++ b/boot/uboot/uboot.mk @@ -111,19 +111,11 @@ endef $(eval $(call GENTARGETS,boot,uboot)) -############################################################# -# -# Toplevel Makefile options -# -############################################################# ifeq ($(BR2_TARGET_UBOOT),y) -TARGETS+=u-boot - # we NEED a board name unless we're at make source ifeq ($(filter source,$(MAKECMDGOALS)),) ifeq ($(UBOOT_BOARD_NAME),) $(error NO U-Boot board name set. Check your BR2_TARGET_UBOOT_BOARDNAME setting) endif endif - endif diff --git a/linux/linux.mk b/linux/linux.mk index f8eac6fb43..25db1f5ea5 100644 --- a/linux/linux.mk +++ b/linux/linux.mk @@ -189,11 +189,6 @@ $(LINUX_DIR)/.stamp_initramfs_rebuilt: $(LINUX_DIR)/.stamp_target_installed $(LI # after it generated the initramfs list of files. linux-rebuild-with-initramfs linux26-rebuild-with-initramfs: $(LINUX_DIR)/.stamp_initramfs_rebuilt - -ifeq ($(BR2_LINUX_KERNEL),y) -TARGETS+=linux -endif - # Checks to give errors that the user can understand ifeq ($(filter source,$(MAKECMDGOALS)),) ifeq ($(BR2_LINUX_KERNEL_USE_DEFCONFIG),y) diff --git a/package/Makefile.package.in b/package/Makefile.package.in index aa5c0d7e81..faa7ff1f38 100644 --- a/package/Makefile.package.in +++ b/package/Makefile.package.in @@ -522,10 +522,20 @@ $$($(2)_TARGET_UNINSTALL): PKG=$(2) $$($(2)_TARGET_CLEAN): PKG=$(2) $$($(2)_TARGET_DIRCLEAN): PKG=$(2) +# Compute the name of the Kconfig option that correspond to the +# package being enabled. We handle three cases: the special Linux +# kernel case, the bootloaders case, and the normal packages case. +ifeq ($(1),linux) +$(2)_KCONFIG_VAR = BR2_LINUX_KERNEL +else ifeq ($(4),boot) +$(2)_KCONFIG_VAR = BR2_TARGET_$(2) +else +$(2)_KCONFIG_VAR = BR2_PACKAGE_$(2) +endif + # add package to the general list of targets if requested by the buildroot # configuration - -ifeq ($$(BR2_PACKAGE_$(2)),y) +ifeq ($$($$($(2)_KCONFIG_VAR)),y) TARGETS += $(1) @@ -537,7 +547,7 @@ else ifeq ($$($(2)_SITE_METHOD),bzr) DL_TOOLS_DEPENDENCIES += bzr endif # SITE_METHOD -endif # BR2_PACKAGE_$(2) +endif # $(2)_KCONFIG_VAR endef # GENTARGETS_INNER ################################################################################ -- 2.30.2