diff options
| author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2016-12-10 15:58:55 -0500 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2016-12-12 03:47:31 -0500 |
| commit | 8cd16166b0002df427ed39e4970e0be4830199b2 (patch) | |
| tree | c694d53bc91d088b718ed9f8a4f16a3f3740e104 | |
| parent | ba1660f1791f31a76da247caebf273b3f3d5071c (diff) | |
HID: fix missing irq field
commit ba18a9314a94 ("Revert "HID: i2c-hid: Add support for ACPI GPIO
interrupts"") removed the need for storing the irq in struct i2c_hid.
But then commit de3c99488609 ("HID: i2c-hid: Disable IRQ before freeing
buffers") forgot to update the location of the irq.
Fix this by using the actual I2C client irq.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
| -rw-r--r-- | drivers/hid/i2c-hid/i2c-hid.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c index 5b90ddaa9350..78fb32a7b103 100644 --- a/drivers/hid/i2c-hid/i2c-hid.c +++ b/drivers/hid/i2c-hid/i2c-hid.c | |||
| @@ -770,11 +770,11 @@ static int i2c_hid_start(struct hid_device *hid) | |||
| 770 | i2c_hid_find_max_report(hid, HID_FEATURE_REPORT, &bufsize); | 770 | i2c_hid_find_max_report(hid, HID_FEATURE_REPORT, &bufsize); |
| 771 | 771 | ||
| 772 | if (bufsize > ihid->bufsize) { | 772 | if (bufsize > ihid->bufsize) { |
| 773 | disable_irq(ihid->irq); | 773 | disable_irq(client->irq); |
| 774 | i2c_hid_free_buffers(ihid); | 774 | i2c_hid_free_buffers(ihid); |
| 775 | 775 | ||
| 776 | ret = i2c_hid_alloc_buffers(ihid, bufsize); | 776 | ret = i2c_hid_alloc_buffers(ihid, bufsize); |
| 777 | enable_irq(ihid->irq); | 777 | enable_irq(client->irq); |
| 778 | 778 | ||
| 779 | if (ret) | 779 | if (ret) |
| 780 | return ret; | 780 | return ret; |
