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 <[email protected]>
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
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 <[email protected]>
-S: Maintained
-F: drivers/dma/dma-jz4780.c
-
INGENIC JZ4780 NAND DRIVER
M: Harvey Hunt <[email protected]>
L: [email protected]
--
2.17.1
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 <[email protected]>
Acked-by: Paul Cercueil <[email protected]>
---
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 362863cae239..d1cc0afe0762 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8544,11 +8544,13 @@ F: samples/bpf/ibumad_user.c
INGENIC JZ4780 NAND DRIVER
M: Harvey Hunt <[email protected]>
L: [email protected]
+L: [email protected]
S: Maintained
F: drivers/mtd/nand/raw/ingenic/
INGENIC JZ47xx SoCs
M: Paul Cercueil <[email protected]>
+L: [email protected]
S: Maintained
F: arch/mips/boot/dts/ingenic/
F: arch/mips/include/asm/mach-jz4740/
--
2.17.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 <[email protected]>
> -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
Hi Maciej,
Le sam. 1 ao?t 2020 ? 23:30, Maciej W. Rozycki <[email protected]> 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 <[email protected]>
>> -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
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
On Sun, 2 Aug 2020 at 14:43, Maciej W. Rozycki <[email protected]> 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