Re: [PATCH] powerpc: Xserve G5 thermal control fixes

From: Olof Johansson
Date: Thu Jul 06 2006 - 01:46:06 EST


On Thu, Jul 06, 2006 at 03:06:34PM +1000, Benjamin Herrenschmidt wrote:
> The thermal control for the Xserve G5s had a few issues. For one, the
> way to program the RPM fans speeds into the FCU is different between it
> and the desktop models, which I didn't figure out until recently, and it
> was missing a control loop for the slots fan, running it too fast. Both
> of those problems were causing the machine to be much more noisy than
> necessary. This patch also changes the fixed value of the slots fan for
> desktop G5s to 40% instead of 50%. It seems to still have a pretty good
> airflow that way and is much less noisy.
>
> Signed-off-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
>
> Index: linux-irq-work/drivers/macintosh/therm_pm72.c
> ===================================================================
> --- linux-irq-work.orig/drivers/macintosh/therm_pm72.c 2006-07-05 14:46:11.000000000 +1000
> +++ linux-irq-work/drivers/macintosh/therm_pm72.c 2006-07-06 14:42:31.000000000 +1000
> @@ -95,6 +95,14 @@
> * - Use min/max macros here or there
> * - Latest darwin updated U3H min fan speed to 20% PWM
> *
> + * July. 06, 2006 : 1.3
> + * - Fix setting of RPM fans on Xserve G5 (they were going too fast)
> + * - Add missing slots fan control loop for Xserve G5
> + * - Lower fixed slots fan speed from 50% to 40% on desktop G5s. We
> + * still can't properly implement the control loop for these, so let's
> + * reduce the noise a little bit, it appears that 40% still gives us
> + * a pretty good air flow
> + *

Doesn't it make more sense to keep this in the GIT log instead of in
the file? (I seem to remember Freescale getting similar comments on some
posted code just a few days ago :-)

Keeping version numbers for in-tree-only drivers isn't all that useful
either.


-Olof
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/