Re: [PATCH] MAINTAINERS: Remove dead greybus/timesync entry
From: Bryan O'Donoghue
Date: Mon Jul 24 2017 - 11:26:16 EST
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@xxxxxxxxxxxxxxxxx>
---
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@xxxxxxxxxxxxxxxxx>
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@xxxxxxxxxx>
Thanks,
Johan
Already applied I believe.
commit 7683e9e529258d01ce99216ad3be21f59eff83ec
Author: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
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