Jean Delvare wrote: > Hi Kevin, Hi Jean, > On Mon, 26 Nov 2007 09:38:54 +0800, Kevin Lo wrote: > > On Sun, 2007-11-25 at 13:39 +0100, Jean Delvare wrote: > > > On Fri, 23 Nov 2007 09:31:52 +0800, Kevin Lo wrote: > > > > Here is the revised patch which fixes the issues that you mentioned. > > > > > > > > Signed-off-by: Kevin Lo <kevlo at kevlo.org> > > > > > > Dave, can you please test this new iteration of Kevin's driver and > > > confirm that it works fine for you? > > > > > > The patch is here if you're not subscribed to the list: > > > http://lists.lm-sensors.org/pipermail/lm-sensors/2007-November/021975.html > > > > > > Kevin, did you test your driver with lm-sensors 3.0.0? > > > > No, but I tested my patch with lm-sensors 2.10.5, it worked fine. > > I don't get it. For lm-sensors 2.10.5, you need chip-specific code for > every new driver, and there's no such code for the W83L786NG yet. So I > don't quite see how you could have tested it. Or do you have a patch > for lm-sensors that adds support? If so, please post it so that it can > be reviewed and applied. First off, sorry for the confusion. I had the w83l786ng driver patch for lm-sensors 2.10.5. I lost the patch due to my hard disk crash, that's why I didn't submit it :-( > Anyway, lm-sensors 3.0.0 only supports the standard kernel interface, > so it is a very interesting test to make sure that your driver > implements the standard interface properly. Please give it a try. I'll try lm-sensors 3.0.0 in a couple of days, thanks. > Thanks, > -- > Jean Delvare Regards, Kevin.
