Re: [ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-03-15 Thread Len Brown
On Sat, 14 Mar 2009, Henrique de Moraes Holschuh wrote: > On Sat, 14 Mar 2009, Mathieu Chouquet-Stringer wrote: > > On Sat, Mar 14, 2009 at 04:14:19PM +0100, Kay Sievers wrote: > > > On Sat, Mar 14, 2009 at 11:42, Mathieu Chouquet-Stringer > > > wrote: > > > > > > > While for older Thinkpads, y

Re: [ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-03-15 Thread Len Brown
applied thanks, Len Brown, Intel Open Source Technology Center On Sun, 15 Feb 2009, Henrique de Moraes Holschuh wrote: > From: Mathieu Chouquet-Stringer > > Looking at the source, there seems to be a missing * to match my DMI > string. I mean for newer IBM and Lenovo's laptops you match either

Re: [ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-03-15 Thread Mathieu Chouquet-Stringer
On Sat, Mar 14, 2009 at 04:14:19PM +0100, Kay Sievers wrote: > On Sat, Mar 14, 2009 at 11:42, Mathieu Chouquet-Stringer > wrote: > > > While for older Thinkpads, you do this (for instance): > > IBM_BIOS_MODULE_ALIAS("1[0,3,6,8,A-G,I,K,M-P,S,T]"); > > Aliases are shell-style globs. Any idea what

Re: [ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-03-15 Thread Kay Sievers
On Sat, Mar 14, 2009 at 11:42, Mathieu Chouquet-Stringer wrote: > While for older Thinkpads, you do this (for instance): > IBM_BIOS_MODULE_ALIAS("1[0,3,6,8,A-G,I,K,M-P,S,T]"); Aliases are shell-style globs. Any idea what the ',' are doing in a character class? Confused ... Thanks, Kay

Re: [ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-03-14 Thread Henrique de Moraes Holschuh
On Sat, 14 Mar 2009, Mathieu Chouquet-Stringer wrote: > On Sat, Mar 14, 2009 at 04:14:19PM +0100, Kay Sievers wrote: > > On Sat, Mar 14, 2009 at 11:42, Mathieu Chouquet-Stringer > > wrote: > > > > > While for older Thinkpads, you do this (for instance): > > > IBM_BIOS_MODULE_ALIAS("1[0,3,6,8,A-G,

[ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-03-14 Thread Mathieu Chouquet-Stringer
Hello Linus, Looks like this one liner patch never made it... Hopefully, it'll go in before you release rc9 or 2.6.29. Best, Mathieu - Forwarded message from Henrique de Moraes Holschuh - Delivered-To: online.fr-mchou...@free.fr X-Sasl-enc: J0pR0vt90JACc2icrP/7acPg7Vn8fC4jw2hS

[ibm-acpi-devel] [PATCH] thinkpad-acpi: fix module autoloading for older models

2009-02-15 Thread Henrique de Moraes Holschuh
From: Mathieu Chouquet-Stringer Looking at the source, there seems to be a missing * to match my DMI string. I mean for newer IBM and Lenovo's laptops you match either one of the following: MODULE_ALIAS("dmi:bvnIBM:*:svnIBM:*:pvrThinkPad*:rvnIBM:*"); MODULE_ALIAS("dmi:bvnLENOVO:*:svnLENOVO:*:pvr