| Message ID | 4b3d0f2cad0.58555470@auth.smtp.1and1.co.uk |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [RESEND] i2c/busses/pasemi: Remove hardcoded bus numbers on smbus | expand |
On Sun, Dec 31, 2017 at 08:53:55PM +0000, Darren Stevens wrote: > The pasemi smbus controller uses PCI_FUNC(dev->devfn) to define which > number bus to attach to, however this fails when something else is > probed first, for example an ATI Radeon graphics card will claim 9 or > 10 busses, including the ones the pasemi wants. > Patch the driver to call i2c_add_adapter rather than > i2c_add_numbered_adapter. > > Signed-off-by: Darren Stevens <darren@stevens-zone.net> > Applied to for-next, thanks! Disclaimer: I usually do not like to change the bus numbering because some people may rely on that. But numbering based on PCI functions seems really weak and all known users of pasemi seem to have issues here, so I make an exception. Thanks to Michael Ellerman for the additional info.
diff --git a/drivers/i2c/busses/i2c-pasemi.c b/drivers/i2c/busses/i2c-pasemi.c index df1dbc9..05847fd 100644 --- a/drivers/i2c/busses/i2c-pasemi.c +++ b/drivers/i2c/busses/i2c-pasemi.c @@ -365,7 +365,6 @@ static int pasemi_smb_probe(struct pci_dev *dev, smbus->adapter.class = I2C_CLASS_HWMON | I2C_CLASS_SPD; smbus->adapter.algo = &smbus_algorithm; smbus->adapter.algo_data = smbus; - smbus->adapter.nr = PCI_FUNC(dev->devfn); /* set up the sysfs linkage to our parent device */ smbus->adapter.dev.parent = &dev->dev; @@ -373,7 +372,7 @@ static int pasemi_smb_probe(struct pci_dev *dev, reg_write(smbus, REG_CTL, (CTL_MTR | CTL_MRR | (CLK_100K_DIV & CTL_CLK_M))); - error = i2c_add_numbered_adapter(&smbus->adapter); + error = i2c_add_adapter(&smbus->adapter); if (error) goto out_release_region;
The pasemi smbus controller uses PCI_FUNC(dev->devfn) to define which number bus to attach to, however this fails when something else is probed first, for example an ATI Radeon graphics card will claim 9 or 10 busses, including the ones the pasemi wants. Patch the driver to call i2c_add_adapter rather than i2c_add_numbered_adapter. Signed-off-by: Darren Stevens <darren@stevens-zone.net> ---