[PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Krzysztof Kozlowski]

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(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index b0a742ce8f2c..9ab3a7fd6451 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8580,11 +8580,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
-- 
2.17.1

[PATCH v2 2/2] MAINTAINERS: Add linux-mips mailing list to JZ47xx entries

[Krzysztof Kozlowski]

The entries for JZ47xx SoCs and its drivers lacked MIPS mailing list.
Only MTD NAND driver pointed linux-mtd.  Add linux-mips so the relevant
patches will get attention of MIPS developers.

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

---

Changes since v1:
1. Do not update DMA driver entry
2. Add ack
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 9ab3a7fd6451..33d0ea35d8b4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8583,11 +8583,13 @@ F:	samples/bpf/ibumad_user.c
 INGENIC JZ4780 NAND DRIVER
 M:	Harvey Hunt <harveyhuntnexus@gmail.com>
 L:	linux-mtd@lists.infradead.org
+L:	linux-mips@vger.kernel.org
 S:	Maintained
 F:	drivers/mtd/nand/raw/ingenic/
 
 INGENIC JZ47xx SoCs
 M:	Paul Cercueil <paul@crapouillou.net>
+L:	linux-mips@vger.kernel.org
 S:	Maintained
 F:	arch/mips/boot/dts/ingenic/
 F:	arch/mips/include/asm/mach-jz4740/
-- 
2.17.1

Re: [PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Krzysztof Kozlowski]

On Mon, Aug 17, 2020 at 08:41:30PM +0200, Krzysztof Kozlowski wrote:
> 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(-)
> 

Hi all,

Any comments on these two patches?

Best regards,
Krzysztof

[PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Krzysztof Kozlowski]

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(-)

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
-- 
2.17.1

Re: [PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Maciej W. Rozycki]

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

Re: [PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Paul Cercueil]

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

Re: [PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Maciej W. Rozycki]

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

Re: [PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry

[Krzysztof Kozlowski]

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