2024-02-15 23:52:10

by Samuel Holland

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: Update SiFive driver maintainers

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 <[email protected]>
---

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 <[email protected]>
+M: Samuel Holland <[email protected]>
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 <[email protected]>
M: Greentime Hu <[email protected]>
+M: Samuel Holland <[email protected]>
L: [email protected]
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 <[email protected]>
M: Paul Walmsley <[email protected]>
+M: Samuel Holland <[email protected]>
L: [email protected]
S: Supported
+F: drivers/dma/sf-pdma/
N: sifive
+K: fu[57]40
K: [^@]sifive

-SIFIVE CACHE DRIVER
-M: Conor Dooley <[email protected]>
-L: [email protected]
-S: Maintained
-F: Documentation/devicetree/bindings/cache/sifive,ccache0.yaml
-F: drivers/cache/sifive_ccache.c
-
-SIFIVE FU540 SYSTEM-ON-CHIP
-M: Paul Walmsley <[email protected]>
-M: Palmer Dabbelt <[email protected]>
-L: [email protected]
-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 <[email protected]>
-S: Maintained
-F: Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml
-F: drivers/dma/sf-pdma/
-
-
SILEAD TOUCHSCREEN DRIVER
M: Hans de Goede <[email protected]>
L: [email protected]
--
2.43.0



2024-02-16 00:44:11

by Paul Walmsley

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

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 <[email protected]>

Acked-by: Paul Walmsley <[email protected]>

- Paul


2024-02-16 09:39:30

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

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 <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Thanks for doing this :)

Cheers,
Conor.


Attachments:
(No filename) (0.98 kB)
signature.asc (235.00 B)
Download all attachments

2024-02-16 18:47:13

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

On Thu, 15 Feb 2024 15:49:11 PST (-0800), [email protected] 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 <[email protected]>
> ---
>
> 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 <[email protected]>
> +M: Samuel Holland <[email protected]>
> 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 <[email protected]>
> M: Greentime Hu <[email protected]>
> +M: Samuel Holland <[email protected]>
> L: [email protected]
> 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 <[email protected]>
> M: Paul Walmsley <[email protected]>
> +M: Samuel Holland <[email protected]>
> L: [email protected]
> S: Supported
> +F: drivers/dma/sf-pdma/
> N: sifive
> +K: fu[57]40
> K: [^@]sifive
>
> -SIFIVE CACHE DRIVER
> -M: Conor Dooley <[email protected]>
> -L: [email protected]
> -S: Maintained
> -F: Documentation/devicetree/bindings/cache/sifive,ccache0.yaml
> -F: drivers/cache/sifive_ccache.c
> -
> -SIFIVE FU540 SYSTEM-ON-CHIP
> -M: Paul Walmsley <[email protected]>
> -M: Palmer Dabbelt <[email protected]>
> -L: [email protected]
> -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 <[email protected]>
> -S: Maintained
> -F: Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml
> -F: drivers/dma/sf-pdma/
> -
> -
> SILEAD TOUCHSCREEN DRIVER
> M: Hans de Goede <[email protected]>
> L: [email protected]

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 <[email protected]>

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.

2024-02-16 19:22:26

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

On Fri, Feb 16, 2024 at 10:43:31AM -0800, Palmer Dabbelt wrote:
> On Thu, 15 Feb 2024 15:49:11 PST (-0800), [email protected] 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 <[email protected]>
> > ---
> >
> > 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 <[email protected]>
> > +M: Samuel Holland <[email protected]>
> > 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 <[email protected]>
> > M: Greentime Hu <[email protected]>
> > +M: Samuel Holland <[email protected]>
> > L: [email protected]
> > 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 <[email protected]>
> > M: Paul Walmsley <[email protected]>
> > +M: Samuel Holland <[email protected]>
> > L: [email protected]
> > S: Supported
> > +F: drivers/dma/sf-pdma/
> > N: sifive
> > +K: fu[57]40
> > K: [^@]sifive
> >
> > -SIFIVE CACHE DRIVER
> > -M: Conor Dooley <[email protected]>
> > -L: [email protected]
> > -S: Maintained
> > -F: Documentation/devicetree/bindings/cache/sifive,ccache0.yaml
> > -F: drivers/cache/sifive_ccache.c
> > -
> > -SIFIVE FU540 SYSTEM-ON-CHIP
> > -M: Paul Walmsley <[email protected]>
> > -M: Palmer Dabbelt <[email protected]>
> > -L: [email protected]
> > -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 <[email protected]>
> > -S: Maintained
> > -F: Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml
> > -F: drivers/dma/sf-pdma/
> > -
> > -
> > SILEAD TOUCHSCREEN DRIVER
> > M: Hans de Goede <[email protected]>
> > L: [email protected]
>
> 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 <[email protected]>
>
> 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.


Attachments:
(No filename) (3.72 kB)
signature.asc (235.00 B)
Download all attachments

2024-02-27 10:09:33

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

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 <[email protected]>

Can you take this please?

Thanks,
Conor.


Attachments:
(No filename) (888.00 B)
signature.asc (235.00 B)
Download all attachments
Subject: Re: [PATCH] MAINTAINERS: Update SiFive driver maintainers

Hello:

This patch was applied to riscv/linux.git (fixes)
by Palmer Dabbelt <[email protected]>:

On Thu, 15 Feb 2024 15:49:11 -0800 you 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.
>
> [...]

Here is the summary with links:
- MAINTAINERS: Update SiFive driver maintainers
https://git.kernel.org/riscv/c/680945f0aa50

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html