> This is not a bug in acpi (Ubuntu) as this is just a software package that 
> displays
>  information. Instead, this would be a bug in the linux kernel (ex. 
> samsung-laptop.c) 
> for not handling the laptop lid logic correctly.

No, you are confounding two issues.

This is a buggy behaviour in samsung's EC embedded controller when going to 
sleep and resuming, that can
only be fixed in linux kernel ACPI handling of it ( 
<linuxkernel>/drivers/acpi/ec.c ).
There is a patch ready in the kernel bugzilla, that adds a few lines to 
"acpi/ec.c", that I personally
tested and confirmed that does the same as my userspace workaround (unstuck the 
samsung
ACPI embedded controller):

      https://bugzilla.kernel.org/show_bug.cgi?id=44161     (comments
133 and 135)


samsung-laptop.c has nothing to do with ACPI or the LID detection or this bug. 
samsung-laptop.c uses a SABI interface.
The reason samsung-laptop.c came up, was because that file disables keyboard 
backlight under certain
conditions, and some people said that now that the LID and AC status works with 
my fix, that
now keyboard backlight was the only thing missing for linux compatibility. I 
just brought up
samsung-laptop.c to point to how to fix that last, separate, independant 
remaining problem
of keyboard backlight. 

LID detection and AC status detection, is handled by standard ACPI
drivers in these laptops.

So, this _is_ a bug that is fixed by patching ACPI's drivers/acpi/ec.c.
(or by my userspace program that pokes two EC ports, post #103).

Cheers
--
Juan Manuel Cabo

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/971061

Title:
  acpi reports battery state incorrectly

To manage notifications about this bug go to:
https://bugs.launchpad.net/acpi/+bug/971061/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to