Re: [PATCH 0/7] MIPS: Move device-tree files to a common location

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 08/22/2014 01:42 PM, Florian Fainelli wrote:
On Aug 21, 2014 3:05 PM, "Andrew Bresticker" <abrestic@xxxxxxxxxxxx
<mailto:abrestic@xxxxxxxxxxxx>> wrote:
 >
 > To be consistent with other architectures and to avoid unnecessary
 > makefile duplication, move all MIPS device-trees to arch/mips/boot/dts
 > and build them with a common makefile.

I recall reading that the ARM organization for DTS files was a bit
unfortunate and should have been something like:

arch/arm/boot/dts/<vendor>/

Is this something we should do for the MIPS and update the other
architectures to follow that scheme?

If we choose not to intermingle .dts files from all the vendors in a single directory, why do anything at all? Currently the .dts files for a vendor are nicely segregated with the rest of the vendors code under a single directory.

Personally I think things are fine as they are. Any common code remaining in the Makefiles could be moved to the scripts directory for a smaller change.



 >
 > Patch 1 sets up the makefiles for building the DTs in arch/mips/boot/dts
 > and introduces the config option BUILTIN_DTB for platforms that require
 > it.
 >
 > Patch 2 introduces the 'dtbs' makefile target to allow building of just
 > the DT binaries.
 >
 > Patches 3-7 move the DTs out of the platform directores.
 >
 > I've build tested this on all affected platforms (Octeon, Lantiq, SEAD3,
 > Netlogic, and Ralink) as well as Malta.  For platforms where builtin DTBs
 > are optional (Netlogic and Ralink), I built with and without the builtin
 > DTBs.
 >
 > Based on 3.17-rc1.
 >
 > Andrew Bresticker (7):
 >   MIPS: Create common infrastructure for building built-in device-trees
 >   MIPS: Add support for building device-tree binaries
 >   MIPS: Octeon: Move device-trees to arch/mips/boot/dts/
 >   MIPS: Lantiq: Move device-trees to arch/mips/boot/dts/
 >   MIPS: sead3: Move device-trees to arch/mips/boot/dts/
 >   MIPS: Netlogic: Move device-trees to arch/mips/boot/dts/
 >   MIPS: ralink: Move device-trees to arch/mips/boot/dts/
 >
 >  arch/mips/Kconfig                                    |  5 +++++
 >  arch/mips/Makefile                                   | 11 +++++++++++
 >  arch/mips/boot/.gitignore                            |  1 +
 >  arch/mips/boot/dts/Makefile                          | 20
++++++++++++++++++++
 >  arch/mips/{lantiq => boot}/dts/danube.dtsi           |  0
 >  arch/mips/{lantiq => boot}/dts/easy50712.dts         |  0
 >  arch/mips/{ralink => boot}/dts/mt7620a.dtsi          |  0
 >  arch/mips/{ralink => boot}/dts/mt7620a_eval.dts      |  0
 >  .../mips/{cavium-octeon => boot/dts}/octeon_3xxx.dts |  0
 >  .../mips/{cavium-octeon => boot/dts}/octeon_68xx.dts |  0
 >  arch/mips/{ralink => boot}/dts/rt2880.dtsi           |  0
 >  arch/mips/{ralink => boot}/dts/rt2880_eval.dts       |  0
 >  arch/mips/{ralink => boot}/dts/rt3050.dtsi           |  0
 >  arch/mips/{ralink => boot}/dts/rt3052_eval.dts       |  0
 >  arch/mips/{ralink => boot}/dts/rt3883.dtsi           |  0
 >  arch/mips/{ralink => boot}/dts/rt3883_eval.dts       |  0
 >  arch/mips/{mti-sead3 => boot/dts}/sead3.dts          |  0
 >  arch/mips/{netlogic => boot}/dts/xlp_evp.dts         |  0
 >  arch/mips/{netlogic => boot}/dts/xlp_fvp.dts         |  0
 >  arch/mips/{netlogic => boot}/dts/xlp_gvp.dts         |  0
 >  arch/mips/{netlogic => boot}/dts/xlp_svp.dts         |  0
 >  arch/mips/cavium-octeon/.gitignore                   |  2 --
 >  arch/mips/cavium-octeon/Makefile                     | 10 ----------
 >  arch/mips/lantiq/Kconfig                             |  1 +
 >  arch/mips/lantiq/Makefile                            |  2 --
 >  arch/mips/lantiq/dts/Makefile                        |  1 -
 >  arch/mips/mti-sead3/Makefile                         |  4 ----
 >  arch/mips/netlogic/Kconfig                           |  4 ++++
 >  arch/mips/netlogic/Makefile                          |  1 -
 >  arch/mips/netlogic/dts/Makefile                      |  4 ----
 >  arch/mips/ralink/Kconfig                             |  4 ++++
 >  arch/mips/ralink/Makefile                            |  2 --
 >  arch/mips/ralink/dts/Makefile                        |  4 ----
 >  33 files changed, 46 insertions(+), 30 deletions(-)
 >  create mode 100644 arch/mips/boot/dts/Makefile
 >  rename arch/mips/{lantiq => boot}/dts/danube.dtsi (100%)
 >  rename arch/mips/{lantiq => boot}/dts/easy50712.dts (100%)
 >  rename arch/mips/{ralink => boot}/dts/mt7620a.dtsi (100%)
 >  rename arch/mips/{ralink => boot}/dts/mt7620a_eval.dts (100%)
 >  rename arch/mips/{cavium-octeon => boot/dts}/octeon_3xxx.dts (100%)
 >  rename arch/mips/{cavium-octeon => boot/dts}/octeon_68xx.dts (100%)
 >  rename arch/mips/{ralink => boot}/dts/rt2880.dtsi (100%)
 >  rename arch/mips/{ralink => boot}/dts/rt2880_eval.dts (100%)
 >  rename arch/mips/{ralink => boot}/dts/rt3050.dtsi (100%)
 >  rename arch/mips/{ralink => boot}/dts/rt3052_eval.dts (100%)
 >  rename arch/mips/{ralink => boot}/dts/rt3883.dtsi (100%)
 >  rename arch/mips/{ralink => boot}/dts/rt3883_eval.dts (100%)
 >  rename arch/mips/{mti-sead3 => boot/dts}/sead3.dts (100%)
 >  rename arch/mips/{netlogic => boot}/dts/xlp_evp.dts (100%)
 >  rename arch/mips/{netlogic => boot}/dts/xlp_fvp.dts (100%)
 >  rename arch/mips/{netlogic => boot}/dts/xlp_gvp.dts (100%)
 >  rename arch/mips/{netlogic => boot}/dts/xlp_svp.dts (100%)
 >  delete mode 100644 arch/mips/cavium-octeon/.gitignore
 >  delete mode 100644 arch/mips/lantiq/dts/Makefile
 >  delete mode 100644 arch/mips/netlogic/dts/Makefile
 >  delete mode 100644 arch/mips/ralink/dts/Makefile
 >
 > --
 > 2.1.0.rc2.206.gedb03e5
 >
 > --
 > To unsubscribe from this list: send the line "unsubscribe devicetree" in
 > the body of a message to majordomo@xxxxxxxxxxxxxxx
<mailto:majordomo@xxxxxxxxxxxxxxx>
 > More majordomo info at http://vger.kernel.org/majordomo-info.html


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux