* [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
@ 2021-11-15 5:53 Rafał Miłecki
2021-11-15 5:53 ` [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block Rafał Miłecki
2021-12-06 7:50 ` [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
0 siblings, 2 replies; 21+ messages in thread
From: Rafał Miłecki @ 2021-11-15 5:53 UTC (permalink / raw)
To: Lee Jones, Wim Van Sebroeck, Guenter Roeck, Rob Herring
Cc: Florian Fainelli, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
From: Rafał Miłecki <rafal@milecki.pl>
This helps validating DTS files.
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
V2: Use valid $id
V4: Add "clocks" maxItems and include Rob's Reviewed-by
RESEND: Patchwork lost 1/2, marc.info lost 2/2
---
.../bindings/watchdog/brcm,bcm7038-wdt.txt | 19 ---------
.../bindings/watchdog/brcm,bcm7038-wdt.yaml | 41 +++++++++++++++++++
2 files changed, 41 insertions(+), 19 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.txt
create mode 100644 Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.txt b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.txt
deleted file mode 100644
index 84122270be8f..000000000000
--- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.txt
+++ /dev/null
@@ -1,19 +0,0 @@
-BCM7038 Watchdog timer
-
-Required properties:
-
-- compatible : should be "brcm,bcm7038-wdt"
-- reg : Specifies base physical address and size of the registers.
-
-Optional properties:
-
-- clocks: The clock running the watchdog. If no clock is found the
- driver will default to 27000000 Hz.
-
-Example:
-
-watchdog@f040a7e8 {
- compatible = "brcm,bcm7038-wdt";
- clocks = <&upg_fixed>;
- reg = <0xf040a7e8 0x16>;
-};
diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
new file mode 100644
index 000000000000..ed6210666ead
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
@@ -0,0 +1,41 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/watchdog/brcm,bcm7038-wdt.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: BCM7038 watchdog timer
+
+allOf:
+ - $ref: "watchdog.yaml#"
+
+maintainers:
+ - Florian Fainelli <f.fainelli@gmail.com>
+ - Justin Chen <justinpopo6@gmail.com>
+ - Rafał Miłecki <rafal@milecki.pl>
+
+properties:
+ compatible:
+ const: brcm,bcm7038-wdt
+
+ reg:
+ maxItems: 1
+
+ clocks:
+ maxItems: 1
+ description: >
+ The clock running the watchdog. If no clock is found the driver will
+ default to 27000000 Hz.
+
+unevaluatedProperties: false
+
+required:
+ - reg
+
+examples:
+ - |
+ watchdog@f040a7e8 {
+ compatible = "brcm,bcm7038-wdt";
+ reg = <0xf040a7e8 0x16>;
+ clocks = <&upg_fixed>;
+ };
--
2.31.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block
2021-11-15 5:53 [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
@ 2021-11-15 5:53 ` Rafał Miłecki
2021-11-18 22:45 ` Rob Herring
2021-12-29 9:44 ` Lee Jones
2021-12-06 7:50 ` [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
1 sibling, 2 replies; 21+ messages in thread
From: Rafał Miłecki @ 2021-11-15 5:53 UTC (permalink / raw)
To: Lee Jones, Wim Van Sebroeck, Guenter Roeck, Rob Herring
Cc: Florian Fainelli, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki
From: Rafał Miłecki <rafal@milecki.pl>
It's a block implementing few time related functions depending on a
(SoC specific) variant. At this point there is ready binding for a
watchdog only. Work on remaining subblocks (e.g. "reg" based reboot) is
in progress.
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
V2: Update $id, description, compatible, example & commit message
V3: Drop "brcm,twd" from compatible list per Rob's review
RESEND: Patchwork lost 1/2, marc.info lost 2/2
---
.../devicetree/bindings/mfd/brcm,twd.yaml | 61 +++++++++++++++++++
1 file changed, 61 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/brcm,twd.yaml
diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
new file mode 100644
index 000000000000..634526f790b8
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
@@ -0,0 +1,61 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/brcm,twd.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Broadcom's Timer-Watchdog (aka TWD)
+
+maintainers:
+ - Rafał Miłecki <rafal@milecki.pl>
+
+description: |
+ Broadcom has a Timer-Watchdog block used in multiple SoCs (e.g., BCM4908,
+ BCM63xx, BCM7038). There are few variants available (they differ slightly in
+ registers layout). This block consists of: timers, watchdog and optionally a
+ software reset handler.
+
+properties:
+ compatible:
+ items:
+ - enum:
+ - brcm,bcm4908-twd
+ - brcm,bcm7038-twd
+ - const: simple-mfd
+ - const: syscon
+
+ reg:
+ maxItems: 1
+
+ ranges: true
+
+ "#address-cells":
+ const: 1
+
+ "#size-cells":
+ const: 1
+
+patternProperties:
+ '^watchdog@[a-f0-9]+$':
+ $ref: /schemas/watchdog/brcm,bcm7038-wdt.yaml
+
+additionalProperties: false
+
+required:
+ - reg
+
+examples:
+ - |
+ timer-mfd@ff800400 {
+ compatible = "brcm,bcm4908-twd", "simple-mfd", "syscon";
+ reg = <0xff800400 0x4c>;
+ ranges = <0x00000000 0xff800400 0x4c>;
+
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ watchdog@28 {
+ compatible = "brcm,bcm7038-wdt";
+ reg = <0x28 0x8>;
+ };
+ };
--
2.31.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block
2021-11-15 5:53 ` [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block Rafał Miłecki
@ 2021-11-18 22:45 ` Rob Herring
2021-12-29 9:44 ` Lee Jones
1 sibling, 0 replies; 21+ messages in thread
From: Rob Herring @ 2021-11-18 22:45 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Justin Chen, bcm-kernel-feedback-list, linux-mips,
Rafał Miłecki, Lee Jones, linux-watchdog, Rob Herring,
Wim Van Sebroeck, Guenter Roeck, devicetree, linux-arm-kernel,
Florian Fainelli
On Mon, 15 Nov 2021 06:53:54 +0100, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> It's a block implementing few time related functions depending on a
> (SoC specific) variant. At this point there is ready binding for a
> watchdog only. Work on remaining subblocks (e.g. "reg" based reboot) is
> in progress.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> V2: Update $id, description, compatible, example & commit message
> V3: Drop "brcm,twd" from compatible list per Rob's review
> RESEND: Patchwork lost 1/2, marc.info lost 2/2
> ---
> .../devicetree/bindings/mfd/brcm,twd.yaml | 61 +++++++++++++++++++
> 1 file changed, 61 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block
2021-11-15 5:53 ` [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block Rafał Miłecki
2021-11-18 22:45 ` Rob Herring
@ 2021-12-29 9:44 ` Lee Jones
1 sibling, 0 replies; 21+ messages in thread
From: Lee Jones @ 2021-12-29 9:44 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki
On Mon, 15 Nov 2021, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> It's a block implementing few time related functions depending on a
> (SoC specific) variant. At this point there is ready binding for a
> watchdog only. Work on remaining subblocks (e.g. "reg" based reboot) is
> in progress.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> V2: Update $id, description, compatible, example & commit message
> V3: Drop "brcm,twd" from compatible list per Rob's review
> RESEND: Patchwork lost 1/2, marc.info lost 2/2
> ---
> .../devicetree/bindings/mfd/brcm,twd.yaml | 61 +++++++++++++++++++
> 1 file changed, 61 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/brcm,twd.yaml
Applied, thanks.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-11-15 5:53 [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
2021-11-15 5:53 ` [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block Rafał Miłecki
@ 2021-12-06 7:50 ` Rafał Miłecki
2021-12-06 8:44 ` Lee Jones
1 sibling, 1 reply; 21+ messages in thread
From: Rafał Miłecki @ 2021-12-06 7:50 UTC (permalink / raw)
To: Lee Jones, Wim Van Sebroeck, Guenter Roeck, Rob Herring
Cc: Florian Fainelli, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
Wim, Lee,
On 15.11.2021 06:53, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> This helps validating DTS files.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
I'm not familiar with handling multi-subsystem patchsets (here: watchdog
& MFD).
Please kindly let me know: how to proceed with this patchset now to get
it queued for Linus?
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 7:50 ` [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
@ 2021-12-06 8:44 ` Lee Jones
2021-12-06 8:53 ` Rafał Miłecki
0 siblings, 1 reply; 21+ messages in thread
From: Lee Jones @ 2021-12-06 8:44 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On Mon, 06 Dec 2021, Rafał Miłecki wrote:
> Wim, Lee,
>
> On 15.11.2021 06:53, Rafał Miłecki wrote:
> > From: Rafał Miłecki <rafal@milecki.pl>
> >
> > This helps validating DTS files.
> >
> > Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> > Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> > Reviewed-by: Rob Herring <robh@kernel.org>
>
> I'm not familiar with handling multi-subsystem patchsets (here: watchdog
> & MFD).
>
> Please kindly let me know: how to proceed with this patchset now to get
> it queued for Linus?
What is the requirement for these to be merged together?
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 8:44 ` Lee Jones
@ 2021-12-06 8:53 ` Rafał Miłecki
2021-12-06 9:05 ` Lee Jones
0 siblings, 1 reply; 21+ messages in thread
From: Rafał Miłecki @ 2021-12-06 8:53 UTC (permalink / raw)
To: Lee Jones
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 06.12.2021 09:44, Lee Jones wrote:
> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>> On 15.11.2021 06:53, Rafał Miłecki wrote:
>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>
>>> This helps validating DTS files.
>>>
>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>
>> I'm not familiar with handling multi-subsystem patchsets (here: watchdog
>> & MFD).
>>
>> Please kindly let me know: how to proceed with this patchset now to get
>> it queued for Linus?
>
> What is the requirement for these to be merged together?
If you merge 2/2 without 1/2 then people running "make dt_binding_check"
may see 1 extra warning until both patches meet in Linus's tree.
So it all comes to how much you care about amount of warnings produced
by "dt_binding_check".
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 8:53 ` Rafał Miłecki
@ 2021-12-06 9:05 ` Lee Jones
2021-12-06 17:20 ` Florian Fainelli
0 siblings, 1 reply; 21+ messages in thread
From: Lee Jones @ 2021-12-06 9:05 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On Mon, 06 Dec 2021, Rafał Miłecki wrote:
> On 06.12.2021 09:44, Lee Jones wrote:
> > On Mon, 06 Dec 2021, Rafał Miłecki wrote:
> > > On 15.11.2021 06:53, Rafał Miłecki wrote:
> > > > From: Rafał Miłecki <rafal@milecki.pl>
> > > >
> > > > This helps validating DTS files.
> > > >
> > > > Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> > > > Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> > > > Reviewed-by: Rob Herring <robh@kernel.org>
> > >
> > > I'm not familiar with handling multi-subsystem patchsets (here: watchdog
> > > & MFD).
> > >
> > > Please kindly let me know: how to proceed with this patchset now to get
> > > it queued for Linus?
> >
> > What is the requirement for these to be merged together?
>
> If you merge 2/2 without 1/2 then people running "make dt_binding_check"
> may see 1 extra warning until both patches meet in Linus's tree.
>
> So it all comes to how much you care about amount of warnings produced
> by "dt_binding_check".
In -next, I don't, but I know Rob gets excited about it.
Rob, what is your final word on this? Is it a forced requirement for
all interconnected document changes to go in together?
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 9:05 ` Lee Jones
@ 2021-12-06 17:20 ` Florian Fainelli
2021-12-06 18:55 ` Lee Jones
0 siblings, 1 reply; 21+ messages in thread
From: Florian Fainelli @ 2021-12-06 17:20 UTC (permalink / raw)
To: Lee Jones, Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 12/6/21 1:05 AM, Lee Jones wrote:
> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>
>> On 06.12.2021 09:44, Lee Jones wrote:
>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>> On 15.11.2021 06:53, Rafał Miłecki wrote:
>>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>>
>>>>> This helps validating DTS files.
>>>>>
>>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>>>
>>>> I'm not familiar with handling multi-subsystem patchsets (here: watchdog
>>>> & MFD).
>>>>
>>>> Please kindly let me know: how to proceed with this patchset now to get
>>>> it queued for Linus?
>>>
>>> What is the requirement for these to be merged together?
>>
>> If you merge 2/2 without 1/2 then people running "make dt_binding_check"
>> may see 1 extra warning until both patches meet in Linus's tree.
>>
>> So it all comes to how much you care about amount of warnings produced
>> by "dt_binding_check".
>
> In -next, I don't, but I know Rob gets excited about it.
>
> Rob, what is your final word on this? Is it a forced requirement for
> all interconnected document changes to go in together?
The first patch is queued up in Guenter's watchdog tree here:
https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/commit/?h=watchdog-next&id=a5b2ebc8f6e67b5c81023e8bde6b19ff48ffdb02
and will be submitted to Wim shortly I believe, so I suppose we should
take patch #2 via Guenter and Wim's tree as well logically.
--
Florian
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 17:20 ` Florian Fainelli
@ 2021-12-06 18:55 ` Lee Jones
2021-12-06 19:10 ` Guenter Roeck
0 siblings, 1 reply; 21+ messages in thread
From: Lee Jones @ 2021-12-06 18:55 UTC (permalink / raw)
To: Florian Fainelli
Cc: Rafał Miłecki, Wim Van Sebroeck, Guenter Roeck,
Rob Herring, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On Mon, 06 Dec 2021, Florian Fainelli wrote:
> On 12/6/21 1:05 AM, Lee Jones wrote:
> > On Mon, 06 Dec 2021, Rafał Miłecki wrote:
> >
> >> On 06.12.2021 09:44, Lee Jones wrote:
> >>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
> >>>> On 15.11.2021 06:53, Rafał Miłecki wrote:
> >>>>> From: Rafał Miłecki <rafal@milecki.pl>
> >>>>>
> >>>>> This helps validating DTS files.
> >>>>>
> >>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> >>>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> >>>>> Reviewed-by: Rob Herring <robh@kernel.org>
> >>>>
> >>>> I'm not familiar with handling multi-subsystem patchsets (here: watchdog
> >>>> & MFD).
> >>>>
> >>>> Please kindly let me know: how to proceed with this patchset now to get
> >>>> it queued for Linus?
> >>>
> >>> What is the requirement for these to be merged together?
> >>
> >> If you merge 2/2 without 1/2 then people running "make dt_binding_check"
> >> may see 1 extra warning until both patches meet in Linus's tree.
> >>
> >> So it all comes to how much you care about amount of warnings produced
> >> by "dt_binding_check".
> >
> > In -next, I don't, but I know Rob gets excited about it.
> >
> > Rob, what is your final word on this? Is it a forced requirement for
> > all interconnected document changes to go in together?
>
> The first patch is queued up in Guenter's watchdog tree here:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/commit/?h=watchdog-next&id=a5b2ebc8f6e67b5c81023e8bde6b19ff48ffdb02
>
> and will be submitted to Wim shortly I believe, so I suppose we should
> take patch #2 via Guenter and Wim's tree as well logically.
If that happens, I would like a PR to an immutable branch.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 18:55 ` Lee Jones
@ 2021-12-06 19:10 ` Guenter Roeck
2021-12-06 19:13 ` Florian Fainelli
0 siblings, 1 reply; 21+ messages in thread
From: Guenter Roeck @ 2021-12-06 19:10 UTC (permalink / raw)
To: Lee Jones, Florian Fainelli
Cc: Rafał Miłecki, Wim Van Sebroeck, Rob Herring,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 12/6/21 10:55 AM, Lee Jones wrote:
> On Mon, 06 Dec 2021, Florian Fainelli wrote:
>
>> On 12/6/21 1:05 AM, Lee Jones wrote:
>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>
>>>> On 06.12.2021 09:44, Lee Jones wrote:
>>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>>> On 15.11.2021 06:53, Rafał Miłecki wrote:
>>>>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>
>>>>>>> This helps validating DTS files.
>>>>>>>
>>>>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>>>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>>>>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>>>>>
>>>>>> I'm not familiar with handling multi-subsystem patchsets (here: watchdog
>>>>>> & MFD).
>>>>>>
>>>>>> Please kindly let me know: how to proceed with this patchset now to get
>>>>>> it queued for Linus?
>>>>>
>>>>> What is the requirement for these to be merged together?
>>>>
>>>> If you merge 2/2 without 1/2 then people running "make dt_binding_check"
>>>> may see 1 extra warning until both patches meet in Linus's tree.
>>>>
>>>> So it all comes to how much you care about amount of warnings produced
>>>> by "dt_binding_check".
>>>
>>> In -next, I don't, but I know Rob gets excited about it.
>>>
>>> Rob, what is your final word on this? Is it a forced requirement for
>>> all interconnected document changes to go in together?
>>
>> The first patch is queued up in Guenter's watchdog tree here:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/commit/?h=watchdog-next&id=a5b2ebc8f6e67b5c81023e8bde6b19ff48ffdb02
>>
>> and will be submitted to Wim shortly I believe, so I suppose we should
>> take patch #2 via Guenter and Wim's tree as well logically.
>
> If that happens, I would like a PR to an immutable branch.
>
I don't entirely see the point of that complexity for dt changes,
but whatever. Since my tree is not the official watchdog-next tree,
that means I can not take the entire series (which goes way beyond
the dt changes and also drops the bcm63xx driver). Unless I hear
otherwise, I'll drop the series from my tree for the time being
and wait for the dt changes to be sorted out.
Guenter
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 19:10 ` Guenter Roeck
@ 2021-12-06 19:13 ` Florian Fainelli
2021-12-06 19:37 ` Guenter Roeck
0 siblings, 1 reply; 21+ messages in thread
From: Florian Fainelli @ 2021-12-06 19:13 UTC (permalink / raw)
To: Guenter Roeck, Lee Jones, Florian Fainelli
Cc: Rafał Miłecki, Wim Van Sebroeck, Rob Herring,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 12/6/21 11:10 AM, Guenter Roeck wrote:
> On 12/6/21 10:55 AM, Lee Jones wrote:
>> On Mon, 06 Dec 2021, Florian Fainelli wrote:
>>
>>> On 12/6/21 1:05 AM, Lee Jones wrote:
>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>
>>>>> On 06.12.2021 09:44, Lee Jones wrote:
>>>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>>>> On 15.11.2021 06:53, Rafał Miłecki wrote:
>>>>>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>>
>>>>>>>> This helps validating DTS files.
>>>>>>>>
>>>>>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>>>>>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>>>>>>
>>>>>>> I'm not familiar with handling multi-subsystem patchsets (here:
>>>>>>> watchdog
>>>>>>> & MFD).
>>>>>>>
>>>>>>> Please kindly let me know: how to proceed with this patchset now
>>>>>>> to get
>>>>>>> it queued for Linus?
>>>>>>
>>>>>> What is the requirement for these to be merged together?
>>>>>
>>>>> If you merge 2/2 without 1/2 then people running "make
>>>>> dt_binding_check"
>>>>> may see 1 extra warning until both patches meet in Linus's tree.
>>>>>
>>>>> So it all comes to how much you care about amount of warnings produced
>>>>> by "dt_binding_check".
>>>>
>>>> In -next, I don't, but I know Rob gets excited about it.
>>>>
>>>> Rob, what is your final word on this? Is it a forced requirement for
>>>> all interconnected document changes to go in together?
>>>
>>> The first patch is queued up in Guenter's watchdog tree here:
>>>
>>> https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/commit/?h=watchdog-next&id=a5b2ebc8f6e67b5c81023e8bde6b19ff48ffdb02
>>>
>>>
>>> and will be submitted to Wim shortly I believe, so I suppose we should
>>> take patch #2 via Guenter and Wim's tree as well logically.
>>
>> If that happens, I would like a PR to an immutable branch.
>>
>
> I don't entirely see the point of that complexity for dt changes,
> but whatever. Since my tree is not the official watchdog-next tree,
> that means I can not take the entire series (which goes way beyond
> the dt changes and also drops the bcm63xx driver). Unless I hear
> otherwise, I'll drop the series from my tree for the time being
> and wait for the dt changes to be sorted out.
There is simply no rush in getting the bcm7038-wdt driver to support
4908 *just now*, so why don't you just take the bcm63xx-wdt series that
I posed, and Rafal posts an updated series that adds support for the
4908 watchdog for the 5.18 cycle?
--
Florian
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 19:13 ` Florian Fainelli
@ 2021-12-06 19:37 ` Guenter Roeck
2021-12-06 19:43 ` Florian Fainelli
0 siblings, 1 reply; 21+ messages in thread
From: Guenter Roeck @ 2021-12-06 19:37 UTC (permalink / raw)
To: Florian Fainelli, Lee Jones
Cc: Rafał Miłecki, Wim Van Sebroeck, Rob Herring,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 12/6/21 11:13 AM, Florian Fainelli wrote:
> On 12/6/21 11:10 AM, Guenter Roeck wrote:
>> On 12/6/21 10:55 AM, Lee Jones wrote:
>>> On Mon, 06 Dec 2021, Florian Fainelli wrote:
>>>
>>>> On 12/6/21 1:05 AM, Lee Jones wrote:
>>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>>
>>>>>> On 06.12.2021 09:44, Lee Jones wrote:
>>>>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>>>>> On 15.11.2021 06:53, Rafał Miłecki wrote:
>>>>>>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>>>
>>>>>>>>> This helps validating DTS files.
>>>>>>>>>
>>>>>>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>>>>>>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>>>>>>>
>>>>>>>> I'm not familiar with handling multi-subsystem patchsets (here:
>>>>>>>> watchdog
>>>>>>>> & MFD).
>>>>>>>>
>>>>>>>> Please kindly let me know: how to proceed with this patchset now
>>>>>>>> to get
>>>>>>>> it queued for Linus?
>>>>>>>
>>>>>>> What is the requirement for these to be merged together?
>>>>>>
>>>>>> If you merge 2/2 without 1/2 then people running "make
>>>>>> dt_binding_check"
>>>>>> may see 1 extra warning until both patches meet in Linus's tree.
>>>>>>
>>>>>> So it all comes to how much you care about amount of warnings produced
>>>>>> by "dt_binding_check".
>>>>>
>>>>> In -next, I don't, but I know Rob gets excited about it.
>>>>>
>>>>> Rob, what is your final word on this? Is it a forced requirement for
>>>>> all interconnected document changes to go in together?
>>>>
>>>> The first patch is queued up in Guenter's watchdog tree here:
>>>>
>>>> https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/commit/?h=watchdog-next&id=a5b2ebc8f6e67b5c81023e8bde6b19ff48ffdb02
>>>>
>>>>
>>>> and will be submitted to Wim shortly I believe, so I suppose we should
>>>> take patch #2 via Guenter and Wim's tree as well logically.
>>>
>>> If that happens, I would like a PR to an immutable branch.
>>>
>>
>> I don't entirely see the point of that complexity for dt changes,
>> but whatever. Since my tree is not the official watchdog-next tree,
>> that means I can not take the entire series (which goes way beyond
>> the dt changes and also drops the bcm63xx driver). Unless I hear
>> otherwise, I'll drop the series from my tree for the time being
>> and wait for the dt changes to be sorted out.
>
> There is simply no rush in getting the bcm7038-wdt driver to support
> 4908 *just now*, so why don't you just take the bcm63xx-wdt series that
> I posed, and Rafal posts an updated series that adds support for the
> 4908 watchdog for the 5.18 cycle?
>
Your series includes the patch discussed here, and it is the first patch
of your series. The second patch in your series depends on it. Are you
telling me that I should drop those two patches from your series ?
For reference, the patches are
079a2959e68b dt-bindings: watchdog: Add BCM6345 compatible to BCM7038 binding
a5b2ebc8f6e6 dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
in my watchdog-next branch.
Guenter
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 19:37 ` Guenter Roeck
@ 2021-12-06 19:43 ` Florian Fainelli
2021-12-06 19:51 ` Guenter Roeck
2021-12-06 21:14 ` Rafał Miłecki
0 siblings, 2 replies; 21+ messages in thread
From: Florian Fainelli @ 2021-12-06 19:43 UTC (permalink / raw)
To: Guenter Roeck, Florian Fainelli, Lee Jones
Cc: Rafał Miłecki, Wim Van Sebroeck, Rob Herring,
Justin Chen, bcm-kernel-feedback-list, linux-watchdog,
linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 12/6/21 11:37 AM, Guenter Roeck wrote:
> On 12/6/21 11:13 AM, Florian Fainelli wrote:
>> On 12/6/21 11:10 AM, Guenter Roeck wrote:
>>> On 12/6/21 10:55 AM, Lee Jones wrote:
>>>> On Mon, 06 Dec 2021, Florian Fainelli wrote:
>>>>
>>>>> On 12/6/21 1:05 AM, Lee Jones wrote:
>>>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>>>
>>>>>>> On 06.12.2021 09:44, Lee Jones wrote:
>>>>>>>> On Mon, 06 Dec 2021, Rafał Miłecki wrote:
>>>>>>>>> On 15.11.2021 06:53, Rafał Miłecki wrote:
>>>>>>>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>>>>
>>>>>>>>>> This helps validating DTS files.
>>>>>>>>>>
>>>>>>>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>>>>>>>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>>>>>>>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>>>>>>>>
>>>>>>>>> I'm not familiar with handling multi-subsystem patchsets (here:
>>>>>>>>> watchdog
>>>>>>>>> & MFD).
>>>>>>>>>
>>>>>>>>> Please kindly let me know: how to proceed with this patchset now
>>>>>>>>> to get
>>>>>>>>> it queued for Linus?
>>>>>>>>
>>>>>>>> What is the requirement for these to be merged together?
>>>>>>>
>>>>>>> If you merge 2/2 without 1/2 then people running "make
>>>>>>> dt_binding_check"
>>>>>>> may see 1 extra warning until both patches meet in Linus's tree.
>>>>>>>
>>>>>>> So it all comes to how much you care about amount of warnings
>>>>>>> produced
>>>>>>> by "dt_binding_check".
>>>>>>
>>>>>> In -next, I don't, but I know Rob gets excited about it.
>>>>>>
>>>>>> Rob, what is your final word on this? Is it a forced requirement for
>>>>>> all interconnected document changes to go in together?
>>>>>
>>>>> The first patch is queued up in Guenter's watchdog tree here:
>>>>>
>>>>> https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git/commit/?h=watchdog-next&id=a5b2ebc8f6e67b5c81023e8bde6b19ff48ffdb02
>>>>>
>>>>>
>>>>>
>>>>> and will be submitted to Wim shortly I believe, so I suppose we should
>>>>> take patch #2 via Guenter and Wim's tree as well logically.
>>>>
>>>> If that happens, I would like a PR to an immutable branch.
>>>>
>>>
>>> I don't entirely see the point of that complexity for dt changes,
>>> but whatever. Since my tree is not the official watchdog-next tree,
>>> that means I can not take the entire series (which goes way beyond
>>> the dt changes and also drops the bcm63xx driver). Unless I hear
>>> otherwise, I'll drop the series from my tree for the time being
>>> and wait for the dt changes to be sorted out.
>>
>> There is simply no rush in getting the bcm7038-wdt driver to support
>> 4908 *just now*, so why don't you just take the bcm63xx-wdt series that
>> I posed, and Rafal posts an updated series that adds support for the
>> 4908 watchdog for the 5.18 cycle?
>>
>
> Your series includes the patch discussed here, and it is the first patch
> of your series. The second patch in your series depends on it. Are you
> telling me that I should drop those two patches from your series ?
No, quite the contrary, I want you to keep the entire 7 patches that
converted the bcm7038-wdt binding to YAML and get rid of the bcm63xx-wdt
changes, the branch that you have right now is good in that regard.
I don't see why you should be creating an immutable branch for Lee and
not simply merge Rafal's "[PATCH V4 RESEND 2/2] dt-bindings: mfd: add
Broadcom's Timer-Watchdog block" patch with Lee's ack directly. This is
a new file, so I don't see how it would create conflicts as long as we
don't pile up changes on top.
--
Florian
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 19:43 ` Florian Fainelli
@ 2021-12-06 19:51 ` Guenter Roeck
2021-12-07 10:03 ` Lee Jones
2021-12-06 21:14 ` Rafał Miłecki
1 sibling, 1 reply; 21+ messages in thread
From: Guenter Roeck @ 2021-12-06 19:51 UTC (permalink / raw)
To: Florian Fainelli
Cc: Lee Jones, Rafał Miłecki, Wim Van Sebroeck,
Rob Herring, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On Mon, Dec 06, 2021 at 11:43:31AM -0800, Florian Fainelli wrote:
[ ... ]
> >
> > Your series includes the patch discussed here, and it is the first patch
> > of your series. The second patch in your series depends on it. Are you
> > telling me that I should drop those two patches from your series ?
>
> No, quite the contrary, I want you to keep the entire 7 patches that
> converted the bcm7038-wdt binding to YAML and get rid of the bcm63xx-wdt
> changes, the branch that you have right now is good in that regard.
>
> I don't see why you should be creating an immutable branch for Lee and
> not simply merge Rafal's "[PATCH V4 RESEND 2/2] dt-bindings: mfd: add
> Broadcom's Timer-Watchdog block" patch with Lee's ack directly. This is
> a new file, so I don't see how it would create conflicts as long as we
> don't pile up changes on top.
It sounded to me like Lee wanted an immutable branch for that, which I
can't do for watchdog. That means the options are to either drop the dt
changes, or to drop everything until after the dt issues are sorted out.
Guenter
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 19:51 ` Guenter Roeck
@ 2021-12-07 10:03 ` Lee Jones
2021-12-07 15:29 ` Guenter Roeck
0 siblings, 1 reply; 21+ messages in thread
From: Lee Jones @ 2021-12-07 10:03 UTC (permalink / raw)
To: Guenter Roeck
Cc: Florian Fainelli, Rafał Miłecki, Wim Van Sebroeck,
Rob Herring, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
Florian Fainelli wrote:
> I don't see why you should be creating an immutable branch for Lee and
> not simply merge Rafal's "[PATCH V4 RESEND 2/2] dt-bindings: mfd: add
> Broadcom's Timer-Watchdog block" patch with Lee's ack directly. This is
> a new file, so I don't see how it would create conflicts as long as we
> don't pile up changes on top.
Rafał Miłecki wrote:
> would that be OK for you to simply ack 2/2? So Guenter can pick my
> patch without the whole immutable branch & PR thing?
Guenter Roeck wrote:
> I don't entirely see the point of that complexity for dt changes,
> but whatever. Since my tree is not the official watchdog-next tree,
> that means I can not take the entire series (which goes way beyond
> the dt changes and also drops the bcm63xx driver). Unless I hear
> otherwise, I'll drop the series from my tree for the time being
> and wait for the dt changes to be sorted out.
If Rob wants `dt_binding_check` to run cleanly in -next, we have to
treat the DT documentation in the same manner we do for real code
when build dependencies exist between patches. Simply sucking them up
through a single repo is just dandy until subsequent changes are
required, which unfortunately is often the case.
Being the Maintainer of MFD, which is often the centre point of
cross-subsystems patch sets, I've been bitten by this too many times.
Hence my hesitancy to 'just Ack it and be done'.
I've been pushing back on the requirement for clean `dt_binding_check`
runs in -next for a while and would much prefer to treat it the same
way we do `checkpatch.pl`, whereby a clean run is not a hard
requirement. Instead it is used as one of many tools to check for
inconsistencies prior to submission (as possibly against patch-sets
once they are posted onto the list). However, just as we see false
positives in `checkpatch.pl` we should see them in `dt_binding_check`
where patches have simply been applied into different trees and may
lag each other by a week or two.
> It sounded to me like Lee wanted an immutable branch for that
Not exactly, I said:
"> Suppose we should take patch #2 via [Watchdog] as well.
If that happens, I would like a PR to an immutable branch."
The alternative is that I take the patch and provide an immutable
branch to you, which I am in a position to do.
Of course all of this hassle just goes away if the clean
`dt_binding_check` run on -next requirement is laxed and we can just
take our own patches without fear of wrath.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-07 10:03 ` Lee Jones
@ 2021-12-07 15:29 ` Guenter Roeck
2021-12-07 15:44 ` Lee Jones
0 siblings, 1 reply; 21+ messages in thread
From: Guenter Roeck @ 2021-12-07 15:29 UTC (permalink / raw)
To: Lee Jones
Cc: Florian Fainelli, Rafał Miłecki, Wim Van Sebroeck,
Rob Herring, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On 12/7/21 2:03 AM, Lee Jones wrote:
[ ... ]
>> It sounded to me like Lee wanted an immutable branch for that
>
> Not exactly, I said:
>
> "> Suppose we should take patch #2 via [Watchdog] as well.
>
> If that happens, I would like a PR to an immutable branch."
>
> The alternative is that I take the patch and provide an immutable
> branch to you, which I am in a position to do.
>
I understand, only I am not in a position to take it since my tree
isn't the official watchdog-next tree, and it doesn't show up in -next.
If Wim takes it into the official watchdog-next tree or not would be
completely up to him.
I personally don't care if the bindings check is clean in my inofficial
tree, so maybe this is a non-issue.
Guenter
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-07 15:29 ` Guenter Roeck
@ 2021-12-07 15:44 ` Lee Jones
2021-12-28 9:21 ` Wim Van Sebroeck
0 siblings, 1 reply; 21+ messages in thread
From: Lee Jones @ 2021-12-07 15:44 UTC (permalink / raw)
To: Guenter Roeck
Cc: Florian Fainelli, Rafał Miłecki, Wim Van Sebroeck,
Rob Herring, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On Tue, 07 Dec 2021, Guenter Roeck wrote:
> On 12/7/21 2:03 AM, Lee Jones wrote:
> [ ... ]
> > > It sounded to me like Lee wanted an immutable branch for that
> >
> > Not exactly, I said:
> >
> > "> Suppose we should take patch #2 via [Watchdog] as well.
> >
> > If that happens, I would like a PR to an immutable branch."
> >
> > The alternative is that I take the patch and provide an immutable
> > branch to you, which I am in a position to do.
> >
>
> I understand, only I am not in a position to take it since my tree
> isn't the official watchdog-next tree, and it doesn't show up in -next.
> If Wim takes it into the official watchdog-next tree or not would be
> completely up to him.
>
> I personally don't care if the bindings check is clean in my inofficial
> tree, so maybe this is a non-issue.
That doesn't help, sadly.
I think the best course of action is for Wim to let me know when this
patch makes it into his tree. I'll take the MFD one at the same time
and the two shall meet in -next.
Honestly, this is all such a faff.
Just to keep a script happy that 3 people care about.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-07 15:44 ` Lee Jones
@ 2021-12-28 9:21 ` Wim Van Sebroeck
2021-12-29 9:45 ` Lee Jones
0 siblings, 1 reply; 21+ messages in thread
From: Wim Van Sebroeck @ 2021-12-28 9:21 UTC (permalink / raw)
To: Lee Jones
Cc: Guenter Roeck, Florian Fainelli, Rafał Miłecki,
Wim Van Sebroeck, Rob Herring, Justin Chen,
bcm-kernel-feedback-list, linux-watchdog, linux-arm-kernel,
devicetree, linux-mips, Rafał Miłecki, Rob Herring
Hi Lee,
> On Tue, 07 Dec 2021, Guenter Roeck wrote:
>
> > On 12/7/21 2:03 AM, Lee Jones wrote:
> > [ ... ]
> > > > It sounded to me like Lee wanted an immutable branch for that
> > >
> > > Not exactly, I said:
> > >
> > > "> Suppose we should take patch #2 via [Watchdog] as well.
> > >
> > > If that happens, I would like a PR to an immutable branch."
> > >
> > > The alternative is that I take the patch and provide an immutable
> > > branch to you, which I am in a position to do.
> > >
> >
> > I understand, only I am not in a position to take it since my tree
> > isn't the official watchdog-next tree, and it doesn't show up in -next.
> > If Wim takes it into the official watchdog-next tree or not would be
> > completely up to him.
> >
> > I personally don't care if the bindings check is clean in my inofficial
> > tree, so maybe this is a non-issue.
>
> That doesn't help, sadly.
>
> I think the best course of action is for Wim to let me know when this
> patch makes it into his tree. I'll take the MFD one at the same time
> and the two shall meet in -next.
>
> Honestly, this is all such a faff.
>
> Just to keep a script happy that 3 people care about.
It's going in today.
Kind regards,
Wim.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-28 9:21 ` Wim Van Sebroeck
@ 2021-12-29 9:45 ` Lee Jones
0 siblings, 0 replies; 21+ messages in thread
From: Lee Jones @ 2021-12-29 9:45 UTC (permalink / raw)
To: Wim Van Sebroeck
Cc: Guenter Roeck, Florian Fainelli, Rafał Miłecki,
Rob Herring, Justin Chen, bcm-kernel-feedback-list,
linux-watchdog, linux-arm-kernel, devicetree, linux-mips,
Rafał Miłecki, Rob Herring
On Tue, 28 Dec 2021, Wim Van Sebroeck wrote:
> Hi Lee,
>
> > On Tue, 07 Dec 2021, Guenter Roeck wrote:
> >
> > > On 12/7/21 2:03 AM, Lee Jones wrote:
> > > [ ... ]
> > > > > It sounded to me like Lee wanted an immutable branch for that
> > > >
> > > > Not exactly, I said:
> > > >
> > > > "> Suppose we should take patch #2 via [Watchdog] as well.
> > > >
> > > > If that happens, I would like a PR to an immutable branch."
> > > >
> > > > The alternative is that I take the patch and provide an immutable
> > > > branch to you, which I am in a position to do.
> > > >
> > >
> > > I understand, only I am not in a position to take it since my tree
> > > isn't the official watchdog-next tree, and it doesn't show up in -next.
> > > If Wim takes it into the official watchdog-next tree or not would be
> > > completely up to him.
> > >
> > > I personally don't care if the bindings check is clean in my inofficial
> > > tree, so maybe this is a non-issue.
> >
> > That doesn't help, sadly.
> >
> > I think the best course of action is for Wim to let me know when this
> > patch makes it into his tree. I'll take the MFD one at the same time
> > and the two shall meet in -next.
> >
> > Honestly, this is all such a faff.
> >
> > Just to keep a script happy that 3 people care about.
>
> It's going in today.
Also applied.
Thanks Wim.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema
2021-12-06 19:43 ` Florian Fainelli
2021-12-06 19:51 ` Guenter Roeck
@ 2021-12-06 21:14 ` Rafał Miłecki
1 sibling, 0 replies; 21+ messages in thread
From: Rafał Miłecki @ 2021-12-06 21:14 UTC (permalink / raw)
To: Florian Fainelli, Guenter Roeck, Lee Jones
Cc: Wim Van Sebroeck, Rob Herring, Justin Chen,
bcm-kernel-feedback-list, linux-watchdog, linux-arm-kernel,
devicetree, linux-mips, Rafał Miłecki, Rob Herring
On 06.12.2021 20:43, Florian Fainelli wrote:
> On 12/6/21 11:37 AM, Guenter Roeck wrote:
>> On 12/6/21 11:13 AM, Florian Fainelli wrote:
>>> There is simply no rush in getting the bcm7038-wdt driver to support
>>> 4908 *just now*, so why don't you just take the bcm63xx-wdt series that
>>> I posed, and Rafal posts an updated series that adds support for the
>>> 4908 watchdog for the 5.18 cycle?
>>>
>>
>> Your series includes the patch discussed here, and it is the first patch
>> of your series. The second patch in your series depends on it. Are you
>> telling me that I should drop those two patches from your series ?
>
> No, quite the contrary, I want you to keep the entire 7 patches that
> converted the bcm7038-wdt binding to YAML and get rid of the bcm63xx-wdt
> changes, the branch that you have right now is good in that regard.
>
> I don't see why you should be creating an immutable branch for Lee and
> not simply merge Rafal's "[PATCH V4 RESEND 2/2] dt-bindings: mfd: add
> Broadcom's Timer-Watchdog block" patch with Lee's ack directly. This is
> a new file, so I don't see how it would create conflicts as long as we
> don't pile up changes on top.
I'd like that pretty much as I wouldn't need to wait few extra months.
Lee: would that be OK for you to simply ack 2/2? So Guenter can pick my
patch without the whole immutable branch & PR thing?
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2021-12-29 9:46 UTC | newest]
Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15 5:53 [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
2021-11-15 5:53 ` [PATCH V4 RESEND 2/2] dt-bindings: mfd: add Broadcom's Timer-Watchdog block Rafał Miłecki
2021-11-18 22:45 ` Rob Herring
2021-12-29 9:44 ` Lee Jones
2021-12-06 7:50 ` [PATCH V4 RESEND 1/2] dt-bindings: watchdog: convert Broadcom's WDT to the json-schema Rafał Miłecki
2021-12-06 8:44 ` Lee Jones
2021-12-06 8:53 ` Rafał Miłecki
2021-12-06 9:05 ` Lee Jones
2021-12-06 17:20 ` Florian Fainelli
2021-12-06 18:55 ` Lee Jones
2021-12-06 19:10 ` Guenter Roeck
2021-12-06 19:13 ` Florian Fainelli
2021-12-06 19:37 ` Guenter Roeck
2021-12-06 19:43 ` Florian Fainelli
2021-12-06 19:51 ` Guenter Roeck
2021-12-07 10:03 ` Lee Jones
2021-12-07 15:29 ` Guenter Roeck
2021-12-07 15:44 ` Lee Jones
2021-12-28 9:21 ` Wim Van Sebroeck
2021-12-29 9:45 ` Lee Jones
2021-12-06 21:14 ` Rafał Miłecki
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).