* [RFC 0/2] Convert omap.txt to DT schema
@ 2023-02-16 15:33 Andrew Davis
2023-02-16 15:33 ` [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible Andrew Davis
` (2 more replies)
0 siblings, 3 replies; 14+ messages in thread
From: Andrew Davis @ 2023-02-16 15:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Benoît Cousson,
Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann
Cc: devicetree, linux-kernel, linux-omap, linux-arm-kernel,
Andrew Davis
Hello all,
I saw there was some more talk on the lists about converting
the omap.txt over to DT schema. I remembered that I had made
an attempt at this and wanted to post it as a WIP example.
It is not perfect or even very good, but might serve as
a base for someone.
Thanks,
Andrew
Andrew Davis (2):
ARM: dts: omap: Drop ti,omap36xx compatible
WIP: dt-bindings: omap: Convert omap.txt to yaml
.../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++
arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +-
arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +-
arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +-
arch/arm/boot/dts/omap3-igep0020.dts | 2 +-
arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +-
arch/arm/boot/dts/omap3-igep0030.dts | 2 +-
arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +-
arch/arm/boot/dts/omap3-n9.dts | 2 +-
arch/arm/boot/dts/omap3-n950.dts | 2 +-
.../arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +-
.../boot/dts/omap3-overo-storm-chestnut43.dts | 2 +-
.../boot/dts/omap3-overo-storm-gallop43.dts | 2 +-
.../arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +-
.../arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +-
.../arm/boot/dts/omap3-overo-storm-summit.dts | 2 +-
arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +-
.../boot/dts/omap3-overo-storm-tobiduo.dts | 2 +-
arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +-
arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +-
arch/arm/boot/dts/omap3-sniper.dts | 2 +-
arch/arm/boot/dts/omap3-zoom3.dts | 2 +-
22 files changed, 195 insertions(+), 21 deletions(-)
create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml
--
2.39.1
^ permalink raw reply [flat|nested] 14+ messages in thread* [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible 2023-02-16 15:33 [RFC 0/2] Convert omap.txt to DT schema Andrew Davis @ 2023-02-16 15:33 ` Andrew Davis 2023-03-27 19:18 ` Andreas Kemnade 2023-02-16 15:33 ` [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml Andrew Davis 2023-02-17 5:48 ` [RFC 0/2] Convert omap.txt to DT schema Tony Lindgren 2 siblings, 1 reply; 14+ messages in thread From: Andrew Davis @ 2023-02-16 15:33 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann Cc: devicetree, linux-kernel, linux-omap, linux-arm-kernel, Andrew Davis This was not matched anywhere and provides no additional information. Signed-off-by: Andrew Davis <afd@ti.com> --- arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +- arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +- arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +- arch/arm/boot/dts/omap3-igep0020.dts | 2 +- arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +- arch/arm/boot/dts/omap3-igep0030.dts | 2 +- arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +- arch/arm/boot/dts/omap3-n9.dts | 2 +- arch/arm/boot/dts/omap3-n950.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-gallop43.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-summit.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +- arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts | 2 +- arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +- arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +- arch/arm/boot/dts/omap3-sniper.dts | 2 +- arch/arm/boot/dts/omap3-zoom3.dts | 2 +- 21 files changed, 21 insertions(+), 21 deletions(-) diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index 35eced6521efb..1a085bc013177 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -8,7 +8,7 @@ / { model = "TI OMAP3 BeagleBoard xM"; - compatible = "ti,omap3-beagle-xm", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "ti,omap3-beagle-xm", "ti,omap3630", "ti,omap3"; cpus { cpu@0 { diff --git a/arch/arm/boot/dts/omap3-cm-t3730.dts b/arch/arm/boot/dts/omap3-cm-t3730.dts index 48e48b0c81901..e1b1a047f77a0 100644 --- a/arch/arm/boot/dts/omap3-cm-t3730.dts +++ b/arch/arm/boot/dts/omap3-cm-t3730.dts @@ -9,7 +9,7 @@ / { model = "CompuLab CM-T3730"; - compatible = "compulab,omap3-cm-t3730", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "compulab,omap3-cm-t3730", "ti,omap3630", "ti,omap3"; wl12xx_vmmc2: wl12xx_vmmc2 { compatible = "regulator-fixed"; diff --git a/arch/arm/boot/dts/omap3-igep0020-rev-f.dts b/arch/arm/boot/dts/omap3-igep0020-rev-f.dts index 9dca5bfc87ab2..eadb5b857f48b 100644 --- a/arch/arm/boot/dts/omap3-igep0020-rev-f.dts +++ b/arch/arm/boot/dts/omap3-igep0020-rev-f.dts @@ -10,7 +10,7 @@ / { model = "IGEPv2 Rev. F (TI OMAP AM/DM37x)"; - compatible = "isee,omap3-igep0020-rev-f", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "isee,omap3-igep0020-rev-f", "ti,omap3630", "ti,omap3"; /* Regulator to trigger the WL_EN signal of the Wifi module */ lbep5clwmc_wlen: regulator-lbep5clwmc-wlen { diff --git a/arch/arm/boot/dts/omap3-igep0020.dts b/arch/arm/boot/dts/omap3-igep0020.dts index c6f863bc03adf..3f0197ceae09e 100644 --- a/arch/arm/boot/dts/omap3-igep0020.dts +++ b/arch/arm/boot/dts/omap3-igep0020.dts @@ -10,7 +10,7 @@ / { model = "IGEPv2 Rev. C (TI OMAP AM/DM37x)"; - compatible = "isee,omap3-igep0020", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "isee,omap3-igep0020", "ti,omap3630", "ti,omap3"; vmmcsdio_fixed: fixedregulator-mmcsdio { compatible = "regulator-fixed"; diff --git a/arch/arm/boot/dts/omap3-igep0030-rev-g.dts b/arch/arm/boot/dts/omap3-igep0030-rev-g.dts index 8e9c12cf51a7b..bc95a8df2e6a1 100644 --- a/arch/arm/boot/dts/omap3-igep0030-rev-g.dts +++ b/arch/arm/boot/dts/omap3-igep0030-rev-g.dts @@ -10,7 +10,7 @@ / { model = "IGEP COM MODULE Rev. G (TI OMAP AM/DM37x)"; - compatible = "isee,omap3-igep0030-rev-g", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "isee,omap3-igep0030-rev-g", "ti,omap3630", "ti,omap3"; /* Regulator to trigger the WL_EN signal of the Wifi module */ lbep5clwmc_wlen: regulator-lbep5clwmc-wlen { diff --git a/arch/arm/boot/dts/omap3-igep0030.dts b/arch/arm/boot/dts/omap3-igep0030.dts index 5188f96f431e7..d36ceecb7328c 100644 --- a/arch/arm/boot/dts/omap3-igep0030.dts +++ b/arch/arm/boot/dts/omap3-igep0030.dts @@ -10,7 +10,7 @@ / { model = "IGEP COM MODULE Rev. E (TI OMAP AM/DM37x)"; - compatible = "isee,omap3-igep0030", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "isee,omap3-igep0030", "ti,omap3630", "ti,omap3"; vmmcsdio_fixed: fixedregulator-mmcsdio { compatible = "regulator-fixed"; diff --git a/arch/arm/boot/dts/omap3-lilly-dbb056.dts b/arch/arm/boot/dts/omap3-lilly-dbb056.dts index ecb4ef738e07b..f6bbea2be54c5 100644 --- a/arch/arm/boot/dts/omap3-lilly-dbb056.dts +++ b/arch/arm/boot/dts/omap3-lilly-dbb056.dts @@ -8,7 +8,7 @@ / { model = "INCOstartec LILLY-DBB056 (DM3730)"; - compatible = "incostartec,omap3-lilly-dbb056", "incostartec,omap3-lilly-a83x", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "incostartec,omap3-lilly-dbb056", "incostartec,omap3-lilly-a83x", "ti,omap3630", "ti,omap3"; }; &twl { diff --git a/arch/arm/boot/dts/omap3-n9.dts b/arch/arm/boot/dts/omap3-n9.dts index d211bcc311745..a3cf3f443785f 100644 --- a/arch/arm/boot/dts/omap3-n9.dts +++ b/arch/arm/boot/dts/omap3-n9.dts @@ -12,7 +12,7 @@ / { model = "Nokia N9"; - compatible = "nokia,omap3-n9", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "nokia,omap3-n9", "ti,omap3630", "ti,omap3"; }; &i2c2 { diff --git a/arch/arm/boot/dts/omap3-n950.dts b/arch/arm/boot/dts/omap3-n950.dts index b2f480022ff6e..cbaf79c4e8422 100644 --- a/arch/arm/boot/dts/omap3-n950.dts +++ b/arch/arm/boot/dts/omap3-n950.dts @@ -12,7 +12,7 @@ / { model = "Nokia N950"; - compatible = "nokia,omap3-n950", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "nokia,omap3-n950", "ti,omap3630", "ti,omap3"; keys { compatible = "gpio-keys"; diff --git a/arch/arm/boot/dts/omap3-overo-storm-alto35.dts b/arch/arm/boot/dts/omap3-overo-storm-alto35.dts index 7f04dfad82034..3eb935df04dcc 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-alto35.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-alto35.dts @@ -14,5 +14,5 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Alto35"; - compatible = "gumstix,omap3-overo-alto35", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-alto35", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; diff --git a/arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts b/arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts index bc5a04e033362..3af8d10d72247 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts @@ -14,7 +14,7 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Chestnut43"; - compatible = "gumstix,omap3-overo-chestnut43", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-chestnut43", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; &omap3_pmx_core2 { diff --git a/arch/arm/boot/dts/omap3-overo-storm-gallop43.dts b/arch/arm/boot/dts/omap3-overo-storm-gallop43.dts index 065c31cbf0e29..813e3c9fe3b64 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-gallop43.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-gallop43.dts @@ -14,7 +14,7 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Gallop43"; - compatible = "gumstix,omap3-overo-gallop43", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-gallop43", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; &omap3_pmx_core2 { diff --git a/arch/arm/boot/dts/omap3-overo-storm-palo35.dts b/arch/arm/boot/dts/omap3-overo-storm-palo35.dts index e38c1c51392c5..8405bd9262de6 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-palo35.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-palo35.dts @@ -14,7 +14,7 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Palo35"; - compatible = "gumstix,omap3-overo-palo35", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-palo35", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; &omap3_pmx_core2 { diff --git a/arch/arm/boot/dts/omap3-overo-storm-palo43.dts b/arch/arm/boot/dts/omap3-overo-storm-palo43.dts index e6dc23159c4d9..b9558d736e795 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-palo43.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-palo43.dts @@ -14,7 +14,7 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Palo43"; - compatible = "gumstix,omap3-overo-palo43", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-palo43", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; &omap3_pmx_core2 { diff --git a/arch/arm/boot/dts/omap3-overo-storm-summit.dts b/arch/arm/boot/dts/omap3-overo-storm-summit.dts index 587c08ce282d3..fcfc449f2abea 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-summit.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-summit.dts @@ -14,7 +14,7 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Summit"; - compatible = "gumstix,omap3-overo-summit", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-summit", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; &omap3_pmx_core2 { diff --git a/arch/arm/boot/dts/omap3-overo-storm-tobi.dts b/arch/arm/boot/dts/omap3-overo-storm-tobi.dts index f57de6010994e..6d14466c180af 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-tobi.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-tobi.dts @@ -14,6 +14,6 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on Tobi"; - compatible = "gumstix,omap3-overo-tobi", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-tobi", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; diff --git a/arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts b/arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts index 281af6c113beb..bcf20ff3f2813 100644 --- a/arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts +++ b/arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts @@ -14,5 +14,5 @@ / { model = "OMAP36xx/AM37xx/DM37xx Gumstix Overo on TobiDuo"; - compatible = "gumstix,omap3-overo-tobiduo", "gumstix,omap3-overo", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "gumstix,omap3-overo-tobiduo", "gumstix,omap3-overo", "ti,omap3630", "ti,omap3"; }; diff --git a/arch/arm/boot/dts/omap3-pandora-1ghz.dts b/arch/arm/boot/dts/omap3-pandora-1ghz.dts index ea509956d7aca..c0252f8a798aa 100644 --- a/arch/arm/boot/dts/omap3-pandora-1ghz.dts +++ b/arch/arm/boot/dts/omap3-pandora-1ghz.dts @@ -16,7 +16,7 @@ / { model = "Pandora Handheld Console 1GHz"; - compatible = "openpandora,omap3-pandora-1ghz", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "openpandora,omap3-pandora-1ghz", "ti,omap3630", "ti,omap3"; }; &omap3_pmx_core2 { diff --git a/arch/arm/boot/dts/omap3-sbc-t3730.dts b/arch/arm/boot/dts/omap3-sbc-t3730.dts index eb3893b9535e6..4c36bde62491d 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3730.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3730.dts @@ -8,7 +8,7 @@ / { model = "CompuLab SBC-T3730 with CM-T3730"; - compatible = "compulab,omap3-sbc-t3730", "compulab,omap3-cm-t3730", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "compulab,omap3-sbc-t3730", "compulab,omap3-cm-t3730", "ti,omap3630", "ti,omap3"; aliases { display0 = &dvi0; diff --git a/arch/arm/boot/dts/omap3-sniper.dts b/arch/arm/boot/dts/omap3-sniper.dts index b6879cdc5c13e..0591af4941847 100644 --- a/arch/arm/boot/dts/omap3-sniper.dts +++ b/arch/arm/boot/dts/omap3-sniper.dts @@ -9,7 +9,7 @@ / { model = "LG Optimus Black"; - compatible = "lg,omap3-sniper", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "lg,omap3-sniper", "ti,omap3630", "ti,omap3"; cpus { cpu@0 { diff --git a/arch/arm/boot/dts/omap3-zoom3.dts b/arch/arm/boot/dts/omap3-zoom3.dts index ce58b1f208e81..ab52e8d68f760 100644 --- a/arch/arm/boot/dts/omap3-zoom3.dts +++ b/arch/arm/boot/dts/omap3-zoom3.dts @@ -9,7 +9,7 @@ / { model = "TI Zoom3"; - compatible = "ti,omap3-zoom3", "ti,omap3630", "ti,omap36xx", "ti,omap3"; + compatible = "ti,omap3-zoom3", "ti,omap3630", "ti,omap3"; cpus { cpu@0 { -- 2.39.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible 2023-02-16 15:33 ` [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible Andrew Davis @ 2023-03-27 19:18 ` Andreas Kemnade 2023-03-28 5:01 ` Tony Lindgren 0 siblings, 1 reply; 14+ messages in thread From: Andreas Kemnade @ 2023-03-27 19:18 UTC (permalink / raw) To: Andrew Davis Cc: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel On Thu, 16 Feb 2023 09:33:38 -0600 Andrew Davis <afd@ti.com> wrote: > This was not matched anywhere and provides no additional information. > > Signed-off-by: Andrew Davis <afd@ti.com> > --- > arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +- > arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +- > arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +- > arch/arm/boot/dts/omap3-igep0020.dts | 2 +- > arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +- > arch/arm/boot/dts/omap3-igep0030.dts | 2 +- > arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +- > arch/arm/boot/dts/omap3-n9.dts | 2 +- > arch/arm/boot/dts/omap3-n950.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-gallop43.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-summit.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +- > arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts | 2 +- > arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +- > arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +- > arch/arm/boot/dts/omap3-sniper.dts | 2 +- > arch/arm/boot/dts/omap3-zoom3.dts | 2 +- > 21 files changed, 21 insertions(+), 21 deletions(-) > hmm, we have drivers/clk/ti/dpll.c: of_machine_is_compatible("ti,omap36xx")) but that is more completely if ((of_machine_is_compatible("ti,omap3630") || of_machine_is_compatible("ti,omap36xx")) && so missing omap36xx will not harm if 3630 is there. SO this should be probably ok. Regards, Andreas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible 2023-03-27 19:18 ` Andreas Kemnade @ 2023-03-28 5:01 ` Tony Lindgren 2023-03-28 7:06 ` Andreas Kemnade 0 siblings, 1 reply; 14+ messages in thread From: Tony Lindgren @ 2023-03-28 5:01 UTC (permalink / raw) To: Andreas Kemnade Cc: Andrew Davis, Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel * Andreas Kemnade <andreas@kemnade.info> [230327 19:18]: > On Thu, 16 Feb 2023 09:33:38 -0600 > Andrew Davis <afd@ti.com> wrote: > > > This was not matched anywhere and provides no additional information. > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > --- > > arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +- > > arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +- > > arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +- > > arch/arm/boot/dts/omap3-igep0020.dts | 2 +- > > arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +- > > arch/arm/boot/dts/omap3-igep0030.dts | 2 +- > > arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +- > > arch/arm/boot/dts/omap3-n9.dts | 2 +- > > arch/arm/boot/dts/omap3-n950.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-gallop43.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-summit.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +- > > arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts | 2 +- > > arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +- > > arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +- > > arch/arm/boot/dts/omap3-sniper.dts | 2 +- > > arch/arm/boot/dts/omap3-zoom3.dts | 2 +- > > 21 files changed, 21 insertions(+), 21 deletions(-) > > > hmm, we have > drivers/clk/ti/dpll.c: of_machine_is_compatible("ti,omap36xx")) > > but that is more completely > if ((of_machine_is_compatible("ti,omap3630") || > of_machine_is_compatible("ti,omap36xx")) && > > so missing omap36xx will not harm if 3630 is there. SO this should > be probably ok. Looks like we still have these that should be patched away first: drivers/cpufreq/ti-cpufreq.c- { .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, }, drivers/cpufreq/ti-cpufreq.c: { .compatible = "ti,omap36xx", .data = &omap36xx_soc_data, }, Regards, Tony ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible 2023-03-28 5:01 ` Tony Lindgren @ 2023-03-28 7:06 ` Andreas Kemnade 2023-03-28 7:08 ` Tony Lindgren 0 siblings, 1 reply; 14+ messages in thread From: Andreas Kemnade @ 2023-03-28 7:06 UTC (permalink / raw) To: Tony Lindgren Cc: Andrew Davis, Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel On Tue, 28 Mar 2023 08:01:15 +0300 Tony Lindgren <tony@atomide.com> wrote: > * Andreas Kemnade <andreas@kemnade.info> [230327 19:18]: > > On Thu, 16 Feb 2023 09:33:38 -0600 > > Andrew Davis <afd@ti.com> wrote: > > > > > This was not matched anywhere and provides no additional information. > > > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > > --- > > > arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +- > > > arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +- > > > arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +- > > > arch/arm/boot/dts/omap3-igep0020.dts | 2 +- > > > arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +- > > > arch/arm/boot/dts/omap3-igep0030.dts | 2 +- > > > arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +- > > > arch/arm/boot/dts/omap3-n9.dts | 2 +- > > > arch/arm/boot/dts/omap3-n950.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-gallop43.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-summit.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +- > > > arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts | 2 +- > > > arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +- > > > arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +- > > > arch/arm/boot/dts/omap3-sniper.dts | 2 +- > > > arch/arm/boot/dts/omap3-zoom3.dts | 2 +- > > > 21 files changed, 21 insertions(+), 21 deletions(-) > > > > > hmm, we have > > drivers/clk/ti/dpll.c: of_machine_is_compatible("ti,omap36xx")) > > > > but that is more completely > > if ((of_machine_is_compatible("ti,omap3630") || > > of_machine_is_compatible("ti,omap36xx")) && > > > > so missing omap36xx will not harm if 3630 is there. SO this should > > be probably ok. > > Looks like we still have these that should be patched away first: > > drivers/cpufreq/ti-cpufreq.c- { .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, }, > drivers/cpufreq/ti-cpufreq.c: { .compatible = "ti,omap36xx", .data = &omap36xx_soc_data, }, > seen that but there is also { .compatible = "ti,omap3430", .data = &omap34xx_soc_data, }, { .compatible = "ti,omap3630", .data = &omap36xx_soc_data, }, so, no trouble will appear when omap36xx is removed. Regards, Andreas ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible 2023-03-28 7:06 ` Andreas Kemnade @ 2023-03-28 7:08 ` Tony Lindgren 2023-03-28 7:27 ` Tony Lindgren 0 siblings, 1 reply; 14+ messages in thread From: Tony Lindgren @ 2023-03-28 7:08 UTC (permalink / raw) To: Andreas Kemnade Cc: Andrew Davis, Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel * Andreas Kemnade <andreas@kemnade.info> [230328 07:06]: > On Tue, 28 Mar 2023 08:01:15 +0300 > Tony Lindgren <tony@atomide.com> wrote: > > > * Andreas Kemnade <andreas@kemnade.info> [230327 19:18]: > > > On Thu, 16 Feb 2023 09:33:38 -0600 > > > Andrew Davis <afd@ti.com> wrote: > > > > > > > This was not matched anywhere and provides no additional information. > > > > > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > > > --- > > > > arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +- > > > > arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +- > > > > arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +- > > > > arch/arm/boot/dts/omap3-igep0020.dts | 2 +- > > > > arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +- > > > > arch/arm/boot/dts/omap3-igep0030.dts | 2 +- > > > > arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +- > > > > arch/arm/boot/dts/omap3-n9.dts | 2 +- > > > > arch/arm/boot/dts/omap3-n950.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-gallop43.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-summit.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +- > > > > arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts | 2 +- > > > > arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +- > > > > arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +- > > > > arch/arm/boot/dts/omap3-sniper.dts | 2 +- > > > > arch/arm/boot/dts/omap3-zoom3.dts | 2 +- > > > > 21 files changed, 21 insertions(+), 21 deletions(-) > > > > > > > hmm, we have > > > drivers/clk/ti/dpll.c: of_machine_is_compatible("ti,omap36xx")) > > > > > > but that is more completely > > > if ((of_machine_is_compatible("ti,omap3630") || > > > of_machine_is_compatible("ti,omap36xx")) && > > > > > > so missing omap36xx will not harm if 3630 is there. SO this should > > > be probably ok. > > > > Looks like we still have these that should be patched away first: > > > > drivers/cpufreq/ti-cpufreq.c- { .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, }, > > drivers/cpufreq/ti-cpufreq.c: { .compatible = "ti,omap36xx", .data = &omap36xx_soc_data, }, > > > seen that but there is also > { .compatible = "ti,omap3430", .data = &omap34xx_soc_data, }, > { .compatible = "ti,omap3630", .data = &omap36xx_soc_data, }, > > so, no trouble will appear when omap36xx is removed. Oh OK, thanks for pointing that out, looks like I missed it. Seems like we should apply this patch already if no issues. Regards, Tony ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible 2023-03-28 7:08 ` Tony Lindgren @ 2023-03-28 7:27 ` Tony Lindgren 0 siblings, 0 replies; 14+ messages in thread From: Tony Lindgren @ 2023-03-28 7:27 UTC (permalink / raw) To: Andreas Kemnade Cc: Andrew Davis, Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel * Tony Lindgren <tony@atomide.com> [230328 07:08]: > * Andreas Kemnade <andreas@kemnade.info> [230328 07:06]: > > On Tue, 28 Mar 2023 08:01:15 +0300 > > Tony Lindgren <tony@atomide.com> wrote: > > > > > * Andreas Kemnade <andreas@kemnade.info> [230327 19:18]: > > > > On Thu, 16 Feb 2023 09:33:38 -0600 > > > > Andrew Davis <afd@ti.com> wrote: > > > > > > > > > This was not matched anywhere and provides no additional information. > > > > > > > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > > > > --- > > > > > arch/arm/boot/dts/omap3-beagle-xm.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-cm-t3730.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-igep0020.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-igep0030.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-lilly-dbb056.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-n9.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-n950.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-alto35.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-chestnut43.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-gallop43.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-palo35.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-palo43.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-summit.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-tobi.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-overo-storm-tobiduo.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-pandora-1ghz.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-sbc-t3730.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-sniper.dts | 2 +- > > > > > arch/arm/boot/dts/omap3-zoom3.dts | 2 +- > > > > > 21 files changed, 21 insertions(+), 21 deletions(-) > > > > > > > > > hmm, we have > > > > drivers/clk/ti/dpll.c: of_machine_is_compatible("ti,omap36xx")) > > > > > > > > but that is more completely > > > > if ((of_machine_is_compatible("ti,omap3630") || > > > > of_machine_is_compatible("ti,omap36xx")) && > > > > > > > > so missing omap36xx will not harm if 3630 is there. SO this should > > > > be probably ok. > > > > > > Looks like we still have these that should be patched away first: > > > > > > drivers/cpufreq/ti-cpufreq.c- { .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, }, > > > drivers/cpufreq/ti-cpufreq.c: { .compatible = "ti,omap36xx", .data = &omap36xx_soc_data, }, > > > > > seen that but there is also > > { .compatible = "ti,omap3430", .data = &omap34xx_soc_data, }, > > { .compatible = "ti,omap3630", .data = &omap36xx_soc_data, }, > > > > so, no trouble will appear when omap36xx is removed. > > Oh OK, thanks for pointing that out, looks like I missed it. Seems like > we should apply this patch already if no issues. Applied this one to omap-for-v6.4/dt and pushed out. Regards, Tony ^ permalink raw reply [flat|nested] 14+ messages in thread
* [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml 2023-02-16 15:33 [RFC 0/2] Convert omap.txt to DT schema Andrew Davis 2023-02-16 15:33 ` [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible Andrew Davis @ 2023-02-16 15:33 ` Andrew Davis 2023-02-16 16:08 ` H. Nikolaus Schaller 2023-02-16 16:41 ` Rob Herring 2023-02-17 5:48 ` [RFC 0/2] Convert omap.txt to DT schema Tony Lindgren 2 siblings, 2 replies; 14+ messages in thread From: Andrew Davis @ 2023-02-16 15:33 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann Cc: devicetree, linux-kernel, linux-omap, linux-arm-kernel, Andrew Davis Signed-off-by: Andrew Davis <afd@ti.com> --- .../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++ 1 file changed, 174 insertions(+) create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml diff --git a/Documentation/devicetree/bindings/arm/omap/omap.yaml b/Documentation/devicetree/bindings/arm/omap/omap.yaml new file mode 100644 index 0000000000000..cf07a7a7df279 --- /dev/null +++ b/Documentation/devicetree/bindings/arm/omap/omap.yaml @@ -0,0 +1,174 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/arm/omap/omap.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Texas Instruments OMAP SoC architecture device tree bindings + +maintainers: + - Tony Lindgren <tony@atomide.com> + +description: | + Platforms based on Texas Instruments OMAP SoC architecture. + +properties: + $nodename: + const: '/' + compatible: + oneOf: + + - description: TI OMAP2420 SoC based platforms + items: + - enum: + - ti,omap2420-h4 # TI OMAP2420 H4 board + - nokia,n800 # Nokia N800 + - nokia,n810 # Nokia N810 + - nokia,n810-wimax # Nokia N810 WiMax + - const: ti,omap2420 + - const: ti,omap2 + + - description: TI OMAP2430 SoC based platforms + items: + - enum: + - ti,omap2430-sdp # TI OMAP2430 SDP + - const: ti,omap2430 + - const: ti,omap2 + + - description: TI OMAP3430 SoC based platforms + items: + - enum: + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom OMAP35xx SOM-LV Development Kit + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit + - ti,omap3430-sdp # TI OMAP3430 SDP + - ti,omap3-beagle # TI OMAP3 BeagleBoard + - compulab,omap3-cm-t3530 # CompuLab CM-T3530 + - timll,omap3-devkit8000 # TimLL OMAP3 Devkit8000 + - ti,omap3-evm # TI OMAP35XX EVM (TMDSEVM3530) + - ti,omap3-ldp # TI OMAP3430 LDP (Zoom1 Labrador) + - nokia,omap3-n900 # Nokia N900 + - const: ti,omap3430 + - const: ti,omap3 + + - description: TI OMAP3630 SoC based platforms + items: + - enum: + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom DM3730 SOM-LV Development Kit + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit + - ti,omap3-beagle-xm # TI OMAP3 BeagleBoard xM + - compulab,omap3-cm-t3730 # CompuLab CM-T3730 + - amazon,omap3-echo # Amazon Echo (first generation) + - ti,omap3-evm-37xx # TI OMAP37XX EVM (TMDSEVM3730) + - ti,omap3-gta04 # OMAP3 GTA04 + - nokia,omap3-n9 # Nokia N9 + - nokia,omap3-n950 # Nokia N950 + - lg,omap3-sniper # LG Optimus Black + - ti,omap3-zoom3 # TI Zoom3 + - const: ti,omap3630 + - const: ti,omap3 + + - description: TI AM35 SoC based platforms + items: + - enum: + - teejet,mt_ventoux # TeeJet Mt.Ventoux + - ti,am3517-craneboard # TI AM3517 CraneBoard (TMDSEVM3517) + - ti,am3517-evm # TI AM3517 EVM (AM3517/05 TMDSEVM3517) + - compulab,omap3-sbc-t3517 # CompuLab SBC-T3517 with CM-T3517 + - const: ti,am3517 + - const: ti,omap3 + + - description: TI OMAP4430 SoC based platforms + items: + - enum: + - motorola,droid4 # Motorola Droid 4 XT894 + - motorola,droid-bionic # Motorola Droid Bionic XT875 + - amazon,omap4-kc1 # Amazon Kindle Fire (first generation) + - ti,omap4-panda # TI OMAP4 PandaBoard + - ti,omap4-sdp # TI OMAP4 SDP board + - const: ti,omap4430 + - const: ti,omap4 + + - description: TI OMAP4460 SoC based platforms + items: + - enum: + - ti,omap4-panda-es # TI OMAP4 PandaBoard-ES + - const: ti,omap4460 + - const: ti,omap4 + + - description: TI OMAP543 SoC based platforms + items: + - enum: + - compulab,omap5-cm-t54 # CompuLab CM-T54 + - isee,omap5-igep0050 # IGEPv5 + - ti,omap5-uevm # TI OMAP5 uEVM board + - const: ti,omap5 + + - description: TI AM33 SoC based platforms + items: + - enum: + - ti,am335x-bone # TI AM335x BeagleBone + - compulab,cm-t335 # CompuLab CM-T335 + - ti,am335x-evm # TI AM335x EVM + - ti,am335x-evmsk # TI AM335x EVM-SK + - bosch,am335x-guardian # Bosch AM335x Guardian + - ti,am3359-icev2 # TI AM3359 ICE-V2 + - novatech,am335x-lxm # NovaTech OrionLXm + - moxa,uc-2101 # Moxa UC-2101 + - moxa,uc-8100-me-t # Moxa UC-8100-ME-T + - gumstix,am335x-pepper # Gumstix Pepper + - tcl,am335x-sl50 # Toby Churchill SL50 Series + - pattern: '^ti,am33(5[1246789]|xx)$' # ti,am33xx is legacy please use full SoC name + + - description: TI AM43 SoC based platforms + items: + - enum: + - compulab,am437x-cm-t43 # CompuLab CM-T43 + - ti,am437x-gp-evm # TI AM437x GP EVM + - ti,am437x-idk-evm # TI AM437x Industrial Development Kit + - ti,am437x-sk-evm # TI AM437x SK EVM + - pattern: '^ti,am4372[26789]$' + - const: ti,am43 + + - description: TI AM57 SoC based platforms + items: + - enum: + - beagle,am5729-beagleboneai # BeagleBoard.org BeagleBone AI + - compulab,cl-som-am57x # CompuLab CL-SOM-AM57x + - ti,am5718-idk # TI AM5718 IDK + - ti,am5728-idk # TI AM5728 IDK + - ti,am5748-idk # TI AM5748 IDK + - pattern: '^ti,am57[0124][689]$' + - const: ti,am57 + + - description: TI DRA7 SoC based platforms + items: + - enum: + - ti,dra718-evm # TI DRA718 EVM + - ti,dra722-evm # TI DRA722 EVM + - ti,dra742-evm # TI DRA742 EVM + - ti,dra762-evm # TI DRA762 EVM + - pattern: '^ti,dra7[12456][024568p]$' + - const: ti,dra7 + +additionalProperties: true + +examples: + - | + / { + model = "TI OMAP2430 SDP (Software Development Board)"; + compatible = "ti,omap2430-sdp", "ti,omap2430", "ti,omap2"; + + #address-cells = <1>; + #size-cells = <1>; + }; + + - | + / { + model = "TI DRA762 EVM"; + compatible = "ti,dra762-evm", "ti,dra762", "ti,dra7"; + + #address-cells = <1>; + #size-cells = <1>; + }; + +... -- 2.39.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml 2023-02-16 15:33 ` [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml Andrew Davis @ 2023-02-16 16:08 ` H. Nikolaus Schaller 2023-02-16 16:19 ` Andrew Davis 2023-02-16 16:41 ` Rob Herring 1 sibling, 1 reply; 14+ messages in thread From: H. Nikolaus Schaller @ 2023-02-16 16:08 UTC (permalink / raw) To: Andrew Davis Cc: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann, OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, linux Kernel Mailing List, Linux-OMAP, Linux ARM, Discussions about the Letux Kernel Hi, > Am 16.02.2023 um 16:33 schrieb Andrew Davis <afd@ti.com>: > > Signed-off-by: Andrew Davis <afd@ti.com> > --- > .../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++ > 1 file changed, 174 insertions(+) > create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml > > diff --git a/Documentation/devicetree/bindings/arm/omap/omap.yaml b/Documentation/devicetree/bindings/arm/omap/omap.yaml > new file mode 100644 > index 0000000000000..cf07a7a7df279 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/omap/omap.yaml > @@ -0,0 +1,174 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/arm/omap/omap.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Texas Instruments OMAP SoC architecture device tree bindings > + > +maintainers: > + - Tony Lindgren <tony@atomide.com> > + > +description: | > + Platforms based on Texas Instruments OMAP SoC architecture. > + > +properties: > + $nodename: > + const: '/' > + compatible: > + oneOf: > + > + - description: TI OMAP2420 SoC based platforms > + items: > + - enum: > + - ti,omap2420-h4 # TI OMAP2420 H4 board > + - nokia,n800 # Nokia N800 > + - nokia,n810 # Nokia N810 > + - nokia,n810-wimax # Nokia N810 WiMax > + - const: ti,omap2420 > + - const: ti,omap2 > + > + - description: TI OMAP2430 SoC based platforms > + items: > + - enum: > + - ti,omap2430-sdp # TI OMAP2430 SDP > + - const: ti,omap2430 > + - const: ti,omap2 > + > + - description: TI OMAP3430 SoC based platforms > + items: > + - enum: > + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom OMAP35xx SOM-LV Development Kit > + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit > + - ti,omap3430-sdp # TI OMAP3430 SDP > + - ti,omap3-beagle # TI OMAP3 BeagleBoard > + - compulab,omap3-cm-t3530 # CompuLab CM-T3530 > + - timll,omap3-devkit8000 # TimLL OMAP3 Devkit8000 > + - ti,omap3-evm # TI OMAP35XX EVM (TMDSEVM3530) > + - ti,omap3-ldp # TI OMAP3430 LDP (Zoom1 Labrador) > + - nokia,omap3-n900 # Nokia N900 > + - const: ti,omap3430 > + - const: ti,omap3 > + > + - description: TI OMAP3630 SoC based platforms > + items: > + - enum: > + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom DM3730 SOM-LV Development Kit > + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit > + - ti,omap3-beagle-xm # TI OMAP3 BeagleBoard xM > + - compulab,omap3-cm-t3730 # CompuLab CM-T3730 > + - amazon,omap3-echo # Amazon Echo (first generation) > + - ti,omap3-evm-37xx # TI OMAP37XX EVM (TMDSEVM3730) > + - ti,omap3-gta04 # OMAP3 GTA04 precisely the GTA04 was not a TI product, but from Golden Delicious Computers. It got this compatible because it originally shared a lot of device tree with the ti,omap3-beagle-xm. Well, we could add a vendor prefix ("goldelico") and update the device trees to "goldelico,gta04". Generally, there is also the OpenPandora with both OMAP3 variants (3430 and 3630): compatible = "openpandora,omap3-pandora-600mhz", "ti,omap3430", "ti,omap3"; compatible = "openpandora,omap3-pandora-1ghz", "ti,omap3630", "ti,omap36xx", "ti,omap3"; But it is also missing a vendor prefix. Should I send a patch for both topics? Before the scheme is merged or afterwards? > + - nokia,omap3-n9 # Nokia N9 > + - nokia,omap3-n950 # Nokia N950 > + - lg,omap3-sniper # LG Optimus Black > + - ti,omap3-zoom3 # TI Zoom3 > + - const: ti,omap3630 > + - const: ti,omap3 > + > + - description: TI AM35 SoC based platforms > + items: > + - enum: > + - teejet,mt_ventoux # TeeJet Mt.Ventoux > + - ti,am3517-craneboard # TI AM3517 CraneBoard (TMDSEVM3517) > + - ti,am3517-evm # TI AM3517 EVM (AM3517/05 TMDSEVM3517) > + - compulab,omap3-sbc-t3517 # CompuLab SBC-T3517 with CM-T3517 > + - const: ti,am3517 > + - const: ti,omap3 > + > + - description: TI OMAP4430 SoC based platforms > + items: > + - enum: > + - motorola,droid4 # Motorola Droid 4 XT894 > + - motorola,droid-bionic # Motorola Droid Bionic XT875 > + - amazon,omap4-kc1 # Amazon Kindle Fire (first generation) > + - ti,omap4-panda # TI OMAP4 PandaBoard > + - ti,omap4-sdp # TI OMAP4 SDP board > + - const: ti,omap4430 > + - const: ti,omap4 > + > + - description: TI OMAP4460 SoC based platforms > + items: > + - enum: > + - ti,omap4-panda-es # TI OMAP4 PandaBoard-ES > + - const: ti,omap4460 > + - const: ti,omap4 > + > + - description: TI OMAP543 SoC based platforms s/TI OMAP543 SoC/TI OMAP5 SoC/ > + items: > + - enum: > + - compulab,omap5-cm-t54 # CompuLab CM-T54 > + - isee,omap5-igep0050 # IGEPv5 > + - ti,omap5-uevm # TI OMAP5 uEVM board > + - const: ti,omap5 > + > + - description: TI AM33 SoC based platforms > + items: > + - enum: > + - ti,am335x-bone # TI AM335x BeagleBone > + - compulab,cm-t335 # CompuLab CM-T335 > + - ti,am335x-evm # TI AM335x EVM > + - ti,am335x-evmsk # TI AM335x EVM-SK > + - bosch,am335x-guardian # Bosch AM335x Guardian > + - ti,am3359-icev2 # TI AM3359 ICE-V2 > + - novatech,am335x-lxm # NovaTech OrionLXm > + - moxa,uc-2101 # Moxa UC-2101 > + - moxa,uc-8100-me-t # Moxa UC-8100-ME-T > + - gumstix,am335x-pepper # Gumstix Pepper > + - tcl,am335x-sl50 # Toby Churchill SL50 Series > + - pattern: '^ti,am33(5[1246789]|xx)$' # ti,am33xx is legacy please use full SoC name > + > + - description: TI AM43 SoC based platforms > + items: > + - enum: > + - compulab,am437x-cm-t43 # CompuLab CM-T43 > + - ti,am437x-gp-evm # TI AM437x GP EVM > + - ti,am437x-idk-evm # TI AM437x Industrial Development Kit > + - ti,am437x-sk-evm # TI AM437x SK EVM > + - pattern: '^ti,am4372[26789]$' > + - const: ti,am43 > + > + - description: TI AM57 SoC based platforms > + items: > + - enum: > + - beagle,am5729-beagleboneai # BeagleBoard.org BeagleBone AI > + - compulab,cl-som-am57x # CompuLab CL-SOM-AM57x > + - ti,am5718-idk # TI AM5718 IDK > + - ti,am5728-idk # TI AM5728 IDK > + - ti,am5748-idk # TI AM5748 IDK > + - pattern: '^ti,am57[0124][689]$' > + - const: ti,am57 > + > + - description: TI DRA7 SoC based platforms > + items: > + - enum: > + - ti,dra718-evm # TI DRA718 EVM > + - ti,dra722-evm # TI DRA722 EVM > + - ti,dra742-evm # TI DRA742 EVM > + - ti,dra762-evm # TI DRA762 EVM > + - pattern: '^ti,dra7[12456][024568p]$' > + - const: ti,dra7 > + > +additionalProperties: true > + > +examples: > + - | > + / { > + model = "TI OMAP2430 SDP (Software Development Board)"; > + compatible = "ti,omap2430-sdp", "ti,omap2430", "ti,omap2"; > + > + #address-cells = <1>; > + #size-cells = <1>; > + }; > + > + - | > + / { > + model = "TI DRA762 EVM"; > + compatible = "ti,dra762-evm", "ti,dra762", "ti,dra7"; > + > + #address-cells = <1>; > + #size-cells = <1>; > + }; > + > +... > -- > 2.39.1 > BR, Nikolaus ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml 2023-02-16 16:08 ` H. Nikolaus Schaller @ 2023-02-16 16:19 ` Andrew Davis 2023-02-16 16:43 ` H. Nikolaus Schaller 0 siblings, 1 reply; 14+ messages in thread From: Andrew Davis @ 2023-02-16 16:19 UTC (permalink / raw) To: H. Nikolaus Schaller Cc: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann, OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, linux Kernel Mailing List, Linux-OMAP, Linux ARM, Discussions about the Letux Kernel On 2/16/23 10:08 AM, H. Nikolaus Schaller wrote: > Hi, > >> Am 16.02.2023 um 16:33 schrieb Andrew Davis <afd@ti.com>: >> >> Signed-off-by: Andrew Davis <afd@ti.com> >> --- >> .../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++ >> 1 file changed, 174 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml >> >> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.yaml b/Documentation/devicetree/bindings/arm/omap/omap.yaml >> new file mode 100644 >> index 0000000000000..cf07a7a7df279 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/arm/omap/omap.yaml >> @@ -0,0 +1,174 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/arm/omap/omap.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Texas Instruments OMAP SoC architecture device tree bindings >> + >> +maintainers: >> + - Tony Lindgren <tony@atomide.com> >> + >> +description: | >> + Platforms based on Texas Instruments OMAP SoC architecture. >> + >> +properties: >> + $nodename: >> + const: '/' >> + compatible: >> + oneOf: >> + >> + - description: TI OMAP2420 SoC based platforms >> + items: >> + - enum: >> + - ti,omap2420-h4 # TI OMAP2420 H4 board >> + - nokia,n800 # Nokia N800 >> + - nokia,n810 # Nokia N810 >> + - nokia,n810-wimax # Nokia N810 WiMax >> + - const: ti,omap2420 >> + - const: ti,omap2 >> + >> + - description: TI OMAP2430 SoC based platforms >> + items: >> + - enum: >> + - ti,omap2430-sdp # TI OMAP2430 SDP >> + - const: ti,omap2430 >> + - const: ti,omap2 >> + >> + - description: TI OMAP3430 SoC based platforms >> + items: >> + - enum: >> + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom OMAP35xx SOM-LV Development Kit >> + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit >> + - ti,omap3430-sdp # TI OMAP3430 SDP >> + - ti,omap3-beagle # TI OMAP3 BeagleBoard >> + - compulab,omap3-cm-t3530 # CompuLab CM-T3530 >> + - timll,omap3-devkit8000 # TimLL OMAP3 Devkit8000 >> + - ti,omap3-evm # TI OMAP35XX EVM (TMDSEVM3530) >> + - ti,omap3-ldp # TI OMAP3430 LDP (Zoom1 Labrador) >> + - nokia,omap3-n900 # Nokia N900 >> + - const: ti,omap3430 >> + - const: ti,omap3 >> + >> + - description: TI OMAP3630 SoC based platforms >> + items: >> + - enum: >> + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom DM3730 SOM-LV Development Kit >> + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit >> + - ti,omap3-beagle-xm # TI OMAP3 BeagleBoard xM >> + - compulab,omap3-cm-t3730 # CompuLab CM-T3730 >> + - amazon,omap3-echo # Amazon Echo (first generation) >> + - ti,omap3-evm-37xx # TI OMAP37XX EVM (TMDSEVM3730) >> + - ti,omap3-gta04 # OMAP3 GTA04 > > precisely the GTA04 was not a TI product, but from Golden Delicious Computers. > It got this compatible because it originally shared a lot of device tree with the ti,omap3-beagle-xm. > Well, we could add a vendor prefix ("goldelico") and update the device trees to "goldelico,gta04". > > Generally, there is also the OpenPandora with both OMAP3 variants (3430 and 3630): > > compatible = "openpandora,omap3-pandora-600mhz", "ti,omap3430", "ti,omap3"; > compatible = "openpandora,omap3-pandora-1ghz", "ti,omap3630", "ti,omap36xx", "ti,omap3"; > > But it is also missing a vendor prefix. Should I send a patch for both topics? > Before the scheme is merged or afterwards? > I'd suggest we get as many fixes into the actual DT files before the schemas are solidified. Doing it after would mean changes in two places vs one. >> + - nokia,omap3-n9 # Nokia N9 >> + - nokia,omap3-n950 # Nokia N950 >> + - lg,omap3-sniper # LG Optimus Black >> + - ti,omap3-zoom3 # TI Zoom3 >> + - const: ti,omap3630 >> + - const: ti,omap3 >> + >> + - description: TI AM35 SoC based platforms >> + items: >> + - enum: >> + - teejet,mt_ventoux # TeeJet Mt.Ventoux >> + - ti,am3517-craneboard # TI AM3517 CraneBoard (TMDSEVM3517) >> + - ti,am3517-evm # TI AM3517 EVM (AM3517/05 TMDSEVM3517) >> + - compulab,omap3-sbc-t3517 # CompuLab SBC-T3517 with CM-T3517 >> + - const: ti,am3517 >> + - const: ti,omap3 >> + >> + - description: TI OMAP4430 SoC based platforms >> + items: >> + - enum: >> + - motorola,droid4 # Motorola Droid 4 XT894 >> + - motorola,droid-bionic # Motorola Droid Bionic XT875 >> + - amazon,omap4-kc1 # Amazon Kindle Fire (first generation) >> + - ti,omap4-panda # TI OMAP4 PandaBoard >> + - ti,omap4-sdp # TI OMAP4 SDP board >> + - const: ti,omap4430 >> + - const: ti,omap4 >> + >> + - description: TI OMAP4460 SoC based platforms >> + items: >> + - enum: >> + - ti,omap4-panda-es # TI OMAP4 PandaBoard-ES >> + - const: ti,omap4460 >> + - const: ti,omap4 >> + >> + - description: TI OMAP543 SoC based platforms > > s/TI OMAP543 SoC/TI OMAP5 SoC/ > ACK Andrew >> + items: >> + - enum: >> + - compulab,omap5-cm-t54 # CompuLab CM-T54 >> + - isee,omap5-igep0050 # IGEPv5 >> + - ti,omap5-uevm # TI OMAP5 uEVM board >> + - const: ti,omap5 >> + >> + - description: TI AM33 SoC based platforms >> + items: >> + - enum: >> + - ti,am335x-bone # TI AM335x BeagleBone >> + - compulab,cm-t335 # CompuLab CM-T335 >> + - ti,am335x-evm # TI AM335x EVM >> + - ti,am335x-evmsk # TI AM335x EVM-SK >> + - bosch,am335x-guardian # Bosch AM335x Guardian >> + - ti,am3359-icev2 # TI AM3359 ICE-V2 >> + - novatech,am335x-lxm # NovaTech OrionLXm >> + - moxa,uc-2101 # Moxa UC-2101 >> + - moxa,uc-8100-me-t # Moxa UC-8100-ME-T >> + - gumstix,am335x-pepper # Gumstix Pepper >> + - tcl,am335x-sl50 # Toby Churchill SL50 Series >> + - pattern: '^ti,am33(5[1246789]|xx)$' # ti,am33xx is legacy please use full SoC name >> + >> + - description: TI AM43 SoC based platforms >> + items: >> + - enum: >> + - compulab,am437x-cm-t43 # CompuLab CM-T43 >> + - ti,am437x-gp-evm # TI AM437x GP EVM >> + - ti,am437x-idk-evm # TI AM437x Industrial Development Kit >> + - ti,am437x-sk-evm # TI AM437x SK EVM >> + - pattern: '^ti,am4372[26789]$' >> + - const: ti,am43 >> + >> + - description: TI AM57 SoC based platforms >> + items: >> + - enum: >> + - beagle,am5729-beagleboneai # BeagleBoard.org BeagleBone AI >> + - compulab,cl-som-am57x # CompuLab CL-SOM-AM57x >> + - ti,am5718-idk # TI AM5718 IDK >> + - ti,am5728-idk # TI AM5728 IDK >> + - ti,am5748-idk # TI AM5748 IDK >> + - pattern: '^ti,am57[0124][689]$' >> + - const: ti,am57 >> + >> + - description: TI DRA7 SoC based platforms >> + items: >> + - enum: >> + - ti,dra718-evm # TI DRA718 EVM >> + - ti,dra722-evm # TI DRA722 EVM >> + - ti,dra742-evm # TI DRA742 EVM >> + - ti,dra762-evm # TI DRA762 EVM >> + - pattern: '^ti,dra7[12456][024568p]$' >> + - const: ti,dra7 >> + >> +additionalProperties: true >> + >> +examples: >> + - | >> + / { >> + model = "TI OMAP2430 SDP (Software Development Board)"; >> + compatible = "ti,omap2430-sdp", "ti,omap2430", "ti,omap2"; >> + >> + #address-cells = <1>; >> + #size-cells = <1>; >> + }; >> + >> + - | >> + / { >> + model = "TI DRA762 EVM"; >> + compatible = "ti,dra762-evm", "ti,dra762", "ti,dra7"; >> + >> + #address-cells = <1>; >> + #size-cells = <1>; >> + }; >> + >> +... >> -- >> 2.39.1 >> > > BR, > Nikolaus > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml 2023-02-16 16:19 ` Andrew Davis @ 2023-02-16 16:43 ` H. Nikolaus Schaller 0 siblings, 0 replies; 14+ messages in thread From: H. Nikolaus Schaller @ 2023-02-16 16:43 UTC (permalink / raw) To: Andrew Davis Cc: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann, OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, linux Kernel Mailing List, Linux-OMAP, Linux ARM, Discussions about the Letux Kernel Hi Andrew, > Am 16.02.2023 um 17:19 schrieb Andrew Davis <afd@ti.com>: > > On 2/16/23 10:08 AM, H. Nikolaus Schaller wrote: >> Hi, >>> Am 16.02.2023 um 16:33 schrieb Andrew Davis <afd@ti.com>: >>> >>> Signed-off-by: Andrew Davis <afd@ti.com> >>> --- >>> .../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++ >>> 1 file changed, 174 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml >>> >>> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.yaml b/Documentation/devicetree/bindings/arm/omap/omap.yaml >>> new file mode 100644 >>> index 0000000000000..cf07a7a7df279 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/arm/omap/omap.yaml >>> @@ -0,0 +1,174 @@ >>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/arm/omap/omap.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: Texas Instruments OMAP SoC architecture device tree bindings >>> + >>> +maintainers: >>> + - Tony Lindgren <tony@atomide.com> >>> + >>> +description: | >>> + Platforms based on Texas Instruments OMAP SoC architecture. >>> + >>> +properties: >>> + $nodename: >>> + const: '/' >>> + compatible: >>> + oneOf: >>> + >>> + - description: TI OMAP2420 SoC based platforms >>> + items: >>> + - enum: >>> + - ti,omap2420-h4 # TI OMAP2420 H4 board >>> + - nokia,n800 # Nokia N800 >>> + - nokia,n810 # Nokia N810 >>> + - nokia,n810-wimax # Nokia N810 WiMax >>> + - const: ti,omap2420 >>> + - const: ti,omap2 >>> + >>> + - description: TI OMAP2430 SoC based platforms >>> + items: >>> + - enum: >>> + - ti,omap2430-sdp # TI OMAP2430 SDP >>> + - const: ti,omap2430 >>> + - const: ti,omap2 >>> + >>> + - description: TI OMAP3430 SoC based platforms >>> + items: >>> + - enum: >>> + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom OMAP35xx SOM-LV Development Kit >>> + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit >>> + - ti,omap3430-sdp # TI OMAP3430 SDP >>> + - ti,omap3-beagle # TI OMAP3 BeagleBoard >>> + - compulab,omap3-cm-t3530 # CompuLab CM-T3530 >>> + - timll,omap3-devkit8000 # TimLL OMAP3 Devkit8000 >>> + - ti,omap3-evm # TI OMAP35XX EVM (TMDSEVM3530) >>> + - ti,omap3-ldp # TI OMAP3430 LDP (Zoom1 Labrador) >>> + - nokia,omap3-n900 # Nokia N900 >>> + - const: ti,omap3430 >>> + - const: ti,omap3 >>> + >>> + - description: TI OMAP3630 SoC based platforms >>> + items: >>> + - enum: >>> + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom DM3730 SOM-LV Development Kit >>> + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit >>> + - ti,omap3-beagle-xm # TI OMAP3 BeagleBoard xM >>> + - compulab,omap3-cm-t3730 # CompuLab CM-T3730 >>> + - amazon,omap3-echo # Amazon Echo (first generation) >>> + - ti,omap3-evm-37xx # TI OMAP37XX EVM (TMDSEVM3730) >>> + - ti,omap3-gta04 # OMAP3 GTA04 >> precisely the GTA04 was not a TI product, but from Golden Delicious Computers. >> It got this compatible because it originally shared a lot of device tree with the ti,omap3-beagle-xm. >> Well, we could add a vendor prefix ("goldelico") and update the device trees to "goldelico,gta04". >> Generally, there is also the OpenPandora with both OMAP3 variants (3430 and 3630): >> compatible = "openpandora,omap3-pandora-600mhz", "ti,omap3430", "ti,omap3"; >> compatible = "openpandora,omap3-pandora-1ghz", "ti,omap3630", "ti,omap36xx", "ti,omap3"; >> But it is also missing a vendor prefix. Should I send a patch for both topics? >> Before the scheme is merged or afterwards? > > I'd suggest we get as many fixes into the actual DT files before the schemas > are solidified. Doing it after would mean changes in two places vs one. Ok, I'll prepare and send a patch set asap. BR and thanks, Nikolaus ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml 2023-02-16 15:33 ` [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml Andrew Davis 2023-02-16 16:08 ` H. Nikolaus Schaller @ 2023-02-16 16:41 ` Rob Herring 2023-02-16 17:30 ` Andrew Davis 1 sibling, 1 reply; 14+ messages in thread From: Rob Herring @ 2023-02-16 16:41 UTC (permalink / raw) To: Andrew Davis Cc: Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel On Thu, Feb 16, 2023 at 9:34 AM Andrew Davis <afd@ti.com> wrote: > > Signed-off-by: Andrew Davis <afd@ti.com> Thanks for posting this! > --- > .../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++ > 1 file changed, 174 insertions(+) > create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml Move this to arm/ti,omap.yaml or arm/ti.yaml. The rest of omap/ dir should get moved elsewhere eventually. > > diff --git a/Documentation/devicetree/bindings/arm/omap/omap.yaml b/Documentation/devicetree/bindings/arm/omap/omap.yaml > new file mode 100644 > index 0000000000000..cf07a7a7df279 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/omap/omap.yaml > @@ -0,0 +1,174 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/arm/omap/omap.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Texas Instruments OMAP SoC architecture device tree bindings Drop 'device tree bindings' > + > +maintainers: > + - Tony Lindgren <tony@atomide.com> > + > +description: | Drop '|' > + Platforms based on Texas Instruments OMAP SoC architecture. > + > +properties: > + $nodename: > + const: '/' > + compatible: > + oneOf: > + > + - description: TI OMAP2420 SoC based platforms > + items: > + - enum: > + - ti,omap2420-h4 # TI OMAP2420 H4 board > + - nokia,n800 # Nokia N800 > + - nokia,n810 # Nokia N810 > + - nokia,n810-wimax # Nokia N810 WiMax A bit more whitespace before the comments would be nice. Personally I don't see much value in pretty names that just reformat the compatible string, but either way is fine. > + - const: ti,omap2420 > + - const: ti,omap2 > + > + - description: TI OMAP2430 SoC based platforms > + items: > + - enum: > + - ti,omap2430-sdp # TI OMAP2430 SDP > + - const: ti,omap2430 > + - const: ti,omap2 > + > + - description: TI OMAP3430 SoC based platforms > + items: > + - enum: > + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom OMAP35xx SOM-LV Development Kit > + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit > + - ti,omap3430-sdp # TI OMAP3430 SDP > + - ti,omap3-beagle # TI OMAP3 BeagleBoard > + - compulab,omap3-cm-t3530 # CompuLab CM-T3530 > + - timll,omap3-devkit8000 # TimLL OMAP3 Devkit8000 > + - ti,omap3-evm # TI OMAP35XX EVM (TMDSEVM3530) > + - ti,omap3-ldp # TI OMAP3430 LDP (Zoom1 Labrador) > + - nokia,omap3-n900 # Nokia N900 > + - const: ti,omap3430 > + - const: ti,omap3 > + > + - description: TI OMAP3630 SoC based platforms > + items: > + - enum: > + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom DM3730 SOM-LV Development Kit > + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit > + - ti,omap3-beagle-xm # TI OMAP3 BeagleBoard xM > + - compulab,omap3-cm-t3730 # CompuLab CM-T3730 > + - amazon,omap3-echo # Amazon Echo (first generation) > + - ti,omap3-evm-37xx # TI OMAP37XX EVM (TMDSEVM3730) > + - ti,omap3-gta04 # OMAP3 GTA04 > + - nokia,omap3-n9 # Nokia N9 > + - nokia,omap3-n950 # Nokia N950 > + - lg,omap3-sniper # LG Optimus Black > + - ti,omap3-zoom3 # TI Zoom3 > + - const: ti,omap3630 > + - const: ti,omap3 > + > + - description: TI AM35 SoC based platforms > + items: > + - enum: > + - teejet,mt_ventoux # TeeJet Mt.Ventoux > + - ti,am3517-craneboard # TI AM3517 CraneBoard (TMDSEVM3517) > + - ti,am3517-evm # TI AM3517 EVM (AM3517/05 TMDSEVM3517) > + - compulab,omap3-sbc-t3517 # CompuLab SBC-T3517 with CM-T3517 > + - const: ti,am3517 > + - const: ti,omap3 > + > + - description: TI OMAP4430 SoC based platforms > + items: > + - enum: > + - motorola,droid4 # Motorola Droid 4 XT894 > + - motorola,droid-bionic # Motorola Droid Bionic XT875 > + - amazon,omap4-kc1 # Amazon Kindle Fire (first generation) > + - ti,omap4-panda # TI OMAP4 PandaBoard > + - ti,omap4-sdp # TI OMAP4 SDP board > + - const: ti,omap4430 > + - const: ti,omap4 > + > + - description: TI OMAP4460 SoC based platforms > + items: > + - enum: > + - ti,omap4-panda-es # TI OMAP4 PandaBoard-ES > + - const: ti,omap4460 > + - const: ti,omap4 > + > + - description: TI OMAP543 SoC based platforms > + items: > + - enum: > + - compulab,omap5-cm-t54 # CompuLab CM-T54 > + - isee,omap5-igep0050 # IGEPv5 > + - ti,omap5-uevm # TI OMAP5 uEVM board > + - const: ti,omap5 > + > + - description: TI AM33 SoC based platforms > + items: > + - enum: > + - ti,am335x-bone # TI AM335x BeagleBone > + - compulab,cm-t335 # CompuLab CM-T335 > + - ti,am335x-evm # TI AM335x EVM > + - ti,am335x-evmsk # TI AM335x EVM-SK > + - bosch,am335x-guardian # Bosch AM335x Guardian > + - ti,am3359-icev2 # TI AM3359 ICE-V2 > + - novatech,am335x-lxm # NovaTech OrionLXm > + - moxa,uc-2101 # Moxa UC-2101 > + - moxa,uc-8100-me-t # Moxa UC-8100-ME-T > + - gumstix,am335x-pepper # Gumstix Pepper > + - tcl,am335x-sl50 # Toby Churchill SL50 Series > + - pattern: '^ti,am33(5[1246789]|xx)$' # ti,am33xx is legacy please use full SoC name > + > + - description: TI AM43 SoC based platforms > + items: > + - enum: > + - compulab,am437x-cm-t43 # CompuLab CM-T43 > + - ti,am437x-gp-evm # TI AM437x GP EVM > + - ti,am437x-idk-evm # TI AM437x Industrial Development Kit > + - ti,am437x-sk-evm # TI AM437x SK EVM > + - pattern: '^ti,am4372[26789]$' > + - const: ti,am43 > + > + - description: TI AM57 SoC based platforms > + items: > + - enum: > + - beagle,am5729-beagleboneai # BeagleBoard.org BeagleBone AI > + - compulab,cl-som-am57x # CompuLab CL-SOM-AM57x > + - ti,am5718-idk # TI AM5718 IDK > + - ti,am5728-idk # TI AM5728 IDK > + - ti,am5748-idk # TI AM5748 IDK > + - pattern: '^ti,am57[0124][689]$' > + - const: ti,am57 > + > + - description: TI DRA7 SoC based platforms > + items: > + - enum: > + - ti,dra718-evm # TI DRA718 EVM > + - ti,dra722-evm # TI DRA722 EVM > + - ti,dra742-evm # TI DRA742 EVM > + - ti,dra762-evm # TI DRA762 EVM > + - pattern: '^ti,dra7[12456][024568p]$' > + - const: ti,dra7 > + > +additionalProperties: true > + > +examples: > + - | > + / { > + model = "TI OMAP2430 SDP (Software Development Board)"; > + compatible = "ti,omap2430-sdp", "ti,omap2430", "ti,omap2"; > + > + #address-cells = <1>; > + #size-cells = <1>; > + }; > + > + - | > + / { You can't have 2 root node examples. Well, you can, but they are just merged together... We normally don't have examples for these either. > + model = "TI DRA762 EVM"; > + compatible = "ti,dra762-evm", "ti,dra762", "ti,dra7"; > + > + #address-cells = <1>; > + #size-cells = <1>; > + }; > + > +... > -- > 2.39.1 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml 2023-02-16 16:41 ` Rob Herring @ 2023-02-16 17:30 ` Andrew Davis 0 siblings, 0 replies; 14+ messages in thread From: Andrew Davis @ 2023-02-16 17:30 UTC (permalink / raw) To: Rob Herring Cc: Krzysztof Kozlowski, Benoît Cousson, Tony Lindgren, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel On 2/16/23 10:41 AM, Rob Herring wrote: > On Thu, Feb 16, 2023 at 9:34 AM Andrew Davis <afd@ti.com> wrote: >> >> Signed-off-by: Andrew Davis <afd@ti.com> > > Thanks for posting this! > >> --- >> .../devicetree/bindings/arm/omap/omap.yaml | 174 ++++++++++++++++++ >> 1 file changed, 174 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/arm/omap/omap.yaml > > Move this to arm/ti,omap.yaml or arm/ti.yaml. The rest of omap/ dir > should get moved elsewhere eventually. > ACK >> >> diff --git a/Documentation/devicetree/bindings/arm/omap/omap.yaml b/Documentation/devicetree/bindings/arm/omap/omap.yaml >> new file mode 100644 >> index 0000000000000..cf07a7a7df279 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/arm/omap/omap.yaml >> @@ -0,0 +1,174 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/arm/omap/omap.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Texas Instruments OMAP SoC architecture device tree bindings > > Drop 'device tree bindings' > ACK >> + >> +maintainers: >> + - Tony Lindgren <tony@atomide.com> >> + >> +description: | > > Drop '|' > ACK >> + Platforms based on Texas Instruments OMAP SoC architecture. >> + >> +properties: >> + $nodename: >> + const: '/' >> + compatible: >> + oneOf: >> + >> + - description: TI OMAP2420 SoC based platforms >> + items: >> + - enum: >> + - ti,omap2420-h4 # TI OMAP2420 H4 board >> + - nokia,n800 # Nokia N800 >> + - nokia,n810 # Nokia N810 >> + - nokia,n810-wimax # Nokia N810 WiMax > > A bit more whitespace before the comments would be nice. Personally I > don't see much value in pretty names that just reformat the compatible > string, but either way is fine. > I'm not a fan either, but this at least gives them all something to be consistent. Will align them for v2. >> + - const: ti,omap2420 >> + - const: ti,omap2 >> + >> + - description: TI OMAP2430 SoC based platforms >> + items: >> + - enum: >> + - ti,omap2430-sdp # TI OMAP2430 SDP >> + - const: ti,omap2430 >> + - const: ti,omap2 >> + >> + - description: TI OMAP3430 SoC based platforms >> + items: >> + - enum: >> + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom OMAP35xx SOM-LV Development Kit >> + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit >> + - ti,omap3430-sdp # TI OMAP3430 SDP >> + - ti,omap3-beagle # TI OMAP3 BeagleBoard >> + - compulab,omap3-cm-t3530 # CompuLab CM-T3530 >> + - timll,omap3-devkit8000 # TimLL OMAP3 Devkit8000 >> + - ti,omap3-evm # TI OMAP35XX EVM (TMDSEVM3530) >> + - ti,omap3-ldp # TI OMAP3430 LDP (Zoom1 Labrador) >> + - nokia,omap3-n900 # Nokia N900 >> + - const: ti,omap3430 >> + - const: ti,omap3 >> + >> + - description: TI OMAP3630 SoC based platforms >> + items: >> + - enum: >> + - logicpd,dm3730-som-lv-devkit # LogicPD Zoom DM3730 SOM-LV Development Kit >> + - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit >> + - ti,omap3-beagle-xm # TI OMAP3 BeagleBoard xM >> + - compulab,omap3-cm-t3730 # CompuLab CM-T3730 >> + - amazon,omap3-echo # Amazon Echo (first generation) >> + - ti,omap3-evm-37xx # TI OMAP37XX EVM (TMDSEVM3730) >> + - ti,omap3-gta04 # OMAP3 GTA04 >> + - nokia,omap3-n9 # Nokia N9 >> + - nokia,omap3-n950 # Nokia N950 >> + - lg,omap3-sniper # LG Optimus Black >> + - ti,omap3-zoom3 # TI Zoom3 >> + - const: ti,omap3630 >> + - const: ti,omap3 >> + >> + - description: TI AM35 SoC based platforms >> + items: >> + - enum: >> + - teejet,mt_ventoux # TeeJet Mt.Ventoux >> + - ti,am3517-craneboard # TI AM3517 CraneBoard (TMDSEVM3517) >> + - ti,am3517-evm # TI AM3517 EVM (AM3517/05 TMDSEVM3517) >> + - compulab,omap3-sbc-t3517 # CompuLab SBC-T3517 with CM-T3517 >> + - const: ti,am3517 >> + - const: ti,omap3 >> + >> + - description: TI OMAP4430 SoC based platforms >> + items: >> + - enum: >> + - motorola,droid4 # Motorola Droid 4 XT894 >> + - motorola,droid-bionic # Motorola Droid Bionic XT875 >> + - amazon,omap4-kc1 # Amazon Kindle Fire (first generation) >> + - ti,omap4-panda # TI OMAP4 PandaBoard >> + - ti,omap4-sdp # TI OMAP4 SDP board >> + - const: ti,omap4430 >> + - const: ti,omap4 >> + >> + - description: TI OMAP4460 SoC based platforms >> + items: >> + - enum: >> + - ti,omap4-panda-es # TI OMAP4 PandaBoard-ES >> + - const: ti,omap4460 >> + - const: ti,omap4 >> + >> + - description: TI OMAP543 SoC based platforms >> + items: >> + - enum: >> + - compulab,omap5-cm-t54 # CompuLab CM-T54 >> + - isee,omap5-igep0050 # IGEPv5 >> + - ti,omap5-uevm # TI OMAP5 uEVM board >> + - const: ti,omap5 >> + >> + - description: TI AM33 SoC based platforms >> + items: >> + - enum: >> + - ti,am335x-bone # TI AM335x BeagleBone >> + - compulab,cm-t335 # CompuLab CM-T335 >> + - ti,am335x-evm # TI AM335x EVM >> + - ti,am335x-evmsk # TI AM335x EVM-SK >> + - bosch,am335x-guardian # Bosch AM335x Guardian >> + - ti,am3359-icev2 # TI AM3359 ICE-V2 >> + - novatech,am335x-lxm # NovaTech OrionLXm >> + - moxa,uc-2101 # Moxa UC-2101 >> + - moxa,uc-8100-me-t # Moxa UC-8100-ME-T >> + - gumstix,am335x-pepper # Gumstix Pepper >> + - tcl,am335x-sl50 # Toby Churchill SL50 Series >> + - pattern: '^ti,am33(5[1246789]|xx)$' # ti,am33xx is legacy please use full SoC name >> + >> + - description: TI AM43 SoC based platforms >> + items: >> + - enum: >> + - compulab,am437x-cm-t43 # CompuLab CM-T43 >> + - ti,am437x-gp-evm # TI AM437x GP EVM >> + - ti,am437x-idk-evm # TI AM437x Industrial Development Kit >> + - ti,am437x-sk-evm # TI AM437x SK EVM >> + - pattern: '^ti,am4372[26789]$' >> + - const: ti,am43 >> + >> + - description: TI AM57 SoC based platforms >> + items: >> + - enum: >> + - beagle,am5729-beagleboneai # BeagleBoard.org BeagleBone AI >> + - compulab,cl-som-am57x # CompuLab CL-SOM-AM57x >> + - ti,am5718-idk # TI AM5718 IDK >> + - ti,am5728-idk # TI AM5728 IDK >> + - ti,am5748-idk # TI AM5748 IDK >> + - pattern: '^ti,am57[0124][689]$' >> + - const: ti,am57 >> + >> + - description: TI DRA7 SoC based platforms >> + items: >> + - enum: >> + - ti,dra718-evm # TI DRA718 EVM >> + - ti,dra722-evm # TI DRA722 EVM >> + - ti,dra742-evm # TI DRA742 EVM >> + - ti,dra762-evm # TI DRA762 EVM >> + - pattern: '^ti,dra7[12456][024568p]$' >> + - const: ti,dra7 >> + >> +additionalProperties: true >> + >> +examples: >> + - | >> + / { >> + model = "TI OMAP2430 SDP (Software Development Board)"; >> + compatible = "ti,omap2430-sdp", "ti,omap2430", "ti,omap2"; >> + >> + #address-cells = <1>; >> + #size-cells = <1>; >> + }; >> + >> + - | >> + / { > > You can't have 2 root node examples. Well, you can, but they are just > merged together... > > We normally don't have examples for these either. > Added just for my personal testing, as testing on the real DTBs gives far too many other errors :) Will drop them for v2. Thanks, Andrew >> + model = "TI DRA762 EVM"; >> + compatible = "ti,dra762-evm", "ti,dra762", "ti,dra7"; >> + >> + #address-cells = <1>; >> + #size-cells = <1>; >> + }; >> + >> +... >> -- >> 2.39.1 >> ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [RFC 0/2] Convert omap.txt to DT schema 2023-02-16 15:33 [RFC 0/2] Convert omap.txt to DT schema Andrew Davis 2023-02-16 15:33 ` [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible Andrew Davis 2023-02-16 15:33 ` [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml Andrew Davis @ 2023-02-17 5:48 ` Tony Lindgren 2 siblings, 0 replies; 14+ messages in thread From: Tony Lindgren @ 2023-02-17 5:48 UTC (permalink / raw) To: Andrew Davis Cc: Rob Herring, Krzysztof Kozlowski, Benoît Cousson, Colin Foster, Olof Johansson, Arnd Bergmann, devicetree, linux-kernel, linux-omap, linux-arm-kernel * Andrew Davis <afd@ti.com> [230216 15:33]: > I saw there was some more talk on the lists about converting > the omap.txt over to DT schema. I remembered that I had made > an attempt at this and wanted to post it as a WIP example. > It is not perfect or even very good, but might serve as > a base for someone. Hey that's great :) Tony ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2023-03-28 7:27 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-02-16 15:33 [RFC 0/2] Convert omap.txt to DT schema Andrew Davis 2023-02-16 15:33 ` [RFC 1/2] ARM: dts: omap: Drop ti,omap36xx compatible Andrew Davis 2023-03-27 19:18 ` Andreas Kemnade 2023-03-28 5:01 ` Tony Lindgren 2023-03-28 7:06 ` Andreas Kemnade 2023-03-28 7:08 ` Tony Lindgren 2023-03-28 7:27 ` Tony Lindgren 2023-02-16 15:33 ` [RFC 2/2] WIP: dt-bindings: omap: Convert omap.txt to yaml Andrew Davis 2023-02-16 16:08 ` H. Nikolaus Schaller 2023-02-16 16:19 ` Andrew Davis 2023-02-16 16:43 ` H. Nikolaus Schaller 2023-02-16 16:41 ` Rob Herring 2023-02-16 17:30 ` Andrew Davis 2023-02-17 5:48 ` [RFC 0/2] Convert omap.txt to DT schema Tony Lindgren
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).