Re: [PATCH v2 05/13] arm64: enable building of all dtbs

From	Rob Herring <robh@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 05/13] arm64: enable building of all dtbs
Date	2015-10-08 21:00 +0200
Message-ID	<qhoA2-3B3-23@gated-at.bofh.it> (permalink)
References	<qhnDY-2fG-3@gated-at.bofh.it> <qhnDZ-2fG-25@gated-at.bofh.it> <qhnXk-2RM-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Oct 8, 2015 at 1:15 PM, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Thu, Oct 8, 2015 at 7:53 PM, Rob Herring <robh@kernel.org> 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. This
>> is simpler for arm64 which has a bunch of sub-dirs.
>
> This doesn't look simpler than the arm(32) version?

I discussed this with Olof in the last version. In the ARM version, I
could have done just one line "dtb-$(CONFIG_OF_ALL_DTBS) += $(dtb-)"
to add all the disabled dtbs. On arm64, I would have to do that for
each sub-dir Makefile as well as make sure any new sub-dirs have this
line. In the end, I also prefer to build everything in the tree and
not rely on the dtbs to be explicitly listed.

Rob
--
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/

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