From: Liang Chen <[email protected]>
add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
a rk3568 platform to pwm-rockchip.yaml.
Signed-off-by: Liang Chen <[email protected]>
---
Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml b/Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml
index 5596bee70509..81a54a4e8e3e 100644
--- a/Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml
+++ b/Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml
@@ -29,6 +29,7 @@ properties:
- enum:
- rockchip,px30-pwm
- rockchip,rk3308-pwm
+ - rockchip,rk3568-pwm
- const: rockchip,rk3328-pwm
reg:
--
2.17.1
On Wed, Jun 23, 2021 at 10:13:03AM +0800, [email protected] wrote:
> From: Liang Chen <[email protected]>
>
> add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
> a rk3568 platform to pwm-rockchip.yaml.
Looks good to me
Acked-by: Uwe Kleine-K?nig <[email protected]>
Who is supposed to apply this patch? Does this need blessing by Rob?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |
On Mon, 05 Jul 2021, Uwe Kleine-König wrote:
> On Wed, Jun 23, 2021 at 10:13:03AM +0800, [email protected] wrote:
> > From: Liang Chen <[email protected]>
> >
> > add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
> > a rk3568 platform to pwm-rockchip.yaml.
>
> Looks good to me
>
> Acked-by: Uwe Kleine-König <[email protected]>
>
> Who is supposed to apply this patch? Does this need blessing by Rob?
There is no standard. Rob will usually have some kind of arrangement
with the associated maintainer(s). If this is a big functional
change, I would suggest letting Rob and his army of bots give it the
once over before it is applied in any case.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
Hello Lee,
On Mon, Jul 05, 2021 at 08:34:18AM +0100, Lee Jones wrote:
> On Mon, 05 Jul 2021, Uwe Kleine-K?nig wrote:
> > On Wed, Jun 23, 2021 at 10:13:03AM +0800, [email protected] wrote:
> > > From: Liang Chen <[email protected]>
> > >
> > > add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
> > > a rk3568 platform to pwm-rockchip.yaml.
> >
> > [...]
> >
> > Who is supposed to apply this patch? Does this need blessing by Rob?
>
> There is no standard. [...]
I'm aware of that. That's why I asked to prevent that everybody thinks
some other maintainer will care for it.
Thanks
Uwe
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |
Am Montag, 5. Juli 2021, 09:44:05 CEST schrieb Uwe Kleine-K?nig:
> Hello Lee,
>
> On Mon, Jul 05, 2021 at 08:34:18AM +0100, Lee Jones wrote:
> > On Mon, 05 Jul 2021, Uwe Kleine-K?nig wrote:
> > > On Wed, Jun 23, 2021 at 10:13:03AM +0800, [email protected] wrote:
> > > > From: Liang Chen <[email protected]>
> > > >
> > > > add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
> > > > a rk3568 platform to pwm-rockchip.yaml.
> > >
> > > [...]
> > >
> > > Who is supposed to apply this patch? Does this need blessing by Rob?
> >
> > There is no standard. [...]
>
> I'm aware of that. That's why I asked to prevent that everybody thinks
> some other maintainer will care for it.
I suppose Thierry would be the one responsible?
I.e. bindings normally go through the subsystem tree and
$ scripts/get_maintainer.pl drivers/pwm
returns:
Thierry Reding <[email protected]> (maintainer:PWM SUBSYSTEM)
"Uwe Kleine-K?nig" <[email protected]> (reviewer:PWM SUBSYSTEM)
Lee Jones <[email protected]> (maintainer:PWM SUBSYSTEM)
[email protected] (open list:PWM SUBSYSTEM)
[email protected] (open list)
See my reply to the original pwm binding patch from 2021-06-21.
Heiko
On Mon, Jul 05, 2021 at 08:34:18AM +0100, Lee Jones wrote:
> On Mon, 05 Jul 2021, Uwe Kleine-K?nig wrote:
>
> > On Wed, Jun 23, 2021 at 10:13:03AM +0800, [email protected] wrote:
> > > From: Liang Chen <[email protected]>
> > >
> > > add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
> > > a rk3568 platform to pwm-rockchip.yaml.
> >
> > Looks good to me
> >
> > Acked-by: Uwe Kleine-K?nig <[email protected]>
> >
> > Who is supposed to apply this patch? Does this need blessing by Rob?
>
> There is no standard. Rob will usually have some kind of arrangement
> with the associated maintainer(s). If this is a big functional
> change, I would suggest letting Rob and his army of bots give it the
> once over before it is applied in any case.
The documented standard[1] is subsystem maintainers take bindings. Are
there exceptions? Yes. Usually that's standalone patches (which netdev
maintainers just ignore for example).
One line compatible string changes don't need to wait for me.
Rob
[1] https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/tree/Documentation/devicetree/bindings/submitting-patches.rst#n67
On Wed, 23 Jun 2021 10:13:03 +0800, [email protected] wrote:
> From: Liang Chen <[email protected]>
>
> add "rockchip,rk3568-pwm", "rockchip,rk3328-pwm" for pwm nodes on
> a rk3568 platform to pwm-rockchip.yaml.
>
> Signed-off-by: Liang Chen <[email protected]>
> ---
> Documentation/devicetree/bindings/pwm/pwm-rockchip.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
Acked-by: Rob Herring <[email protected]>