diff options
| author | Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> | 2016-10-21 18:48:41 -0400 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2016-11-05 10:57:38 -0400 |
| commit | 021afd55e2191248b471b29b9d0a96c267939a4d (patch) | |
| tree | 32d7dbbc6c7d2aaf4316e2dc7d9c5e8d49cf2ef6 | |
| parent | 2a1e3b932c5606e2b3671b82eb63929937eb1e0b (diff) | |
HID: intel-ish-hid: request_irq failure
On some platforms ISH interrupt is shared, which causes request_irq to
fail. This requires IRQF_SHARED irq flag.
But IRQF_NO_SUSPEND and IRQF_SHARED should not be used together, so
removed IRQF_NO_SUSPEND flag. Anyway this driver doesn't require
IRQF_NO_SUSPEND, as this interrupt is not required during "noirq" phases
of suspending and resuming devices as well as during the time when
nonboot CPUs are taken offline and brought back online.
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
| -rw-r--r-- | drivers/hid/intel-ish-hid/ipc/pci-ish.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/intel-ish-hid/ipc/pci-ish.c b/drivers/hid/intel-ish-hid/ipc/pci-ish.c index 7e3622a724aa..20d647d2dd2c 100644 --- a/drivers/hid/intel-ish-hid/ipc/pci-ish.c +++ b/drivers/hid/intel-ish-hid/ipc/pci-ish.c | |||
| @@ -146,7 +146,7 @@ static int ish_probe(struct pci_dev *pdev, const struct pci_device_id *ent) | |||
| 146 | pdev->dev_flags |= PCI_DEV_FLAGS_NO_D3; | 146 | pdev->dev_flags |= PCI_DEV_FLAGS_NO_D3; |
| 147 | 147 | ||
| 148 | /* request and enable interrupt */ | 148 | /* request and enable interrupt */ |
| 149 | ret = request_irq(pdev->irq, ish_irq_handler, IRQF_NO_SUSPEND, | 149 | ret = request_irq(pdev->irq, ish_irq_handler, IRQF_SHARED, |
| 150 | KBUILD_MODNAME, dev); | 150 | KBUILD_MODNAME, dev); |
| 151 | if (ret) { | 151 | if (ret) { |
| 152 | dev_err(&pdev->dev, "ISH: request IRQ failure (%d)\n", | 152 | dev_err(&pdev->dev, "ISH: request IRQ failure (%d)\n", |
