MAINTAINERS: Update SiFive driver maintainers

[PATCH] MAINTAINERS: Update SiFive driver maintainers

Posted by Samuel Holland 1 year, 11 months ago

Add myself as a maintainer for the various SiFive drivers, since I have
been performing cleanup activity on these drivers and reviewing patches
to them for a while now. Remove Palmer as a maintainer, as he is focused
on overall RISC-V architecture support.

Collapse some duplicate entries into the main SiFive drivers entry:
 - Conor is already maintainer of standalone cache drivers as a whole,
   and these files are also covered by the "sifive" file name regex.
 - Paul's git tree has not been updated since 2018, and all file names
   matching the "fu540" pattern also match the "sifive" pattern.
 - Green has not been active on the LKML for a couple of years.

Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
---

 MAINTAINERS | 29 +++++------------------------
 1 file changed, 5 insertions(+), 24 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 73d898383e51..f1bbb0f82664 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1395,6 +1395,7 @@ F:	drivers/hwmon/max31760.c
 
 ANALOGBITS PLL LIBRARIES
 M:	Paul Walmsley <paul.walmsley@sifive.com>
+M:	Samuel Holland <samuel.holland@sifive.com>
 S:	Supported
 F:	drivers/clk/analogbits/*
 F:	include/linux/clk/analogbits*
@@ -16725,6 +16726,7 @@ F:	drivers/pci/controller/dwc/*layerscape*
 PCI DRIVER FOR FU740
 M:	Paul Walmsley <paul.walmsley@sifive.com>
 M:	Greentime Hu <greentime.hu@sifive.com>
+M:	Samuel Holland <samuel.holland@sifive.com>
 L:	linux-pci@vger.kernel.org
 S:	Maintained
 F:	Documentation/devicetree/bindings/pci/sifive,fu740-pcie.yaml
@@ -19968,36 +19970,15 @@ S:	Maintained
 F:	drivers/watchdog/simatic-ipc-wdt.c
 
 SIFIVE DRIVERS
-M:	Palmer Dabbelt <palmer@dabbelt.com>
 M:	Paul Walmsley <paul.walmsley@sifive.com>
+M:	Samuel Holland <samuel.holland@sifive.com>
 L:	linux-riscv@lists.infradead.org
 S:	Supported
+F:	drivers/dma/sf-pdma/
 N:	sifive
+K:	fu[57]40
 K:	[^@]sifive
 
-SIFIVE CACHE DRIVER
-M:	Conor Dooley <conor@kernel.org>
-L:	linux-riscv@lists.infradead.org
-S:	Maintained
-F:	Documentation/devicetree/bindings/cache/sifive,ccache0.yaml
-F:	drivers/cache/sifive_ccache.c
-
-SIFIVE FU540 SYSTEM-ON-CHIP
-M:	Paul Walmsley <paul.walmsley@sifive.com>
-M:	Palmer Dabbelt <palmer@dabbelt.com>
-L:	linux-riscv@lists.infradead.org
-S:	Supported
-T:	git git://git.kernel.org/pub/scm/linux/kernel/git/pjw/sifive.git
-N:	fu540
-K:	fu540
-
-SIFIVE PDMA DRIVER
-M:	Green Wan <green.wan@sifive.com>
-S:	Maintained
-F:	Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml
-F:	drivers/dma/sf-pdma/
-
-
 SILEAD TOUCHSCREEN DRIVER
 M:	Hans de Goede <hdegoede@redhat.com>
 L:	linux-input@vger.kernel.org
-- 
2.43.0

Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

Posted by Conor Dooley 1 year, 11 months ago

Palmer,

On Thu, Feb 15, 2024 at 03:49:11PM -0800, Samuel Holland wrote:
> Add myself as a maintainer for the various SiFive drivers, since I have
> been performing cleanup activity on these drivers and reviewing patches
> to them for a while now. Remove Palmer as a maintainer, as he is focused
> on overall RISC-V architecture support.
> 
> Collapse some duplicate entries into the main SiFive drivers entry:
>  - Conor is already maintainer of standalone cache drivers as a whole,
>    and these files are also covered by the "sifive" file name regex.
>  - Paul's git tree has not been updated since 2018, and all file names
>    matching the "fu540" pattern also match the "sifive" pattern.
>  - Green has not been active on the LKML for a couple of years.
> 
> Signed-off-by: Samuel Holland <samuel.holland@sifive.com>

Can you take this please?

Thanks,
Conor.

Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

Posted by Palmer Dabbelt 1 year, 11 months ago

On Thu, 15 Feb 2024 15:49:11 PST (-0800), samuel.holland@sifive.com wrote:
> Add myself as a maintainer for the various SiFive drivers, since I have
> been performing cleanup activity on these drivers and reviewing patches
> to them for a while now. Remove Palmer as a maintainer, as he is focused
> on overall RISC-V architecture support.
>
> Collapse some duplicate entries into the main SiFive drivers entry:
>  - Conor is already maintainer of standalone cache drivers as a whole,
>    and these files are also covered by the "sifive" file name regex.
>  - Paul's git tree has not been updated since 2018, and all file names
>    matching the "fu540" pattern also match the "sifive" pattern.
>  - Green has not been active on the LKML for a couple of years.
>
> Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
> ---
>
>  MAINTAINERS | 29 +++++------------------------
>  1 file changed, 5 insertions(+), 24 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 73d898383e51..f1bbb0f82664 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1395,6 +1395,7 @@ F:	drivers/hwmon/max31760.c
>
>  ANALOGBITS PLL LIBRARIES
>  M:	Paul Walmsley <paul.walmsley@sifive.com>
> +M:	Samuel Holland <samuel.holland@sifive.com>
>  S:	Supported
>  F:	drivers/clk/analogbits/*
>  F:	include/linux/clk/analogbits*
> @@ -16725,6 +16726,7 @@ F:	drivers/pci/controller/dwc/*layerscape*
>  PCI DRIVER FOR FU740
>  M:	Paul Walmsley <paul.walmsley@sifive.com>
>  M:	Greentime Hu <greentime.hu@sifive.com>
> +M:	Samuel Holland <samuel.holland@sifive.com>
>  L:	linux-pci@vger.kernel.org
>  S:	Maintained
>  F:	Documentation/devicetree/bindings/pci/sifive,fu740-pcie.yaml
> @@ -19968,36 +19970,15 @@ S:	Maintained
>  F:	drivers/watchdog/simatic-ipc-wdt.c
>
>  SIFIVE DRIVERS
> -M:	Palmer Dabbelt <palmer@dabbelt.com>
>  M:	Paul Walmsley <paul.walmsley@sifive.com>
> +M:	Samuel Holland <samuel.holland@sifive.com>
>  L:	linux-riscv@lists.infradead.org
>  S:	Supported
> +F:	drivers/dma/sf-pdma/
>  N:	sifive
> +K:	fu[57]40
>  K:	[^@]sifive
>
> -SIFIVE CACHE DRIVER
> -M:	Conor Dooley <conor@kernel.org>
> -L:	linux-riscv@lists.infradead.org
> -S:	Maintained
> -F:	Documentation/devicetree/bindings/cache/sifive,ccache0.yaml
> -F:	drivers/cache/sifive_ccache.c
> -
> -SIFIVE FU540 SYSTEM-ON-CHIP
> -M:	Paul Walmsley <paul.walmsley@sifive.com>
> -M:	Palmer Dabbelt <palmer@dabbelt.com>
> -L:	linux-riscv@lists.infradead.org
> -S:	Supported
> -T:	git git://git.kernel.org/pub/scm/linux/kernel/git/pjw/sifive.git
> -N:	fu540
> -K:	fu540
> -
> -SIFIVE PDMA DRIVER
> -M:	Green Wan <green.wan@sifive.com>
> -S:	Maintained
> -F:	Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml
> -F:	drivers/dma/sf-pdma/
> -
> -
>  SILEAD TOUCHSCREEN DRIVER
>  M:	Hans de Goede <hdegoede@redhat.com>
>  L:	linux-input@vger.kernel.org

I'm fine not having to maintain the SiFive drivers, the SOC stuff was 
never really my thing anyway so that's all a bit easier on my end.  So

Acked-by: Palmer Dabbelt <palmer@rivosinc.com>

I do end up picking up some random patches though, mostly because stuff 
seems to slip through the cracks.  Are you going to have an SOC subtree 
and send stuff over there?  That seems more manageable on my end, as 
it's a bit scattered today.

Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

Posted by Conor Dooley 1 year, 11 months ago

On Fri, Feb 16, 2024 at 10:43:31AM -0800, Palmer Dabbelt wrote:
> On Thu, 15 Feb 2024 15:49:11 PST (-0800), samuel.holland@sifive.com wrote:
> > Add myself as a maintainer for the various SiFive drivers, since I have
> > been performing cleanup activity on these drivers and reviewing patches
> > to them for a while now. Remove Palmer as a maintainer, as he is focused
> > on overall RISC-V architecture support.
> > 
> > Collapse some duplicate entries into the main SiFive drivers entry:
> >  - Conor is already maintainer of standalone cache drivers as a whole,
> >    and these files are also covered by the "sifive" file name regex.
> >  - Paul's git tree has not been updated since 2018, and all file names
> >    matching the "fu540" pattern also match the "sifive" pattern.
> >  - Green has not been active on the LKML for a couple of years.
> > 
> > Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
> > ---
> > 
> >  MAINTAINERS | 29 +++++------------------------
> >  1 file changed, 5 insertions(+), 24 deletions(-)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 73d898383e51..f1bbb0f82664 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1395,6 +1395,7 @@ F:	drivers/hwmon/max31760.c
> > 
> >  ANALOGBITS PLL LIBRARIES
> >  M:	Paul Walmsley <paul.walmsley@sifive.com>
> > +M:	Samuel Holland <samuel.holland@sifive.com>
> >  S:	Supported
> >  F:	drivers/clk/analogbits/*
> >  F:	include/linux/clk/analogbits*
> > @@ -16725,6 +16726,7 @@ F:	drivers/pci/controller/dwc/*layerscape*
> >  PCI DRIVER FOR FU740
> >  M:	Paul Walmsley <paul.walmsley@sifive.com>
> >  M:	Greentime Hu <greentime.hu@sifive.com>
> > +M:	Samuel Holland <samuel.holland@sifive.com>
> >  L:	linux-pci@vger.kernel.org
> >  S:	Maintained
> >  F:	Documentation/devicetree/bindings/pci/sifive,fu740-pcie.yaml
> > @@ -19968,36 +19970,15 @@ S:	Maintained
> >  F:	drivers/watchdog/simatic-ipc-wdt.c
> > 
> >  SIFIVE DRIVERS
> > -M:	Palmer Dabbelt <palmer@dabbelt.com>
> >  M:	Paul Walmsley <paul.walmsley@sifive.com>
> > +M:	Samuel Holland <samuel.holland@sifive.com>
> >  L:	linux-riscv@lists.infradead.org
> >  S:	Supported
> > +F:	drivers/dma/sf-pdma/
> >  N:	sifive
> > +K:	fu[57]40
> >  K:	[^@]sifive
> > 
> > -SIFIVE CACHE DRIVER
> > -M:	Conor Dooley <conor@kernel.org>
> > -L:	linux-riscv@lists.infradead.org
> > -S:	Maintained
> > -F:	Documentation/devicetree/bindings/cache/sifive,ccache0.yaml
> > -F:	drivers/cache/sifive_ccache.c
> > -
> > -SIFIVE FU540 SYSTEM-ON-CHIP
> > -M:	Paul Walmsley <paul.walmsley@sifive.com>
> > -M:	Palmer Dabbelt <palmer@dabbelt.com>
> > -L:	linux-riscv@lists.infradead.org
> > -S:	Supported
> > -T:	git git://git.kernel.org/pub/scm/linux/kernel/git/pjw/sifive.git
> > -N:	fu540
> > -K:	fu540
> > -
> > -SIFIVE PDMA DRIVER
> > -M:	Green Wan <green.wan@sifive.com>
> > -S:	Maintained
> > -F:	Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml
> > -F:	drivers/dma/sf-pdma/
> > -
> > -
> >  SILEAD TOUCHSCREEN DRIVER
> >  M:	Hans de Goede <hdegoede@redhat.com>
> >  L:	linux-input@vger.kernel.org
> 
> I'm fine not having to maintain the SiFive drivers, the SOC stuff was never
> really my thing anyway so that's all a bit easier on my end.  So
> 
> Acked-by: Palmer Dabbelt <palmer@rivosinc.com>
> 
> I do end up picking up some random patches though, mostly because stuff
> seems to slip through the cracks.  Are you going to have an SOC subtree and
> send stuff over there?  That seems more manageable on my end, as it's a bit
> scattered today.

I don't think anything in this patch was "supposed" to go via the riscv
tree as things stood. Probably more a case of harassing subsystem
maintainers (or sending them PRs) than sending things to Arnd I think.

Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

Posted by Conor Dooley 1 year, 11 months ago

On Thu, Feb 15, 2024 at 03:49:11PM -0800, Samuel Holland wrote:
> Add myself as a maintainer for the various SiFive drivers, since I have
> been performing cleanup activity on these drivers and reviewing patches
> to them for a while now. Remove Palmer as a maintainer, as he is focused
> on overall RISC-V architecture support.
> 
> Collapse some duplicate entries into the main SiFive drivers entry:
>  - Conor is already maintainer of standalone cache drivers as a whole,
>    and these files are also covered by the "sifive" file name regex.

And the binding isn't affected since I get all binding patches anyway.

>  - Paul's git tree has not been updated since 2018, and all file names
>    matching the "fu540" pattern also match the "sifive" pattern.
>  - Green has not been active on the LKML for a couple of years.
> 
> Signed-off-by: Samuel Holland <samuel.holland@sifive.com>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks for doing this :)

Cheers,
Conor.

Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

Posted by Paul Walmsley 1 year, 11 months ago

On Thu, 15 Feb 2024, Samuel Holland wrote:

> Add myself as a maintainer for the various SiFive drivers, since I have
> been performing cleanup activity on these drivers and reviewing patches
> to them for a while now. Remove Palmer as a maintainer, as he is focused
> on overall RISC-V architecture support.
> 
> Collapse some duplicate entries into the main SiFive drivers entry:
>  - Conor is already maintainer of standalone cache drivers as a whole,
>    and these files are also covered by the "sifive" file name regex.
>  - Paul's git tree has not been updated since 2018, and all file names
>    matching the "fu540" pattern also match the "sifive" pattern.
>  - Green has not been active on the LKML for a couple of years.
> 
> Signed-off-by: Samuel Holland <samuel.holland@sifive.com>

Acked-by: Paul Walmsley <paul.walmsley@sifive.com>

- Paul