Bug 24592

Summary:	Backlight setting regression from 392a13923eb4d44a8bdb204922230e126b188fae
Product:	xorg	Reporter:	Rafał Miłecki <zajec5>
Component:	Driver/radeonhd	Assignee:	Luc Verhaegen <lverhaegen>
Status:	RESOLVED FIXED	QA Contact:	Xorg Project Team <xorg-team>
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Rafał Miłecki 2009-10-17 08:30:39 UTC

Starting from 392a13923eb4d44a8bdb204922230e126b188fae I can not set Backlight for my PANEL:

> xrandr --output PANEL --set Backlight 150
X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  150 (RANDR)
  Minor opcode of failed request:  13 (RRChangeOutputProperty)
  Value in failed request:  0x3c
  Serial number of failed request:  28
  Current serial number in output stream:  29

Comment 1 Matthias Hopf 2009-10-19 06:05:28 UTC

Please report the output of 'xrandr --props'

Comment 2 Rafał Miłecki 2009-10-19 06:17:05 UTC

Using f695445b386050de346211baec5e487813187cfd

PANEL connected 1600x900+0+0 (normal left inverted right x axis y axis) 423mm x 238mm
        Backlight: 0 (0x00000000)       range:  (0,255)
        _PanningArea:
        _OutputNumber: 2 (0x00000002)
        ConnectorNumber: 2 (0x00000002)
        ConnectorType:  Panel
        SignalFormat:   LVDS
                supported: LVDS

You may wonder about "0" value. That's due to bug #24617, however controlling works without problem (and reporting backlight level after first change).

Comment 3 Rafał Miłecki 2009-10-19 06:22:22 UTC

Using 392a13923eb4d44a8bdb204922230e126b188fae:


PANEL connected 1600x900+0+0 (normal left inverted right x axis y axis) 423mm x 238mm
        Backlight: 255 (0x000000ff)     range:  (0,255)                              
        _PanningArea:                                                                
        _OutputNumber: 2 (0x00000002)                                                
        ConnectorNumber: 2 (0x00000002)                                              
        ConnectorType:  Panel                                                        
        SignalFormat:   LVDS                                                         
                supported: LVDS

Comment 4 Egbert Eich 2009-10-19 07:15:43 UTC

Can you provide a verbose log file, please?

Comment 5 Egbert Eich 2009-10-19 08:45:31 UTC

Found it. Fixed.
commit a838114e998a779ce64b894bf8cc34f93647c001
Author: Egbert Eich <eich@freedesktop.org>
Date:   Mon Oct 19 17:36:16 2009 +0200

    Fix backlight readout thru AtomBIOS.
    
    The correct value was not read out as 'Set' was called instead of 'Get'.

Comment 6 Rafał Miłecki 2009-10-19 09:24:16 UTC

(In reply to comment #5)
> Found it. Fixed.
> commit a838114e998a779ce64b894bf8cc34f93647c001

Thanks.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.