Hi Rudolf, On 8/22/06, Rudolf Marek <r.marek at sh.cvut.cz> wrote: > Hello Jean, > > This patch adds support for the temperature sensor(s) found in AMD K8 CPUs. > > Signed-off-by: Rudolf Marek <r.marek at sh.cvut.cz> > > All fixed & tested on quad CPU - dual dual core ;), please apply. This line has me thinking... + u32 temp[2][2]; /* core, place */ AMD and Intel are both coming out with quad-core processors soon. Some could be backward-compatible with existing dual-core sockets. In addition, some servers have four sockets (quad processor, or 8-core, etc.). Maybe this array should be a little bigger? It will save you some trouble later. If not, it's okay to ignore this message... HTH, David
