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

[Akhila YS]

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@xxxxxxxxxxxxx>
>> Signed-off-by: Akhila YS <akhilayalmati@xxxxxxxxx>
>> ---
>>   .../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@xxxxxxxxxxxxx>
>> +  - Claudiu Beznea <claudiu.beznea@xxxxxxxxx>
>> +
>> +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@xxxxxxxxx> 
>
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +
>> +unevaluatedProperties: false
>> +
>> +examples:
>> +  - |
>> +    chipid@fc069000 {
>> +        compatible = "atmel,sama5d2-chipid";
>> +        reg = <0xfc069000 0x8>;
>> +    };
>> +...
>>
>
-- 
Best Regards,
Akhila.