public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: bus: ti,da850-mstpri: Convert to YAML
@ 2026-03-13 17:31 Zeynep Dicle
  2026-03-13 17:41 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Zeynep Dicle @ 2026-03-13 17:31 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt
  Cc: bartosz.golaszewski, devicetree, linux-kernel, daniel.baluta,
	simona.toaca, d-gole, m-chawdhry, Zeynep Dicle

Convert the TI DA850 master peripheral priority binding from txt
to YAML format to enable automated validation of device tree sources.

Signed-off-by: Zeynep Dicle <zeynep.dicle.dev@gmail.com>
---
 .../bindings/bus/ti,da850-mstpri.txt          | 20 -------------
 .../bindings/bus/ti,da850-mstpri.yaml         | 28 +++++++++++++++++++
 2 files changed, 28 insertions(+), 20 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/bus/ti,da850-mstpri.txt
 create mode 100644 Documentation/devicetree/bindings/bus/ti,da850-mstpri.yaml

diff --git a/Documentation/devicetree/bindings/bus/ti,da850-mstpri.txt b/Documentation/devicetree/bindings/bus/ti,da850-mstpri.txt
deleted file mode 100644
index 72daefc6b..000000000
--- a/Documentation/devicetree/bindings/bus/ti,da850-mstpri.txt
+++ /dev/null
@@ -1,20 +0,0 @@
-* Device tree bindings for Texas Instruments da8xx master peripheral
-  priority driver
-
-DA8XX SoCs feature a set of registers allowing to change the priority of all
-peripherals classified as masters.
-
-Documentation:
-OMAP-L138 (DA850) - http://www.ti.com/lit/ug/spruh82c/spruh82c.pdf
-
-Required properties:
-
-- compatible:		"ti,da850-mstpri" - for da850 based boards
-- reg:			offset and length of the mstpri registers
-
-Example for da850-lcdk is shown below.
-
-mstpri {
-	compatible = "ti,da850-mstpri";
-	reg = <0x14110 0x0c>;
-};
diff --git a/Documentation/devicetree/bindings/bus/ti,da850-mstpri.yaml b/Documentation/devicetree/bindings/bus/ti,da850-mstpri.yaml
new file mode 100644
index 000000000..b89bfebd1
--- /dev/null
+++ b/Documentation/devicetree/bindings/bus/ti,da850-mstpri.yaml
@@ -0,0 +1,28 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/bus/ti,da850-mstpri.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+title: Texas Instruments DA850 Master Peripheral Priority
+maintainers:
+  - Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
+description: |
+  DA8XX SoCs feature a set of registers allowing to change the priority of all
+  peripherals classified as masters.
+properties:
+  compatible:
+    const: ti,da850-mstpri
+  reg:
+    maxItems: 1
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    mstpri@14110 {
+        compatible = "ti,da850-mstpri";
+        reg = <0x14110 0x0c>;
+    };
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: bus: ti,da850-mstpri: Convert to YAML
  2026-03-13 17:31 [PATCH] dt-bindings: bus: ti,da850-mstpri: Convert to YAML Zeynep Dicle
@ 2026-03-13 17:41 ` Krzysztof Kozlowski
  2026-03-13 18:40   ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2026-03-13 17:41 UTC (permalink / raw)
  To: Zeynep Dicle, robh, krzk+dt, conor+dt
  Cc: bartosz.golaszewski, devicetree, linux-kernel, daniel.baluta,
	simona.toaca, d-gole, m-chawdhry

On 13/03/2026 18:31, Zeynep Dicle wrote:
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/bus/ti,da850-mstpri.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +title: Texas Instruments DA850 Master Peripheral Priority
> +maintainers:
> +  - Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> +description: |
> +  DA8XX SoCs feature a set of registers allowing to change the priority of all
> +  peripherals classified as masters.
> +properties:
> +  compatible:
> +    const: ti,da850-mstpri
> +  reg:
> +    maxItems: 1
> +required:
> +  - compatible
> +  - reg

I don't believe this passed your GSoC mentors review. Please don't send
AI output but actually follow existing kernel/DT style.

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: bus: ti,da850-mstpri: Convert to YAML
  2026-03-13 17:41 ` Krzysztof Kozlowski
@ 2026-03-13 18:40   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2026-03-13 18:40 UTC (permalink / raw)
  To: Zeynep Dicle, robh, krzk+dt, conor+dt
  Cc: bartosz.golaszewski, devicetree, linux-kernel, daniel.baluta,
	simona.toaca, d-gole, m-chawdhry

On 13/03/2026 18:41, Krzysztof Kozlowski wrote:
> On 13/03/2026 18:31, Zeynep Dicle wrote:
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/bus/ti,da850-mstpri.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +title: Texas Instruments DA850 Master Peripheral Priority
>> +maintainers:
>> +  - Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
>> +description: |
>> +  DA8XX SoCs feature a set of registers allowing to change the priority of all
>> +  peripherals classified as masters.
>> +properties:
>> +  compatible:
>> +    const: ti,da850-mstpri
>> +  reg:
>> +    maxItems: 1
>> +required:
>> +  - compatible
>> +  - reg
> 
> I don't believe this passed your GSoC mentors review. Please don't send
> AI output but actually follow existing kernel/DT style.

And let me remind, that what you sent is not better what Rob already
prepared a year ago:

https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/commit/?h=dt-convert&id=9f0479e32bcd132e85899b40185c62754ff93aed

So what is the point of posting unsupervised and poor AI output if we
have it all done?

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-03-13 18:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-13 17:31 [PATCH] dt-bindings: bus: ti,da850-mstpri: Convert to YAML Zeynep Dicle
2026-03-13 17:41 ` Krzysztof Kozlowski
2026-03-13 18:40   ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox