Re: [PATCH 2/3] dt-bindings: timer: Document arm,cortex-a7-timer for arch timer

[Krzysztof Kozlowski]

On 16/03/2022 10:54, Kuldeep Singh wrote:
> Renesas RZ/N1D platform uses compatible "arm,cortex-a7-timer" in
> conjugation with "arm,armv7-timer". Since, initial entry is not
> documented, it start raising dtbs_check warnings.
> 
> ['arm,cortex-a7-timer', 'arm,armv7-timer'] is too long
> 'arm,cortex-a7-timer' is not one of ['arm,armv7-timer', 'arm,armv8-timer']
> 'arm,cortex-a7-timer' is not one of ['arm,cortex-a15-timer']
> 
> In general, removing an existing entry is mostly devastating considering
> backward compatibility. Therefore, document it.

How removing undocumented and unused compatible is devastating for
backwards compatibility? I don't see it.

DTS is rather wrong, not bindings. Otherwise please explain it more, why
DTS is correct.



Best regards,
Krzysztof