Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: zz8DiT5V4B+jjq/WpgSqBToxu/XDOQk2JTMBWP+xzbiXiyVLxEZsQDLuiAexLODUF61C2NSxxl
 vPsmu/kXiFTAECAHjr48i+s1PYZjx9vyC265yMnzds+nEUK9b0qnMfjx8BQtihhp3dCBOX7bRX
 vD4Prx+7jf4DQYLmHdBTQgI9kY9mEy0JXMg2WvUEjNahM65/tH0SUh2P1CkEddAUVZzbyPfUm3
 nnF46Zy6k53+zizrMRjiIkFxMcdNoGVy5uSt6dvM+A3+5Xaur6UORPZCvdIHuBwdpfBxzNk8CA
 hNE=
IronPort-SDR: 9p6TmkzxglvRTDTpzjFNzfDBQ2/mN63pPxpRxVMJeTXldRABGNSXOw/E+6eIRosQpQIre04eId
 8uAC0X/qVQ0A==
IronPort-SDR: vCKJjUggEeJrcizyNn1jQEEb8EnrrmDjgAlxUjyCHJTuklGiuPukmrXAUrqyCDpeYKy07LZKrf
 MqBed3i3K7Kg==
WDCIronportException: Internal
Date:   Sun, 2 Aug 2020 13:42:51 +0100 (BST)
From:   "Maciej W. Rozycki" <macro@wdc.com>
To:     Paul Cercueil <paul@crapouillou.net>
cc:     Krzysztof Kozlowski <krzk@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        linux-mips@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        "Maciej W. Rozycki" <macro@linux-mips.org>
Subject: Re: [PATCH v2 1/2] MAINTAINERS: Remove JZ4780 DMA driver entry
In-Reply-To: <DGFFEQ.RMDW72DK0U5B1@crapouillou.net>
Message-ID: <alpine.LFD.2.21.2008021339110.24175@redsun52.ssa.fujisawa.hgst.com>
References: <20200726155559.4650-1-krzk@kernel.org> <alpine.LFD.2.21.2008012327070.24175@redsun52.ssa.fujisawa.hgst.com> <DGFFEQ.RMDW72DK0U5B1@crapouillou.net>
User-Agent: Alpine 2.21 (LFD 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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