[PATCH] MAINTAINERS: Remove dead greybus/timesync entry

public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed

* [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
@ 2017-07-24 10:05 Bryan O'Donoghue
  2017-07-24 11:27 ` Johan Hovold
  0 siblings, 1 reply; 7+ messages in thread
From: Bryan O'Donoghue @ 2017-07-24 10:05 UTC (permalink / raw)
  To: linux-kernel; +Cc: 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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
  2017-07-24 10:05 [PATCH] MAINTAINERS: Remove dead greybus/timesync entry Bryan O'Donoghue
@ 2017-07-24 11:27 ` Johan Hovold
  2017-07-24 15:19   ` Bryan O'Donoghue
  0 siblings, 1 reply; 7+ messages in thread
From: Johan Hovold @ 2017-07-24 11:27 UTC (permalink / raw)
  To: Bryan O'Donoghue; +Cc: linux-kernel, Joe Perches, Greg Kroah-Hartman

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
  2017-07-24 11:27 ` Johan Hovold
@ 2017-07-24 15:19   ` Bryan O'Donoghue
  2017-07-24 15:16     ` Joe Perches
  0 siblings, 1 reply; 7+ messages in thread
From: Bryan O'Donoghue @ 2017-07-24 15:19 UTC (permalink / raw)
  To: Johan Hovold; +Cc: linux-kernel, Joe Perches, Greg Kroah-Hartman

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
  2017-07-24 15:19   ` Bryan O'Donoghue
@ 2017-07-24 15:16     ` Joe Perches
  2017-07-24 15:32       ` Bryan O'Donoghue
  0 siblings, 1 reply; 7+ messages in thread
From: Joe Perches @ 2017-07-24 15:16 UTC (permalink / raw)
  To: Bryan O'Donoghue, Johan Hovold; +Cc: linux-kernel, Greg Kroah-Hartman

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.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
  2017-07-24 15:16     ` Joe Perches
@ 2017-07-24 15:32       ` Bryan O'Donoghue
  2017-07-24 15:31         ` Joe Perches
  0 siblings, 1 reply; 7+ messages in thread
From: Bryan O'Donoghue @ 2017-07-24 15:32 UTC (permalink / raw)
  To: Joe Perches, Johan Hovold; +Cc: linux-kernel, Greg Kroah-Hartman

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
  2017-07-24 15:32       ` Bryan O'Donoghue
@ 2017-07-24 15:31         ` Joe Perches
  2017-07-24 15:50           ` Bryan O'Donoghue
  0 siblings, 1 reply; 7+ messages in thread
From: Joe Perches @ 2017-07-24 15:31 UTC (permalink / raw)
  To: Bryan O'Donoghue, Johan Hovold; +Cc: linux-kernel, Greg Kroah-Hartman

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.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
  2017-07-24 15:31         ` Joe Perches
@ 2017-07-24 15:50           ` Bryan O'Donoghue
  0 siblings, 0 replies; 7+ messages in thread
From: Bryan O'Donoghue @ 2017-07-24 15:50 UTC (permalink / raw)
  To: Joe Perches, Johan Hovold; +Cc: linux-kernel, Greg Kroah-Hartman

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-07-24 15:45 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-24 10:05 [PATCH] MAINTAINERS: Remove dead greybus/timesync entry Bryan O'Donoghue
2017-07-24 11:27 ` Johan Hovold
2017-07-24 15:19   ` Bryan O'Donoghue
2017-07-24 15:16     ` Joe Perches
2017-07-24 15:32       ` Bryan O'Donoghue
2017-07-24 15:31         ` Joe Perches
2017-07-24 15:50           ` Bryan O'Donoghue

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox