From mboxrd@z Thu Jan 1 00:00:00 1970 From: swarren@wwwdotorg.org (Stephen Warren) Date: Mon, 10 Sep 2012 23:41:37 -0600 Subject: [PATCH 07/12] ARM: move all dtb targets out of Makefile.boot In-Reply-To: <20120911044907.GF31430@game.jcrosoft.org> References: <1346962375-26163-1-git-send-email-robherring2@gmail.com> <1346962375-26163-8-git-send-email-robherring2@gmail.com> <20120910155224.GE31430@game.jcrosoft.org> <504E28B0.5040404@gmail.com> <20120911044907.GF31430@game.jcrosoft.org> Message-ID: <504ECF11.2000206@wwwdotorg.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 09/10/2012 10:49 PM, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 12:51 Mon 10 Sep , Rob Herring wrote: >> On 09/10/2012 10:52 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: >>> On 15:12 Thu 06 Sep , Rob Herring wrote: >>>> From: Rob Herring >>>> >>>> In preparation to support multi-platform kernels, move all the dtb targets >>>> out of the mach Makefile.boot and into the arch/arm/boot/dts/Makefile >>>> which is closer to the sources. >>>> >>>> Signed-off-by: Rob Herring >>>> Cc: Russell King >>>> Cc: Andrew Victor >>>> Cc: Nicolas Ferre >>>> Cc: Jean-Christophe Plagniol-Villard >>>> Cc: Kukjin Kim >>>> Cc: Sascha Hauer >>>> Cc: Jason Cooper >>>> Cc: Andrew Lunn >>>> Cc: Gregory Clement >>>> Cc: Shawn Guo >>>> Cc: Viresh Kumar >>>> Cc: Shiraz Hashim >>>> Cc: Rajeev Kumar >>>> Cc: Stephen Warren >>>> Cc: Srinidhi Kasagar >>>> Cc: Linus Walleij >>>> --- >>>> arch/arm/boot/Makefile | 2 ++ >>>> arch/arm/boot/dts/Makefile | 41 +++++++++++++++++++++++++++++++++ >>>> arch/arm/mach-at91/Makefile.boot | 24 ------------------- >>>> arch/arm/mach-exynos/Makefile.boot | 3 --- >>>> arch/arm/mach-imx/Makefile.boot | 7 ------ >>>> arch/arm/mach-kirkwood/Makefile.boot | 11 --------- >>>> arch/arm/mach-lpc32xx/Makefile.boot | 2 -- >>>> arch/arm/mach-mvebu/Makefile.boot | 2 -- >>>> arch/arm/mach-mxs/Makefile.boot | 9 -------- >>>> arch/arm/mach-spear13xx/Makefile.boot | 3 --- >>>> arch/arm/mach-spear3xx/Makefile.boot | 4 ---- >>>> arch/arm/mach-spear6xx/Makefile.boot | 2 -- >>>> arch/arm/mach-tegra/Makefile.boot | 8 ------- >>>> arch/arm/mach-ux500/Makefile.boot | 2 -- >>>> arch/arm/mach-vexpress/Makefile.boot | 5 ---- >>>> 15 files changed, 43 insertions(+), 82 deletions(-) >>>> create mode 100644 arch/arm/boot/dts/Makefile >>>> >>>> diff --git a/arch/arm/boot/Makefile b/arch/arm/boot/Makefile >>>> index c877087..3fdab01 100644 >>>> --- a/arch/arm/boot/Makefile >>>> +++ b/arch/arm/boot/Makefile >>>> @@ -15,6 +15,8 @@ ifneq ($(MACHINE),) >>>> include $(srctree)/$(MACHINE)/Makefile.boot >>>> endif >>>> >>>> +include $(srctree)/arch/arm/boot/dts/Makefile >>>> + >>>> # Note: the following conditions must always be true: >>>> # ZRELADDR == virt_to_phys(PAGE_OFFSET + TEXT_OFFSET) >>>> # PARAMS_PHYS must be within 4MB of ZRELADDR >>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >>>> new file mode 100644 >>>> index 0000000..e59586b >>>> --- /dev/null >>>> +++ b/arch/arm/boot/dts/Makefile >>>> @@ -0,0 +1,41 @@ >>>> +dtb-$(CONFIG_ARCH_HIGHBANK) += highbank.dtb >>> >>> sorry I see no need of the dtb-y >>> >>> just do all the time >> >> That's not how it works today. dtb's only for all enabled platforms are >> built. Changing the behavior should be a different patch. > > no I see no point to check if the platform is enable or not just build all > > This will allow to check them Why would I want to build 100 .dtbs that I don't care about? If I want to check them, I can either build all the defconfigs, or simply write a trivial sh for loop to loop over *.dts in the boot/dts directory.