[v2,1/2] MAINTAINERS: Remove JZ4780 DMA driver entry
diff mbox series

Message ID 20200726155559.4650-1-krzk@kernel.org
State Superseded
Headers show
Series
  • [v2,1/2] MAINTAINERS: Remove JZ4780 DMA driver entry
Related show

Commit Message

Krzysztof Kozlowski July 26, 2020, 3:55 p.m. UTC
The entry for MIPS Ingenic JZ4780 DMA driver is not up to date anymore.
Zubair Lutfullah Kakakhel's email bounces and no maintenance is
provided.

Suggested-by: Paul Cercueil <paul@crapouillou.net>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. New patch
---
 MAINTAINERS | 5 -----
 1 file changed, 5 deletions(-)

Comments

Maciej W. Rozycki Aug. 1, 2020, 10:30 p.m. UTC | #1
On Sun, 26 Jul 2020, Krzysztof Kozlowski wrote:

> diff --git a/MAINTAINERS b/MAINTAINERS
> index cba0ed77775b..362863cae239 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -8541,11 +8541,6 @@ F:	include/uapi/rdma/
>  F:	samples/bpf/ibumad_kern.c
>  F:	samples/bpf/ibumad_user.c
>  
> -INGENIC JZ4780 DMA Driver
> -M:	Zubair Lutfullah Kakakhel <Zubair.Kakakhel@imgtec.com>
> -S:	Maintained
> -F:	drivers/dma/dma-jz4780.c
> -

 FYI, the usual approach would be marking the entry "Orphan" rather than 
removing it entirely, so that the mapping remains and makes it easy for 
someone to pick it up.

  Maciej
Paul Cercueil Aug. 2, 2020, 7:57 a.m. UTC | #2
Hi Maciej,

Le sam. 1 août 2020 à 23:30, Maciej W. Rozycki <macro@wdc.com> a 
écrit :
> On Sun, 26 Jul 2020, Krzysztof Kozlowski wrote:
> 
>>  diff --git a/MAINTAINERS b/MAINTAINERS
>>  index cba0ed77775b..362863cae239 100644
>>  --- a/MAINTAINERS
>>  +++ b/MAINTAINERS
>>  @@ -8541,11 +8541,6 @@ F:	include/uapi/rdma/
>>   F:	samples/bpf/ibumad_kern.c
>>   F:	samples/bpf/ibumad_user.c
>> 
>>  -INGENIC JZ4780 DMA Driver
>>  -M:	Zubair Lutfullah Kakakhel <Zubair.Kakakhel@imgtec.com>
>>  -S:	Maintained
>>  -F:	drivers/dma/dma-jz4780.c
>>  -
> 
>  FYI, the usual approach would be marking the entry "Orphan" rather 
> than
> removing it entirely, so that the mapping remains and makes it easy 
> for
> someone to pick it up.

This is OK in this case, as the driver is also listed as part of the 
"INGENIC JZ47xx SoCs".

-Paul
Maciej W. Rozycki Aug. 2, 2020, 12:42 p.m. UTC | #3
On Sun, 2 Aug 2020, Paul Cercueil wrote:

> >  FYI, the usual approach would be marking the entry "Orphan" rather 
> > than
> > removing it entirely, so that the mapping remains and makes it easy 
> > for
> > someone to pick it up.
> 
> This is OK in this case, as the driver is also listed as part of the 
> "INGENIC JZ47xx SoCs".

 Fair enough (and odd enough too to have two entries for one piece), 
though in this case I think the commit description ought to mention it.  
As it stands it makes one think the driver is not going to have a 
maintainer anymore: "[...] and no maintenance is provided."

  Maciej
Krzysztof Kozlowski Aug. 2, 2020, 4:46 p.m. UTC | #4
On Sun, 2 Aug 2020 at 14:43, Maciej W. Rozycki <macro@wdc.com> wrote:
>
> On Sun, 2 Aug 2020, Paul Cercueil wrote:
>
> > >  FYI, the usual approach would be marking the entry "Orphan" rather
> > > than
> > > removing it entirely, so that the mapping remains and makes it easy
> > > for
> > > someone to pick it up.
> >
> > This is OK in this case, as the driver is also listed as part of the
> > "INGENIC JZ47xx SoCs".
>
>  Fair enough (and odd enough too to have two entries for one piece),
> though in this case I think the commit description ought to mention it.
> As it stands it makes one think the driver is not going to have a
> maintainer anymore: "[...] and no maintenance is provided."

Sure, the message could be improved. How about:

"The entry for MIPS Ingenic JZ4780 DMA driver is not up to date
anymore. Zubair Lutfullah Kakakhel's email bounces. The maintenance is
provided through Ingenic SoC maintainers entry."

If it is OK, I can either resend or it could be fixed up when applying.

Best regards,
Krzysztof

Patch
diff mbox series

diff --git a/MAINTAINERS b/MAINTAINERS
index cba0ed77775b..362863cae239 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8541,11 +8541,6 @@  F:	include/uapi/rdma/
 F:	samples/bpf/ibumad_kern.c
 F:	samples/bpf/ibumad_user.c
 
-INGENIC JZ4780 DMA Driver
-M:	Zubair Lutfullah Kakakhel <Zubair.Kakakhel@imgtec.com>
-S:	Maintained
-F:	drivers/dma/dma-jz4780.c
-
 INGENIC JZ4780 NAND DRIVER
 M:	Harvey Hunt <harveyhuntnexus@gmail.com>
 L:	linux-mtd@lists.infradead.org