* [PATCH] dt-bindings: watchdog: fsl-imx7ulp-wdt: Fix assigned-clock-parents
@ 2022-01-20 17:23 Rob Herring
2022-01-20 19:58 ` Guenter Roeck
2022-01-23 15:29 ` Rob Herring
0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2022-01-20 17:23 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Anson Huang
Cc: linux-watchdog, devicetree, linux-arm-kernel, linux-kernel
The schema has a typo with 'assigned-clocks-parents'. As it is not
required to list assigned clocks in bindings, just drop the assigned-clocks
property definitions to fix this.
Signed-off-by: Rob Herring <robh@kernel.org>
---
.../devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml b/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml
index fb603a20e396..4ca8a31359a5 100644
--- a/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml
@@ -29,12 +29,6 @@ properties:
clocks:
maxItems: 1
- assigned-clocks:
- maxItems: 1
-
- assigned-clocks-parents:
- maxItems: 1
-
timeout-sec: true
required:
@@ -56,7 +50,7 @@ examples:
interrupts = <GIC_SPI 55 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pcc2 IMX7ULP_CLK_WDG1>;
assigned-clocks = <&pcc2 IMX7ULP_CLK_WDG1>;
- assigned-clocks-parents = <&scg1 IMX7ULP_CLK_FIRC_BUS_CLK>;
+ assigned-clock-parents = <&scg1 IMX7ULP_CLK_FIRC_BUS_CLK>;
timeout-sec = <40>;
};
--
2.32.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: watchdog: fsl-imx7ulp-wdt: Fix assigned-clock-parents
2022-01-20 17:23 [PATCH] dt-bindings: watchdog: fsl-imx7ulp-wdt: Fix assigned-clock-parents Rob Herring
@ 2022-01-20 19:58 ` Guenter Roeck
2022-01-23 15:29 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Guenter Roeck @ 2022-01-20 19:58 UTC (permalink / raw)
To: Rob Herring, Wim Van Sebroeck, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Anson Huang
Cc: linux-watchdog, devicetree, linux-arm-kernel, linux-kernel
On 1/20/22 9:23 AM, Rob Herring wrote:
> The schema has a typo with 'assigned-clocks-parents'. As it is not
> required to list assigned clocks in bindings, just drop the assigned-clocks
> property definitions to fix this.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Guenter Roeck <linux@roeck-us.net>
> ---
> .../devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml b/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml
> index fb603a20e396..4ca8a31359a5 100644
> --- a/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml
> @@ -29,12 +29,6 @@ properties:
> clocks:
> maxItems: 1
>
> - assigned-clocks:
> - maxItems: 1
> -
> - assigned-clocks-parents:
> - maxItems: 1
> -
> timeout-sec: true
>
> required:
> @@ -56,7 +50,7 @@ examples:
> interrupts = <GIC_SPI 55 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&pcc2 IMX7ULP_CLK_WDG1>;
> assigned-clocks = <&pcc2 IMX7ULP_CLK_WDG1>;
> - assigned-clocks-parents = <&scg1 IMX7ULP_CLK_FIRC_BUS_CLK>;
> + assigned-clock-parents = <&scg1 IMX7ULP_CLK_FIRC_BUS_CLK>;
> timeout-sec = <40>;
> };
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: watchdog: fsl-imx7ulp-wdt: Fix assigned-clock-parents
2022-01-20 17:23 [PATCH] dt-bindings: watchdog: fsl-imx7ulp-wdt: Fix assigned-clock-parents Rob Herring
2022-01-20 19:58 ` Guenter Roeck
@ 2022-01-23 15:29 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2022-01-23 15:29 UTC (permalink / raw)
To: Rob Herring
Cc: Guenter Roeck, Shawn Guo, NXP Linux Team, Wim Van Sebroeck,
linux-arm-kernel, linux-kernel, Anson Huang, linux-watchdog,
Sascha Hauer, Fabio Estevam, devicetree, Pengutronix Kernel Team
On Thu, 20 Jan 2022 11:23:32 -0600, Rob Herring wrote:
> The schema has a typo with 'assigned-clocks-parents'. As it is not
> required to list assigned clocks in bindings, just drop the assigned-clocks
> property definitions to fix this.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../devicetree/bindings/watchdog/fsl-imx7ulp-wdt.yaml | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-23 15:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-20 17:23 [PATCH] dt-bindings: watchdog: fsl-imx7ulp-wdt: Fix assigned-clock-parents Rob Herring
2022-01-20 19:58 ` Guenter Roeck
2022-01-23 15:29 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).