Bug 10537 - Support LCD brightness on Fujitsu laptops
Summary: Support LCD brightness on Fujitsu laptops
Status: RESOLVED NOTOURBUG
Alias: None
Product: hal
Classification: Unclassified
Component: hald (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: medium enhancement
Assignee: David Zeuthen (not reading bugmail)
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-04-05 17:06 UTC by Jody Goldberg
Modified: 2008-03-11 10:37 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
The above-mentioned patch (3.65 KB, patch)
2007-04-05 17:48 UTC, Jody Goldberg
Details | Splinter Review

Description Jody Goldberg 2007-04-05 17:06:02 UTC
The brightness controls are similar to toshiba, but are located in a different subdirectory.  The attached patch adds the necessary adjustments.

NOTE : the fujitsu controls are not part of the mainline kernel, they require a patch (http://devrandom.com/fujitsu/).  However, the patch seems to cover most S and P models in the last 4 years.
Comment 1 Jody Goldberg 2007-04-05 17:48:14 UTC
Created attachment 9491 [details] [review]
The above-mentioned patch

This compiles but I have not run the replacement hald on the laptop, only manually tested the input/output.
Comment 2 Danny Kukawka 2007-04-06 10:02:46 UTC
1) If this is not mainline we can't/won't add this to HAL (the patch is from 2004 and not mainline atm).
2) The most FSC machines are already supported by the acpi-video module, but you can't detect currently if this really work, so we can't add this to HAL atm.
3) The ACPI video stuff should be sometime in the future available via the the 
/sys/class/backlight interface. If the FSC machines support this in the mainline kernel this would work out of the box. 

IMO we should not support this not mainline driver, but I can may take a look at the patch and if it would be possible to port this to the new backlight interface, or the author of the patch should do this.
Comment 3 Jody Goldberg 2007-04-26 13:14:21 UTC
Point me at the specs for the /sys/class/backlight iface and I'll tweak the patch.
Comment 4 Benjamin Close 2008-01-11 02:38:25 UTC
Bugzilla Upgrade Mass Bug Change

NEEDSINFO state was removed in Bugzilla 3.x, reopening any bugs previously listed as NEEDSINFO.

  - benjsc
    fd.o Wrangler
Comment 5 Danny Kukawka 2008-03-11 10:37:36 UTC
(In reply to comment #3)
> Point me at the specs for the /sys/class/backlight iface and I'll tweak the
> patch.

Simply check the kernel source package for the doc. I'm not a kernel developer.

Close the bug now since it is not a HAL bug.


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.