[PATCH 1/3] olpc-battery: add support for CHARGE_FULL_DESIGN

[Andres Salomon]

On Fri, 10 Dec 2010 14:56:15 -1000
Mitch Bradley <wmb at 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<dwmw2 at 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;
> >>> +}
> >>