Quoting Chew Hong Gunn <hgchew at gunnet.org>: > > * Your new driver defines two different names for the two > > different releases of the chip. This breaks compatibility with > > libsensors, and would better be fixed (I will do too). > > Ok. It was there to differentiate the two revisions. The intention was good, but libsensors won't like it, sorry. > I have tested rc2 and there are no problems with it. I did however > make two minor changes to it: > * added swap_bytes function declaration (unless you deleted it > intentionally) Yes, deletion was intentional. This is a utility function and doesn't belong to the interface, so to say. > * removed voltage_in[0-2] = 0 for rev00 chips > these are initialised to 0 on init, so there should be no need > to set them at each update. I couldn't find where it had been initialized so decided to play it safe, but you're right, there's a memset at the very beginning, so this part can get away. Looks like we are not that far from a "final" version? Maybe some changes to bring comments, authorship and copyrights in line with the other drivers, and we should have something good for inclusion. Greg, we're listening to your comments if you have any. And thanks a lot for your initial review which was obviously needed ;) -- Jean Delvare http://www.ensicaen.ismra.fr/~delvare/
