Re: [PATCH] backlight/lp855x: Don't clear level on suspend/blank

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux