On Wed, 24 Oct 2012 20:14:41 +0200, Harald Judt wrote: > Hi Jean, > > Am 24.10.2012 20:05, schrieb Harald Judt: > [...] > > Thank you for your patch. I've applied it on 3.6.2, and it seems to work > > fine. The values are saved and restored correctly, and they also keep > > changing after resume. Further it gave me a little insight in how > > suspend/resume code works. > > Sorry, I stand corrected. Not all min/max values have been restored: > > Before suspend: > in1: +1.84 V (min = +0.00 V, max = +0.00 V) ALARM > AVCC: +3.34 V (min = +2.98 V, max = +3.63 V) > +3.3V: +3.34 V (min = +2.98 V, max = +3.63 V) > in4: +1.05 V (min = +0.00 V, max = +0.00 V) ALARM > in5: +1.68 V (min = +0.00 V, max = +0.00 V) ALARM > > After resume: > in1: +1.84 V (min = +0.00 V, max = +0.00 V) ALARM > AVCC: +3.34 V (min = +0.00 V, max = +0.00 V) ALARM > +3.3V: +3.34 V (min = +0.00 V, max = +0.00 V) ALARM > in4: +1.05 V (min = +0.00 V, max = +0.00 V) ALARM > in5: +1.68 V (min = +0.00 V, max = +0.00 V) ALARM > > I somehow missed the min/max differences in AVCC and +3.3V, too many > numbers ;-) Odd, I'll take a look. > But the rest is fine, double-checked now. To be really sure, it would be good if you could set arbitrary non-zero limits to in1, in4, in5 etc. otherwise we can't be sure these limits have been restored either. > BTW: cpu0_vid is always +0.000 V, am I right assuming this simply isn't > supported on my board? Yes, that's the most likely. With CPUs reporting their VID through MSRs these days, board manufacturers tend to stop wiring the VID pins to save space on the PCB. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
