Messages in this thread | | | Date | Sat, 7 Nov 2009 16:37:42 +0100 | From | Willy Tarreau <> | Subject | Re: [Stable-review] [78/99] hwmon: (it87) Fix VID reading on IT8718F/IT8720F |
| |
Hi Greg,
On Fri, Nov 06, 2009 at 02:15:16PM -0800, Greg KH wrote: > 2.6.31-stable review patch. If anyone has any objections, please let us know.
I noticed this one was not merged into 2.6.27.39. I suspect it is just because it did not apply since IT8720F was not present there. However I think the test still needs fixing, though I'm not well qualified to judge about that.
Jean, do you have any opinion on this matter ?
Thanks, Willy
> ------------------ > From: Jean Delvare <khali@linux-fr.org> > > commit 371dc4a6d8c3c74a9a1c74b87c2affb3fcef6500 upstream. > > Comparing apples to bananas doesn't seem right. Consistently use the > chips enum for chip type comparisons, to avoid such bugs in the > future. > > The bug has been there since support for the IT8718F was added, so > VID never worked for this chip nor for the similar IT8720F. > > Signed-off-by: Jean Delvare <khali@linux-fr.org> > Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> > > --- > drivers/hwmon/it87.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > --- a/drivers/hwmon/it87.c > +++ b/drivers/hwmon/it87.c > @@ -1028,12 +1028,11 @@ static int __init it87_find(unsigned sho > chip_type, *address, sio_data->revision); > > /* Read GPIO config and VID value from LDN 7 (GPIO) */ > - if (chip_type != IT8705F_DEVID) { > + if (sio_data->type != it87) { > int reg; > > superio_select(GPIO); > - if ((chip_type == it8718) || > - (chip_type == it8720)) > + if (sio_data->type == it8718 || sio_data->type == it8720) > sio_data->vid_value = superio_inb(IT87_SIO_VID_REG); > > reg = superio_inb(IT87_SIO_PINX2_REG); > > > _______________________________________________ > Stable-review mailing list > Stable-review@linux.kernel.org > http://linux.kernel.org/mailman/listinfo/stable-review
| |