lkml.org 
[lkml]   [2016]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 02/17] RTC: ds1307: Disable square wave and timers as default
On 21/06/2016 at 00:22:35 -0700, Andrey Smirnov wrote :
> Disable square wave and timers as default for DS1337/39/41 and
> DS3231. The rationale being that configuring a chip this way puts it
> into a known state with lower power consumption. While it is not very
> likely it is still possible that the code controlling RTCs that ran
> before this driver configured it to produce square wave and left it in
> such a state.
>

The main issue being that if it has been configured that way (from the
bootloader for example). It is probably necessary for the board. There
may be users of that clock and this patch definitively breaks them.
The proper way of doing that is to add CCF support in the driver. See:
http://patchwork.ozlabs.org/patch/576201/

> Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
> ---
> drivers/rtc/rtc-ds1307.c | 12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/rtc/rtc-ds1307.c b/drivers/rtc/rtc-ds1307.c
> index 7e65e2e..c618c22 100644
> --- a/drivers/rtc/rtc-ds1307.c
> +++ b/drivers/rtc/rtc-ds1307.c
> @@ -1320,19 +1320,17 @@ static int ds1307_probe(struct i2c_client *client,
> ds1307->regs[0] &= ~DS1337_BIT_nEOSC;
>
> /*
> - * Using IRQ or defined as wakeup-source?
> * Disable the square wave and both alarms.
> * For some variants, be sure alarms can trigger when we're
> * running on Vbackup (BBSQI/BBSQW)
> */
> - if (chip->alarm && (ds1307->client->irq > 0 ||
> - ds1307_can_wakeup_device)) {
> - ds1307->regs[0] |= DS1337_BIT_INTCN
> - | bbsqi_bitpos[ds1307->type];
> - ds1307->regs[0] &= ~(DS1337_BIT_A2IE | DS1337_BIT_A1IE);
> + ds1307->regs[0] |= DS1337_BIT_INTCN
> + | bbsqi_bitpos[ds1307->type];
> + ds1307->regs[0] &= ~(DS1337_BIT_A2IE | DS1337_BIT_A1IE);
>
> + if (chip->alarm && (ds1307->client->irq > 0 ||
> + ds1307_can_wakeup_device))
> want_irq = true;
> - }
>
> i2c_smbus_write_byte_data(client, DS1337_REG_CONTROL,
> ds1307->regs[0]);
> --
> 2.5.5
>

--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

\
 
 \ /
  Last update: 2016-06-22 00:01    [W:0.202 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site