[PATCH v4 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema

Akhila YS posted 5 patches 1 month ago
[PATCH v4 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema
Posted by Akhila YS 1 month ago
Convert Atmel system registers binding to YAML format.

Acked-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
---
 .../devicetree/bindings/arm/atmel-sysregs.txt      |  5 ---
 .../bindings/arm/microchip,sama7g5-chipid.yaml     | 41 ++++++++++++++++++++++
 2 files changed, 41 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
index 5ce54f9befe6..4ee18112586d 100644
--- a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
+++ b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
@@ -1,10 +1,5 @@
 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
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..4fdb068be929
--- /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.beznea@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
Re: [PATCH v4 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema
Posted by Claudiu Beznea 3 weeks, 5 days ago
Hi, Akhila,

On 2/27/26 17:14, Akhila YS wrote:
> Convert Atmel system registers binding to YAML format.
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
> ---
>   .../devicetree/bindings/arm/atmel-sysregs.txt      |  5 ---
>   .../bindings/arm/microchip,sama7g5-chipid.yaml     | 41 ++++++++++++++++++++++
>   2 files changed, 41 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
> index 5ce54f9befe6..4ee18112586d 100644
> --- a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
> +++ b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
> @@ -1,10 +1,5 @@
>   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
> 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..4fdb068be929
> --- /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.beznea@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

I'll keep this alphanumerically sorted, thus "microchip,sama7d64-chipid" in the 
middle. I'll adjust it while applying. Other than that:

Reviewed-by: Claudiu Beznea <claudiu.beznea@tuxon.dev>

> +
> +  reg:
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    chipid@fc069000 {
> +        compatible = "atmel,sama5d2-chipid";
> +        reg = <0xfc069000 0x8>;
> +    };
> +...
>
Re: [PATCH v4 1/5] dt-bindings: arm: microchip,sama7g5-chipid : convert to DT schema
Posted by Akhila YS 3 weeks, 5 days ago
On 07-03-2026 20:00, Claudiu Beznea wrote:
> Hi, Akhila,
>
> On 2/27/26 17:14, Akhila YS wrote:
>> Convert Atmel system registers binding to YAML format.
>>
>> Acked-by: Conor Dooley <conor.dooley@microchip.com>
>> Signed-off-by: Akhila YS <akhilayalmati@gmail.com>
>> ---
>>   .../devicetree/bindings/arm/atmel-sysregs.txt      |  5 ---
>>   .../bindings/arm/microchip,sama7g5-chipid.yaml     | 41
>> ++++++++++++++++++++++
>>   2 files changed, 41 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> index 5ce54f9befe6..4ee18112586d 100644
>> --- a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> +++ b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> @@ -1,10 +1,5 @@
>>   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
>> 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..4fdb068be929
>> --- /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.beznea@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
>
> I'll keep this alphanumerically sorted, thus
> "microchip,sama7d64-chipid" in the middle. I'll adjust it while
> applying. Other than that: 
>


Thank you claudiu.


> Reviewed-by: Claudiu Beznea <claudiu.beznea@tuxon.dev> 
>
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +
>> +unevaluatedProperties: false
>> +
>> +examples:
>> +  - |
>> +    chipid@fc069000 {
>> +        compatible = "atmel,sama5d2-chipid";
>> +        reg = <0xfc069000 0x8>;
>> +    };
>> +...
>>
>
-- 
Best Regards,
Akhila.