Re: [PATCH 02/10] dt-bindings: net: can: grcan: Convert GRCAN CAN controllers binding from txt to YAML

[Krzysztof Kozlowski <krzk@kernel.org>]

On 24/11/2025 16:26, Arun Muthusamy wrote:
> Thank you for the review. I  wish to clarify a few details concerning 
> the DT binding and seek your guidance on the preferred approach
> 
> On 11/18/2025 12:01 PM, Krzysztof Kozlowski wrote:
>>>> +      Fallback on node name matching for systems that don't provide compatible.
>>>> +    enum:
>>>> +      - GAISLER_GRCAN
>>>> +      - 01_03d
>>>> +      - GAISLER_GRHCAN
>>>> +      - "01_034"
>>> This does not really work. Are you really defining here "name" property?
> 
> PROM-based LEON systems identify uses the "node name" property, while 
> DTS-based NOEL systems use proper "|compatible"| strings.
> 

No, either node name or name property. There is no such thing as node
name property.

> The updated schema now uses:
> properties:
>    $nodename:
>      pattern: "^(GAISLER_GRCAN|01_03d|GAISLER_GRHCAN|01_034)$"

So node name?

Anyway, names are wrong - they do not follow DTS coding style at all.
You need to come rationale why incorrect style has to be used.

> 
>    compatible:
>      enum:
>        - gaisler,grcan
>        - gaisler,grcanfd
> 
> Please advice me if its the preferred  is a preferred way to describe or 
> structure this dual-matching approach.

Nothing above usually - you need specific device compatible. I also do
not know what is the difference between one and another - in terms of
hardware (don't mention drivers please).


Best regards,
Krzysztof