| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sun, 13 Mar 2011 16:28:12 +0000 From: Matthew Garrett <mjg59@...f.ucam.org> To: Greg KH <greg@...ah.com> Cc: Nikolai Kondrashov <spbnick@...il.com>, Randy Dunlap <rdunlap@...otime.net>, linux-kernel@...r.kernel.org, platform-driver-x86@...r.kernel.org Subject: Re: [PATCH] Platform: add Samsung Laptop platform driver On Sun, Mar 13, 2011 at 09:17:53AM -0700, Greg KH wrote: > On Sun, Mar 13, 2011 at 12:03:20PM +0200, Nikolai Kondrashov wrote: > > Sometime ago I've found this hack on some forum, which I use currently > > to set the brightness: > > > > setpci -s 00:02.0 f4.b="$HEX_BYTE" > > > > It allows finer control of the brightness, but it seems it operates directly > > on the video card PCI register. I assume there is no sane way to use > > this interface from the platform driver, right? > > Yes, do not do this, it causes all sorts of problems in the end, not the > least being the BIOS doesn't know what is going on at all as you are > changing the brightness without it knowing it. It's also modifying configuration that belongs to a device being managed by another driver, so it's the kind of thing we frown upon. Having said that, the Intel driver should gain native backlight support in .39. -- Matthew Garrett | mjg59@...f.ucam.org -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists