Messages in this thread | | | Date | Thu, 6 Jul 2006 00:46:39 -0500 | Subject | Re: [PATCH] powerpc: Xserve G5 thermal control fixes | From | Olof Johansson <> |
| |
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@kernel.crashing.org> > > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |