Hi Alan, On Wed, Jul 23, 2014 at 01:06:57PM +0100, Alan Cox wrote: > This may appear as PCI or ACPI depending upon the firmware so we > have to list both. All share the same ACPI identifier but not > the same PCI identifier. > > Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx> > --- > drivers/i2c/busses/i2c-designware-pcidrv.c | 9 +++++++++ > drivers/i2c/busses/i2c-designware-platdrv.c | 1 + > 2 files changed, 10 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c b/drivers/i2c/busses/i2c-designware-pcidrv.c > index 3356f7a..d31d313 100644 > --- a/drivers/i2c/busses/i2c-designware-pcidrv.c > +++ b/drivers/i2c/busses/i2c-designware-pcidrv.c > @@ -188,6 +188,7 @@ static struct dw_pci_controller dw_pci_controllers[] = { > .scl_sda_cfg = &hsw_config, > }, > }; > + > static struct i2c_algorithm i2c_dw_algo = { > .master_xfer = i2c_dw_xfer, > .functionality = i2c_dw_func, > @@ -350,6 +351,14 @@ static const struct pci_device_id i2_designware_pci_ids[] = { > /* Haswell */ > { PCI_VDEVICE(INTEL, 0x9c61), haswell }, > { PCI_VDEVICE(INTEL, 0x9c62), haswell }, > + /* Braswell / Cherrytrail */ > + { PCI_VDEVICE(INTEL, 0x22C1), baytrail,}, > + { PCI_VDEVICE(INTEL, 0x22C2), baytrail }, > + { PCI_VDEVICE(INTEL, 0x22C3), baytrail }, > + { PCI_VDEVICE(INTEL, 0x22C4), baytrail }, > + { PCI_VDEVICE(INTEL, 0x22C5), baytrail }, > + { PCI_VDEVICE(INTEL, 0x22C6), baytrail }, > + { PCI_VDEVICE(INTEL, 0x22C7), baytrail }, > { 0,} > }; > MODULE_DEVICE_TABLE(pci, i2_designware_pci_ids); > diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c > index 402ec39..915ce6c2 100644 > --- a/drivers/i2c/busses/i2c-designware-platdrv.c > +++ b/drivers/i2c/busses/i2c-designware-platdrv.c > @@ -106,6 +106,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { > { "INT3432", 0 }, > { "INT3433", 0 }, > { "80860F41", 0 }, > + { "808622C1", 0 }, Why are 22c2 up to 22c7 not added here? Thanks, Wolfram
Attachment:
signature.asc
Description: Digital signature