[lkml]   [2006]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] [RESEND] Add Dell laptop backlight brightness display
On Tue, 2006-02-07 at 12:32 +0000, Matthew Garrett wrote:
+ /* The backlight interface doesn't give us a means of providing
+ more than one brightness value, so we put the AC value in the
+ top bits of the brightness and the DC value in the bottom bits */

This is total abuse of the backlight class. The idea is that
cat /sys/class/backlight/ccc/brightness returns the *current* backlight
brightness. On AC power it should return the AC brightness value and on
DC power return the DC value.

If you want to know the DC and AC values as stored in the BIOS they
should be device specific attributes, not generic class ones.

I have a patch in the pipeline to change the backlight class slightly to
provide both the user requested brightness and the current brightness as
the two can differ (the Zaurus handhelds limit the backlight intensity
on low power).


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2006-02-07 14:09    [W:0.047 / U:2.708 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site