Re: [stable] [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Greg KH
On Wed, Feb 06, 2008 at 11:18:50PM -0200, Henrique de Moraes Holschuh wrote: > On Thu, 07 Feb 2008, Roel Kluin wrote: > > It's a oneliner and the patch is from linus' tree. > > Roel, better to just let me and Len Brown handle it. I will add your patch > to my thinkpad-acpi queue and send it to

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Len Brown
Applied. thanks, -Len On Monday 04 February 2008 18:24, Roel Kluin wrote: > Roland Dreier wrote: > > > /* safety net should the EC not support AUTO > > > * or FULLSPEED mode bits and just ignore them */ > > > if (level & TP_EC_FAN_FULLSPEED) >

Re: [stable] [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Greg KH
On Thu, Feb 07, 2008 at 12:48:24AM +0100, Roel Kluin wrote: > Greg KH wrote: > > On Tue, Feb 05, 2008 at 09:34:54AM +0100, Roel Kluin wrote: > >> Henrique de Moraes Holschuh wrote: > >>> On Tue, 05 Feb 2008, Roel Kluin wrote: > Roland Dreier wrote: > > > > Note the duplicate test 'if

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Henrique de Moraes Holschuh
On Thu, 07 Feb 2008, Roel Kluin wrote: > It's a oneliner and the patch is from linus' tree. Roel, better to just let me and Len Brown handle it. I will add your patch to my thinkpad-acpi queue and send it to Len soon, along with other simple patches that are pending. Either I or Len will notify

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Roel Kluin
Greg KH wrote: > On Tue, Feb 05, 2008 at 09:34:54AM +0100, Roel Kluin wrote: >> Henrique de Moraes Holschuh wrote: >>> On Tue, 05 Feb 2008, Roel Kluin wrote: Roland Dreier wrote: > > Note the duplicate test 'if (level & TP_EC_FAN_FULLSPEED)'. should > > this be replaced by >

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Greg KH
On Tue, Feb 05, 2008 at 09:34:54AM +0100, Roel Kluin wrote: > Henrique de Moraes Holschuh wrote: > > On Tue, 05 Feb 2008, Roel Kluin wrote: > >> Roland Dreier wrote: > >>> > /* safety net should the EC not support AUTO > >>> > * or FULLSPEED mode bits and just

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Greg KH
On Tue, Feb 05, 2008 at 09:34:54AM +0100, Roel Kluin wrote: Henrique de Moraes Holschuh wrote: On Tue, 05 Feb 2008, Roel Kluin wrote: Roland Dreier wrote: /* safety net should the EC not support AUTO * or FULLSPEED mode bits and just ignore them */

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Roel Kluin
Greg KH wrote: On Tue, Feb 05, 2008 at 09:34:54AM +0100, Roel Kluin wrote: Henrique de Moraes Holschuh wrote: On Tue, 05 Feb 2008, Roel Kluin wrote: Roland Dreier wrote: Note the duplicate test 'if (level TP_EC_FAN_FULLSPEED)'. should this be replaced by Actually I suspect one of the

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Henrique de Moraes Holschuh
On Thu, 07 Feb 2008, Roel Kluin wrote: It's a oneliner and the patch is from linus' tree. Roel, better to just let me and Len Brown handle it. I will add your patch to my thinkpad-acpi queue and send it to Len soon, along with other simple patches that are pending. Either I or Len will notify

Re: [stable] [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Greg KH
On Thu, Feb 07, 2008 at 12:48:24AM +0100, Roel Kluin wrote: Greg KH wrote: On Tue, Feb 05, 2008 at 09:34:54AM +0100, Roel Kluin wrote: Henrique de Moraes Holschuh wrote: On Tue, 05 Feb 2008, Roel Kluin wrote: Roland Dreier wrote: Note the duplicate test 'if (level

Re: [stable] [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-06 Thread Greg KH
On Wed, Feb 06, 2008 at 11:18:50PM -0200, Henrique de Moraes Holschuh wrote: On Thu, 07 Feb 2008, Roel Kluin wrote: It's a oneliner and the patch is from linus' tree. Roel, better to just let me and Len Brown handle it. I will add your patch to my thinkpad-acpi queue and send it to Len

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-05 Thread Roel Kluin
Henrique de Moraes Holschuh wrote: > On Tue, 05 Feb 2008, Roel Kluin wrote: >> Roland Dreier wrote: >>> > /* safety net should the EC not support AUTO >>> > * or FULLSPEED mode bits and just ignore them */ >>> > if (level & TP_EC_FAN_FULLSPEED)

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-05 Thread Roel Kluin
Henrique de Moraes Holschuh wrote: On Tue, 05 Feb 2008, Roel Kluin wrote: Roland Dreier wrote: /* safety net should the EC not support AUTO * or FULLSPEED mode bits and just ignore them */ if (level TP_EC_FAN_FULLSPEED)

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-04 Thread Henrique de Moraes Holschuh
On Tue, 05 Feb 2008, Roel Kluin wrote: > Roland Dreier wrote: > > > /* safety net should the EC not support AUTO > > > * or FULLSPEED mode bits and just ignore them */ > > > if (level & TP_EC_FAN_FULLSPEED) > > > level |=

[PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level & TP_EC_FAN_FULLSPEED)

2008-02-04 Thread Roel Kluin
Roland Dreier wrote: > > /* safety net should the EC not support AUTO > > * or FULLSPEED mode bits and just ignore them */ > > if (level & TP_EC_FAN_FULLSPEED) > > level |= 7; /* safety min speed 7 */ > >

[PATCH][drivers/misc/thinkpad_acpi.c] duplicate test 'if (level & TP_EC_FAN_FULLSPEED)'

2008-02-04 Thread Roel kluin
in drivers/misc/thinkpad_acpi.c, lines 4137-4142 it reads: /* safety net should the EC not support AUTO * or FULLSPEED mode bits and just ignore them */ if (level & TP_EC_FAN_FULLSPEED) level |= 7; /* safety min speed 7

[PATCH][drivers/misc/thinkpad_acpi.c] duplicate test 'if (level TP_EC_FAN_FULLSPEED)'

2008-02-04 Thread Roel kluin
in drivers/misc/thinkpad_acpi.c, lines 4137-4142 it reads: /* safety net should the EC not support AUTO * or FULLSPEED mode bits and just ignore them */ if (level TP_EC_FAN_FULLSPEED) level |= 7; /* safety min speed 7

[PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-04 Thread Roel Kluin
Roland Dreier wrote: /* safety net should the EC not support AUTO * or FULLSPEED mode bits and just ignore them */ if (level TP_EC_FAN_FULLSPEED) level |= 7; /* safety min speed 7 */ else

Re: [PATCH][drivers/misc/thinkpad_acpi.c] duplicate test if (level TP_EC_FAN_FULLSPEED)

2008-02-04 Thread Henrique de Moraes Holschuh
On Tue, 05 Feb 2008, Roel Kluin wrote: Roland Dreier wrote: /* safety net should the EC not support AUTO * or FULLSPEED mode bits and just ignore them */ if (level TP_EC_FAN_FULLSPEED) level |= 7; /*