lkml.org 
[lkml]   [2006]   [Aug]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: RTC: add RTC class interface to m41t00 driver
From
On Fri, 4 Aug 2006 19:33:39 -0700, David Brownell <david-b@pacbell.net> wrote:
> Actually, it'd be worth trying drivers/rtc/rtc-ds1307.c ... the M41T00 is
> one of a family of mostly-compatible RTC chips, and the ds1307 driver
> should be pretty much the least-common-denominator there. They all use
> the same I2C address, and the same register layout for the calendar/time
> function.
>
> I'd expect rtc-ds1307 to handle the m41t00 already, or with at most minor
> tweaks to recognize whatever's different. It should already be ignoring
> the bits in the clock/calendar registers that vary, as well as the SRAM
> and (for some other chips) the alarm. (Not that I2C has ways to tell us
> what IRQ the alarm would use, but that's a different tale!)

Thanks for your suggestion. I have looked rtc-ds1307 too before I
tried to modify m41t00 driver.

It seems some works are still needed to support M41Txx chips by the
driver.

1. The driver contains ds_1340 (or st m41t00) definition, but it seems
no way to select the ds_type.

2. As m41t00_chip_info_tbl[] in m41t00 driver shows, M41T81 and M41T85
have different register layout.

3. It lacks some features (ST bit, HT bit, SQW freq.) in m41t00
driver, though I personally does not need these features.

I choose changing m41t00 driver by (1) and (2).

If we really need a super generic driver, I suppose adding ds13xx
support to new m41txx driver is less hard. I think having separate
drivers are good enough for now.

---
Atsushi Nemoto
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-08-05 18:31    [W:0.053 / U:0.712 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site