Messages in this thread | | | Date | Tue, 7 Feb 2006 04:09:35 +0000 | From | Matthew Garrett <> | Subject | Re: [PATCH] [RESEND] Add Dell laptop backlight brightness display |
| |
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@srcf.ucam.org - 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/
| |