• Dirk Eibach's avatar
    i2c: fix broken ds1337 initialization · 763d9c04
    Dirk Eibach authored
    On a custom board with ds1337 RTC I found that upgrade from 2.6.15 to
    2.6.18 broke RTC support.
    
    The main problem are changes to ds1337_init_client().
    When a ds1337 recognizes a problem (e.g. power or clock failure) bit 7
    in status register is set. This has to be reset by writing 0 to status
    register. But since there are only 16 byte written to the chip and the
    first byte is interpreted as an address, the status register (which is
    the 16th) is never written.
    The other problem is, that initializing all registers to zero is not
    valid for day, date and month register. Funny enough this is checked by
    ds1337_detect(), which depends on this values not being zero. So then
    treated by ds1337_init_client() the ds1337 is not detected anymore,
    whereas the failure bit in the status register is still set.
    Signed-off-by: default avatarDirk Stieler <stieler@gdsys.de>
    Signed-off-by: default avatarDirk Eibach <eibach@gdsys.de>
    Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
    763d9c04
ds1337.c 9.92 KB