On Wed, Aug 27, 2008 at 1:44 PM, Ilya Petrov <[EMAIL PROTECTED]> wrote: > 2008/8/27 Daniel Ribeiro <[EMAIL PROTECTED]>: >> Could you please try the change i suggested on the other thread? > > yes that patch solved problem > >> Also, is there any change on your tree i should be aware of? > > no changes at all. gen2bp was not applied > > $ cat /sys/class/spi_master/spi1/device/spi1.0/adc_battcurr > voltage: 622 > current: 4294755 > > please, dont do so - create separate files for voltage and current.
Check the code. There is no specific current adc channel. The pcap only allows us to read current when doing it simultaneously with the voltage reading. (battcurr only calls start_adc() once). Doing simultaneous voltage/current measurement is good because you can consider current drain on battery level algorithm. The higher the drained current, the lower will be the voltage reported for the same battery charge level. If you want just battery voltage, you can read from adc_battery. Also, your current value seems completelly bogus. My a1200 reads ~170 with the backlight off, and ~290 with the backlight on. Please, change the do_batt_adc() polarity argument to 0, and see if you get sane values with it. Please, report back if the polarity change makes any difference for you. -- Daniel Ribeiro
