Sugar battery capacity does not update

[rsmith]

The capacity indicator in sugar does not update as the capacity of the battery changes. I think the reason for this is because the 1% SCI events are masked to prevent being woken up from sleep.

The eventual solution to this is to mask the SCI events when we go to sleep and then to unmask them during normal operation. OHM also needs to start deciding when the system is idle and put the system to sleep.

Until these things are in place sugar will have to poll for battery capacity changes. I think that every 3 to 5 minutes should be enough. 1% capacity changes happen about every 100 seconds.

[marco]

Sugar is not involved at all here, it just listen on HAL property changes. I'm assigning to power-manager which, even if not completely correct, seem a better component for this. Jim, if you actually want me to look into this problem at the HAL level please reassign to me.

[rsmith]

Replying to kimquirk:

Yes. Marco please look into this. We just need _some_ sort of workaround. So it does not matter if its in Sugar or HAL. Wherever its easiest to apply and then un-apply once proper idle detection works and the battery capacity SCI's are unmasked. I would just fix it myself but my initial poking around didn't go anywhere. I'll be happy to come up with some sort of fix if you can get me in the general vicinity of the code that is responsible for this.

[marco]

I think rsmith is trying to solve this for real at ohm/kernel level.

[dilinger]

Longer term, we need to fix OHM or sugar. In the short term, we'll work around it in the kernel by enabling SoC SCI events during normal operation, but shut them prior to suspend.

Patch applied to stable: http://dev.laptop.org/git?p=olpc-2.6;a=commitdiff;h=ca9cca5c8f9a66a350d13c159b8fd1279406dbd6

[dilinger]

(should probably be downgraded in priority?)

[wmb@…]

Need someone to test this, cjb can give recipe.