On Wed, Apr 8, 2009 at 3:27 PM, Alan Jenkins <alan-jenkins@xxxxxxxxxxxxxx> wrote: > 1) Buggy firmware can change the RFKILL state by itself. This is easily > detected. The RFKILL API states that in such cases, we should call > rfkill_force_state() to notify the core. > > I have reported the bug to Asus. I believe this is the right thing > to do for robustness, even if this particular firmware bug is fixed. > > 2) The same bug causes the wireless toggle key to be reported as 0x11 > instead of 0x10. 0x11 is otherwise unused, so it should be safe to > add this as a new keycode. > > The bug is triggered by removing the laptop battery while hibernated. > > On resume, the wireless toggle key causes the firmware to toggle the > wireless state itself. (Also, the key is reported as 0x11 when the > current wireless state is OFF). > > This is very poor behaviour because the OS can't predict whether the > firmware is controlling the RFKILL state. > > Without this workaround, the bug means users have to press the wireless > toggle key twice to enable, due to the OS/firmware conflict. (Assuming > rfkill-input or equivalent is being used). The workaround avoids this. > > I believe that acpid scripts which toggle the value of the sysfs state file > when the toggle key is pressed will be rendered ineffective by the bug, > regardless of this workaround. If they simply toggle the state, when the > firmware has already toggled it, then you will never see a state change. > > Tested on "EEEPC 4G" only. Hi, Thanks for the patch, applied on acpi4asus tree. Could someone with EEEPC 900/1000 test this patch ? Thanks -- Corentin Chary http://xf.iksaif.net -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html