On 12/04/2022 11:10, Tony Lindgren wrote: >> Apart from what Grygorii said, it would be useful if you add "if:then:" >> case constraining it per implementation (as I understood from commit msg >> only some use double mapping). >> >> This looks like: >> https://elixir.bootlin.com/linux/v5.18-rc2/source/Documentation/devicetree/bindings/clock/samsung,exynos7885-clock.yaml#L53 > > Yes I tried, but see the above. The compatible property is the same for > single mapped and dual mapped timers on the same SoC for omap4/5. But all on others this is single mapped? Then at least reduce them to one item inside allOf:if:then. The omap4/5 will accept both - one and two items. Best regards, Krzysztof
