Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry

[Joe Perches <joe@perches.com>]

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.