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

[Mitch Bradley]

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.

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;
>>> +}
>>
>> I don't much like hard-coding it in the kernel. Can the firmware
>> expose these values in the device-tree?
>>
>
>
> It there is, it's not at all clear.  The values are fetched from the
> EC, which get them from the EEPROM.  The DT has a battery entry, but it
> contains nothing useful:
>
> /proc/device-tree/battery at 0/name
> /proc/device-tree/battery at 0/reg
> /proc/device-tree/battery at 0/.node
>
> Nor is there anything in the DT related to the battery EEPROM.