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