2016-05-19 9:52 GMT+02:00 Pali Rohár <pali.rohar@xxxxxxxxx>: > For me whole patch looks like a big hack because LED_CORE_SUSPENDRESUME > does not work correctly... I would rather see fixed support for flag > LED_CORE_SUSPENDRESUME, not adding another suspend/resume hook if > possible. Any idea? As I wrote you in private mail I think the only way we have is to keep the led_cdev->brightness in sync with actual state. And to do this, we can listen the ACPI event, keeping this like a private event that we just consume to update led brightness value, without delivering it to userspace. However, the custom code needed for getting this done, is probably way more than the one I added here. Maybe it could be useful to others to read the brightness level before suspending, and in this case we could add a new flag to the led subsystem, but not sure it's really worth. -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
