Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

dt-bindings: watchdog: sunxi: clarify clock support

Most Allwinner SoCs have just one input clock to drive the watchdog
peripheral. So far this is the 24 MHz "HOSC" oscillator, divided down
internally to 32 KHz.
The F1C100 series watchdog however uses the unchanged 32 KHz "LOSC" as
its only clock input, which has the same effect, but let's the binding
description mismatch.

Change the binding description to name the clocks more loosely, so both
the LOSC and divided HOSC match the description. As the fixed clock names
now make less sense, drop them from SoCs supporting just one clock
input, they were not used by any DT anyway.

For the newer SoCs, supporting a choice of two input clocks, we keep
both the description and clock-names requirement.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Samuel Holland <samuel@sholland.org>
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Link: https://lore.kernel.org/r/20220317162349.739636-3-andre.przywara@arm.com

authored by

Andre Przywara and committed by
Jernej Skrabec
42b91bb1 6d41e130

+8 -12
+8 -12
Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
··· 39 39 clocks: 40 40 minItems: 1 41 41 items: 42 - - description: High-frequency oscillator input, divided internally 43 - - description: Low-frequency oscillator input, only found on some variants 44 - 45 - clock-names: 46 - minItems: 1 47 - items: 48 - - const: hosc 49 - - const: losc 42 + - description: 32 KHz input clock 43 + - description: secondary clock source 50 44 51 45 interrupts: 52 46 maxItems: 1 ··· 65 71 properties: 66 72 clocks: 67 73 minItems: 2 74 + items: 75 + - description: High-frequency oscillator input, divided internally 76 + - description: Low-frequency oscillator input 68 77 69 78 clock-names: 70 - minItems: 2 79 + items: 80 + - const: hosc 81 + - const: losc 71 82 72 83 required: 73 84 - clock-names ··· 80 81 else: 81 82 properties: 82 83 clocks: 83 - maxItems: 1 84 - 85 - clock-names: 86 84 maxItems: 1 87 85 88 86 unevaluatedProperties: false