| Message ID | 20240823150057.56141-10-detlev.casanova@collabora.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add device tree for ArmSoM Sige 5 board | expand |
On Fri, Aug 23, 2024 at 10:52:36AM -0400, Detlev Casanova wrote: > It is compatible with the other rockchip SoCs. > > Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Fri, Aug 23, 2024 at 10:52:36AM -0400, Detlev Casanova wrote: > It is compatible with the other rockchip SoCs. > > Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> Acked-by: Guenter Roeck <linux@roeck-us.net> > --- > Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml b/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > index c7aab0418a320..b5a3dc3770706 100644 > --- a/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > +++ b/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > @@ -29,6 +29,7 @@ properties: > - rockchip,rk3368-wdt > - rockchip,rk3399-wdt > - rockchip,rk3568-wdt > + - rockchip,rk3576-wdt > - rockchip,rk3588-wdt > - rockchip,rv1108-wdt > - const: snps,dw-wdt > -- > 2.46.0 >
Hi Guenter, Am Samstag, 24. August 2024, 18:44:29 CEST schrieb Guenter Roeck: > On Fri, Aug 23, 2024 at 10:52:36AM -0400, Detlev Casanova wrote: > > It is compatible with the other rockchip SoCs. > > > > Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> > > Acked-by: Guenter Roeck <linux@roeck-us.net> For my understanding, does this Ack mean you expect the patch to go in with the other patches? Because in theory this patch could also be picked and go through the watchdog tree, similar to how the saradc binding went into the iio tree already. Thanks Heiko
On 8/27/24 00:20, Heiko Stübner wrote: > Hi Guenter, > > Am Samstag, 24. August 2024, 18:44:29 CEST schrieb Guenter Roeck: >> On Fri, Aug 23, 2024 at 10:52:36AM -0400, Detlev Casanova wrote: >>> It is compatible with the other rockchip SoCs. >>> >>> Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> >> >> Acked-by: Guenter Roeck <linux@roeck-us.net> > > For my understanding, does this Ack mean you expect the patch to go in > with the other patches? > Yes > Because in theory this patch could also be picked and go through the > watchdog tree, similar to how the saradc binding went into the > iio tree already. > I thought it was all supposed to be pushed together. Guenter
Hi Guenter, Am Dienstag, 27. August 2024, 21:38:35 CEST schrieb Guenter Roeck: > On 8/27/24 00:20, Heiko Stübner wrote: > > Am Samstag, 24. August 2024, 18:44:29 CEST schrieb Guenter Roeck: > >> On Fri, Aug 23, 2024 at 10:52:36AM -0400, Detlev Casanova wrote: > >>> It is compatible with the other rockchip SoCs. > >>> > >>> Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> > >> > >> Acked-by: Guenter Roeck <linux@roeck-us.net> > > > > For my understanding, does this Ack mean you expect the patch to go in > > with the other patches? > > > Yes > > > Because in theory this patch could also be picked and go through the > > watchdog tree, similar to how the saradc binding went into the > > iio tree already. > > > > I thought it was all supposed to be pushed together. I think at this point the rk3576 has still quite a number of separate pieces. A series for clocks, one for mmc and I think there are more. So for stuff that is obvious, like the saradc compatible Jonathan already, picked or this watchdog compatible, a strategy of "someone picks the individual" patch also is helpful, because it reduces the number of pieces on the "chess board" ;-) . So I guess both ways (wdt binding getting applied, or me just applying it with the rest) are doable and we'll just follow what you feel comfortable with doing. Heiko
diff --git a/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml b/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml index c7aab0418a320..b5a3dc3770706 100644 --- a/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml +++ b/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml @@ -29,6 +29,7 @@ properties: - rockchip,rk3368-wdt - rockchip,rk3399-wdt - rockchip,rk3568-wdt + - rockchip,rk3576-wdt - rockchip,rk3588-wdt - rockchip,rv1108-wdt - const: snps,dw-wdt
It is compatible with the other rockchip SoCs. Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> --- Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml | 1 + 1 file changed, 1 insertion(+)