[PATCH v7 0/2] dt-bindings: omap: Convert omap.txt to yaml

[PATCH v7 0/2] dt-bindings: omap: Convert omap.txt to yaml

[Andreas Kemnade]

Convert board compatibles to yaml and add the new yaml file to
MAINTAINERS so that emails are properly distributed

Changes in V7:
 - checked for lost compatibles
 - remove conversions with pattern matches probably allowing
   too much

Changes in V6:
 - reflect the rename also in the file header

Changes in V5:
 - renamed the new file to ti/omap.yaml

Changes in V4:
 - fix order 
 - re-add dra7 to .txt to have it sorted out later

Changes in V3:
 - update MAINTAINERS
 - remove converted stuff from .txt

Changes in V2:
- renamed file
- fixed gta04
- added Openpandora, Epson Moverio BT-200
- drop example
- remove descriptions if just reformatting the name

Andreas Kemnade (1):
  MAINTAINERS: add board bindings list to OMAP2+ files

Andrew Davis (1):
  dt-bindings: omap: Partially convert omap.txt to yaml

 .../devicetree/bindings/arm/omap/omap.txt     |  99 ----------
 .../devicetree/bindings/arm/ti/omap.yaml      | 176 ++++++++++++++++++
 MAINTAINERS                                   |   1 +
 3 files changed, 177 insertions(+), 99 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/arm/ti/omap.yaml

-- 
2.39.2

[PATCH v7 1/2] dt-bindings: omap: Partially convert omap.txt to yaml

[Andreas Kemnade]

From: Andrew Davis <afd@ti.com>

Convert omap.txt to yaml.

CC: linux-omap@vger.kernel.org
Signed-off-by: Andrew Davis <afd@ti.com>
[reduced to only OMAP3/4/5 and AM3, adding Epson Moverio BT-200]
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
 .../devicetree/bindings/arm/omap/omap.txt     |  99 ----------
 .../devicetree/bindings/arm/ti/omap.yaml      | 176 ++++++++++++++++++
 2 files changed, 176 insertions(+), 99 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/arm/ti/omap.yaml

diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt b/Documentation/devicetree/bindings/arm/omap/omap.txt
index fa8b31660cadd..c863ec07cbbb1 100644
--- a/Documentation/devicetree/bindings/arm/omap/omap.txt
+++ b/Documentation/devicetree/bindings/arm/omap/omap.txt
@@ -41,14 +41,6 @@ SoC Type (optional):
 
 SoC Families:
 
-- OMAP2 generic - defaults to OMAP2420
-  compatible = "ti,omap2"
-- OMAP3 generic
-  compatible = "ti,omap3"
-- OMAP4 generic - defaults to OMAP4430
-  compatible = "ti,omap4"
-- OMAP5 generic - defaults to OMAP5430
-  compatible = "ti,omap5"
 - DRA7 generic - defaults to DRA742
   compatible = "ti,dra7"
 - AM33x generic
@@ -58,32 +50,6 @@ SoC Families:
 
 SoCs:
 
-- OMAP2420
-  compatible = "ti,omap2420", "ti,omap2"
-- OMAP2430
-  compatible = "ti,omap2430", "ti,omap2"
-
-- OMAP3430
-  compatible = "ti,omap3430", "ti,omap3"
-  legacy: "ti,omap34xx" - please do not use any more
-- AM3517
-  compatible = "ti,am3517", "ti,omap3"
-- OMAP3630
-  compatible = "ti,omap3630", "ti,omap3"
-  legacy: "ti,omap36xx" - please do not use any more
-- AM335x
-  compatible = "ti,am33xx"
-
-- OMAP4430
-  compatible = "ti,omap4430", "ti,omap4"
-- OMAP4460
-  compatible = "ti,omap4460", "ti,omap4"
-
-- OMAP5430
-  compatible = "ti,omap5430", "ti,omap5"
-- OMAP5432
-  compatible = "ti,omap5432", "ti,omap5"
-
 - DRA762
   compatible = "ti,dra762", "ti,dra7"
 
@@ -116,65 +82,6 @@ SoCs:
 
 Boards (incomplete list of examples):
 
-- OMAP3 BeagleBoard : Low cost community board
-  compatible = "ti,omap3-beagle", "ti,omap3430", "ti,omap3"
-
-- OMAP3 BeagleBoard A to B4 : Early BeagleBoard revisions A to B4 with a timer quirk
-  compatible = "ti,omap3-beagle-ab4", "ti,omap3-beagle", "ti,omap3430", "ti,omap3"
-
-- OMAP3 Tobi with Overo : Commercial expansion board with daughter board
-  compatible = "gumstix,omap3-overo-tobi", "gumstix,omap3-overo", "ti,omap3430", "ti,omap3"
-
-- OMAP4 SDP : Software Development Board
-  compatible = "ti,omap4-sdp", "ti,omap4430", "ti,omap4"
-
-- OMAP4 PandaBoard : Low cost community board
-  compatible = "ti,omap4-panda", "ti,omap4430", "ti,omap4"
-
-- OMAP4 DuoVero with Parlor : Commercial expansion board with daughter board
-  compatible = "gumstix,omap4-duovero-parlor", "gumstix,omap4-duovero", "ti,omap4430", "ti,omap4";
-
-- OMAP4 VAR-STK-OM44 : Commercial dev kit with VAR-OM44CustomBoard and VAR-SOM-OM44 w/WLAN
-  compatible = "variscite,var-stk-om44", "variscite,var-som-om44", "ti,omap4460", "ti,omap4";
-
-- OMAP4 VAR-DVK-OM44 : Commercial dev kit with VAR-OM44CustomBoard, VAR-SOM-OM44 w/WLAN and LCD touchscreen
-  compatible = "variscite,var-dvk-om44", "variscite,var-som-om44", "ti,omap4460", "ti,omap4";
-
-- OMAP3 EVM : Software Development Board for OMAP35x, AM/DM37x
-  compatible = "ti,omap3-evm", "ti,omap3630", "ti,omap3"
-
-- AM335X EVM : Software Development Board for AM335x
-  compatible = "ti,am335x-evm", "ti,am33xx"
-
-- AM335X Bone : Low cost community board
-  compatible = "ti,am335x-bone", "ti,am33xx"
-
-- AM3359 ICEv2 : Low cost Industrial Communication Engine EVM.
-  compatible = "ti,am3359-icev2", "ti,am33xx"
-
-- AM335X OrionLXm : Substation Automation Platform
-  compatible = "novatech,am335x-lxm", "ti,am33xx"
-
-- AM335X phyBOARD-WEGA: Single Board Computer dev kit
-  compatible = "phytec,am335x-wega", "phytec,am335x-phycore-som", "ti,am33xx"
-
-- AM335X CM-T335 : System On Module, built around the Sitara AM3352/4
-  compatible = "compulab,cm-t335", "ti,am33xx"
-
-- AM335X SBC-T335 : single board computer, built around the Sitara AM3352/4
-  compatible = "compulab,sbc-t335", "compulab,cm-t335", "ti,am33xx"
-
-- AM335X phyCORE-AM335x: Development kit
-  compatible = "phytec,am335x-pcm-953", "phytec,am335x-phycore-som", "ti,am33xx"
-
-- AM335x phyBOARD-REGOR: Single Board Computer
-  compatible = "phytec,am335x-regor", "phytec,am335x-phycore-som", "ti,am33xx"
-
-- AM335X UC-8100-ME-T: Communication-centric industrial computing platform
-  compatible = "moxa,uc-8100-me-t", "ti,am33xx";
-
-- OMAP5 EVM : Evaluation Module
-  compatible = "ti,omap5-evm", "ti,omap5"
 
 - AM437x CM-T43
   compatible = "compulab,am437x-cm-t43", "ti,am4372", "ti,am43"
@@ -217,9 +124,3 @@ Boards (incomplete list of examples):
 
 - DRA718 EVM: Software Development Board for DRA718
   compatible = "ti,dra718-evm", "ti,dra718", "ti,dra722", "ti,dra72", "ti,dra7"
-
-- DM3730 Logic PD Torpedo + Wireless: Commercial System on Module with WiFi and Bluetooth
-  compatible = "logicpd,dm3730-torpedo-devkit", "ti,omap3630", "ti,omap3"
-
-- DM3730 Logic PD SOM-LV: Commercial System on Module with WiFi and Bluetooth
-  compatible = "logicpd,dm3730-som-lv-devkit", "ti,omap3630", "ti,omap3"
diff --git a/Documentation/devicetree/bindings/arm/ti/omap.yaml b/Documentation/devicetree/bindings/arm/ti/omap.yaml
new file mode 100644
index 0000000000000..b18fc046390a4
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/ti/omap.yaml
@@ -0,0 +1,176 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/ti/omap.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments OMAP SoC architecture
+
+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:
+              - nokia,n800
+              - nokia,n810
+              - nokia,n810-wimax
+              - ti,omap2420-h4
+          - 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:
+              - compulab,omap3-cm-t3530
+              - logicpd,dm3730-som-lv-devkit  # LogicPD Zoom OMAP35xx SOM-LV Development Kit
+              - logicpd,dm3730-torpedo-devkit # LogicPD Zoom OMAP35xx Torpedo Development Kit
+              - nokia,omap3-n900
+              - openpandora,omap3-pandora-600mhz
+              - ti,omap3430-sdp
+              - ti,omap3-beagle
+              - ti,omap3-evm                  # TI OMAP35XX EVM (TMDSEVM3530)
+              - ti,omap3-ldp                  # TI OMAP3430 LDP (Zoom1 Labrador)
+              - timll,omap3-devkit8000
+          - const: ti,omap3430
+          - const: ti,omap3
+
+      - description: Early BeagleBoard revisions A to B4 with a timer quirk
+        items:
+          - const: ti,omap3-beagle-ab4
+          - const: ti,omap3-beagle
+          - const: ti,omap3430
+          - const: ti,omap3
+
+      - description: Gumstix Overo TI OMAP 3430/3630 boards + expansion boards
+        items:
+          - enum:
+              - gumstix,omap3-overo-alto35
+              - gumstix,omap3-overo-chestnut43
+              - gumstix,omap3-overo-gallop43
+              - gumstix,omap3-overo-palo35
+              - gumstix,omap3-overo-palo43
+              - gumstix,omap3-overo-summit
+              - gumstix,omap3-overo-tobi
+              - gumstix,omap3-overo-tobiduo
+          - const: gumstix,omap3-overo
+          - enum:
+              - ti,omap3430
+              - ti,omap3630
+
+      - description: TI OMAP3630 SoC based platforms
+        items:
+          - enum:
+              - amazon,omap3-echo             # Amazon Echo (first generation)
+              - compulab,omap3-cm-t3730
+              - goldelico,gta04
+              - lg,omap3-sniper               # LG Optimus Black
+              - logicpd,dm3730-som-lv-devkit  # LogicPD Zoom DM3730 SOM-LV Development Kit
+              - logicpd,dm3730-torpedo-devkit # LogicPD Zoom DM3730 Torpedo + Wireless Development Kit
+              - nokia,omap3-n9
+              - nokia,omap3-n950
+              - openpandora,omap3-pandora-1ghz
+              - ti,omap3-beagle-xm
+              - ti,omap3-evm-37xx             # TI OMAP37XX EVM (TMDSEVM3730)
+              - ti,omap3-zoom3
+          - const: ti,omap3630
+          - const: ti,omap3
+
+      - description: TI AM35 SoC based platforms
+        items:
+          - enum:
+              - compulab,omap3-sbc-t3517 # CompuLab SBC-T3517 with CM-T3517
+              - teejet,mt_ventoux
+              - ti,am3517-craneboard     # TI AM3517 CraneBoard (TMDSEVM3517)
+              - ti,am3517-evm            # TI AM3517 EVM (AM3517/05 TMDSEVM3517)
+          - const: ti,am3517
+          - const: ti,omap3
+
+      - description: TI AM33 based platform
+        items:
+          - enum:
+              - compulab,cm-t335
+              - moxa,uc-8100-me-t
+              - novatech,am335x-lxm
+              - ti,am335x-bone
+              - ti,am335x-evm
+              - ti,am3359-icev2
+          - const: ti,am33xx
+
+      - description: Compulab board variants based on TI AM33
+        items:
+          - enum:
+              - compulab,sbc-t335
+          - const: compulab,cm-t335
+          - const: ti,am33xx
+
+      - description: Phytec boards based on TI AM33
+        items:
+          - enum:
+              - phytec,am335x-wega
+              - phytec,am335x-pcm-953
+              - phytec,am335x-regor
+          - const: phytec,am335x-phycore-som
+          - const: ti,am33xx
+
+      - description: TI OMAP4430 SoC based platforms
+        items:
+          - enum:
+              - amazon,omap4-kc1        # Amazon Kindle Fire (first generation)
+              - motorola,droid4         # Motorola Droid 4 XT894
+              - motorola,droid-bionic   # Motorola Droid Bionic XT875
+              - ti,omap4-panda
+              - ti,omap4-sdp
+          - const: ti,omap4430
+          - const: ti,omap4
+
+      - description: OMAP4 DuoVero with Parlor expansion board/daughter board
+        items:
+          - const: gumstix,omap4-duovero-parlor
+          - const: gumstix,omap4-duovero
+          - const: ti,omap4430
+          - const: ti,omap4
+
+      - description: TI OMAP4460 SoC based platforms
+        items:
+          - enum:
+              - epson,embt2ws    # Epson Moverio BT-200
+              - ti,omap4-panda-es
+          - const: ti,omap4460
+          - const: ti,omap4
+
+      - description: VAR-OM44 boards
+        items:
+          - enum:
+              - variscite,var-dvk-om44
+              - variscite,var-stk-om44
+          - const: variscite,var-som-om44
+          - const: ti,omap4460
+          - const: ti,omap4
+
+      - description: TI OMAP5 SoC based platforms
+        items:
+          - enum:
+              - compulab,omap5-cm-t54
+              - isee,omap5-igep0050
+              - ti,omap5-uevm
+          - const: ti,omap5
+
+additionalProperties: true
+
+...
-- 
2.39.2

Re: [PATCH v7 1/2] dt-bindings: omap: Partially convert omap.txt to yaml

[Krzysztof Kozlowski]

On 15/05/2023 09:45, Andreas Kemnade wrote:
> From: Andrew Davis <afd@ti.com>
> 
> Convert omap.txt to yaml.
> 
> CC: linux-omap@vger.kernel.org
> Signed-off-by: Andrew Davis <afd@ti.com>
> [reduced to only OMAP3/4/5 and AM3, adding Epson Moverio BT-200]
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
> ---


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

Re: [PATCH v7 1/2] dt-bindings: omap: Partially convert omap.txt to yaml

[Tony Lindgren]

* Andreas Kemnade <andreas@kemnade.info> [230515 07:45]:
> From: Andrew Davis <afd@ti.com>
> 
> Convert omap.txt to yaml.

Looks good to me, thanks a lot for doing this:

Reviewed-by: Tony Lindgren <tony@atomide.com>

Anreas, for future patch reference, care to summarize what's still
blocking updating the rest of the TI SoCs for the binding? I think
it may have been discussed earlier here and there already so apologies
if I'm the only one who lost track :)

Regards,

Tony

Re: [PATCH v7 1/2] dt-bindings: omap: Partially convert omap.txt to yaml

[Andreas Kemnade]

Hi Tony,

On Wed, 17 May 2023 09:40:31 +0300
Tony Lindgren <tony@atomide.com> wrote:

> * Andreas Kemnade <andreas@kemnade.info> [230515 07:45]:
> > From: Andrew Davis <afd@ti.com>
> > 
> > Convert omap.txt to yaml.  
> 
> Looks good to me, thanks a lot for doing this:
> 
> Reviewed-by: Tony Lindgren <tony@atomide.com>
> 
> Anreas, for future patch reference, care to summarize what's still
> blocking updating the rest of the TI SoCs for the binding? I think
> it may have been discussed earlier here and there already so apologies
> if I'm the only one who lost track :)
> 
The original patch by Andrew allows probably too many combinations of
compatibles for the rest of the SoCs (e.g.  pattern: '^ti,dra7[12456][024568p]).
I do not know the intention of that and nobody commented, so I would have to
do a lot of research, so I decided to split the work.

Regards,
Andreas

Re: [PATCH v7 1/2] dt-bindings: omap: Partially convert omap.txt to yaml

[Tony Lindgren]

* Andreas Kemnade <andreas@kemnade.info> [230517 08:10]:
> Hi Tony,
> 
> On Wed, 17 May 2023 09:40:31 +0300
> Tony Lindgren <tony@atomide.com> wrote:
> 
> > * Andreas Kemnade <andreas@kemnade.info> [230515 07:45]:
> > > From: Andrew Davis <afd@ti.com>
> > > 
> > > Convert omap.txt to yaml.  
> > 
> > Looks good to me, thanks a lot for doing this:
> > 
> > Reviewed-by: Tony Lindgren <tony@atomide.com>
> > 
> > Anreas, for future patch reference, care to summarize what's still
> > blocking updating the rest of the TI SoCs for the binding? I think
> > it may have been discussed earlier here and there already so apologies
> > if I'm the only one who lost track :)
> > 
> The original patch by Andrew allows probably too many combinations of
> compatibles for the rest of the SoCs (e.g.  pattern: '^ti,dra7[12456][024568p]).
> I do not know the intention of that and nobody commented, so I would have to
> do a lot of research, so I decided to split the work.

OK makes sense, thanks.

Tony

[PATCH v7 2/2] MAINTAINERS: add board bindings list to OMAP2+ files

[Andreas Kemnade]

Add list of OMAP2+ boards to the corresponding section

CC: linux-omap@vger.kernel.org
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index e0ad886d31632..3c85808bb2785 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15404,6 +15404,7 @@ W:	http://www.muru.com/linux/omap/
 W:	http://linux.omap.com/
 Q:	http://patchwork.kernel.org/project/linux-omap/list/
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap.git
+F:	Documentation/devicetree/bindings/arm/ti/omap.yaml
 F:	arch/arm/configs/omap2plus_defconfig
 F:	arch/arm/mach-omap2/
 F:	drivers/bus/ti-sysc.c
-- 
2.39.2

Re: [PATCH v7 2/2] MAINTAINERS: add board bindings list to OMAP2+ files

[Tony Lindgren]

* Andreas Kemnade <andreas@kemnade.info> [230515 07:45]:
> Add list of OMAP2+ boards to the corresponding section

Reviewed-by: Tony Lindgren <tony@atomide.com>

Re: [PATCH v7 2/2] MAINTAINERS: add board bindings list to OMAP2+ files

[Tony Lindgren]

* Andreas Kemnade <andreas@kemnade.info> [230515 07:45]:
> Add list of OMAP2+ boards to the corresponding section

Applying this patch into omap-for-v6.6/maintainers thanks.

Tony

Re: [PATCH v7 0/2] dt-bindings: omap: Convert omap.txt to yaml

[Andreas Kemnade]

Hi,

any action still expected from my side?
people gave R-bys...
So looks like it is ready for the dt-folks to pick it up.

Regards,
Andreas

On Mon, 15 May 2023 09:45:10 +0200
Andreas Kemnade <andreas@kemnade.info> wrote:

> Convert board compatibles to yaml and add the new yaml file to
> MAINTAINERS so that emails are properly distributed
> 
> Changes in V7:
>  - checked for lost compatibles
>  - remove conversions with pattern matches probably allowing
>    too much
> 
> Changes in V6:
>  - reflect the rename also in the file header
> 
> Changes in V5:
>  - renamed the new file to ti/omap.yaml
> 
> Changes in V4:
>  - fix order 
>  - re-add dra7 to .txt to have it sorted out later
> 
> Changes in V3:
>  - update MAINTAINERS
>  - remove converted stuff from .txt
> 
> Changes in V2:
> - renamed file
> - fixed gta04
> - added Openpandora, Epson Moverio BT-200
> - drop example
> - remove descriptions if just reformatting the name
> 
> Andreas Kemnade (1):
>   MAINTAINERS: add board bindings list to OMAP2+ files
> 
> Andrew Davis (1):
>   dt-bindings: omap: Partially convert omap.txt to yaml
> 
>  .../devicetree/bindings/arm/omap/omap.txt     |  99 ----------
>  .../devicetree/bindings/arm/ti/omap.yaml      | 176 ++++++++++++++++++
>  MAINTAINERS                                   |   1 +
>  3 files changed, 177 insertions(+), 99 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/arm/ti/omap.yaml
> 

Re: [PATCH v7 0/2] dt-bindings: omap: Convert omap.txt to yaml

[Krzysztof Kozlowski]

On 13/06/2023 19:32, Andreas Kemnade wrote:
> Hi,
> 
> any action still expected from my side?
> people gave R-bys...
> So looks like it is ready for the dt-folks to pick it up.
> 

It's SoC file, isn't it? Then goes via SoC tree.

Best regards,
Krzysztof

Re: [PATCH v7 0/2] dt-bindings: omap: Convert omap.txt to yaml

[Andreas Kemnade]

On Tue, 13 Jun 2023 20:43:55 +0200
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:

> On 13/06/2023 19:32, Andreas Kemnade wrote:
> > Hi,
> > 
> > any action still expected from my side?
> > people gave R-bys...
> > So looks like it is ready for the dt-folks to pick it up.
> >   
> 
> It's SoC file, isn't it? Then goes via SoC tree.
> 
Apparently there was confusion about it. Quoting
Tony:"And thanks for getting the
omap.yaml conversion going, that will get merged by the dt folks."

I am fine with either, 

Regards
Andreas

Re: [PATCH v7 0/2] dt-bindings: omap: Convert omap.txt to yaml

[Tony Lindgren]

* Andreas Kemnade <andreas@kemnade.info> [230616 16:53]:
> On Tue, 13 Jun 2023 20:43:55 +0200
> Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
> 
> > On 13/06/2023 19:32, Andreas Kemnade wrote:
> > > Hi,
> > > 
> > > any action still expected from my side?
> > > people gave R-bys...
> > > So looks like it is ready for the dt-folks to pick it up.
> > >   
> > 
> > It's SoC file, isn't it? Then goes via SoC tree.
> > 
> Apparently there was confusion about it. Quoting
> Tony:"And thanks for getting the
> omap.yaml conversion going, that will get merged by the dt folks."
> 
> I am fine with either, 

OK I'll apply this into omap-for-v6.6/dt-bindings thanks.

Regards,

Tony