[PATCH] MAINTAINERS: Remove dead greybus/timesync entry

[PATCH] MAINTAINERS: Remove dead greybus/timesync entry

[Bryan O'Donoghue]

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
 
 GREYBUS PLATFORM DRIVERS
 M:	Vaibhav Hiremath <hvaibhav.linux@gmail.com>
-- 
2.7.4

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

[Johan Hovold]

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

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

[Joe Perches]

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.

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

[Bryan O'Donoghue]

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

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

[Joe Perches]

On Mon, 2017-07-24 at 16:32 +0100, Bryan O'Donoghue wrote:
> Ah yes I remember now, those are two separate patches. Anyway my greybus 
> filter caught your mail which is what prompted me to send this out, I 
> wasn't sure if you wanted others to clean up their own crap, or if you 
> wanted to yourself.

Everyone learns in kindergarten to clean up their own messes.
And my sweet wife tells me I'm not very good at it.

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

[Bryan O'Donoghue]

On 24/07/17 16:16, Joe Perches wrote:
> 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.
> 

Ah yes I remember now, those are two separate patches. Anyway my greybus 
filter caught your mail which is what prompted me to send this out, I 
wasn't sure if you wanted others to clean up their own crap, or if you 
wanted to yourself.

Feel free to add my Acked-by if you perfer to send the whole lot out as one.

---
bod

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

[Bryan O'Donoghue]

On 24/07/17 16:31, Joe Perches wrote:
> On Mon, 2017-07-24 at 16:32 +0100, Bryan O'Donoghue wrote:
>> Ah yes I remember now, those are two separate patches. Anyway my greybus
>> filter caught your mail which is what prompted me to send this out, I
>> wasn't sure if you wanted others to clean up their own crap, or if you
>> wanted to yourself.
> 
> Everyone learns in kindergarten to clean up their own messes.
> And my sweet wife tells me I'm not very good at it.
> 

Personally I'm 100% fine living in reams of trash/take-away 
boxes/kitty-litter.

/meh - TMI