public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML
@ 2026-02-24 14:46 Akhila YS
  2026-02-24 14:46 ` [PATCH v2 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema Akhila YS
                   ` (5 more replies)
  0 siblings, 6 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-24 14:46 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea
  Cc: devicetree, linux-kernel, linux-arm-kernel, Akhila YS

Convert various legacy .txt bindings for Microchip (formerly Atmel) AT91/SAMA
family system peripherals to proper YAML schemas. This includes:

- CHIPID (SoC ID register block)
- PIT (Period Interval Timer, old style)
- PIT64B (64-bit Period Interval Timer, newer parts)
- ST (System Timer, including watchdog subnode)
- RAMC/SDRAMC/DDRAMC/UDDRC (SDRAM/DDR memory controller

Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
Changes in v2:
- Change email for maintainers.
- microchip,sam9x60-pit64b: modify compatible and clock-names in properties.  
- Link to v1: https://lore.kernel.org/r/20260217-arm-microchip-v1-0-ae5d907e10e3@gmail.com

---
Akhila YS (5):
      dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema
      dt-bindings: arm: atmel,at91sam9260-pit: convert to DT schema
      dt-bindings: arm: microchip,sam9x60-pit64b : convert to DT schema
      dt-bindings: arm: atmel,at91rm9200-st: convert to DT schema
      dt-bindings: arm: atmel,at91rm9200-sdramc: convert to DT schema

 .../bindings/arm/atmel,at91rm9200-sdramc.yaml      | 67 +++++++++++++++++++++
 .../bindings/arm/atmel,at91rm9200-st.yaml          | 65 +++++++++++++++++++++
 .../bindings/arm/atmel,at91sam9260-pit.yaml        | 49 ++++++++++++++++
 .../devicetree/bindings/arm/atmel-sysregs.txt      | 48 ---------------
 .../bindings/arm/microchip,sam9x60-pit64b.yaml     | 68 ++++++++++++++++++++++
 .../bindings/arm/microchip,sama7g5-chipid.yaml     | 41 +++++++++++++
 6 files changed, 290 insertions(+), 48 deletions(-)
---
base-commit: ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
change-id: 20260128-arm-microchip-c0c0515024e6

Best regards,
-- 
Akhila YS <akhilayalmati@gmail.com>



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

* [PATCH v2 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema
  2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
@ 2026-02-24 14:46 ` Akhila YS
  2026-02-24 14:46 ` [PATCH v2 2/5] dt-bindings: arm: atmel,at91sam9260-pit: " Akhila YS
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-24 14:46 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea
  Cc: devicetree, linux-kernel, linux-arm-kernel, Akhila YS

Convert Atmel system registers binding to YAML format.

Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
 .../bindings/arm/microchip,sama7g5-chipid.yaml     | 41 ++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/microchip,sama7g5-chipid.yaml b/Documentation/devicetree/bindings/arm/microchip,sama7g5-chipid.yaml
new file mode 100644
index 000000000000..0228a5505259
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/microchip,sama7g5-chipid.yaml
@@ -0,0 +1,41 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/microchip,sama7g5-chipid.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Atmel/Microchip RAMC SDRAM/DDR Controller
+
+maintainers:
+  - Nicolas Ferre <nicolas.ferre@microchip.com>
+  - Claudiu Beznea <claudiu@tuxon.dev>
+
+description:
+  This binding describes the Atmel/Microchip Chip ID register block used
+  for SoC identification and revision information. It requires compatible
+  strings matching specific SoC families and a reg property defining the
+  register address and size.
+
+properties:
+  compatible:
+    enum:
+      - atmel,sama5d2-chipid
+      - microchip,sama7g5-chipid
+      - microchip,sama7d65-chipid
+
+  reg:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    chipid@fc069000 {
+        compatible = "atmel,sama5d2-chipid";
+        reg = <0xfc069000 0x8>;
+    };
+...

-- 
2.43.0



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

* [PATCH v2 2/5] dt-bindings: arm: atmel,at91sam9260-pit: convert to DT schema
  2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
  2026-02-24 14:46 ` [PATCH v2 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema Akhila YS
@ 2026-02-24 14:46 ` Akhila YS
  2026-02-24 14:46 ` [PATCH v2 3/5] dt-bindings: arm: microchip,sam9x60-pit64b : " Akhila YS
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-24 14:46 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea
  Cc: devicetree, linux-kernel, linux-arm-kernel, Akhila YS

Convert Atmel Periodic interval timer (PIT) binding to YAML format.

Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
 .../bindings/arm/atmel,at91sam9260-pit.yaml        | 49 ++++++++++++++++++++++
 1 file changed, 49 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/atmel,at91sam9260-pit.yaml b/Documentation/devicetree/bindings/arm/atmel,at91sam9260-pit.yaml
new file mode 100644
index 000000000000..dbb5fd843a9b
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/atmel,at91sam9260-pit.yaml
@@ -0,0 +1,49 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/atmel,at91sam9260-pit.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Atmel AT91SAM9260 Periodic Interval Timer (PIT)
+
+maintainers:
+  - Nicolas Ferre <nicolas.ferre@microchip.com>
+  - Claudiu Beznea <claudiu@tuxon.dev>
+
+description:
+  The Periodic Interval Timer (PIT) is part of the System Controller of
+  various Microchip 32-bit ARM-based SoCs (formerly Atmel AT91 series).
+  It is a simple down-counter timer used mainly as the kernel tick source.
+  The PIT is clocked from the slow clock and shares a single IRQ line with
+  other System Controller peripherals.
+
+properties:
+  compatible:
+    const: atmel,at91sam9260-pit
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  clocks:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+  - interrupts
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+    timer@fffffd30 {
+        compatible = "atmel,at91sam9260-pit";
+        reg = <0xfffffd30 0x10>;
+        interrupts = <1 IRQ_TYPE_LEVEL_HIGH>;
+        clocks = <&clk32k>;
+    };
+...

-- 
2.43.0



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

* [PATCH v2 3/5] dt-bindings: arm: microchip,sam9x60-pit64b : convert to DT schema
  2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
  2026-02-24 14:46 ` [PATCH v2 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema Akhila YS
  2026-02-24 14:46 ` [PATCH v2 2/5] dt-bindings: arm: atmel,at91sam9260-pit: " Akhila YS
@ 2026-02-24 14:46 ` Akhila YS
  2026-02-24 14:46 ` [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: " Akhila YS
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-24 14:46 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea
  Cc: devicetree, linux-kernel, linux-arm-kernel, Akhila YS

Convert Atmel Periodic interval timer of 64bit (PIT64b) binding to YAML
format.
Changes during conversion:
- Add missing compatible "microchip,sama7g5-pit64b" along with a fallback
compatible "microchip,sam9x60-pit64b".

Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
 .../bindings/arm/microchip,sam9x60-pit64b.yaml     | 68 ++++++++++++++++++++++
 1 file changed, 68 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/microchip,sam9x60-pit64b.yaml b/Documentation/devicetree/bindings/arm/microchip,sam9x60-pit64b.yaml
new file mode 100644
index 000000000000..cbc0a999748a
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/microchip,sam9x60-pit64b.yaml
@@ -0,0 +1,68 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/microchip,sam9x60-pit64b.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Microchip PIT64B 64-bit Periodic Interval Timer
+
+maintainers:
+  - Nicolas Ferre <nicolas.ferre@microchip.com>
+  - Claudiu Beznea <claudiu@tuxon.dev>
+
+description:
+  The Microchip PIT64B is a 64-bit periodic interval timer used in
+  several modern Microchip ARM SoCs including SAM9X60, SAM9X7 and
+  SAMA7D65 families. It provides extended timing range, flexible
+  clock selection and supports both periodic and one-shot interrupt
+  generation modes.
+
+properties:
+  compatible:
+    oneOf:
+      - const: microchip,sam9x60-pit64b
+      - items:
+          - enum:
+              - microchip,sama7d65-pit64b
+              - microchip,sama7g5-pit64b
+              - microchip,sam9x7-pit64b
+          - const: microchip,sam9x60-pit64b
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  clocks:
+    minItems: 1
+    maxItems: 2
+
+  clock-names:
+    minItems: 1
+    maxItems: 2
+    items:
+      enum:
+        - pclk
+        - gclk
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - clocks
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+    #include <dt-bindings/clock/at91.h>
+    timer@f0028000 {
+        compatible = "microchip,sama7g5-pit64b", "microchip,sam9x60-pit64b";
+        reg = <0xf0028000 0x100>;
+        interrupts = <37 IRQ_TYPE_LEVEL_HIGH 7>;
+        clocks = <&pmc PMC_TYPE_PERIPHERAL 37>, <&pmc PMC_TYPE_GCK 37>;
+        clock-names = "pclk", "gclk";
+    };
+...

-- 
2.43.0



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

* [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: convert to DT schema
  2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
                   ` (2 preceding siblings ...)
  2026-02-24 14:46 ` [PATCH v2 3/5] dt-bindings: arm: microchip,sam9x60-pit64b : " Akhila YS
@ 2026-02-24 14:46 ` Akhila YS
  2026-02-24 16:59   ` Rob Herring (Arm)
  2026-02-24 17:48   ` Conor Dooley
  2026-02-24 14:46 ` [PATCH v2 5/5] dt-bindings: arm: atmel,at91rm9200-sdramc: " Akhila YS
  2026-02-24 17:43 ` [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Conor Dooley
  5 siblings, 2 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-24 14:46 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea
  Cc: devicetree, linux-kernel, linux-arm-kernel, Akhila YS

Convert System Timer binding to YAML format.

Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
 .../bindings/arm/atmel,at91rm9200-st.yaml          | 65 ++++++++++++++++++++++
 1 file changed, 65 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml
new file mode 100644
index 000000000000..ff485b37cba8
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml
@@ -0,0 +1,65 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/atmel,at91rm9200-st.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Atmel System Timer
+
+maintainers:
+  - Nicolas Ferre <nicolas.ferre@microchip.com>
+  - Claudiu Beznea <claudiu@tuxon.dev>
+
+description:
+  The System Timer (ST) module in AT91RM9200 provides periodic tick and
+  alarm capabilities. It is exposed as a simple multi-function device
+  (simple-mfd + syscon) because it shares its register space and interrupt
+  with other System Controller blocks.
+
+properties:
+  compatible:
+    items:
+      - const: atmel,at91rm9200-st
+      - const: syscon
+      - const: simple-mfd
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  clocks:
+    maxItems: 1
+
+  watchdog:
+    type: object
+    description: Watchdog timer subnode.
+    properties:
+      compatible:
+        const: atmel,at91rm9200-wdt
+    required:
+      - compatible
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - clocks
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+    timer@fffffd00 {
+        compatible = "atmel,at91rm9200-st", "syscon", "simple-mfd";
+        reg = <0xfffffd00 0x100>;
+        interrupts = <1 IRQ_TYPE_LEVEL_HIGH 7>;
+        clocks = <&slow_xtal>;
+
+        watchdog {
+                compatible = "atmel,at91rm9200-wdt";
+        };
+    };
+...

-- 
2.43.0



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

* [PATCH v2 5/5] dt-bindings: arm: atmel,at91rm9200-sdramc: convert to DT schema
  2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
                   ` (3 preceding siblings ...)
  2026-02-24 14:46 ` [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: " Akhila YS
@ 2026-02-24 14:46 ` Akhila YS
  2026-02-24 17:43 ` [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Conor Dooley
  5 siblings, 0 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-24 14:46 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea
  Cc: devicetree, linux-kernel, linux-arm-kernel, Akhila YS

Convert RAMC SDRAM/DDR controller binding to YAML format.

Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
 .../bindings/arm/atmel,at91rm9200-sdramc.yaml      | 67 ++++++++++++++++++++++
 .../devicetree/bindings/arm/atmel-sysregs.txt      | 48 ----------------
 2 files changed, 67 insertions(+), 48 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/atmel,at91rm9200-sdramc.yaml b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-sdramc.yaml
new file mode 100644
index 000000000000..2cc07a772063
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-sdramc.yaml
@@ -0,0 +1,67 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/atmel,at91rm9200-sdramc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Microchip (Atmel) SDRAM / DDR Controller (RAMC / DDRAMC / UDDRC)
+
+maintainers:
+  - Nicolas Ferre <nicolas.ferre@microchip.com>
+  - Claudiu Beznea <claudiu@tuxon.dev>
+
+description:
+  The SDRAM/DDR Controller (often called RAMC or DDRAMC) in various
+  Atmel/Microchip ARM9 and Cortex-A5/A7 SoCs  manages external
+  SDRAM / DDR memory. It is typically exposed as a syscon node for
+  register access from other drivers (e.g. for initialization or mode
+  configuration). No interrupts or clocks are usually required in the
+  binding.
+
+properties:
+  compatible:
+    oneOf:
+      - items:
+          - const: atmel,at91rm9200-sdramc
+          - const: syscon
+      - items:
+          - const: microchip,sama7d65-uddrc
+          - const: microchip,sama7g5-uddrc
+      - items:
+          enum:
+            - atmel,at91sam9260-sdramc
+            - atmel,at91sam9g45-ddramc
+            - atmel,sama5d3-ddramc
+            - microchip,sam9x60-ddramc
+            - microchip,sam9x7-ddramc
+            - microchip,sama7g5-uddrc
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    minItems: 1
+    maxItems: 2
+
+  clock-names:
+    minItems: 1
+    items:
+      - const: ddrck
+      - const: mpddr
+
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/clock/at91.h>
+    ramc@ffffe400 {
+        compatible = "atmel,at91sam9g45-ddramc";
+        reg = <0xffffe400 0x200>;
+        clocks = <&pmc PMC_TYPE_SYSTEM 2>;
+        clock-names = "ddrck";
+    };
+...
diff --git a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
deleted file mode 100644
index 5ce54f9befe6..000000000000
--- a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
+++ /dev/null
@@ -1,48 +0,0 @@
-Atmel system registers
-
-Chipid required properties:
-- compatible: Should be "atmel,sama5d2-chipid" or "microchip,sama7g5-chipid"
-			"microchip,sama7d65-chipid"
-- reg : Should contain registers location and length
-
-PIT Timer required properties:
-- compatible: Should be "atmel,at91sam9260-pit"
-- reg: Should contain registers location and length
-- interrupts: Should contain interrupt for the PIT which is the IRQ line
-  shared across all System Controller members.
-
-PIT64B Timer required properties:
-- compatible: Should be "microchip,sam9x60-pit64b" or
-			"microchip,sam9x7-pit64b", "microchip,sam9x60-pit64b"
-			"microchip,sama7d65-pit64b", "microchip,sam9x60-pit64b"
-- reg: Should contain registers location and length
-- interrupts: Should contain interrupt for PIT64B timer
-- clocks: Should contain the available clock sources for PIT64B timer.
-
-System Timer (ST) required properties:
-- compatible: Should be "atmel,at91rm9200-st", "syscon", "simple-mfd"
-- reg: Should contain registers location and length
-- interrupts: Should contain interrupt for the ST which is the IRQ line
-  shared across all System Controller members.
-- clocks: phandle to input clock.
-Its subnodes can be:
-- watchdog: compatible should be "atmel,at91rm9200-wdt"
-
-RAMC SDRAM/DDR Controller required properties:
-- compatible: Should be "atmel,at91rm9200-sdramc", "syscon" or
-			"atmel,at91sam9260-sdramc" or
-			"atmel,at91sam9g45-ddramc" or
-			"atmel,sama5d3-ddramc" or
-			"microchip,sam9x60-ddramc" or
-			"microchip,sama7g5-uddrc" or
-			"microchip,sama7d65-uddrc", "microchip,sama7g5-uddrc" or
-			"microchip,sam9x7-ddramc", "atmel,sama5d3-ddramc".
-- reg: Should contain registers location and length
-
-Examples:
-
-	ramc0: ramc@ffffe800 {
-		compatible = "atmel,at91sam9g45-ddramc";
-		reg = <0xffffe800 0x200>;
-	};
-

-- 
2.43.0



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

* Re: [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: convert to DT schema
  2026-02-24 14:46 ` [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: " Akhila YS
@ 2026-02-24 16:59   ` Rob Herring (Arm)
  2026-02-24 17:48   ` Conor Dooley
  1 sibling, 0 replies; 12+ messages in thread
From: Rob Herring (Arm) @ 2026-02-24 16:59 UTC (permalink / raw)
  To: Akhila YS
  Cc: Claudiu Beznea, linux-arm-kernel, Krzysztof Kozlowski,
	linux-kernel, Alexandre Belloni, Conor Dooley, Nicolas Ferre,
	devicetree, Claudiu Beznea


On Tue, 24 Feb 2026 14:46:49 +0000, Akhila YS wrote:
> Convert System Timer binding to YAML format.
> 
> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
> ---
>  .../bindings/arm/atmel,at91rm9200-st.yaml          | 65 ++++++++++++++++++++++
>  1 file changed, 65 insertions(+)
> 

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.example.dtb: watchdog (atmel,at91rm9200-wdt): 'reg' is a required property
	from schema $id: http://devicetree.org/schemas/watchdog/atmel,at91rm9200-wdt.yaml

doc reference errors (make refcheckdocs):

See https://patchwork.kernel.org/project/devicetree/patch/20260224-arm-microchip-v2-4-8bedacd2cdcb@gmail.com

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.



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

* Re: [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML
  2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
                   ` (4 preceding siblings ...)
  2026-02-24 14:46 ` [PATCH v2 5/5] dt-bindings: arm: atmel,at91rm9200-sdramc: " Akhila YS
@ 2026-02-24 17:43 ` Conor Dooley
  2026-02-24 17:51   ` Conor Dooley
  5 siblings, 1 reply; 12+ messages in thread
From: Conor Dooley @ 2026-02-24 17:43 UTC (permalink / raw)
  To: Akhila YS
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea, devicetree,
	linux-kernel, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 2060 bytes --]

On Tue, Feb 24, 2026 at 02:46:45PM +0000, Akhila YS wrote:
> Convert various legacy .txt bindings for Microchip (formerly Atmel) AT91/SAMA
> family system peripherals to proper YAML schemas. This includes:
> 
> - CHIPID (SoC ID register block)
> - PIT (Period Interval Timer, old style)
> - PIT64B (64-bit Period Interval Timer, newer parts)
> - ST (System Timer, including watchdog subnode)
> - RAMC/SDRAMC/DDRAMC/UDDRC (SDRAM/DDR memory controller
> 
> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
> ---
> Changes in v2:
> - Change email for maintainers.
> - microchip,sam9x60-pit64b: modify compatible and clock-names in properties.  
> - Link to v1: https://lore.kernel.org/r/20260217-arm-microchip-v1-0-ae5d907e10e3@gmail.com

Please give me a chance to respond to your messages before sending new
versions. My comment about partial removal of the text file still
applies. You've got to send another version anyway.

> 
> ---
> Akhila YS (5):
>       dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema
>       dt-bindings: arm: atmel,at91sam9260-pit: convert to DT schema
>       dt-bindings: arm: microchip,sam9x60-pit64b : convert to DT schema
>       dt-bindings: arm: atmel,at91rm9200-st: convert to DT schema
>       dt-bindings: arm: atmel,at91rm9200-sdramc: convert to DT schema
> 
>  .../bindings/arm/atmel,at91rm9200-sdramc.yaml      | 67 +++++++++++++++++++++
>  .../bindings/arm/atmel,at91rm9200-st.yaml          | 65 +++++++++++++++++++++
>  .../bindings/arm/atmel,at91sam9260-pit.yaml        | 49 ++++++++++++++++
>  .../devicetree/bindings/arm/atmel-sysregs.txt      | 48 ---------------
>  .../bindings/arm/microchip,sam9x60-pit64b.yaml     | 68 ++++++++++++++++++++++
>  .../bindings/arm/microchip,sama7g5-chipid.yaml     | 41 +++++++++++++
>  6 files changed, 290 insertions(+), 48 deletions(-)
> ---
> base-commit: ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
> change-id: 20260128-arm-microchip-c0c0515024e6
> 
> Best regards,
> -- 
> Akhila YS <akhilayalmati@gmail.com>
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

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

* Re: [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: convert to DT schema
  2026-02-24 14:46 ` [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: " Akhila YS
  2026-02-24 16:59   ` Rob Herring (Arm)
@ 2026-02-24 17:48   ` Conor Dooley
  2026-02-25 15:10     ` Akhila YS
  1 sibling, 1 reply; 12+ messages in thread
From: Conor Dooley @ 2026-02-24 17:48 UTC (permalink / raw)
  To: Akhila YS
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea, devicetree,
	linux-kernel, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 2575 bytes --]

On Tue, Feb 24, 2026 at 02:46:49PM +0000, Akhila YS wrote:
> Convert System Timer binding to YAML format.
> 
> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
> ---
>  .../bindings/arm/atmel,at91rm9200-st.yaml          | 65 ++++++++++++++++++++++
>  1 file changed, 65 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml
> new file mode 100644
> index 000000000000..ff485b37cba8
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml
> @@ -0,0 +1,65 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/atmel,at91rm9200-st.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Atmel System Timer
> +
> +maintainers:
> +  - Nicolas Ferre <nicolas.ferre@microchip.com>
> +  - Claudiu Beznea <claudiu@tuxon.dev>
> +
> +description:
> +  The System Timer (ST) module in AT91RM9200 provides periodic tick and
> +  alarm capabilities. It is exposed as a simple multi-function device
> +  (simple-mfd + syscon) because it shares its register space and interrupt
> +  with other System Controller blocks.
> +
> +properties:
> +  compatible:
> +    items:
> +      - const: atmel,at91rm9200-st
> +      - const: syscon
> +      - const: simple-mfd
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  clocks:
> +    maxItems: 1
> +
> +  watchdog:
> +    type: object
> +    description: Watchdog timer subnode.
> +    properties:
> +      compatible:
> +        const: atmel,at91rm9200-wdt
> +    required:
> +      - compatible

This should just be a ref to the binding providing the watchdog.

I am guessing you didn't test this against all bindings, only against
/this/ binding and therefore missed the fact that it doesn't comply with
the binding for the watchdog itself.

> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +  - clocks
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +    timer@fffffd00 {
> +        compatible = "atmel,at91rm9200-st", "syscon", "simple-mfd";
> +        reg = <0xfffffd00 0x100>;
> +        interrupts = <1 IRQ_TYPE_LEVEL_HIGH 7>;
> +        clocks = <&slow_xtal>;
> +
> +        watchdog {
> +                compatible = "atmel,at91rm9200-wdt";
> +        };
> +    };
> +...
> 
> -- 
> 2.43.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

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

* Re: [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML
  2026-02-24 17:43 ` [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Conor Dooley
@ 2026-02-24 17:51   ` Conor Dooley
  2026-02-25 15:13     ` Akhila YS
  0 siblings, 1 reply; 12+ messages in thread
From: Conor Dooley @ 2026-02-24 17:51 UTC (permalink / raw)
  To: Akhila YS
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea, devicetree,
	linux-kernel, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 1194 bytes --]

On Tue, Feb 24, 2026 at 05:43:15PM +0000, Conor Dooley wrote:
> On Tue, Feb 24, 2026 at 02:46:45PM +0000, Akhila YS wrote:
> > Convert various legacy .txt bindings for Microchip (formerly Atmel) AT91/SAMA
> > family system peripherals to proper YAML schemas. This includes:
> > 
> > - CHIPID (SoC ID register block)
> > - PIT (Period Interval Timer, old style)
> > - PIT64B (64-bit Period Interval Timer, newer parts)
> > - ST (System Timer, including watchdog subnode)
> > - RAMC/SDRAMC/DDRAMC/UDDRC (SDRAM/DDR memory controller
> > 
> > Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
> > ---
> > Changes in v2:
> > - Change email for maintainers.
> > - microchip,sam9x60-pit64b: modify compatible and clock-names in properties.  
> > - Link to v1: https://lore.kernel.org/r/20260217-arm-microchip-v1-0-ae5d907e10e3@gmail.com
> 
> Please give me a chance to respond to your messages before sending new
> versions. My comment about partial removal of the text file still
> applies. You've got to send another version anyway.

Also, you probably should have checked the what I thought was Claudiu's
email was actually correct. It's actually claudiu.beznea@tuxon.dev!

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

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

* Re: [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: convert to DT schema
  2026-02-24 17:48   ` Conor Dooley
@ 2026-02-25 15:10     ` Akhila YS
  0 siblings, 0 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-25 15:10 UTC (permalink / raw)
  To: Conor Dooley
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea, devicetree,
	linux-kernel, linux-arm-kernel


On 24-02-2026 23:18, Conor Dooley wrote:
> On Tue, Feb 24, 2026 at 02:46:49PM +0000, Akhila YS wrote:
>> Convert System Timer binding to YAML format.
>>
>> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
>> ---
>>  .../bindings/arm/atmel,at91rm9200-st.yaml          | 65 ++++++++++++++++++++++
>>  1 file changed, 65 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml
>> new file mode 100644
>> index 000000000000..ff485b37cba8
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/arm/atmel,at91rm9200-st.yaml
>> @@ -0,0 +1,65 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/arm/atmel,at91rm9200-st.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Atmel System Timer
>> +
>> +maintainers:
>> +  - Nicolas Ferre <nicolas.ferre@microchip.com>
>> +  - Claudiu Beznea <claudiu@tuxon.dev>
>> +
>> +description:
>> +  The System Timer (ST) module in AT91RM9200 provides periodic tick and
>> +  alarm capabilities. It is exposed as a simple multi-function device
>> +  (simple-mfd + syscon) because it shares its register space and interrupt
>> +  with other System Controller blocks.
>> +
>> +properties:
>> +  compatible:
>> +    items:
>> +      - const: atmel,at91rm9200-st
>> +      - const: syscon
>> +      - const: simple-mfd
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  interrupts:
>> +    maxItems: 1
>> +
>> +  clocks:
>> +    maxItems: 1
>> +
>> +  watchdog:
>> +    type: object
>> +    description: Watchdog timer subnode.
>> +    properties:
>> +      compatible:
>> +        const: atmel,at91rm9200-wdt
>> +    required:
>> +      - compatible
> This should just be a ref to the binding providing the watchdog.
>
> I am guessing you didn't test this against all bindings, only against
> /this/ binding and therefore missed the fact that it doesn't comply with
> the binding for the watchdog itself.


Ok, i will take watchdog properties as a ref from different yaml.

>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +  - interrupts
>> +  - clocks
>> +
>> +unevaluatedProperties: false
>> +
>> +examples:
>> +  - |
>> +    #include <dt-bindings/interrupt-controller/irq.h>
>> +    timer@fffffd00 {
>> +        compatible = "atmel,at91rm9200-st", "syscon", "simple-mfd";
>> +        reg = <0xfffffd00 0x100>;
>> +        interrupts = <1 IRQ_TYPE_LEVEL_HIGH 7>;
>> +        clocks = <&slow_xtal>;
>> +
>> +        watchdog {
>> +                compatible = "atmel,at91rm9200-wdt";
>> +        };
>> +    };
>> +...
>>
>> -- 
>> 2.43.0
>>
-- 
Best Regards,
Akhila.



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

* Re: [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML
  2026-02-24 17:51   ` Conor Dooley
@ 2026-02-25 15:13     ` Akhila YS
  0 siblings, 0 replies; 12+ messages in thread
From: Akhila YS @ 2026-02-25 15:13 UTC (permalink / raw)
  To: Conor Dooley
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
	Claudiu Beznea, Alexandre Belloni, Claudiu Beznea, devicetree,
	linux-kernel, linux-arm-kernel


On 24-02-2026 23:21, Conor Dooley wrote:
> On Tue, Feb 24, 2026 at 05:43:15PM +0000, Conor Dooley wrote:
>> On Tue, Feb 24, 2026 at 02:46:45PM +0000, Akhila YS wrote:
>>> Convert various legacy .txt bindings for Microchip (formerly Atmel) AT91/SAMA
>>> family system peripherals to proper YAML schemas. This includes:
>>>
>>> - CHIPID (SoC ID register block)
>>> - PIT (Period Interval Timer, old style)
>>> - PIT64B (64-bit Period Interval Timer, newer parts)
>>> - ST (System Timer, including watchdog subnode)
>>> - RAMC/SDRAMC/DDRAMC/UDDRC (SDRAM/DDR memory controller
>>>
>>> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
>>> ---
>>> Changes in v2:
>>> - Change email for maintainers.
>>> - microchip,sam9x60-pit64b: modify compatible and clock-names in properties.  
>>> - Link to v1: https://lore.kernel.org/r/20260217-arm-microchip-v1-0-ae5d907e10e3@gmail.com
>> Please give me a chance to respond to your messages before sending new
>> versions. My comment about partial removal of the text file still
>> applies. You've got to send another version anyway.

Ok, i partially  remove binding from txt for each patch.

> Also, you probably should have checked the what I thought was Claudiu's
> email was actually correct. It's actually claudiu.beznea@tuxon.dev!

Sure, i will change email id for each patch under maintainer.

-- 
Best Regards,
Akhila.



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

end of thread, other threads:[~2026-02-25 15:13 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-24 14:46 [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Akhila YS
2026-02-24 14:46 ` [PATCH v2 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema Akhila YS
2026-02-24 14:46 ` [PATCH v2 2/5] dt-bindings: arm: atmel,at91sam9260-pit: " Akhila YS
2026-02-24 14:46 ` [PATCH v2 3/5] dt-bindings: arm: microchip,sam9x60-pit64b : " Akhila YS
2026-02-24 14:46 ` [PATCH v2 4/5] dt-bindings: arm: atmel,at91rm9200-st: " Akhila YS
2026-02-24 16:59   ` Rob Herring (Arm)
2026-02-24 17:48   ` Conor Dooley
2026-02-25 15:10     ` Akhila YS
2026-02-24 14:46 ` [PATCH v2 5/5] dt-bindings: arm: atmel,at91rm9200-sdramc: " Akhila YS
2026-02-24 17:43 ` [PATCH v2 0/5] dt-bindings: Microchip/Atmel AT91/SAMA system peripherals: convert to YAML Conor Dooley
2026-02-24 17:51   ` Conor Dooley
2026-02-25 15:13     ` Akhila YS

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