i2c-i801: Enable interrupts for all post-ICH5 chips
authorJean Delvare <khali@linux-fr.org>
Sun, 16 Dec 2012 20:11:55 +0000 (21:11 +0100)
committerJean Delvare <khali@endymion.delvare>
Sun, 16 Dec 2012 20:11:55 +0000 (21:11 +0100)
I did not receive a single bug report after interrupt support was
added for a limited number of chips. So I'd say the code is good and
should be enabled for all supported chips, that is: ICH5 and later.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
drivers/i2c/busses/i2c-i801.c

index e80eafc..1e73638 100644 (file)
@@ -1109,6 +1109,7 @@ static int __devinit i801_probe(struct pci_dev *dev,
                /* fall through */
        default:
                priv->features |= FEATURE_I2C_BLOCK_READ;
+               priv->features |= FEATURE_IRQ;
                /* fall through */
        case PCI_DEVICE_ID_INTEL_82801DB_3:
                priv->features |= FEATURE_SMBUS_PEC;
@@ -1121,16 +1122,6 @@ static int __devinit i801_probe(struct pci_dev *dev,
                break;
        }
 
-       /* IRQ processing tested on CougarPoint PCH, ICH5, ICH7-M and ICH10 */
-       if (dev->device == PCI_DEVICE_ID_INTEL_COUGARPOINT_SMBUS ||
-           dev->device == PCI_DEVICE_ID_INTEL_82801EB_3 ||
-           dev->device == PCI_DEVICE_ID_INTEL_ICH7_17 ||
-           dev->device == PCI_DEVICE_ID_INTEL_ICH8_5 ||
-           dev->device == PCI_DEVICE_ID_INTEL_ICH9_6 ||
-           dev->device == PCI_DEVICE_ID_INTEL_ICH10_4 ||
-           dev->device == PCI_DEVICE_ID_INTEL_ICH10_5)
-               priv->features |= FEATURE_IRQ;
-
        /* Disable features on user request */
        for (i = 0; i < ARRAY_SIZE(i801_feature_names); i++) {
                if (priv->features & disable_features & (1 << i))