[lm-sensors] [PATCH] k8temp - Add the diode offset to reported temperature

Amul Shah 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.

thanks,
Amul





More information about the lm-sensors mailing list