Jean Delvare wrote: > On Thu, 24 May 2007 09:34:37 -0600, Jim Cromie wrote: > >> Jean Delvare wrote: >> >>> I finally found some time to convert the pc87360 driver to a platform >>> driver. Jim (or anyone else with access to supported hardware), could >>> you please give it a try and confirm that it works OK? >>> >> yes it works! >> >> these are with the conf in debian unstable - >> >> soekris:~# uname -a >> Linux soekris 2.6.22-rc2-sk #2 Mon May 21 19:15:36 MDT 2007 i586 GNU/Linux >> soekris:~# >> soekris:~# sensors >> pc87366-isa-6620 >> Adapter: ISA adapter >> in0: +3.01 V (min = +0.00 V, max = +3.01 V) >> in1: +2.01 V (min = +0.00 V, max = +3.01 V) >> in2: +2.48 V (min = +0.00 V, max = +3.01 V) >> in3: +0.66 V (min = +0.00 V, max = +3.01 V) >> in4: +2.46 V (min = +0.00 V, max = +3.01 V) >> in5: +2.49 V (min = +0.00 V, max = +3.01 V) >> in6: +0.00 V (min = +0.00 V, max = +3.01 V) >> Vsb: +3.31 V (min = +0.00 V, max = +6.03 V) >> Vdd: +2.98 V (min = +0.00 V, max = +6.03 V) >> Vbat: +3.01 V (min = +0.00 V, max = +3.01 V) >> AVdd: +3.28 V (min = +0.00 V, max = +6.03 V) >> CPU0 Temp: +127 C (low = -55 C, high = +127 C) OPEN >> CPU0 Crit: +127 C ALARM >> CPU1 Temp: +127 C (low = -55 C, high = +127 C) OPEN >> CPU1 Crit: +127 C ALARM >> S-IO Temp: +93 C (low = -55 C, high = +127 C) >> S-IO Crit: +127 C >> > > Thanks for the report, Jim! Hopefully the output looks the same as > without my patch? Did you try unloading and reloading the driver too? > > Sorry for delay, I thought Id sent this earlier, but no.. I tested using 22-rc1 and 22-rc2 + 360-platform-patch, with 2 different configs - ( my best-config for my board, and debian's dist-config and captured `sensors > files`. Then compared the 2 pairs, using this program, http://jim.cromie.googlepages.com/linux-page which parses the output, and looks for a 5% difference before reporting problems. I find it easier than reading the voltages closely :-) comments, suggestions and complaints welcome.. > This patch is now in -mm. > > >> [ 1903.061714] hwmon-vid: requested unknown VRM version >> vid: +0.000 V (VRM Version 0.0) >> > > Hmmm, we should probably fix this warning, too. Maybe your CPU doesn't > even have VID pins. Patch follows. > > good++ thanks Jean, -jimc
