Re: therm_pm72 units, interface

2013-09-27 Thread Michel Dänzer
On Mon, 2013-08-05 at 20:53 +1000, Benjamin Herrenschmidt wrote: > On Mon, 2013-08-05 at 12:32 +0200, Michel Dänzer wrote: > > > > I did that, sorry should have mentioned that. > > > > > > > > @@ -468,5 +478,3 @@ static struct platform_driver > > i2c_powermac_driver > > > > = > > > > { > > > >

Re: therm_pm72 units, interface

2013-08-05 Thread Benjamin Herrenschmidt
On Mon, 2013-08-05 at 12:32 +0200, Michel Dänzer wrote: > > I did that, sorry should have mentioned that. > > > > > @@ -468,5 +478,3 @@ static struct platform_driver > i2c_powermac_driver > > > = > > > { > > > }; > > > > > > module_platform_driver(i2c_powermac_driver); > > > - > > > -MODULE_A

Re: therm_pm72 units, interface

2013-08-05 Thread Michel Dänzer
On Mon, 2013-08-05 at 19:22 +1000, Benjamin Herrenschmidt wrote: > On Mon, 2013-08-05 at 11:13 +0200, Michel Dänzer wrote: > > > The version below builds, but the module still doesn't get loaded > > automagically (unless I'm missing some command I need to run between > > copying the new module to

Re: therm_pm72 units, interface

2013-08-05 Thread Benjamin Herrenschmidt
On Mon, 2013-08-05 at 11:13 +0200, Michel Dänzer wrote: > > static struct platform_driver i2c_powermac_driver = { > > .probe = i2c_powermac_probe, > > .remove = i2c_powermac_remove, > > + .id_table = *i2c_powermac_id, > > This fails to build: > > CC [M] drivers/i2c/busses/i2c-

Re: therm_pm72 units, interface

2013-08-05 Thread Michel Dänzer
On Sam, 2013-08-03 at 07:02 +1000, Benjamin Herrenschmidt wrote: > On Fri, 2013-08-02 at 18:52 +0200, Michel Dänzer wrote: > > > Thanks for the suggestion. The same windfarm modules were loaded in both > > cases, but i2c_powermac wasn't loaded with the newer kernels. Loading it > > manually fixes

Re: therm_pm72 units, interface

2013-08-02 Thread Ben Hutchings
On Sat, 2013-08-03 at 06:47 +1000, Benjamin Herrenschmidt wrote: > On Fri, 2013-08-02 at 16:47 +0200, Michel Dänzer wrote: > > On Fre, 2013-08-02 at 22:51 +1000, Benjamin Herrenschmidt wrote: > > > On Fri, 2013-08-02 at 12:03 +0200, Michel Dänzer wrote: > > > > Could a similar change fix the same p

Re: therm_pm72 units, interface

2013-08-02 Thread Benjamin Herrenschmidt
On Fri, 2013-08-02 at 22:56 +0200, Ben Hutchings wrote: > > Yes. Do you have all the windfarm modules loaded ? What about > > i2c-powermac ? (It's typical that the latter is missed, I think it > > doesn't auto-load, which we never fixed, we used to request distros > to > > just built it in) > > We

Re: therm_pm72 units, interface

2013-08-02 Thread Benjamin Herrenschmidt
On Fri, 2013-08-02 at 18:52 +0200, Michel Dänzer wrote: > Thanks for the suggestion. The same windfarm modules were loaded in both > cases, but i2c_powermac wasn't loaded with the newer kernels. Loading it > manually fixes the problem. > > How is i2c_powermac supposed to get loaded with current k

Re: therm_pm72 units, interface

2013-08-02 Thread Benjamin Herrenschmidt
On Fri, 2013-08-02 at 16:47 +0200, Michel Dänzer wrote: > On Fre, 2013-08-02 at 22:51 +1000, Benjamin Herrenschmidt wrote: > > On Fri, 2013-08-02 at 12:03 +0200, Michel Dänzer wrote: > > > Could a similar change fix the same problem on desktop G5s? The same > > > values for slots_param in windfarm_

Re: therm_pm72 units, interface

2013-08-02 Thread Michel Dänzer
On Fre, 2013-08-02 at 18:58 +0300, Aaro Koskinen wrote: > On Fri, Aug 02, 2013 at 04:47:51PM +0200, Michel Dänzer wrote: > > On Fre, 2013-08-02 at 22:51 +1000, Benjamin Herrenschmidt wrote: > > > On Fri, 2013-08-02 at 12:03 +0200, Michel Dänzer wrote: > > > > Could a similar change fix the same pro

Re: therm_pm72 units, interface

2013-08-02 Thread Aaro Koskinen
On Fri, Aug 02, 2013 at 04:47:51PM +0200, Michel Dänzer wrote: > On Fre, 2013-08-02 at 22:51 +1000, Benjamin Herrenschmidt wrote: > > On Fri, 2013-08-02 at 12:03 +0200, Michel Dänzer wrote: > > > Could a similar change fix the same problem on desktop G5s? The same > > > values for slots_param in wi

Re: therm_pm72 units, interface

2013-08-02 Thread Michel Dänzer
On Fre, 2013-08-02 at 22:51 +1000, Benjamin Herrenschmidt wrote: > On Fri, 2013-08-02 at 12:03 +0200, Michel Dänzer wrote: > > Could a similar change fix the same problem on desktop G5s? The same > > values for slots_param in windfarm_pm112.c don't help, unfortunately. > > You have a 11,2 and a no

Re: therm_pm72 units, interface

2013-08-02 Thread Benjamin Herrenschmidt
On Fri, 2013-08-02 at 12:03 +0200, Michel Dänzer wrote: > Could a similar change fix the same problem on desktop G5s? The same > values for slots_param in windfarm_pm112.c don't help, unfortunately. You have a 11,2 and a noisy fan ? Odd, mine(s) don't I can dbl check the values vs. what Darwin

Re: therm_pm72 units, interface

2013-08-02 Thread Michel Dänzer
On Sam, 2013-07-20 at 23:33 +0300, Aaro Koskinen wrote: > Hi, > > On Sat, Jul 20, 2013 at 09:16:49AM +1000, Benjamin Herrenschmidt wrote: > > On Fri, 2013-07-19 at 20:43 +0300, Aaro Koskinen wrote: > > > I booted a Xserve today with 3.11-rc1, and noticed the noise is coming > > > from slots-fan (P

Re: therm_pm72 units, interface

2013-07-20 Thread Benjamin Herrenschmidt
On Sat, 2013-07-20 at 23:33 +0300, Aaro Koskinen wrote: > Hi, > > On Sat, Jul 20, 2013 at 09:16:49AM +1000, Benjamin Herrenschmidt wrote: > > On Fri, 2013-07-19 at 20:43 +0300, Aaro Koskinen wrote: > > > I booted a Xserve today with 3.11-rc1, and noticed the noise is coming > > > from slots-fan (P

Re: therm_pm72 units, interface

2013-07-20 Thread Aaro Koskinen
Hi, On Sat, Jul 20, 2013 at 09:16:49AM +1000, Benjamin Herrenschmidt wrote: > On Fri, 2013-07-19 at 20:43 +0300, Aaro Koskinen wrote: > > I booted a Xserve today with 3.11-rc1, and noticed the noise is coming > > from slots-fan (PCI fan?) which is always 99%, although the slots-temp is > > just 33

Re: therm_pm72 units, interface

2013-07-19 Thread Benjamin Herrenschmidt
On Fri, 2013-07-19 at 20:43 +0300, Aaro Koskinen wrote: > I booted a Xserve today with 3.11-rc1, and noticed the noise is coming > from slots-fan (PCI fan?) which is always 99%, although the slots-temp is > just 33. Is it on purpose the rm31 slots PID params are totally different > from therm_pm72

Re: therm_pm72 units, interface

2013-07-19 Thread Aaro Koskinen
On Fri, Jan 18, 2013 at 10:38:05AM +1100, Benjamin Herrenschmidt wrote: > On Thu, 2013-01-17 at 15:38 +0100, Jan Engelhardt wrote: > > Meanwhile, I run Linux 3.7.1 and the software side changed somewhat. > > windfarm_rm31 seems to no longer calm the fans down once loaded, > > whereas therm_pm72 on

Re: therm_pm72 units, interface

2013-01-17 Thread Benjamin Herrenschmidt
On Thu, 2013-01-17 at 15:38 +0100, Jan Engelhardt wrote: > > Meanwhile, I run Linux 3.7.1 and the software side changed somewhat. > windfarm_rm31 seems to no longer calm the fans down once loaded, > whereas therm_pm72 on Linux 3.0 did so. Hrm, that would be a bug ... unfortunately I never got to

Re: therm_pm72 units, interface

2013-01-17 Thread Jan Engelhardt
On Wednesday 2012-08-15 23:36, Benjamin Herrenschmidt wrote: >BTW... On a somewhat related note ... if you happen to have a spare >Xserve G5 PSU I'm interested :-) I tried pulling it out, but the PSU stuck so hard in the ATX-like mainboard power socket that removing the PSU would have probably

Re: therm_pm72 units, interface

2012-08-17 Thread Benjamin Herrenschmidt
On Fri, 2012-08-17 at 15:51 +0200, Jan Engelhardt wrote: > > I can just exploit the fact that the machine will run for about an > hour when it has had a cooldown night. > > Except that 3.5, as I already expected by scary mails read on > linux-kernel, looked dangerous to use. Here is a boot-time >

Re: therm_pm72 units, interface

2012-08-17 Thread Jan Engelhardt
On Thursday 2012-08-16 22:51, Benjamin Herrenschmidt wrote: > >You can try netbooting... OF netboot is limited to 4M sized zImages >which can be a bit tough nowadays, but modern yaboot can netboot larger >files. Another option is USB sticks. I can just exploit the fact that the machine will run f

Re: therm_pm72 units, interface

2012-08-16 Thread Benjamin Herrenschmidt
> If you have more things to print/offer via sysfs, I'm all for it. > > The XsG5 really has (by looking into the casing): 1 PCI Fan, > 6 center fans, 1 PSU intake and 1 PSU outblow fan (this last one > seems rather slow-turning, but maybe that's normal). > It is not quite clear which is which in

Re: therm_pm72 units, interface

2012-08-16 Thread Jan Engelhardt
On Wednesday 2012-08-15 23:35, Benjamin Herrenschmidt wrote: >> XServe G5 of mine started powering off more or less >> randomly > >BTW. There's a new windfarm driver for these in recent kernels... > >Appart from that, the trip points are coming from a calibration EEPROM, >you may want to tweak th

therm_pm72 units, interface

2012-08-15 Thread Jan Engelhardt
About a week ago, an XServe G5 of mine started powering off more or less randomly (after 1 hour, chances were good it for it to occur). A problematic UPS has already been cut from the loop, and today I cleaned the machine inside out with pressurized air. So far it runs, for now at least, with

Re: therm_pm72 units, interface

2012-08-15 Thread Benjamin Herrenschmidt
BTW... On a somewhat related note ... if you happen to have a spare Xserve G5 PSU I'm interested :-) Mine died (well I -think- it's the PSU ... it just won't power up) which means I can't test on these things anymore (the new Windfarm RackMac driver is totally untested for example). Cheers, Ben.

Re: therm_pm72 units, interface

2012-08-15 Thread Benjamin Herrenschmidt
On Wed, 2012-08-15 at 19:53 +0200, Jan Engelhardt wrote: > About a week ago, an XServe G5 of mine started powering off more or less > randomly (after 1 hour, chances were good it for it to occur). A > problematic UPS has already been cut from the loop, and today I cleaned > the machine inside ou