Patchwork conflicting MAINTAINERS entries for RTC subsys

login
register
mail settings
Submitter Wolfram Sang
Date Aug. 17, 2010, 7:21 a.m.
Message ID <20100817072136.GA4207@pengutronix.de>
Download mbox | patch
Permalink /patch/61863/
State New
Headers show

Comments

Wolfram Sang - Aug. 17, 2010, 7:21 a.m.
On Tue, Aug 17, 2010 at 12:45:44AM -0400, Mike Frysinger wrote:
> MAINTAINERS currently lists the following two entries.  should the
> first entry be scrapped ?  or should it be merged into the 2nd
> (presumably by just adding Paul's "M:" entry) ?

Ah, there was something :) I asked a similar question more than a year ago, see
this thread for some explanations:

http://groups.google.com/group/rtc-linux/browse_thread/thread/e9f90bb3d6d47371/bdd94740c69900c4?q=rtc-linux#bdd94740c69900c4

I started to have a look to remove the legacy driver but did not get very far.
Maybe we should apply something like this for now?

Regards,

   Wolfram

From: Wolfram Sang <w.sang@pengutronix.de>
Subject: [PATCH] MAINTAINERS: correct entry for legacy RTC-driver

Because no one dares to remove it so far, let's keep the entry correct, at
least.

Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Cc: Paul Gortmaker <p_gortmaker@yahoo.com>
Cc: Mike Frysinger <vapier.adi@gmail.com>
Cc: Alessandro Zummo <a.zummo@towertech.it>
---
 MAINTAINERS |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)
Mike Frysinger - Aug. 17, 2010, 11:47 a.m.
On Tue, Aug 17, 2010 at 03:21, Wolfram Sang wrote:
> -REAL TIME CLOCK DRIVER
> +REAL TIME CLOCK DRIVER (LEGACY)
>  M:     Paul Gortmaker <p_gortmaker@yahoo.com>
>  S:     Maintained
> -F:     Documentation/rtc.txt
> -F:     drivers/rtc/
> -F:     include/linux/rtc.h
> +F:     drivers/char/rtc.c

ah, this makes a lot more sense.
Acked-by: Mike Frysinger <vapier@gentoo.org>
-mike
Wolfram Sang - Aug. 23, 2010, 3:10 p.m.
On Tue, Aug 17, 2010 at 07:47:07AM -0400, Mike Frysinger wrote:
> On Tue, Aug 17, 2010 at 03:21, Wolfram Sang wrote:
> > -REAL TIME CLOCK DRIVER
> > +REAL TIME CLOCK DRIVER (LEGACY)
> >  M:     Paul Gortmaker <p_gortmaker@yahoo.com>
> >  S:     Maintained
> > -F:     Documentation/rtc.txt
> > -F:     drivers/rtc/
> > -F:     include/linux/rtc.h
> > +F:     drivers/char/rtc.c
> 
> ah, this makes a lot more sense.
> Acked-by: Mike Frysinger <vapier@gentoo.org>

Will this be picked up?
Mike Frysinger - Aug. 23, 2010, 4:11 p.m.
On Mon, Aug 23, 2010 at 11:10, Wolfram Sang <wwrote:
> On Tue, Aug 17, 2010 at 07:47:07AM -0400, Mike Frysinger wrote:
>> On Tue, Aug 17, 2010 at 03:21, Wolfram Sang wrote:
>> > -REAL TIME CLOCK DRIVER
>> > +REAL TIME CLOCK DRIVER (LEGACY)
>> >  M:     Paul Gortmaker <p_gortmaker@yahoo.com>
>> >  S:     Maintained
>> > -F:     Documentation/rtc.txt
>> > -F:     drivers/rtc/
>> > -F:     include/linux/rtc.h
>> > +F:     drivers/char/rtc.c
>>
>> ah, this makes a lot more sense.
>> Acked-by: Mike Frysinger <vapier@gentoo.org>
>
> Will this be picked up?

i'd bounce it over to akpm so it doesnt get lost
-mike

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index b5b8baa..3e275fe 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4820,12 +4820,10 @@  F:	kernel/rcu*
 F:	kernel/srcu*
 X:	kernel/rcutorture.c
 
-REAL TIME CLOCK DRIVER
+REAL TIME CLOCK DRIVER (LEGACY)
 M:	Paul Gortmaker <p_gortmaker@yahoo.com>
 S:	Maintained
-F:	Documentation/rtc.txt
-F:	drivers/rtc/
-F:	include/linux/rtc.h
+F:	drivers/char/rtc.c
 
 REAL TIME CLOCK (RTC) SUBSYSTEM
 M:	Alessandro Zummo <a.zummo@towertech.it>