Re: [PATCH] dt-bindings: can: convert tcan4x5x.txt to DT schema

[Krzysztof Kozlowski <krzk@kernel.org>]

On 05/11/2024 13:24, Sean Nyekjaer wrote:
>>>
>>> If you use fallback for a 4552 then it would enable the use of the
>>> optional pins device-state-gpios and device-wake-gpios. But the chip
>>> doesn't have those so the hw guys would connect them and they won't
>>> be in the DT.
>>>
>>> Honestly I'm confused :/
>>
>> What stops anyone to use tcan4x5x ALONE for 4552? Nothing. And that's
>> the problem here.
>>
>>
> 
> Schema check will fail, but driver wize it will work just fine.

Schema will not fail. That's the problem - no errors will be ever
reported. The entire point of the schema, in contrast to TXT, is to
detect errors and that ridiculous wildcard used as front compatible
affects/reduces detection.

> Agree that is kinda broken.
> If I have time I can try to fix that later.

No, the fix is to drop the wildcard alone, as I said in your RFC.

> 
> Please explain one more time for me. Is this a comment on the if
> sentence or the broken behavior of the driver?

This is just generic comment, nothing to change here because you decided
not to fix that wildcard from old binding.



Best regards,
Krzysztof