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

[PATCH v3 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 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: Convert omap.txt to yaml

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

-- 
2.30.2

[PATCH v3 1/2] dt-bindings: omap: Convert omap.txt to yaml

[Andreas Kemnade]

From: Andrew Davis <afd@ti.com>

Convert omap.txt to yaml.

Signed-off-by: Andrew Davis <afd@ti.com>
[various cleanup, adding Epson Moverio BT-200]
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
 .../devicetree/bindings/arm/omap/omap.txt     | 185 ------------------
 Documentation/devicetree/bindings/arm/ti.yaml | 157 +++++++++++++++
 2 files changed, 157 insertions(+), 185 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/arm/ti.yaml

diff --git a/Documentation/devicetree/bindings/arm/omap/omap.txt b/Documentation/devicetree/bindings/arm/omap/omap.txt
index fa8b31660cad..67858a52be37 100644
--- a/Documentation/devicetree/bindings/arm/omap/omap.txt
+++ b/Documentation/devicetree/bindings/arm/omap/omap.txt
@@ -38,188 +38,3 @@ SoC Type (optional):
   compatible = "ti,gp"
 - High Security devices
   compatible = "ti,hs"
-
-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
-  compatible = "ti,am33xx"
-- AM43x generic - defaults to AM4372
-  compatible = "ti,am43"
-
-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"
-
-- DRA742
-  compatible = "ti,dra742", "ti,dra74", "ti,dra7"
-
-- DRA722
-  compatible = "ti,dra722", "ti,dra72", "ti,dra7"
-
-- DRA718
-  compatible = "ti,dra718", "ti,dra722", "ti,dra72", "ti,dra7"
-
-- AM5748
-  compatible = "ti,am5748", "ti,dra762", "ti,dra7"
-
-- AM5728
-  compatible = "ti,am5728", "ti,dra742", "ti,dra74", "ti,dra7"
-
-- AM5726
-  compatible = "ti,am5726", "ti,dra742", "ti,dra74", "ti,dra7"
-
-- AM5718
-  compatible = "ti,am5718", "ti,dra722", "ti,dra72", "ti,dra7"
-
-- AM5716
-  compatible = "ti,am5716", "ti,dra722", "ti,dra72", "ti,dra7"
-
-- AM4372
-  compatible = "ti,am4372", "ti,am43"
-
-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"
-
-- AM437x SBC-T43
-  compatible = "compulab,am437x-sbc-t43", "compulab,am437x-cm-t43", "ti,am4372", "ti,am43"
-
-- AM43x EPOS EVM
-  compatible = "ti,am43x-epos-evm", "ti,am43", "ti,am438x"
-
-- AM437x GP EVM
-  compatible = "ti,am437x-gp-evm", "ti,am4372", "ti,am43"
-
-- AM437x SK EVM: AM437x StarterKit Evaluation Module
-  compatible = "ti,am437x-sk-evm", "ti,am4372", "ti,am43"
-
-- AM57XX CL-SOM-AM57x
-  compatible = "compulab,cl-som-am57x", "ti,am5728", "ti,dra742", "ti,dra74", "ti,dra7"
-
-- AM57XX SBC-AM57x
-  compatible = "compulab,sbc-am57x", "compulab,cl-som-am57x", "ti,am5728", "ti,dra742", "ti,dra74", "ti,dra7"
-
-- AM5748 IDK
-  compatible = "ti,am5748-idk", "ti,am5748", "ti,dra762", "ti,dra7";
-
-- AM5728 IDK
-  compatible = "ti,am5728-idk", "ti,am5728", "ti,dra742", "ti,dra74", "ti,dra7"
-
-- AM5718 IDK
-  compatible = "ti,am5718-idk", "ti,am5718", "ti,dra7"
-
-- DRA762 EVM:  Software Development Board for DRA762
-  compatible = "ti,dra76-evm", "ti,dra762", "ti,dra7"
-
-- DRA742 EVM:  Software Development Board for DRA742
-  compatible = "ti,dra7-evm", "ti,dra742", "ti,dra74", "ti,dra7"
-
-- DRA722 EVM: Software Development Board for DRA722
-  compatible = "ti,dra72-evm", "ti,dra722", "ti,dra72", "ti,dra7"
-
-- 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.yaml b/Documentation/devicetree/bindings/arm/ti.yaml
new file mode 100644
index 000000000000..735e3f0e8143
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/ti.yaml
@@ -0,0 +1,157 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/ti.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:
+              - ti,omap2420-h4
+              - nokia,n800
+              - nokia,n810
+              - 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,omap3-beagle
+              - compulab,omap3-cm-t3530
+              - timll,omap3-devkit8000
+              - ti,omap3-evm                  # TI OMAP35XX EVM (TMDSEVM3530)
+              - ti,omap3-ldp                  # TI OMAP3430 LDP (Zoom1 Labrador)
+              - nokia,omap3-n900
+              - openpandora,omap3-pandora-600mhz
+          - 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
+              - compulab,omap3-cm-t3730
+              - amazon,omap3-echo             # Amazon Echo (first generation)
+              - ti,omap3-evm-37xx             # TI OMAP37XX EVM (TMDSEVM3730)
+              - goldelico,gta04
+              - nokia,omap3-n9
+              - nokia,omap3-n950
+              - lg,omap3-sniper               # LG Optimus Black
+              - ti,omap3-zoom3
+              - openpandora,omap3-pandora-1ghz
+          - const: ti,omap3630
+          - const: ti,omap3
+
+      - description: TI AM35 SoC based platforms
+        items:
+          - enum:
+              - 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-sdp
+          - const: ti,omap4430
+          - const: ti,omap4
+
+      - description: TI OMAP4460 SoC based platforms
+        items:
+          - enum:
+              - ti,omap4-panda-es
+              - epson,epson,embt2ws    # Epson Moverio BT-200
+          - 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
+
+      - description: TI AM33 SoC based platforms
+        items:
+          - enum:
+              - ti,am335x-bone
+              - compulab,cm-t335
+              - ti,am335x-evm
+              - ti,am335x-evmsk
+              - bosch,am335x-guardian
+              - ti,am3359-icev2
+              - novatech,am335x-lxm  # NovaTech OrionLXm
+              - moxa,uc-2101
+              - moxa,uc-8100-me-t
+              - gumstix,am335x-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
+              - ti,am437x-gp-evm
+              - ti,am437x-idk-evm
+              - ti,am437x-sk-evm
+          - pattern: '^ti,am4372[26789]$'
+          - const: ti,am43
+
+      - description: TI AM57 SoC based platforms
+        items:
+          - enum:
+              - beagle,am5729-beagleboneai
+              - compulab,cl-som-am57x
+              - ti,am5718-idk
+              - ti,am5728-idk
+              - ti,am5748-idk
+          - pattern: '^ti,am57[0124][689]$'
+          - const: ti,am57
+
+      - description: TI DRA7 SoC based platforms
+        items:
+          - enum:
+              - ti,dra718-evm
+              - ti,dra722-evm
+              - ti,dra742-evm
+              - ti,dra762-evm
+          - pattern: '^ti,dra7[12456][024568p]$'
+          - const: ti,dra7
+
+additionalProperties: true
+
+...
-- 
2.30.2

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

[Andreas Kemnade]

Add list of OMAP2+ boards to the corresponding section

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

diff --git a/MAINTAINERS b/MAINTAINERS
index 99b665e85f0a..e6c1d54a4d08 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15354,6 +15354,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.yaml
 F:	arch/arm/configs/omap2plus_defconfig
 F:	arch/arm/mach-omap2/
 F:	drivers/bus/ti-sysc.c
-- 
2.30.2

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

[Krzysztof Kozlowski]

On 01/04/2023 16:02, Andreas Kemnade wrote:
> From: Andrew Davis <afd@ti.com>
> 
> Convert omap.txt to yaml.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>
> [various cleanup, adding Epson Moverio BT-200]
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC.  It might happen, that command when run on an older
kernel, gives you outdated entries.  Therefore please be sure you base
your patches on recent Linux kernel.

Your CC list is broken in few ways. Just use the script, don't re-invent
the addresses.

> ---
>  .../devicetree/bindings/arm/omap/omap.txt     | 185 ------------------
>  Documentation/devicetree/bindings/arm/ti.yaml | 157 +++++++++++++++
>  2 files changed, 157 insertions(+), 185 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/arm/ti.yaml
> 

> +
> +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
> +              - nokia,n800
> +              - nokia,n810
> +              - nokia,n810-wimax

This looks unordered.

> +          - 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,omap3-beagle
> +              - compulab,omap3-cm-t3530
> +              - timll,omap3-devkit8000
> +              - ti,omap3-evm                  # TI OMAP35XX EVM (TMDSEVM3530)
> +              - ti,omap3-ldp                  # TI OMAP3430 LDP (Zoom1 Labrador)
> +              - nokia,omap3-n900
> +              - openpandora,omap3-pandora-600mhz

This section looks in even more random order.

> +          - 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
> +              - compulab,omap3-cm-t3730
> +              - amazon,omap3-echo             # Amazon Echo (first generation)
> +              - ti,omap3-evm-37xx             # TI OMAP37XX EVM (TMDSEVM3730)
> +              - goldelico,gta04
> +              - nokia,omap3-n9
> +              - nokia,omap3-n950
> +              - lg,omap3-sniper               # LG Optimus Black
> +              - ti,omap3-zoom3
> +              - openpandora,omap3-pandora-1ghz
> +          - const: ti,omap3630
> +          - const: ti,omap3
> +
> +      - description: TI AM35 SoC based platforms
> +        items:
> +          - enum:
> +              - 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-sdp
> +          - const: ti,omap4430
> +          - const: ti,omap4
> +
> +      - description: TI OMAP4460 SoC based platforms
> +        items:
> +          - enum:
> +              - ti,omap4-panda-es
> +              - epson,epson,embt2ws    # Epson Moverio BT-200
> +          - 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
> +
> +      - description: TI AM33 SoC based platforms
> +        items:
> +          - enum:
> +              - ti,am335x-bone
> +              - compulab,cm-t335
> +              - ti,am335x-evm
> +              - ti,am335x-evmsk
> +              - bosch,am335x-guardian
> +              - ti,am3359-icev2
> +              - novatech,am335x-lxm  # NovaTech OrionLXm
> +              - moxa,uc-2101
> +              - moxa,uc-8100-me-t
> +              - gumstix,am335x-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
> +              - ti,am437x-gp-evm
> +              - ti,am437x-idk-evm
> +              - ti,am437x-sk-evm
> +          - pattern: '^ti,am4372[26789]$'
> +          - const: ti,am43
> +
> +      - description: TI AM57 SoC based platforms
> +        items:
> +          - enum:
> +              - beagle,am5729-beagleboneai
> +              - compulab,cl-som-am57x
> +              - ti,am5718-idk
> +              - ti,am5728-idk
> +              - ti,am5748-idk
> +          - pattern: '^ti,am57[0124][689]$'
> +          - const: ti,am57
> +
> +      - description: TI DRA7 SoC based platforms
> +        items:
> +          - enum:
> +              - ti,dra718-evm

This is not correct. Maybe you forgot to fix DTS, but anyway it's
confusing to have dra718-evm with dra768 SoC. Is this really proper
combination?


> +              - ti,dra722-evm
> +              - ti,dra742-evm
> +              - ti,dra762-evm
> +          - pattern: '^ti,dra7[12456][024568p]$'
> +          - const: ti,dra7

Best regards,
Krzysztof

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

[Andreas Kemnade]

Hi,

On Sun, 2 Apr 2023 13:26:46 +0200
Krzysztof Kozlowski <krzk@kernel.org> wrote:

> On 01/04/2023 16:02, Andreas Kemnade wrote:
> > From: Andrew Davis <afd@ti.com>
> > 
> > Convert omap.txt to yaml.
> > 
> > Signed-off-by: Andrew Davis <afd@ti.com>
> > [various cleanup, adding Epson Moverio BT-200]
> > Signed-off-by: Andreas Kemnade <andreas@kemnade.info>  
> 
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC.  It might happen, that command when run on an older
> kernel, gives you outdated entries.  Therefore please be sure you base
> your patches on recent Linux kernel.
> 
> Your CC list is broken in few ways. Just use the script, don't re-invent
> the addresses.
> 
well, I am using the script and added a few adresses obviously missing.
But maybe I have used the script from an older kernel. The patch was against
linux-next.

[...]
> > +      - description: TI DRA7 SoC based platforms
> > +        items:
> > +          - enum:
> > +              - ti,dra718-evm  
> 
> This is not correct. Maybe you forgot to fix DTS, but anyway it's
> confusing to have dra718-evm with dra768 SoC. Is this really proper
> combination?
> 
dra718-evm has a lot of compatibles, no idea if that is sane.
And to better discuss such things I added the omap mailinglist (with a typo,
sorry). To avoid that trouble in the future, I added a patch to MAINTAINERS.

But I think it is better to not convert dra7 stuff for now, then
it can be discussed by the right people. 

Regards,
Andreas

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

[Krzysztof Kozlowski]

On 03/04/2023 12:18, Andreas Kemnade wrote:
> Hi,
> 
> On Sun, 2 Apr 2023 13:26:46 +0200
> Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
>> On 01/04/2023 16:02, Andreas Kemnade wrote:
>>> From: Andrew Davis <afd@ti.com>
>>>
>>> Convert omap.txt to yaml.
>>>
>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>> [various cleanup, adding Epson Moverio BT-200]
>>> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>  
>>
>> Please use scripts/get_maintainers.pl to get a list of necessary people
>> and lists to CC.  It might happen, that command when run on an older
>> kernel, gives you outdated entries.  Therefore please be sure you base
>> your patches on recent Linux kernel.
>>
>> Your CC list is broken in few ways. Just use the script, don't re-invent
>> the addresses.
>>
> well, I am using the script and added a few adresses obviously missing.

And one of them (omap@kernel) was invalid which made it impossible to
reply to you...


> But maybe I have used the script from an older kernel. The patch was against
> linux-next.
> 
> [...]
>>> +      - description: TI DRA7 SoC based platforms
>>> +        items:
>>> +          - enum:
>>> +              - ti,dra718-evm  
>>
>> This is not correct. Maybe you forgot to fix DTS, but anyway it's
>> confusing to have dra718-evm with dra768 SoC. Is this really proper
>> combination?
>>
> dra718-evm has a lot of compatibles, no idea if that is sane.
> And to better discuss such things I added the omap mailinglist (with a typo,
> sorry). To avoid that trouble in the future, I added a patch to MAINTAINERS.
> 
> But I think it is better to not convert dra7 stuff for now, then
> it can be discussed by the right people. 


Best regards,
Krzysztof