Re: [PATCH v5] dt-bindings: clk: versaclock5: Miscellaneous fixes and improvements:
From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2021-06-10 07:11:46
Also in:
linux-clk, linux-renesas-soc
Hi Luca, On Wed, Jun 9, 2021 at 10:56 PM Luca Ceresoli [off-list ref] wrote:
On 16/03/21 17:26, Geert Uytterhoeven wrote:quoted
- 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)
Send v6 (patch got smaller again ;-)
quoted
--- 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.
Yeah, silly rebase mistake (I apply fixes with changelogs with --squash).
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds