[lm-sensors] [PATCH] k8temp - Add the diode offset to reported temperature
amul.shah at unisys.com
Tue Apr 17 15:49:20 CEST 2007
On Tue, 2007-04-17 at 18:24 +0200, Rudolf Marek wrote:
> Hello all,
> Amul Shah wrote:
> > This patch adds support to include the CPU diode sensor offset when
> > reporting CPU temperature.
> > Please see http://www.lm-sensors.org/ticket/2200 for the full problem
> > description.
> Well I dont think this is correct. I believe that diode offset register is
> related to analog diode output not the digital sensor. Also I know that AMD is
> using my driver and did not make any changes. This leads me to a conclusion that
> the sensor simply does not work correctly and this error might be only
> compensated for analog part and not the digital. IIRC in some errata AMD states
> that the digital temperature sensor might be wrong and urges users to use the
> analog output.
> What do you think?
Well, I was looking to see what you think. :) Thanks for the
information. I'll go look for the errata and drop that information into
the ticket. After that we can close it as works as designed. :) No
need to fix what isn't broken.
More information about the lm-sensors