Re: [rtc-linux] rv3029 EEPROM handling

["Michael Büsch" <m@bues.ch>]

[-- Attachment #1: Type: text/plain, Size: 1542 bytes --]

On Sun, 28 Feb 2016 11:41:10 +0100
Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:

> ds1339, ds1390 and bq32k are using trickle-resistor-ohms and
> trickle-diode-disable.


The rv3029 datasheet doesn't say anything about a diode (I think it
still has one though). It merely talks about resistors connected
between vdd and the battery pin.
So I'm not sure whether we should use the 'trickle-diode-disable'
property here. One other possibility would be to use
trickle-resistor-ohms=-1 or such to disable the charger.


> > The rv3029 does the compensation internally. It just needs to be
> > enabled via a bit in its EEPROM. We could probably always set that bit
> > though. I'm not sure whether there is a case where one would not want
> > that.
> >   
> 
> If you want to enable it without changing the values, just enable it
> unconditionally.


I think the internal temperature compensation should be enabled by
default and be disabled via DT property ('temp-compensation-disable'
or such).

-- 
Michael

-- 
-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]