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
