On Wed, January 12, 2011 13:07, Chris Wilson wrote: > > Sure, s/intel_panel_get_max_backlight/intel_panel_get_backlight/ and we > get the behaviour we both want - preserving the current backlight unless > none is set. Indeed, I hadn't noticed that shortcut. That's a lot nicer than my ifdefery. > >> While I'm glad this problem is being fixed upstream, it would be nice to get >> some credit for finding the source of the problem. > > Sorry. You found the bug but I felt your rationale was off. However, I was > amiss in not giving you the credit you fully deserved. Thank you very much! The rationale was that intel_panel_set_backlight(0) was somehow called twice, and that the current code unconditionally stored the old backlight, and thus losing the original brightness level. This is exactly what happened. My fix was to prevent backlight_level from being overwritten by zero. Your fix was more structural and properly fixed backlight enabled/disabled state. In the end both have the same effect and solve the bug. Perhaps I was unclear in the bug description. Anyhow, it's a pleasure working with you. I'll try to not bother you too much, you got enough on your plate as it is. I'll leave you alone for a while after you looked into my fix for bug 23472, after that all my Intel graphics are pretty much solved. :-) Take care, Indan _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
