2019-12-04 09:08:49

by Miquel Raynal

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: rockchip: Track more files

The current list misses a lot of drivers not prefixed or suffixed by
"rockchip". For instance, there are plenty drivers called rk808 and
rk805 which are currently not tracked (clk, regulator, pinctrl, RTC,
MFD, includes, bindings). Add them to the list under the Rockchip
entry.

Signed-off-by: Miquel Raynal <[email protected]>
---

Hi Heiko,

You are right we should try to check more often your tree. Also, here
is a patch so that you are Cc'ed for all Rockchip related patches
because the current list is not exhaustive at all (not sure it is
voluntary or not though).

Cheers,
Miquèl

MAINTAINERS | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index cba1095547fd..a9564e6cb872 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2198,12 +2198,16 @@ L: [email protected]
T: git git://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git
S: Maintained
F: Documentation/devicetree/bindings/i2c/i2c-rk3x.txt
+F: Documentation/devicetree/bindings/*/*rk80*
F: arch/arm/boot/dts/rk3*
F: arch/arm/boot/dts/rv1108*
F: arch/arm/mach-rockchip/
+F: include/*/*/rk808.h
+F: include/*/*/*/rk808.h
F: drivers/clk/rockchip/
F: drivers/i2c/busses/i2c-rk3x.c
F: drivers/*/*rockchip*
+F: drivers/*/*rk80*
F: drivers/*/*/*rockchip*
F: sound/soc/rockchip/
N: rockchip
--
2.20.1


2019-12-09 08:24:50

by Kever Yang

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: rockchip: Track more files

Hi Miquel,

On 2019/12/4 下午5:07, Miquel Raynal wrote:
> The current list misses a lot of drivers not prefixed or suffixed by
> "rockchip". For instance, there are plenty drivers called rk808 and
> rk805 which are currently not tracked (clk, regulator, pinctrl, RTC,
> MFD, includes, bindings). Add them to the list under the Rockchip
> entry.
>
> Signed-off-by: Miquel Raynal <[email protected]>
> ---
>
> Hi Heiko,
>
> You are right we should try to check more often your tree. Also, here
> is a patch so that you are Cc'ed for all Rockchip related patches
> because the current list is not exhaustive at all (not sure it is
> voluntary or not though).
>
> Cheers,
> Miquèl
>
> MAINTAINERS | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cba1095547fd..a9564e6cb872 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2198,12 +2198,16 @@ L: [email protected]
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git
> S: Maintained
> F: Documentation/devicetree/bindings/i2c/i2c-rk3x.txt
> +F: Documentation/devicetree/bindings/*/*rk80*
> F: arch/arm/boot/dts/rk3*
> F: arch/arm/boot/dts/rv1108*
> F: arch/arm/mach-rockchip/
> +F: include/*/*/rk808.h
> +F: include/*/*/*/rk808.h
> F: drivers/clk/rockchip/
> F: drivers/i2c/busses/i2c-rk3x.c
> F: drivers/*/*rockchip*
> +F: drivers/*/*rk80*
> F: drivers/*/*/*rockchip*
> F: sound/soc/rockchip/
> N: rockchip
For the Rockchip PMIC, is it better to have a NEW MAINTAINER entry like
"ROCKCHIP PMIC DRIVER"
which share the same mailing list linux-rockchip@, because there is not
only rk808,
but also rk805, rk809, rk817, rk818, and may be more to come.

BTW, we should use 'rk8' instead of 'rk80' to match all the Rockchip PMICs.


Thanks,
- Kever



2019-12-13 10:28:36

by Heiko Stübner

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: rockchip: Track more files

Am Montag, 9. Dezember 2019, 09:18:00 CET schrieb Kever Yang:
> Hi Miquel,
>
> On 2019/12/4 下午5:07, Miquel Raynal wrote:
> > The current list misses a lot of drivers not prefixed or suffixed by
> > "rockchip". For instance, there are plenty drivers called rk808 and
> > rk805 which are currently not tracked (clk, regulator, pinctrl, RTC,
> > MFD, includes, bindings). Add them to the list under the Rockchip
> > entry.
> >
> > Signed-off-by: Miquel Raynal <[email protected]>
> > ---
> >
> > Hi Heiko,
> >
> > You are right we should try to check more often your tree. Also, here
> > is a patch so that you are Cc'ed for all Rockchip related patches
> > because the current list is not exhaustive at all (not sure it is
> > voluntary or not though).
> >
> > Cheers,
> > Miquèl
> >
> > MAINTAINERS | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index cba1095547fd..a9564e6cb872 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -2198,12 +2198,16 @@ L: [email protected]
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git
> > S: Maintained
> > F: Documentation/devicetree/bindings/i2c/i2c-rk3x.txt
> > +F: Documentation/devicetree/bindings/*/*rk80*
> > F: arch/arm/boot/dts/rk3*
> > F: arch/arm/boot/dts/rv1108*
> > F: arch/arm/mach-rockchip/
> > +F: include/*/*/rk808.h
> > +F: include/*/*/*/rk808.h
> > F: drivers/clk/rockchip/
> > F: drivers/i2c/busses/i2c-rk3x.c
> > F: drivers/*/*rockchip*
> > +F: drivers/*/*rk80*
> > F: drivers/*/*/*rockchip*
> > F: sound/soc/rockchip/
> > N: rockchip
> For the Rockchip PMIC, is it better to have a NEW MAINTAINER entry like
> "ROCKCHIP PMIC DRIVER"
> which share the same mailing list linux-rockchip@, because there is not
> only rk808,
> but also rk805, rk809, rk817, rk818, and may be more to come.
>
> BTW, we should use 'rk8' instead of 'rk80' to match all the Rockchip PMICs.

yep, I agree with Kever on this ... also that new pmic-related entry should
not really link to a git tree as patches for it will get distributed through
multiple trees (mfd, regulator, rtc) most of the time.

Miquel can you do a v2 please?

Thanks
Heiko


2019-12-13 10:37:25

by Miquel Raynal

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: rockchip: Track more files

Hi Heiko,

Heiko Stuebner <[email protected]> wrote on Fri, 13 Dec 2019 11:27:29
+0100:

> Am Montag, 9. Dezember 2019, 09:18:00 CET schrieb Kever Yang:
> > Hi Miquel,
> >
> > On 2019/12/4 下午5:07, Miquel Raynal wrote:
> > > The current list misses a lot of drivers not prefixed or suffixed by
> > > "rockchip". For instance, there are plenty drivers called rk808 and
> > > rk805 which are currently not tracked (clk, regulator, pinctrl, RTC,
> > > MFD, includes, bindings). Add them to the list under the Rockchip
> > > entry.
> > >
> > > Signed-off-by: Miquel Raynal <[email protected]>
> > > ---
> > >
> > > Hi Heiko,
> > >
> > > You are right we should try to check more often your tree. Also, here
> > > is a patch so that you are Cc'ed for all Rockchip related patches
> > > because the current list is not exhaustive at all (not sure it is
> > > voluntary or not though).
> > >
> > > Cheers,
> > > Miquèl
> > >
> > > MAINTAINERS | 4 ++++
> > > 1 file changed, 4 insertions(+)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index cba1095547fd..a9564e6cb872 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -2198,12 +2198,16 @@ L: [email protected]
> > > T: git git://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git
> > > S: Maintained
> > > F: Documentation/devicetree/bindings/i2c/i2c-rk3x.txt
> > > +F: Documentation/devicetree/bindings/*/*rk80*
> > > F: arch/arm/boot/dts/rk3*
> > > F: arch/arm/boot/dts/rv1108*
> > > F: arch/arm/mach-rockchip/
> > > +F: include/*/*/rk808.h
> > > +F: include/*/*/*/rk808.h
> > > F: drivers/clk/rockchip/
> > > F: drivers/i2c/busses/i2c-rk3x.c
> > > F: drivers/*/*rockchip*
> > > +F: drivers/*/*rk80*
> > > F: drivers/*/*/*rockchip*
> > > F: sound/soc/rockchip/
> > > N: rockchip
> > For the Rockchip PMIC, is it better to have a NEW MAINTAINER entry like
> > "ROCKCHIP PMIC DRIVER"
> > which share the same mailing list linux-rockchip@, because there is not
> > only rk808,
> > but also rk805, rk809, rk817, rk818, and may be more to come.
> >
> > BTW, we should use 'rk8' instead of 'rk80' to match all the Rockchip PMICs.
>
> yep, I agree with Kever on this ... also that new pmic-related entry should
> not really link to a git tree as patches for it will get distributed through
> multiple trees (mfd, regulator, rtc) most of the time.

I think it is important that the maintainer receive all Rockchip
related patches, as he has the bigger picture in mind. Of course it is
possible to add an additional MAINTAINER entry for Rockchip PMIC
drivers specifically.

So in the end, I will s/rk80/rk8/, shall I remove a specific entry from
the 4 added?

>
> Miquel can you do a v2 please?

Of course, this is planned, I'm just a bit late in my e-mail backlog!

Cheers,
Miquèl