Comment 187 for bug 1098216

Revision history for this message
In , aaron.lu (aaron.lu-linux-kernel-bugs) wrote :

Hi Seth,

I'm sorry I think I missed some code in video.c...

Apparently, for the notification case, the driver indeed changed the brightness level. I don't think this is correct, since it will also report this event to user space, like the no notification case. So we should definitely set brightness_switch_enabled to 0 by default, to give user space a consistent behaviour.

BTW, I've just checked the link you gave, there are several patches. Thanks for the patches.