Hi Alek,
On Tue, 20 May 2008 13:56:35 +0800, Alek Du wrote:
> Modify the patch according to your last comments except for two cases:
>
> > > +
> > > + /* If the SMBus is still busy, we give up */
> > > + if (timeout >= MAX_TIMEOUT) {
> > > + dev_err(&sch_adapter.dev, "SMBus Timeout!\n");
> > > + result = -EPERM;
> >
> > -ETIMEDOUT
>
> there is no -ETIMEOUT defined
ETIMEOUT is indeed not defined, but ETIMEDOUT (what I wrote) is.
> > > + dev_dbg(&dev->dev, "SMBA = 0x%X\n", sch_smba);
>
> > What about bit 31 of this register? It says whether the I/O area is
> > enabled or not. If it's not then you're registering the device but it
> > won't work, that's no good.
>
> As I tested, the bit 31 of this register is always 0...
Well, the datasheet clearly says otherwise. How did you test bit 31?
Your code uses pci_read_config_word() at the moment so you can't read
bit 31, you'd need to use pci_read_config_dword() instead. If Bit 31
really isn't used by the hardware, please report to whoever at Intel is
in charge of the technical documentation for this chip, so that they
can update the datahseet.
> Here is my patch v3, please help to review it:
I'll review it later today as my time permits.
--
Jean Delvare
_______________________________________________
i2c mailing list
[email protected]
http://lists.lm-sensors.org/mailman/listinfo/i2c
