Re: [PATCH] thermal: Fix polling frequency for systems without passive cooling

2009-04-16 Thread Zhang Rui 
On Wed, 2009-04-15 at 03:16 +0800, Matthew Garrett wrote:
 The polling interval (in deciseconds) was accidently interpreted as 
 being in milliseconds in one codepath, resulting in excessively frequent 
 polling. Ensure that the conversion is performed.
 
 Signed-off-by: Matthew Garrett m...@redhat.com

Acked-by: Zhang Rui rui.zh...@intel.com

Len,
I think we can ship this patch in 2.6.30-rc3.

thanks,
rui
 
 ---
 
 diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c
 index 9cd15e8..564ea14 100644
 --- a/drivers/acpi/thermal.c
 +++ b/drivers/acpi/thermal.c
 @@ -909,7 +909,7 @@ static int acpi_thermal_register_thermal_zone(struct 
 acpi_thermal *tz)
   thermal_zone_device_register(acpitz, trips, tz,
acpi_thermal_zone_ops,
0, 0, 0,
 -  tz-polling_frequency);
 +  tz-polling_frequency*100);
   if (IS_ERR(tz-thermal_zone))
   return -ENODEV;
 

--
To unsubscribe from this list: send the line unsubscribe kernel-testers in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] thermal: Fix polling frequency for systems without passive cooling

2009-04-14 Thread Matthew Garrett 
The polling interval (in deciseconds) was accidently interpreted as 
being in milliseconds in one codepath, resulting in excessively frequent 
polling. Ensure that the conversion is performed.

Signed-off-by: Matthew Garrett m...@redhat.com

---

diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c
index 9cd15e8..564ea14 100644
--- a/drivers/acpi/thermal.c
+++ b/drivers/acpi/thermal.c
@@ -909,7 +909,7 @@ static int acpi_thermal_register_thermal_zone(struct 
acpi_thermal *tz)
thermal_zone_device_register(acpitz, trips, tz,
 acpi_thermal_zone_ops,
 0, 0, 0,
-tz-polling_frequency);
+tz-polling_frequency*100);
if (IS_ERR(tz-thermal_zone))
return -ENODEV;

-- 
Matthew Garrett | mj...@srcf.ucam.org
--
To unsubscribe from this list: send the line unsubscribe kernel-testers in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html