Hi David, > 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.). > Four sockets are ok. It will appear as 4 devices, so 4 instances of this driver ;) > Maybe this array should be a little bigger? It will save you some > trouble later. If not, it's okay to ignore this message... Well I dont know how to switch between more than two cores anyway. The driver should handle 4temps/physical CPU. regards Rudolf