Re: [PATCH 1/3] iio: adc: cpcap: Fix default register values and battery temperature

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 6 Jun 2017 23:22:00 -0700
Tony Lindgren <tony@xxxxxxxxxxx> wrote:

> * Jonathan Cameron <jic23@xxxxxxxxxx> [170524 12:22]:
> > On Mon, 22 May 2017 17:51:44 -0700
> > Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> >   
> > > Looking at the register dumps from Android kernel on droid 4, I noticed
> > > the values with the mainline kernel don't match. Let's fix this by
> > > initializing the ADC registers to what Android does.
> > > 
> > > For getting correct values from the battery thermistor, we need to
> > > toggle the CPCAP_BIT_THERMBIAS_EN when measuring battery temperature
> > > to get correct battery temperatures. And looks like we also need to
> > > wait a little bit before reading the battery temperature as otherwise
> > > the results are inaccurate.
> > > 
> > > Cc: Marcel Partap <mpartap@xxxxxxx>
> > > Cc: Michael Scott <michael.scott@xxxxxxxxxx>
> > > Cc: Sebastian Reichel <sre@xxxxxxxxxx>
> > > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>  
> > Applied to the togreg branch of iio.git and will be shortly pushed
> > out as testing for the autobuilders to play with it.  
> 
> Not seeing these in your togreg brach or in next? The
> branch I just checked is the one below, maybe I'm looking
> at a wrong branch or something.
I think I was being a muppet last weekend.  Forgot to actually
push out anything much.

Done now, though still as testing as obviously the autobuilders haven't
looked at it yet.
Sorry about that and thanks for point it out!

Jonathan
> 
> Regards,
> 
> Tony
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git/log/?h=togreg

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux