Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1244579

Re: [PATCH v2 12/13] powerpc: enable building of all dtbs

Show all headers | View raw

On Thu, 2015-10-08 at 12:53 -0500, Rob Herring wrote:
> Enable building all dtb files when CONFIG_OF_ALL_DTBS is enabled. The dtbs
> are not really dependent on a platform being enabled or any other kernel
> config, so for testing coverage it is convenient to build all of the dtbs.
> This builds all dts files in the tree, not just targets listed.
> 
> Supporting this requires adding 'dtbs' make target which was not yet
> supported on powerpc.

I'm not entirely clear why we want this? It just means we can test build all
the dts regardless of what kernel config we are building?

Have you tested the powerpc build with this applied or do you want me to?

> diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile
> index b9b4af2..0ec35ff 100644
> --- a/arch/powerpc/Makefile
> +++ b/arch/powerpc/Makefile
> @@ -273,6 +273,11 @@ bootwrapper_install:
>  %.dtb: scripts
>  	$(Q)$(MAKE) ARCH=ppc64 $(build)=$(boot) $(patsubst %,$(boot)/%,$@)
>  
> +PHONY += dtbs
> +
> +dtbs: prepare scripts
> +	$(Q)$(MAKE) ARCH=ppc64 $(build)=$(boot)/dts

The ARCH should be powerpc, I guess you just copied the one above.

> diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile
> index 4eec430..381df1a 100644
> --- a/arch/powerpc/boot/Makefile
> +++ b/arch/powerpc/boot/Makefile
> @@ -402,7 +402,7 @@ zInstall: $(CONFIGURE) $(addprefix $(obj)/, $(image-y))
>  clean-files += $(image-) $(initrd-) cuImage.* dtbImage.* treeImage.* \
>  	zImage zImage.initrd zImage.chrp zImage.coff zImage.holly \
>  	zImage.miboot zImage.pmac zImage.pseries \
> -	zImage.maple simpleImage.* otheros.bld *.dtb
> +	zImage.maple simpleImage.* otheros.bld

This looks like we used to clean *.dtb in the current dir but now we don't? But
I'm probably just not understanding how clean-files works.

> @@ -410,6 +410,9 @@ clean-kernel += $(addsuffix .gz,$(clean-kernel))
>  # If not absolute clean-files are relative to $(obj).
>  clean-files += $(addprefix $(objtree)/, $(clean-kernel))
>  
> +# Let clean descend into subdirs
> +subdir- := dts
> +
>  WRAPPER_OBJDIR := /usr/lib/kernel-wrapper
>  WRAPPER_DTSDIR := /usr/lib/kernel-wrapper/dts
>  WRAPPER_BINDIR := /usr/sbin
> diff --git a/arch/powerpc/boot/dts/Makefile b/arch/powerpc/boot/dts/Makefile
> new file mode 100644
> index 0000000..f121775
> --- /dev/null
> +++ b/arch/powerpc/boot/dts/Makefile
> @@ -0,0 +1,5 @@
> +dtstree		:= $(srctree)/$(src)
> +dtb-$(CONFIG_OF_ALL_DTBS) := $(patsubst $(dtstree)/%.dts,%.dtb, $(wildcard $(dtstree)/*.dts))
> +
> +always		:= $(dtb-y)

How/is that different from just doing the more normal:

obj-$(CONFIG_OF_ALL_DTBS) := ...

?

cheers


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 00/13] Enable building all dtb files Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 09/13] mips: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 05/13] arm64: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
    Re: [PATCH v2 05/13] arm64: enable building of all dtbs Geert Uytterhoeven <geert@linux-m68k.org> - 2015-10-08 20:20 +0200
      Re: [PATCH v2 05/13] arm64: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 21:00 +0200
  [PATCH v2 13/13] xtensa: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 01/13] of: add config option to enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 02/13] arc: use common make variables for dtb builds Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 06/13] h8300: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 07/13] metag: use common make variables for dtb builds Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 08/13] metag: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
  [PATCH v2 12/13] powerpc: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-08 20:00 +0200
    Re: [PATCH v2 12/13] powerpc: enable building of all dtbs Michael Ellerman <mpe@ellerman.id.au> - 2015-10-12 12:30 +0200
      Re: [PATCH v2 12/13] powerpc: enable building of all dtbs Rob Herring <robh@kernel.org> - 2015-10-13 17:30 +0200

csiph-web