| Message ID | 20200726155559.4650-1-krzk@kernel.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2,1/2] MAINTAINERS: Remove JZ4780 DMA driver entry | expand |
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
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
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 <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
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
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(-)