Comment 12 for bug 249568

(In reply to comment #5)
> Created an attachment (id=21325) [details]
> possible fix
>
> This implements the on chip backlight control, not all oems use it. It might
> work for you, it might not. Also, depending on how the backlight is wired up,
> the levels may be reversed (not sure whether 0 = on or off and 255 = on or
> off).
> if it doesn't work, try switching the levels in the patch. I've got the
> alternate setting commented out below the main one, so just switch the
> comments. Make sure you switch both the enable and disable paths.
>

I tested your patch but when I run Xorg, the brightness was always very low (zero) and the backlight wasn't switch off.

I test the current git repository without patch and the driver works very well but this backlight bug is present.