Comment 24 for bug 810093

Revision history for this message
Felix Vollmer (felixvollmer) wrote :

I can confirm we need a special check for 0 I just tested your patch and it is working. I can't reproduce bug mentioned in #22 (on Samsung N220), but it doesn't do anything bad, so I think it should be used.

But there must be a bug in code, because if I load the module the screen brightness changes. I couldn't see why
set_brightness(initial_level);
isn't really working.

I just got an idea for the general quirk problem. Wouldn't it be the easiest to switch first to level 0 and than to the desired level ? (I'm no expert on this and I didn't tried it but I think this should always work (if switching is fast enough))