Re: [rtc-linux] Re: [PATCH/RFC 0/5] Generic RTC class driver

Geert Uytterhoeven
2009-03-04T08:26:43Z

On Wed, 4 Mar 2009, Paul Mundt wrote:
> On Tue, Mar 03, 2009 at 11:41:23AM +0100, Geert Uytterhoeven wrote:
> > So would you accept a patch series that:
> > 1. Adds the missing module aliases to rtc-parisc (which is a bugfix),
> > 2. Moves the platform device creation out of rtc-ppc and into arch-specific
> > code (which is also a bugfix),
> > 3. Consolidates rtc-parisc and rtc-ppc into rtc-generic (which is a cleanup),
> > 4. Makes rtc-generic dependent on PARISC, PPC, and M68K (the existing
> > [sg]et_rtc_time() users):
> > a. without introducing ARCH_HAS_GENERIC_RTC,
> > b. with a big fat warning in the Kconfig comment not relaxing the
> > dependencies, as it's supposed to go away.
> > 4. Converts the PS3 RTC support into a separate driver, called rtc-ps3
> > (as a bonus ;-)
> >
> > ? If yes, I'll cook it up.
> >
> > Other RTC platform support can be converted into separate drivers later.
> >
> Did you miss the rtc-firmware thread?
>
> http://groups.google.com/group/rtc-linux/browse_thread/thread/53e8d98966048f66/1d730cb4aa2f85f0?lnk=gst&q=rtc-firmware#1d730cb4aa2f85f0
> http://groups.google.com/group/rtc-linux/browse_thread/thread/b3d10115c7e147f2/cb9c1530d9c3a433?lnk=gst&q=rtc-firmware#cb9c1530d9c3a433

Thanks Paul, I wasn't aware of that thread!

Yes, this is almost the same. The only part I don't agree with is the move of
the creation of the platform device from arch-specific code to rtc-firmware.c,
as this makes autoloading the driver more difficult.

Seems like everybody but the RTC maintainer has an interest in having an RTC
class driver on top of [gs]et_rtc_time()... ;-)

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Techsoft Centre Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 · RPR Brussels
Fortis · BIC GEBABEBB · IBAN BE41293037680010