* [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml
@ 2023-10-26 19:53 Nik Bune
2023-10-27 14:14 ` Conor Dooley
0 siblings, 1 reply; 6+ messages in thread
From: Nik Bune @ 2023-10-26 19:53 UTC (permalink / raw)
To: wim, linux, robh+dt, krzysztof.kozlowski+dt, conor+dt, skhan, afd
Cc: Nik Bune, linux-watchdog, devicetree, linux-kernel
Convert txt file to yaml.
Add maintainers list.
Mark clock as required property, by reviewer's suggestion.
Add power-domains as optional property, by reviewer's suggestion.
Signed-off-by: Nik Bune <n2h9z4@gmail.com>
---
Changes in 3 (according to review comments):
- Added Andrew Davis <afd@ti.com> into maintainers list.
- Added power-domains property.
v2 patch: https://lore.kernel.org/all/20231024195839.49607-1-n2h9z4@gmail.com/
.../bindings/watchdog/davinci-wdt.txt | 24 ---------
.../bindings/watchdog/ti,davinci-wdt.yaml | 52 +++++++++++++++++++
2 files changed, 52 insertions(+), 24 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
create mode 100644 Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
diff --git a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt b/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
deleted file mode 100644
index aa10b8ec36e2..000000000000
--- a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
+++ /dev/null
@@ -1,24 +0,0 @@
-Texas Instruments DaVinci/Keystone Watchdog Timer (WDT) Controller
-
-Required properties:
-- compatible : Should be "ti,davinci-wdt", "ti,keystone-wdt"
-- reg : Should contain WDT registers location and length
-
-Optional properties:
-- timeout-sec : Contains the watchdog timeout in seconds
-- clocks : the clock feeding the watchdog timer.
- Needed if platform uses clocks.
- See clock-bindings.txt
-
-Documentation:
-Davinci DM646x - https://www.ti.com/lit/ug/spruer5b/spruer5b.pdf
-Keystone - https://www.ti.com/lit/ug/sprugv5a/sprugv5a.pdf
-
-Examples:
-
-wdt: wdt@2320000 {
- compatible = "ti,davinci-wdt";
- reg = <0x02320000 0x80>;
- timeout-sec = <30>;
- clocks = <&clkwdtimer0>;
-};
diff --git a/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml b/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
new file mode 100644
index 000000000000..17c824f2fc7d
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
@@ -0,0 +1,52 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments DaVinci/Keystone Watchdog Timer (WDT) Controller
+
+description: |
+ Documentation:
+ Davinci DM646x - https://www.ti.com/lit/ug/spruer5b/spruer5b.pdf
+ Keystone - https://www.ti.com/lit/ug/sprugv5a/sprugv5a
+
+maintainers:
+ - Andrew Davis <afd@ti.com>
+
+allOf:
+ - $ref: watchdog.yaml#
+
+properties:
+ compatible:
+ enum:
+ - ti,davinci-wdt
+ - ti,keystone-wdt
+
+ reg:
+ maxItems: 1
+
+ clocks:
+ maxItems: 1
+
+ power-domains:
+ description:
+ A phandle and PM domain specifier as defined by bindings of
+ the power controller specified by phandle.
+ See Documentation/devicetree/bindings/power/power-domain.yaml for details.
+
+required:
+ - compatible
+ - reg
+ - clocks
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ watchdog@2320000 {
+ compatible = "ti,davinci-wdt";
+ reg = <0x02320000 0x80>;
+ timeout-sec = <30>;
+ clocks = <&clkwdtimer0>;
+ };
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml
2023-10-26 19:53 [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml Nik Bune
@ 2023-10-27 14:14 ` Conor Dooley
2023-10-28 17:51 ` Nik Bune
0 siblings, 1 reply; 6+ messages in thread
From: Conor Dooley @ 2023-10-27 14:14 UTC (permalink / raw)
To: Nik Bune
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, conor+dt, skhan, afd,
linux-watchdog, devicetree, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 3859 bytes --]
Hey,
On Thu, Oct 26, 2023 at 09:53:13PM +0200, Nik Bune wrote:
> Convert txt file to yaml.
> Add maintainers list.
> Mark clock as required property, by reviewer's suggestion.
> Add power-domains as optional property, by reviewer's suggestion.
>
> Signed-off-by: Nik Bune <n2h9z4@gmail.com>
> ---
>
> Changes in 3 (according to review comments):
> - Added Andrew Davis <afd@ti.com> into maintainers list.
> - Added power-domains property.
>
> v2 patch: https://lore.kernel.org/all/20231024195839.49607-1-n2h9z4@gmail.com/
>
>
> .../bindings/watchdog/davinci-wdt.txt | 24 ---------
> .../bindings/watchdog/ti,davinci-wdt.yaml | 52 +++++++++++++++++++
> 2 files changed, 52 insertions(+), 24 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
> create mode 100644 Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
>
> diff --git a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt b/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
> deleted file mode 100644
> index aa10b8ec36e2..000000000000
> --- a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
> +++ /dev/null
> @@ -1,24 +0,0 @@
> -Texas Instruments DaVinci/Keystone Watchdog Timer (WDT) Controller
> -
> -Required properties:
> -- compatible : Should be "ti,davinci-wdt", "ti,keystone-wdt"
> -- reg : Should contain WDT registers location and length
> -
> -Optional properties:
> -- timeout-sec : Contains the watchdog timeout in seconds
> -- clocks : the clock feeding the watchdog timer.
> - Needed if platform uses clocks.
> - See clock-bindings.txt
> -
> -Documentation:
> -Davinci DM646x - https://www.ti.com/lit/ug/spruer5b/spruer5b.pdf
> -Keystone - https://www.ti.com/lit/ug/sprugv5a/sprugv5a.pdf
> -
> -Examples:
> -
> -wdt: wdt@2320000 {
> - compatible = "ti,davinci-wdt";
> - reg = <0x02320000 0x80>;
> - timeout-sec = <30>;
> - clocks = <&clkwdtimer0>;
> -};
> diff --git a/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml b/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
> new file mode 100644
> index 000000000000..17c824f2fc7d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
> @@ -0,0 +1,52 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments DaVinci/Keystone Watchdog Timer (WDT) Controller
> +
> +description: |
> + Documentation:
> + Davinci DM646x - https://www.ti.com/lit/ug/spruer5b/spruer5b.pdf
> + Keystone - https://www.ti.com/lit/ug/sprugv5a/sprugv5a
> +
> +maintainers:
> + - Andrew Davis <afd@ti.com>
> +
> +allOf:
> + - $ref: watchdog.yaml#
> +
> +properties:
> + compatible:
> + enum:
> + - ti,davinci-wdt
> + - ti,keystone-wdt
> +
> + reg:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + power-domains:
> + description:
> + A phandle and PM domain specifier as defined by bindings of
> + the power controller specified by phandle.
> + See Documentation/devicetree/bindings/power/power-domain.yaml for details.
I think you missed Rob's comment in reply to Andrew, about constraining
and ordering the power-domains:
https://lore.kernel.org/all/20231026203037.GA327324-robh@kernel.org/
Cheers,
Conor.
> +
> +required:
> + - compatible
> + - reg
> + - clocks
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + watchdog@2320000 {
> + compatible = "ti,davinci-wdt";
> + reg = <0x02320000 0x80>;
> + timeout-sec = <30>;
> + clocks = <&clkwdtimer0>;
> + };
> --
> 2.34.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml
2023-10-27 14:14 ` Conor Dooley
@ 2023-10-28 17:51 ` Nik Bune
2023-10-29 7:31 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Nik Bune @ 2023-10-28 17:51 UTC (permalink / raw)
To: conor+dt
Cc: robh, devicetree, krzysztof.kozlowski+dt, afd, linux-kernel,
linux-watchdog, linux, n2h9z4, skhan, wim
Hello!
Thank you for your review.
>On Fri, 27 Oct 2023 at 16:14, Conor Dooley <conor@kernel.org> wrote:
>Hey,
>
>On Thu, Oct 26, 2023 at 09:53:13PM +0200, Nik Bune wrote:
>> Convert txt file to yaml.
>> Add maintainers list.
>> Mark clock as required property, by reviewer's suggestion.
>> Add power-domains as optional property, by reviewer's suggestion.
>>
>> Signed-off-by: Nik Bune <n2h9z4@gmail.com>
>> ---
>>
>> Changes in 3 (according to review comments):
>> - Added Andrew Davis <afd@ti.com> into maintainers list.
>> - Added power-domains property.
>>
>> v2 patch: https://lore.kernel.org/all/20231024195839.49607-1-n2h9z4@gmail.com/
>>
>>
>> .../bindings/watchdog/davinci-wdt.txt | 24 ---------
>> .../bindings/watchdog/ti,davinci-wdt.yaml | 52 +++++++++++++++++++
>> 2 files changed, 52 insertions(+), 24 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
>> create mode 100644 Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt b/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
>> deleted file mode 100644
>> index aa10b8ec36e2..000000000000
>> --- a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
>> +++ /dev/null
>> @@ -1,24 +0,0 @@
>> -Texas Instruments DaVinci/Keystone Watchdog Timer (WDT) Controller
>> -
>> -Required properties:
>> -- compatible : Should be "ti,davinci-wdt", "ti,keystone-wdt"
>> -- reg : Should contain WDT registers location and length
>> -
>> -Optional properties:
>> -- timeout-sec : Contains the watchdog timeout in seconds
>> -- clocks : the clock feeding the watchdog timer.
>> - Needed if platform uses clocks.
>> - See clock-bindings.txt
>> -
>> -Documentation:
>> -Davinci DM646x - https://www.ti.com/lit/ug/spruer5b/spruer5b.pdf
>> -Keystone - https://www.ti.com/lit/ug/sprugv5a/sprugv5a.pdf
>> -
>> -Examples:
>> -
>> -wdt: wdt@2320000 {
>> - compatible = "ti,davinci-wdt";
>> - reg = <0x02320000 0x80>;
>> - timeout-sec = <30>;
>> - clocks = <&clkwdtimer0>;
>> -};
>> diff --git a/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml b/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
>> new file mode 100644
>> index 000000000000..17c824f2fc7d
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml
>> @@ -0,0 +1,52 @@
>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Texas Instruments DaVinci/Keystone Watchdog Timer (WDT) Controller
>> +
>> +description: |
>> + Documentation:
>> + Davinci DM646x - https://www.ti.com/lit/ug/spruer5b/spruer5b.pdf
>> + Keystone - https://www.ti.com/lit/ug/sprugv5a/sprugv5a
>> +
>> +maintainers:
>> + - Andrew Davis <afd@ti.com>
>> +
>> +allOf:
>> + - $ref: watchdog.yaml#
>> +
>> +properties:
>> + compatible:
>> + enum:
>> + - ti,davinci-wdt
>> + - ti,keystone-wdt
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + clocks:
>> + maxItems: 1
>> +
>> + power-domains:
>> + description:
>> + A phandle and PM domain specifier as defined by bindings of
>> + the power controller specified by phandle.
>> + See Documentation/devicetree/bindings/power/power-domain.yaml for details.
>
>I think you missed Rob's comment in reply to Andrew, about constraining
>and ordering the power-domains:
>https://lore.kernel.org/all/20231026203037.GA327324-robh@kernel.org/
>
>Cheers,
>Conor.
I didn't get what should be done with respect to the comment above:
Should we wait for the response from Andrew,
or should we skip and remove the the power-domains definition for this binding,
or add "maxItems: 1" to power-domains definition?
Thanks, Nik.
>
>> +
>> +required:
>> + - compatible
>> + - reg
>> + - clocks
>> +
>> +unevaluatedProperties: false
>> +
>> +examples:
>> + - |
>> + watchdog@2320000 {
>> + compatible = "ti,davinci-wdt";
>> + reg = <0x02320000 0x80>;
>> + timeout-sec = <30>;
>> + clocks = <&clkwdtimer0>;
>> + };
>> --
>> 2.34.1
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml
2023-10-28 17:51 ` Nik Bune
@ 2023-10-29 7:31 ` Krzysztof Kozlowski
2023-10-29 15:09 ` Nik Bune
0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-10-29 7:31 UTC (permalink / raw)
To: Nik Bune, conor+dt
Cc: robh, devicetree, krzysztof.kozlowski+dt, afd, linux-kernel,
linux-watchdog, linux, skhan, wim
On 28/10/2023 19:51, Nik Bune wrote:
>>> + power-domains:
>>> + description:
>>> + A phandle and PM domain specifier as defined by bindings of
>>> + the power controller specified by phandle.
>>> + See Documentation/devicetree/bindings/power/power-domain.yaml for details.
>>
>> I think you missed Rob's comment in reply to Andrew, about constraining
>> and ordering the power-domains:
>> https://lore.kernel.org/all/20231026203037.GA327324-robh@kernel.org/
>>
>> Cheers,
>> Conor.
>
> I didn't get what should be done with respect to the comment above:
> Should we wait for the response from Andrew,
> or should we skip and remove the the power-domains definition for this binding,
> or add "maxItems: 1" to power-domains definition?
None of the options above allow to send the same code without maxItems.
Sending new version without addressing feedback is basically ignoring
the feedback and next time you will have annoyed reviewers...
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml
2023-10-29 7:31 ` Krzysztof Kozlowski
@ 2023-10-29 15:09 ` Nik Bune
2023-10-29 15:20 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Nik Bune @ 2023-10-29 15:09 UTC (permalink / raw)
To: krzysztof.kozlowski+dt
Cc: robh, conor+dt, devicetree, afd, linux-kernel, linux-watchdog,
linux, n2h9z4, skhan, wim
Hello!
>On Sun, 29 Oct 2023 at 08:31, Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
>On 28/10/2023 19:51, Nik Bune wrote:
>>>> + power-domains:
>>>> + description:
>>>> + A phandle and PM domain specifier as defined by bindings of
>>>> + the power controller specified by phandle.
>>>> + See Documentation/devicetree/bindings/power/power-domain.yaml for details.
>>>
>>> I think you missed Rob's comment in reply to Andrew, about constraining
>>> and ordering the power-domains:
>>> https://lore.kernel.org/all/20231026203037.GA327324-robh@kernel.org/
>>>
>>> Cheers,
>>> Conor.
>>
>> I didn't get what should be done with respect to the comment above:
>> Should we wait for the response from Andrew,
>> or should we skip and remove the the power-domains definition for this binding,
>> or add "maxItems: 1" to power-domains definition?
>
>None of the options above allow to send the same code without maxItems.
>Sending new version without addressing feedback is basically ignoring
>the feedback and next time you will have annoyed reviewers...
>
>Best regards,
>Krzysztof
>
I didn't ignore. v3 patch was sent 37 minutes before the Rob's comment to v2.
Kind regards, Nik.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml
2023-10-29 15:09 ` Nik Bune
@ 2023-10-29 15:20 ` Krzysztof Kozlowski
0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-10-29 15:20 UTC (permalink / raw)
To: Nik Bune, krzysztof.kozlowski+dt
Cc: robh, conor+dt, devicetree, afd, linux-kernel, linux-watchdog,
linux, skhan, wim
On 29/10/2023 16:09, Nik Bune wrote:
> Hello!
>
>> On Sun, 29 Oct 2023 at 08:31, Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
>> On 28/10/2023 19:51, Nik Bune wrote:
>>>>> + power-domains:
>>>>> + description:
>>>>> + A phandle and PM domain specifier as defined by bindings of
>>>>> + the power controller specified by phandle.
>>>>> + See Documentation/devicetree/bindings/power/power-domain.yaml for details.
>>>>
>>>> I think you missed Rob's comment in reply to Andrew, about constraining
>>>> and ordering the power-domains:
>>>> https://lore.kernel.org/all/20231026203037.GA327324-robh@kernel.org/
>>>>
>>>> Cheers,
>>>> Conor.
>>>
>>> I didn't get what should be done with respect to the comment above:
>>> Should we wait for the response from Andrew,
>>> or should we skip and remove the the power-domains definition for this binding,
>>> or add "maxItems: 1" to power-domains definition?
>>
>> None of the options above allow to send the same code without maxItems.
>> Sending new version without addressing feedback is basically ignoring
>> the feedback and next time you will have annoyed reviewers...
>>
>> Best regards,
>> Krzysztof
>>
>
> I didn't ignore. v3 patch was sent 37 minutes before the Rob's comment to v2.
Ah, sorry, it's good then :). Rob's comment still applies, though.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-10-29 15:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-26 19:53 [PATCH v3] dt-bindings: watchdog: davinci-wdt: convert txt to yaml Nik Bune
2023-10-27 14:14 ` Conor Dooley
2023-10-28 17:51 ` Nik Bune
2023-10-29 7:31 ` Krzysztof Kozlowski
2023-10-29 15:09 ` Nik Bune
2023-10-29 15:20 ` Krzysztof Kozlowski
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).