Messages in this thread | | | Date | Wed, 16 Mar 2022 17:28:49 +0100 | Subject | Re: [PATCH 2/3] dt-bindings: timer: Document arm,cortex-a7-timer for arch timer | From | 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
| |