Since updating to kernel 3.16.3 I have noticed that an old bug has returned where EC events like lid open/close and ambient light change are no longer being processed on my Samsung notebook (900X3C). Most of this was previously documented in: Samsung Series 9 np900x4b: ACPI does not seem to reflect charging/discharging... https://bugzilla.kernel.org/show_bug.cgi?id=44161 A workaround was added to the kernel at the time: ad332c8a45330d170bb38b95209de449b31cd1b4 ACPI / EC: Clear stale EC events on Samsung systems 3eba563e280101209bad27d40bfc83ddf1489234 ACPI / EC: Process rather than discard events in acpi_ec_clear That seemed to have solved the problem, but it seems that a recent change between 66b42b78bc1e816f92b662e8888c89195e4199e1 ACPI / EC: Avoid race condition related to advance_transaction() and 558e4736f2e1b0e6323adf7a5e4df77ed6cfc1a4 ACPI / EC: Add support to disallow QR_EC to be issued before completing previous QR_EC Has reintroduced this bug. I have not had time to fully test this myself, but someone else who noticed the same regression has now posted in the kernel bugzilla link above reporting that: > I've narrowed it down to the following commits: > > ACPI / EC: Add support to disallow QR_EC to be issued when SCI_EVT isn't set > 3afcf2ece453e1a8c2c6de19cdf06da3772a1b08 > > ACPI / EC: Add support to disallow QR_EC to be issued before completing previous QR_EC > 558e4736f2e1b0e6323adf7a5e4df77ed6cfc1a4 > > On 3.17rc7, reverting the latter seems to solve the issue for acpi events related to > the power supply, however LID switch is still broken. > Reverting both seems to resolve the issue completely. Can anyone (Lv Zheng) think of why these patches have caused this issue on this hardware? Maybe we need to modify the patches above to do something different in case EC_FLAGS_CLEAR_ON_RESUME is set? Regards, Kieran Clancy -- 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
