Re: [PATCH v5] dt-bindings: clk: versaclock5: Miscellaneous fixes and improvements:
From: Luca Ceresoli <luca@lucaceresoli.net>
Date: 2021-06-09 20:56:46
Also in:
linux-clk, linux-renesas-soc
From: Luca Ceresoli <luca@lucaceresoli.net>
Date: 2021-06-09 20:56:46
Also in:
linux-clk, linux-renesas-soc
Hi All, On 16/03/21 17:26, Geert Uytterhoeven wrote:
- Remove unneeded references for "idt,xtal-load-femtofarads" and
"idt,slew-percent", as vendor specific properties having a standard
unit suffix don't need a type,
- Add missing "additionalProperties: false" for subnodes, to catch
typos in properties,
- Fix property names in example.
Fixes: 45c940184b501fc6 ("dt-bindings: clk: versaclock5: convert to yaml")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Luca Ceresoli <luca@lucaceresoli.net>
Reviewed-by: Rob Herring <robh@kernel.org>
Acked-by: Stephen Boyd <sboyd@kernel.org>This patch seems to have gone MIA. It really should be applied. Geert, would you please rebase&resend in case it does not not apply anymore? (or tell me if you prefer me to take it over)
---
This depends on dt-schema v2021.2.1.
v4:
- Add Reviewed-by, Acked-by,
v3:
- Drop references for "idt,voltage-microvolt" and "idt,slew-percent",
v2:
- Settle on "idt,voltage-microvolt", cfr. commit 4b003f5fcadfa2d0
('clk: vc5: Use "idt,voltage-microvolt" instead of
"idt,voltage-microvolts"'),
- Drop reference to clock.yaml, which is already applied
unconditionally,
- Drop removal of allOf around if condition, as it is unnecessary
churn.
squash! dt-bindings: clk: versaclock5: Miscellaneous fixes and improvements:heh, should be removed, but is's after --- so not a big deal. -- Luca