lkml.org 
[lkml]   [2013]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH REPOST] xtensa: use new common dtc rule
Hi Stephen,

Thanks for the patch. I have added it to the 'for_next' tree. Max, since
you added that feature for Xtensa, any comments before I ask to pull?

-Chris


On 1/2/13 7:27 PM, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
>
> The current rules have the .dtb files build in a different directory
> from the .dts files. This patch changes xtensa to use the generic dtb
> rule which builds .dtb files in the same directory as the source .dts.
>
> This requires moving parts of arch/xtensa/boot/Makefile into newly
> created arch/xtensa/boot/dts/Makefile, and updating arch/xtensa/Makefile
> to call the new Makefile.
>
> Cc: Chris Zankel <chris@zankel.net>
> Cc: Max Filippov <jcmvbkbc@gmail.com>
> Cc: linux-xtensa@linux-xtensa.org
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> This patch applies recent dtc-rule-related changes to the xtensa arch.
> They couldn't be applied along with the similar changes for other archs
> due to dependencies on other xtensa changes. This should apply OK now to
> Linus's tree, or for 3.9.
>
> Note: I have not tested this recently.
>
> arch/xtensa/Makefile | 4 ++--
> arch/xtensa/boot/Makefile | 12 ------------
> arch/xtensa/boot/dts/Makefile | 15 +++++++++++++++
> 3 files changed, 17 insertions(+), 14 deletions(-)
> create mode 100644 arch/xtensa/boot/dts/Makefile
>
> diff --git a/arch/xtensa/Makefile b/arch/xtensa/Makefile
> index 0aa7270..48c1a5b 100644
> --- a/arch/xtensa/Makefile
> +++ b/arch/xtensa/Makefile
> @@ -88,7 +88,7 @@ core-y += $(buildvar) $(buildplf)
> libs-y += arch/xtensa/lib/ $(LIBGCC)
>
> ifneq ($(CONFIG_BUILTIN_DTB),"")
> -core-$(CONFIG_OF) += arch/xtensa/boot/
> +core-$(CONFIG_OF) += arch/xtensa/boot/dts/
> endif
>
> boot := arch/xtensa/boot
> @@ -101,7 +101,7 @@ zImage: vmlinux
> $(Q)$(MAKE) $(build)=$(boot) $@
>
> %.dtb:
> - $(Q)$(MAKE) $(build)=$(boot) $(boot)/$@
> + $(Q)$(MAKE) $(build)=$(boot)/dts $(boot)/dts/$@
>
> define archhelp
> @echo '* zImage - Compressed kernel image (arch/xtensa/boot/images/zImage.*)'
> diff --git a/arch/xtensa/boot/Makefile b/arch/xtensa/boot/Makefile
> index 818647e..64ffc4b 100644
> --- a/arch/xtensa/boot/Makefile
> +++ b/arch/xtensa/boot/Makefile
> @@ -25,18 +25,6 @@ bootdir-$(CONFIG_XTENSA_PLATFORM_ISS) += boot-elf
> bootdir-$(CONFIG_XTENSA_PLATFORM_XT2000) += boot-redboot boot-elf boot-uboot
> bootdir-$(CONFIG_XTENSA_PLATFORM_XTFPGA) += boot-redboot boot-elf boot-uboot
>
> -
> -BUILTIN_DTB := $(patsubst "%",%,$(CONFIG_BUILTIN_DTB)).dtb.o
> -ifneq ($(CONFIG_BUILTIN_DTB),"")
> -obj-$(CONFIG_OF) += $(BUILTIN_DTB)
> -endif
> -
> -# Rule to build device tree blobs
> -$(obj)/%.dtb: $(src)/dts/%.dts FORCE
> - $(call if_changed_dep,dtc)
> -
> -clean-files := *.dtb.S
> -
> zImage Image: $(bootdir-y)
>
> $(bootdir-y): $(addprefix $(obj)/,$(subdir-y)) \
> diff --git a/arch/xtensa/boot/dts/Makefile b/arch/xtensa/boot/dts/Makefile
> new file mode 100644
> index 0000000..5f711bb
> --- /dev/null
> +++ b/arch/xtensa/boot/dts/Makefile
> @@ -0,0 +1,15 @@
> +#
> +# arch/xtensa/boot/dts/Makefile
> +#
> +# This file is subject to the terms and conditions of the GNU General Public
> +# License. See the file "COPYING" in the main directory of this archive
> +# for more details.
> +#
> +#
> +
> +BUILTIN_DTB := $(patsubst "%",%,$(CONFIG_BUILTIN_DTB)).dtb.o
> +ifneq ($(CONFIG_BUILTIN_DTB),"")
> +obj-$(CONFIG_OF) += $(BUILTIN_DTB)
> +endif
> +
> +clean-files := *.dtb.S



\
 
 \ /
  Last update: 2013-01-04 01:01    [W:0.054 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site