Bug 13375 - Intel's backlight control methods need better doc
Summary: Intel's backlight control methods need better doc
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/intel (show other bugs)
Version: 7.3 (2007.09)
Hardware: Other All
: medium normal
Assignee: Jesse Barnes
QA Contact: Xorg Project Team
URL: http://bugs.debian.org/cgi-bin/bugrep...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-24 01:22 UTC by Brice Goglin
Modified: 2007-12-07 16:31 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Brice Goglin 2007-11-24 01:22:52 UTC
Joey Hess wants better doc for the existing backlight control method in Intel driver 2.2. He says:

It's hard to figure out from this documentation what value should be passed
to xrandr to enable the different backlight control methods. It turns out
to be the first word of each phrase ("kernel", "native", "legacy", or
"combination").

      LVDS   Laptop panel.  Properties:
                BACKLIGHT - set backlight level
                BACKLIGHT_CONTROL - set backlight level control method  (i.e.
              use  kernel interfaces, native LVDS power register, legacy reg‐
              ister, or combination)

The documentation in the source is much more helpful than this brief line
in the man page:

+ * Four different methods are available:
+ *   NATIVE:  only ever touch the native backlight control registers
+ *     This method may be susceptible to problem (2) above if the firmware
+ *     modifies the legacy registers.
+ *   LEGACY:  only ever touch the legacy backlight control registers
+ *     This method may be susceptible to problem (1) above if the firmware
+ *     also modifies the legacy registers.
+ *   COMBO:  try to use both sets
+ *     In this case, the driver will try to modify both sets of registers
+ *     if needed.  To avoid problem (2) above it may set the LBB register
+ *     to a non-zero value if the brightness is to be increased.  It's still
+ *     susceptible to problem (1), but to a lesser extent than the LEGACY only
+ *     method.
+ *   KERNEL:  use kernel methods for controlling the backlight
+ *     This is only available on some platforms, but where present this can
+ *     provide the best user experience.
Comment 1 Jesse Barnes 2007-11-26 09:00:14 UTC
Yeah, you're right I should restructure that section of the man page.  That  means I have to learn troff a bit better...
Comment 2 Jesse Barnes 2007-12-07 16:31:21 UTC
Fixed in e2d9fd6425584119f28768d85c4b7d106ecf8742 (added TV output property descriptions too).


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.