Hi Sean, On 5/14/2015 4:33 AM, Sean Paul wrote:
On Mon, May 11, 2015 at 3:48 PM, Kim, Milo <milo.kim@xxxxxx <mailto:milo.kim@xxxxxx>> wrote: Hi Sean, On 5/12/2015 5:32 AM, Sean Paul wrote: Don't clear the backlight level when we're going into suspend or blanking. Instead, just temporarily set the level to 0 so we retain the value when we resume. Could you describe what the problem is? I wrote same code in other backlight drivers. So maybe I need to look into them as well. Hi Milo, Well, I'm not sure what the expected behavior is, but our userspace expects the driver to maintain its brightness across suspend/resume. The way things are coded now, the driver will clear the brightness value on suspend such that when we resume, the backlight is restored to 0.
Got it. Your patch makes sense. Thanks for catching this. Best regards, Milo -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html