Alright, thanks. I guess a comment in the (kernel) source saying as much would be useful.
On Fri, 10 Dec 2010 21:41:00 -0500 Paul Fox <p...@laptop.org> wrote: > i'm pretty sure those numbers (and the corresponding numbers from the > 1.5 DSDT) came from the battery manufacturer. obviously we could move > the data to the EC, but i'm not sure what the advantage of that would > be. i know that the numbers don't come from the battery itself. > > paul > > mitch wrote: > > By not providing more information, I was sending a subtle signal > > that I am busy with something else right now and do not want to go > > into "digging up everything I know or can find out about > > batteries" mode at the moment. > > > > > > On 12/10/2010 3:09 PM, Andres Salomon wrote: > > > On Fri, 10 Dec 2010 14:56:15 -1000 > > > Mitch Bradley<w...@laptop.org> wrote: > > > > > >> There is some battery info in the _BIF (battery info) method in > > >> the BATT node of the ACPI DSDT. I don't remember if it is > > >> correct or not. The numbers below match the DSDT numbers. > > > > > > Wait, so where did *those* numbers come from? A spec somewhere, > > > the EC, or did you actually reverse engineer them? > > > > > > (Note that ACPI is only available on XO-1.5, so pulling them > > > from ACPI on XO-1 isn't an option.) > > > > > > > > > > > >> > > >> On 12/10/2010 2:38 PM, Andres Salomon wrote: > > >>> On Fri, 10 Dec 2010 22:15:10 +0000 > > >>> David Woodhouse<dw...@infradead.org> wrote: > > >>> > > >>>> On Fri, 2010-12-10 at 23:05 +0100, Sascha Silbe wrote: > > >>>>> > > >>>>> + > > >>>>> + switch (tech.intval) { > > >>>>> + case POWER_SUPPLY_TECHNOLOGY_NiMH: > > >>>>> + switch (mfr) { > > >>>>> + case 1: /* Gold Peak */ > > >>>>> + val->intval = 3000000*.8; > > >>>>> + break; > > >>>>> + default: > > >>>>> + return -EIO; > > >>>>> + } > > >>>>> + break; > > >>>>> + > > >>>>> + case POWER_SUPPLY_TECHNOLOGY_LiFe: > > >>>>> + switch (mfr) { > > >>>>> + case 1: /* Gold Peak */ > > >>>>> + val->intval = 2800000; > > >>>>> + break; > > >>>>> + case 2: /* BYD */ > > >>>>> + val->intval = 3100000; > > >>>>> + break; > > >>>>> + default: > > >>>>> + return -EIO; > > >>>>> + } > > >>>>> + break; > > >>>>> + > > >>>>> + default: > > >>>>> + return -EIO; > > >>>>> + } > > >>>>> + > > >>>>> + return ret; > > >>>>> +} > > >>>> > > _______________________________________________ > > Devel mailing list > > Devel@lists.laptop.org > > http://lists.laptop.org/listinfo/devel > > =--------------------- > paul fox, p...@laptop.org _______________________________________________ Devel mailing list Devel@lists.laptop.org http://lists.laptop.org/listinfo/devel