Backport support for ACPI GPIO interrupts for i2c-hid devices

Bug #1501187 reported by Phidias
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
Fix Released
Undecided
Phidias
linux (Ubuntu)
Triaged
Medium
Phidias
Vivid
Fix Released
Undecided
Unassigned

Bug Description

[Impact]

On some platforms HID over I2C uses GPIO interrupts instead of directly connected to the IO-APIC. Without ACPI GPIO interrupts support, i2c-hid can not find IRQ when probing the device, and shows the following message:

i2c_hid i2c-SYNA7508:00: Could not register for SYNA7508:00 interrupt, irq = -1, ret = -22
i2c_hid: probe of i2c-SYNA7508:00 failed with error -22

So far we've found this issue on some SKL platforms, and one BSW platform.

[Fix]

There's a fix in mainline v4.1-rc1, which add the support for ACPI GPIO interrupts for i2c-hid module.

Commit log is as follows:

commit a485923efbb83056b7fb79e4fd2fee05c990ad5e
Author: Mika Westerberg <email address hidden>
Commit: Jiri Kosina <email address hidden>
CommitDate: Tue Feb 24 20:53:30 2015 +0100

    HID: i2c-hid: Add support for ACPI GPIO interrupts

[Test]

1. Boot with 4.1-rc1 kernel and patched 3.19 kernel
2. Both touchscreen should work

Phidias (phidias-chiang)
Changed in hwe-next:
assignee: nobody → Phidias (phidias-chiang)
Phidias (phidias-chiang)
tags: added: vivid
Revision history for this message
Brad Figg (brad-figg) wrote : Missing required logs.

This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:

apport-collect 1501187

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

Changed in linux (Ubuntu):
status: New → Incomplete
Changed in linux (Ubuntu):
assignee: nobody → Phidias (phidias-chiang)
penalvch (penalvch)
Changed in linux (Ubuntu):
importance: Undecided → Medium
Brad Figg (brad-figg)
Changed in linux (Ubuntu):
status: Incomplete → New
status: New → Triaged
Changed in linux (Ubuntu Vivid):
status: New → Fix Committed
Revision history for this message
Brad Figg (brad-figg) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-vivid' to 'verification-done-vivid'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-vivid
Phidias (phidias-chiang)
tags: added: verification-done-vivid
removed: verification-needed-vivid
Revision history for this message
Phidias (phidias-chiang) wrote :

Test:
1. Install proposed kernel
2. Check if touchscreen works -> pass
3. Test if touchscreen works after S3 -> pass

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package linux - 3.19.0-32.37

---------------
linux (3.19.0-32.37) vivid; urgency=low

  [ Luis Henriques ]

  * Release Tracking Bug
    - LP: #1508381

  [ Joseph Salisbury ]

  * SAUCE: storvsc: use small sg_tablesize on x86
    - LP: #1495983

  [ Phidias Chiang ]

  * SAUCE: dma: dw_dmac: Workaround for stop probing on HP X360 laptop v2
    - LP: #1501580

  [ Tim Gardner ]

  * [Config] Add MMC modules sufficient for net booting
    - LP: #1502772

  [ Upstream Kernel Changes ]

  * USB: whiteheat: fix potential null-deref at probe
    - LP: #1478826
    - CVE-2015-5257
  * dcache: Handle escaped paths in prepend_path
    - LP: #1441108
    - CVE-2015-2925
  * vfs: Test for and handle paths that are unreachable from their mnt_root
    - LP: #1441108
    - CVE-2015-2925
  * hv_netvsc: Add support to set MTU reservation from guest side
    - LP: #1494431
  * hv_netvsc: Add close of RNDIS filter into change mtu call
    - LP: #1494431
  * powerpc/eeh: Fix missed PE#0 on P7IOC
    - LP: #1502982
  * powerpc/powernv: display reason for Malfunction Alert HMI.
    - LP: #1482343
  * powerpc/powernv: Pull all HMI events before panic.
    - LP: #1482343
  * powerpc/powernv: Invoke opal_cec_reboot2() on unrecoverable machine
    check errors.
    - LP: #1482343
  * powerpc/powernv: Invoke opal_cec_reboot2() on unrecoverable HMI.
    - LP: #1482343
  * powerpc/eeh: Fix PE#0 check in eeh_add_to_parent_pe()
    - LP: #1502982
  * HID: i2c-hid: The interrupt should be level sensitive v2
    - LP: #1501187
  * HID: i2c-hid: Add support for ACPI GPIO interrupts v2
    - LP: #1501187

 -- Luis Henriques <email address hidden> Wed, 21 Oct 2015 10:30:13 +0100

Changed in linux (Ubuntu Vivid):
status: Fix Committed → Fix Released
Changed in hwe-next:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.