Hi: * Jean Delvare <khali at linux-fr.org> [2007-08-28 12:23:27 +0200]: > Hi Auke, > > On Mon, 27 Aug 2007 16:17:01 -0700, Auke Kok wrote: > > Signed-off-by: Auke Kok <auke-jan.h.kok at intel.com> > > Cc: lm-sensors at lm-sensors.org > > --- > > > > drivers/hwmon/sis5595.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/hwmon/sis5595.c b/drivers/hwmon/sis5595.c > > index 92956eb..3d20a0c 100644 > > --- a/drivers/hwmon/sis5595.c > > +++ b/drivers/hwmon/sis5595.c > > @@ -517,7 +517,7 @@ static int __devinit sis5595_probe(struct platform_device *pdev) > > platform_set_drvdata(pdev, data); > > > > /* Check revision and pin registers to determine whether 4 or 5 voltages */ > > - pci_read_config_byte(s_bridge, PCI_REVISION_ID, &data->revision); > > + data->revision = s_bridge->revision; > > /* 4 voltages, 1 temp */ > > data->maxins = 3; > > if (data->revision >= REV2MIN) { > > I didn't know this revision field existed. Good cleanup, thanks. > > Acked-by: Jean Delvare <khali at linux-fr.org> Applied to hwmon-2.6.git/testing, thanks. -- Mark M. Hoffman mhoffman at lightlink.com