[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#670958: [regression] HP ProBook 451x: udev/power_supply battery oscilates between remove and add



Hi,

> So far I have determined that the following HP laptop models are
> affected:
> - HP Probook 451x
> - HP Compaq 315 (Ralf's)
> - HP 6735-s (alculquicondor from the Archlinux forum thread)
It's a compaq 615 ;-)

> 
> ACPI functionalities that are broken so far:
> - power_supply (battery and AC events - this bug)
> - backlight setting
> 
> I don't know what's the next best course of action. I guess I can start
> getting dirty and bissecting the releases between 2.6.37 and
> 2.6.38~rc6... I hope to have some results soon.
I already email the result of my bisect to the bug, but it did not get added 
so far (maybe it did not like the attachments?). Here it is again:

da8aeb92d4853f37e281f11fddf61f9c7d84c3cd is the first bad commit
commit da8aeb92d4853f37e281f11fddf61f9c7d84c3cd
Author: Rafael J. Wysocki <rjw@sisk.pl>
Date:   Thu Jan 6 23:42:27 2011 +0100

    ACPI / Battery: Update information on info notification and resume
    
    A notification event 0x81 from an ACPI battery device requires us to
    re-read the battery information structure.  Follow this requirement
    and remove and re-create the battery's attibutes in sysfs so that
    they reflect the reporting units used by the battery at the moment
    (those units may actually change sometimes at run time, which happens
    on some Thinkpads).
    
    The approach used in this patch was suggested by Matthew Garrett.
    
    Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
    Reported-by: Matthew Garrett <mjg@redhat.com>
    Signed-off-by: Len Brown <len.brown@intel.com>

:040000 040000 93e2bcfb35018bbdaccc91e712dcadab00b2a2b7 
86ce92164b62d8dd645e582c6323ed4684019775 M      drivers

Looking at that patch makes the behaviour of upower quite plausible ;-) so I 
doubt this is a problem in ACPI.
Backlight control is working here.

Kind regards,
Ralf



Reply to: