Message-ID: <1500909380.28002.5.camel@perches.com>
Subject: Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
From: Joe Perches <joe@perches.com>
To: "Bryan O'Donoghue" <pure.logic@nexus-software.ie>,
        Johan Hovold <johan@kernel.org>
Cc: linux-kernel@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: Mon, 24 Jul 2017 08:16:20 -0700
In-Reply-To: <dd128bba-1c68-4203-2e43-6e8dcc5b9084@nexus-software.ie>
References: <1500890749-19419-1-git-send-email-pure.logic@nexus-software.ie>
         <20170724112738.GH2729@localhost>
         <dd128bba-1c68-4203-2e43-6e8dcc5b9084@nexus-software.ie>
Content-Type: text/plain; charset="ISO-8859-1"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1749
Lines: 48

On Mon, 2017-07-24 at 16:19 +0100, Bryan O'Donoghue wrote:
> On 24/07/17 12:27, Johan Hovold wrote:
> > On Mon, Jul 24, 2017 at 11:05:49AM +0100, Bryan O'Donoghue wrote:
> > > commit bdfb95c4baab ("staging: greybus: remove timesync protocol support")
> > > commit 1e029b836108 ("staging: greybus: arche: remove timesync remains")
> > > 
> > > Subtract timesync from greybus/staging but haven't updated MAINTAINERS
> > > appropriately. Fix that omission now.
> > > 
> > > Signed-off-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
> > > ---
> > >   MAINTAINERS | 2 --
> > >   1 file changed, 2 deletions(-)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index f66488d..ca9e29e 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -5830,8 +5830,6 @@ GREYBUS LOOBACK/TIME PROTOCOLS DRIVERS
> > >   M:	Bryan O'Donoghue <pure.logic@nexus-software.ie>
> > >   S:	Maintained
> > >   F:	drivers/staging/greybus/loopback.c
> > > -F:	drivers/staging/greybus/timesync.c
> > > -F:	drivers/staging/greybus/timesync_platform.c
> > 
> > Joe reported these, and a number of other obsolete or misspelled entries
> > in MAINTAINERS three days ago but wanted to get his sorting patch in
> > before addressing them. I'm not sure whether the intention was to fix
> > them all up in one go or to do it like this.
> > 
> > But either way,
> > 
> > Acked-by: Johan Hovold <johan@kernel.org>
> > 
> > Thanks,
> > Johan
> > 
> 
> Already applied I believe.
> 
> commit 7683e9e529258d01ce99216ad3be21f59eff83ec
> Author: Linus Torvalds <torvalds@linux-foundation.org>
> Date:   Sun Jul 23 16:06:21 2017 -0700
> 
>      Properly alphabetize MAINTAINERS file

No, the sorting did not remove or update invalid F: entries.
It just moved sections around.