From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from bombadil.infradead.org ([198.137.202.133]:52168 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727614AbeJEBjo (ORCPT ); Thu, 4 Oct 2018 21:39:44 -0400 Date: Thu, 4 Oct 2018 15:45:05 -0300 From: Mauro Carvalho Chehab To: Ludovic Desroches Cc: Joe Perches , Eugen Hristev , Josh Wu , Laurent Pinchart , , Songjun Wu Subject: Re: [PATCH] MAINTAINERS: Remove stale file entry for the Atmel ISI driver Message-ID: <20181004154505.0be5f857@coco.lan> In-Reply-To: <20181002063547.ul7htrd54x7iksxy@M43218.corp.atmel.com> References: <20180930065448.5019-1-laurent.pinchart@ideasonboard.com> <20180930063034.1dab99d9@coco.lan> <7150c1de00db05ec3c1a53611c156fb823d7f345.camel@perches.com> <20181001135101.536b4c22@coco.lan> <20181002063547.ul7htrd54x7iksxy@M43218.corp.atmel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Em Tue, 2 Oct 2018 08:35:47 +0200 Ludovic Desroches escreveu: > On Mon, Oct 01, 2018 at 01:51:01PM -0300, Mauro Carvalho Chehab wrote: > > Em Sun, 30 Sep 2018 02:40:35 -0700 > > Joe Perches escreveu: > > > > > On Sun, 2018-09-30 at 06:30 -0300, Mauro Carvalho Chehab wrote: > > > > Em Sun, 30 Sep 2018 09:54:48 +0300 > > > > Laurent Pinchart escreveu: > > > > > > > > > include/media/atmel-isi got removed three years ago without the > > > > > MAINTAINERS file being updated. Remove the stale entry. > > > > > > > > > > Fixes: 40a78f36fc92 ("[media] v4l: atmel-isi: Remove support for platform data") > > > > > Reported-by: Joe Perches > > > > > Signed-off-by: Laurent Pinchart > > > > > --- > > > > > MAINTAINERS | 1 - > > > > > 1 file changed, 1 deletion(-) > > > > > > > > > > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > [] > > > > > @@ -2497,7 +2497,6 @@ M: Ludovic Desroches > > > > > L: linux-media@vger.kernel.org > > > > > S: Supported > > > > > F: drivers/media/platform/atmel/atmel-isi.c > > > > > -F: include/media/atmel-isi.h > > > > > > > > I guess the right fix would be to replace it by: > > > > > > > > F: drivers/media/platform/atmel/atmel-isi.h > > > > > > Or replace both F entries with: > > > > > > F: drivers/media/platform/atmel/atmel-isi.* > > > > > > Or combine the 2 MICROCHIP sections into one > > > > > > MICROCHIP ISC DRIVER > > > M: Eugen Hristev > > > L: linux-media@vger.kernel.org > > > S: Supported > > > F: drivers/media/platform/atmel/atmel-isc.c > > > F: drivers/media/platform/atmel/atmel-isc-regs.h > > > F: devicetree/bindings/media/atmel-isc.txt > > > > > > MICROCHIP ISI DRIVER > > > M: Eugen Hristev > > > L: linux-media@vger.kernel.org > > > S: Supported > > > F: drivers/media/platform/atmel/atmel-isi.c > > > F: include/media/atmel-isi.h > > > > > > and maybe use something like: > > > > > > MICROCHIP MEDIA DRIVERS > > > M: Eugen Hristev > > > L: > > > linux-media@vger.kernel.org > > > S: Supported > > > F: drivers/media/platform/atmel/ > > > F: devicetree/bindings/media/atmel-isc.txt > > > > Yeah, combining both of them seems a good alternative to me. > > > > Eugen/Ludovic/Josh, > > > > Comments? > > I have no strong opinion about it. The devices are different but usually > there is one person per topic so combining them makes sense. Hmm... At media tree, currently, MAINTAINERS entry is different: MICROCHIP / ATMEL ISC DRIVER M: Songjun Wu L: linux-media@vger.kernel.org S: Supported F: drivers/media/platform/atmel/atmel-isc.c F: drivers/media/platform/atmel/atmel-isc-regs.h F: devicetree/bindings/media/atmel-isc.txt ATMEL ISI DRIVER M: Ludovic Desroches L: linux-media@vger.kernel.org S: Supported F: drivers/media/platform/atmel/atmel-isi.c F: include/media/atmel-isi.h Maybe some patch upstream did some recent changes on it via another tree. So, in order to avoid conflicts upstream, for now I would just correct the location for the ISI file. After the merge window, we may revisit and join both entries, if the maintainers are the same. Regards, Mauro - MAINTAINERS: fix location for atmel-isi.h file The location of this file got changed by changeset 40a78f36fc92 ("[media] v4l: atmel-isi: Remove support for platform data"), but MAINTAINERS was not updated accordingly. Fixes: 40a78f36fc92 ("[media] v4l: atmel-isi: Remove support for platform data") Reported-by: Joe Perches Signed-off-by: Mauro Carvalho Chehab diff --git a/MAINTAINERS b/MAINTAINERS index 9989925f658d..385ebe9ca0a2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2496,8 +2496,7 @@ ATMEL ISI DRIVER M: Ludovic Desroches L: linux-media@vger.kernel.org S: Supported -F: drivers/media/platform/atmel/atmel-isi.c -F: include/media/atmel-isi.h +F: drivers/media/platform/atmel/atmel-isi.* ATMEL LCDFB DRIVER M: Nicolas Ferre