[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 13:23 +0000, Matthew Garrett wrote:
    > On Tue, Feb 07, 2006 at 01:06:45PM +0000, Richard Purdie wrote:
    > > 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.
    > Unfortunately, the hardware doesn't seem to give us that option. There's
    > no obvious way of determining whether we're on AC or DC from the kernel
    > without doing very nasty things with ACPI and APM (userspace can work it
    > out fairly easily).
    > Would you be open to adding generic support for displaying separate AC
    > and DC brightnesses? Making it driver specific leaves the potential for
    > inconsistencies.

    The problem is that AC or DC is not a generic property of backlights but
    specific to your problematic hardware case. You're going to have a to
    find a way to tell if its running on AC or not to report the right value
    in the manner the class requires.

    I can't see how you plan to add "generic support for displaying separate
    AC and DC brightnesses" without destroying the point of the class (which
    for the brightness parameter is to display the current backlight


    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:39    [W:0.043 / U:0.956 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site