Re: [PATCH] [RESEND] Add Dell laptop backlight brightness display

[Matthew Garrett]

On Mon, Feb 06, 2006 at 09:43:16PM -0600, Michael E Brown wrote:

> 	I would _strongly_ suggest that this patch _not_ go in. This driver
> uses hardcoded values that are subject to change without notice. It is
> perfectly legitimate for future versions of Dell BIOS to interpret pokes
> to cmos location 0x99 as the command to format your hard drive.

I managed to send the wrong patch - the Dell one only reads from nvram. 
If nvram reads are likely to reformat your hard drive, I think Dell 
needs to reconsider its BIOS design :)

More seriously, a quick scan of libsmbios hasn't revealed any method for 
obtaining the screen brightness. It's possible that I'm blind (I'm 
certainly slightly drunk), but can you give a pointer to the correct 
mechanism for making this call?

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