lkml.org 
[lkml]   [2006]   [Feb]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] [RESEND] Add Dell laptop backlight brightness display
From
Date
On Mon, 2006-02-20 at 17:53 +0100, Pavel Machek wrote:
> Hi!
>
> > Matthew has shown up on the libsmbios-devel mailing list. I sent
> > all the
> > info needed to do a test of Dell LCD brightness control. The main thing
> > left
> > would be to make one utility out of the current separate, unsupported,
> > test
> > utils.
> >
> > As for fixing i8k, I don't have the slightest clue where to
> > begin. You
> > either have to split initialization with userspace to parse and send in
> > the
> > correct io/magic ports to do SMI, or you have to put Dell-specific SMI
> > token
> > parsing in the kernel.
>
> What is wrong with Dell-specific SMI parsing in kernel? Is it _that_
> much code?
>

Pavel,
Sorry for the late response.

No, it isn't that much code. You are welcome to rip the parsing from
libsmbios. There are about three or so C structs necessary to parse it.
The total code is probably about 30-40 lines. (see
libraries/token/TokenDA.cpp, and libraries/common/TokenLowLevel.h).

My only point is that _everything_ that this module does can now be
completely and fully implemented in userspace using dcdbas and
libsmbios. I'm not against fixing the current i8k code, though. I don't
have the time to submit a patch myself, but would be happy to help any
interested volunteers.
--
Michael

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-23 06:20    [W:1.149 / U:1.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site