linux-kernel - Re: [PATCH] Samsung laptop driver

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20090826161827.GB22474@kroah.com>
Date:	Wed, 26 Aug 2009 09:18:27 -0700
From:	Greg KH <greg@...ah.com>
To:	Alan Jenkins <sourcejedi.lkml@...glemail.com>
Cc:	linux-kernel@...r.kernel.org, intel-gfx@...ts.freedesktop.org,
	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Soeren Sonnenburg <bugreports@....de>,
	Jérémie Huchet <jeremie@...ah.info>,
	Matthew Garrett <mjg59@...f.ucam.org>,
	Jesse Barnes <jbarnes@...tuousgeek.org>
Subject: Re: [PATCH] Samsung laptop driver

On Sat, Aug 22, 2009 at 10:23:16AM +0100, Alan Jenkins wrote:
> On 8/21/09, Greg KH <greg@...ah.com> wrote:
> > Well, it turns out that for the Samsung laptops, we aren't supposed to
> > be poking into the raw PCI config space to control the backlight,
> > there's a SMI way to do it instead (Matthew, you were right.)
> >
> > So here's an updated version of the driver, with lots of debugging still
> > left in it, that controls the backlight on a N130 through the "proper"
> > way.
> >
> > It's bigger than the last driver, as I've added lots of documenation
> > about this new interface.  I'm also working on adding wireless LED
> > support, as well as maybe CPU temperature monitoring to the driver, as
> > that can be controlled through this interface.
> >
> > Soeren and Jérémie, I took out support for both of your laptops, as I
> > didn't know if it supported this type of interface.  Could you load the
> > driver with the "force=1" parameter:
> > 	modprobe samsung-laptop force=1
> > and tell me what the kernel log shows for your laptop?  If it works,
> > I'll add the DMI values back to the driver.
> >
> > Jesse, I don't know how this is going to play with your recently added
> > backlight support for the i915 driver.  As you don't have control over
> > the LEDs and other stuff that this driver is going to support, I think
> > we need some way to keep the i915 driver from messing with the backlight
> > values, don't you think?
> >
> > thanks,
> >
> > greg k-h
> >
> > --------------
> >
> >
> > From: Greg Kroah-Hartman <gregkh@...e.de>
> > Subject: Samsung laptop driver
> >
> > This driver implements backlight controls for Samsung laptops that
> > currently do not have ACPI support for this control.
> >
> > It has been tested on the N130 laptop and properly works there.
> >
> > Many thanks to Dmitry Torokhov <dmitry.torokhov@...il.com> for cleanups
> > and other suggestions on how to make the driver simpler.
> >
> > Cc: Soeren Sonnenburg <bugreports@....de>
> > Cc: Jérémie Huchet <jeremie@...ah.info>
> > Cc: Dmitry Torokhov <dmitry.torokhov@...il.com>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@...e.de>
> >
> > ---
> >  drivers/platform/x86/Kconfig          |   12
> >  drivers/platform/x86/Makefile         |    1
> >  drivers/platform/x86/samsung-laptop.c |  420
> > ++++++++++++++++++++++++++++++++++
> >  3 files changed, 433 insertions(+)
> 
> > +static struct dmi_system_id __initdata samsung_dmi_table[] = {
> > +	{
> > +		.ident = "N120",
> > +		.matches = {
> > +			DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> > +			DMI_MATCH(DMI_PRODUCT_NAME, "N120"),
> > +			DMI_MATCH(DMI_BOARD_NAME, "N120"),
> > +		},
> > +		.callback = dmi_check_cb,
> > +	},
> > +	{
> > +		.ident = "N130",
> > +		.matches = {
> > +			DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> > +			DMI_MATCH(DMI_PRODUCT_NAME, "N130"),
> > +			DMI_MATCH(DMI_BOARD_NAME, "N130"),
> > +		},
> > +		.callback = dmi_check_cb,
> > +	},
> > +	{ },
> > +};
> 
> ...
> 
> > +MODULE_ALIAS("dmi:*:svnSAMSUNGELECTRONICSCO.,LTD.:pnN120:*:rnN120:*");
> > +MODULE_ALIAS("dmi:*:svnSAMSUNGELECTRONICSCO.,LTD.:pnN130:*:rnN130:*");
> 
> Can't you just use MODULE_DEVICE_TABLE(dmi, samsung_dmi_table)?

Doh, I didn't realize that, nice.

Hm, it seems not many others realize it either, it's only used by 2
other drivers in the whole kernel.  Might be a good job for some janitor
project to go and clean up...

thanks,

greg k-h
--
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/