#2861 BLOC First D: Battery indicator not changing

[Zarro Boogs per Child]

#2861: Battery indicator not changing
------------------------+---------------------------------------------------
  Reporter:  crschmidt  |       Owner:  dilinger              
      Type:  defect     |      Status:  new                   
  Priority:  blocker    |   Milestone:  First Deployment, V1.0
 Component:  kernel     |     Version:  Build 542             
Resolution:             |    Keywords:                        
  Verified:  0          |  
------------------------+---------------------------------------------------

Comment(by rsmith):

 Replying to [comment:9 kimquirk]:
 > I have seen this as well. Richard said it is likely that the polling of
 battery life was turned off during suspend/resume testing so the
 inidicator on the home page doesn't get updated.
 >

 I think the problem is that to prevent the laptop from waking up from
 resume every on 1% change notify we set the battery event mask so that
 battery % events are suppressed.  If sugar is depending on getting SCIs
 for battery % change then it would never update.
 This is supported by the indicator updating properly when you plug up
 external power which will generate an event that is not masked.

 The fix is to set the 1% mask only when the system goes into suspend and
 then re-enable when we come back out of suspend.

 > The real question is do we actually get any indication of low battery
 before it just shuts off? I did not catch a red led inidication when mine
 shut off.

 Just did a test with q2c28i and it "works for me".

-- 
Ticket URL: <https://dev.laptop.org/ticket/2861#comment:10>
One Laptop Per Child <https://dev.laptop.org>
OLPC bug tracking system