Bug 1786317
| Summary: | Goodix touchscreen Kernel module not working after suspend | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Dahlhoff <benjamin.dahlhoff> | ||||||||
| Component: | kernel | Assignee: | Kernel Maintainer List <kernel-maint> | ||||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||||
| Severity: | medium | Docs Contact: | |||||||||
| Priority: | unspecified | ||||||||||
| Version: | 31 | CC: | airlied, benjamin.dahlhoff, bskeggs, hdegoede, ichavero, itamar, jarodwilson, jeremy, jglisse, john.j5live, jonathan, josef, kernel-maint, linville, masami256, mchehab, mjg59, steved | ||||||||
| Target Milestone: | --- | ||||||||||
| Target Release: | --- | ||||||||||
| Hardware: | x86_64 | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | Environment: | ||||||||||
| Last Closed: | 2020-11-04 12:17:45 UTC | Type: | Bug | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | --- | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Dahlhoff
2019-12-24 11:15:26 UTC
Benjamin, I happen to be working on some goodix.c changes related to issues like this. For starters, can you please provide an ACPI dump of the device: sudo acpidump -o acpidump And then attach the generated acpidump file here? Thanks. Created attachment 1647553 [details]
ACPIDump of Trekstor
acpidump attached
@Hans: I attached the ACPIdump as requested. Many greetings. Benjamin Thank you for the acpidump. I've started a test/scratch-build of a Fedora 31 kernel with some patches which will hopefully help: https://koji.fedoraproject.org/koji/taskinfo?taskID=39964621 This is still building atm (this take a couple of hours), once it is done building please install the kernel, see: https://fedorapeople.org/~jwrdegoede/kernel-test-instructions.txt for instructions on directly installing a kernel from koji. After installing, please boot into the new kernel, do a suspend/resume and after the suspend/resume do: dmesg | grep GDIX And copy and paste the output here. Also please try the touchscreen after the suspend/resume, with some luck it will still work... Problem is solved with Kernel 5.4.6-200.rhbz1786317.fc31.x86_64 Here is the output of ``dmesg | grep GDIX`` : [ 10.432254] Goodix-TS i2c-GDIX1001:00: i2c-GDIX1001:00 supply AVDD28 not found, using dummy regulator [ 10.432304] Goodix-TS i2c-GDIX1001:00: i2c-GDIX1001:00 supply VDDIO not found, using dummy regulator [ 10.432352] Goodix-TS i2c-GDIX1001:00: Using ACPI INTI and INTO methods for IRQ pin access [ 10.432867] Goodix-TS i2c-GDIX1001:00: ID 928, version: 1040 [ 10.438820] input: Goodix Capacitive TouchScreen as /devices/pci0000:00/0000:00:16.3/i2c_designware.3/i2c-8/i2c-GDIX1001:00/input/input11 [ 88.406392] Goodix-TS i2c-GDIX1001:00: Error reading config version: -121, resetting controller Thanks @Hans de Goede. Let me know if you have more to be tested on the Trekstor S11B. Great, thank you for testing. I will submit the patch series fixing this upstream. Once it has been accepted upstream I can also add (temporarily) the patches as downstream patches to the Fedora kernels. For now, as a workaround, it is probably best if you stick with the 5.4.6-200.rhbz1786317.fc31.x86_64 kernel. (In reply to Hans de Goede from comment #6) > Great, thank you for testing. > > I will submit the patch series fixing this upstream. Once it has been > accepted upstream I can also add (temporarily) the patches as downstream > patches to the Fedora kernels. > > For now, as a workaround, it is probably best if you stick with the > 5.4.6-200.rhbz1786317.fc31.x86_64 kernel. Hi Hans, thanks again for your work. Would it be possible to get the source code of your modified goodix.c and build a kernel module with dkms? If thats possible it would be helpful until the changes reached the Fedora Kernel. - Ben Created attachment 1657060 [details] goodix.c driver with patche to fix suspend/resume issues included You can find the individual patches (all 10 of them) in my personal git tree: https://github.com/jwrdegoede/linux-sunxi/commits/master I'm also attaching the reesulting goodix.c file here. This message is a reminder that Fedora 31 is nearing its end of life. Fedora will stop maintaining and issuing updates for Fedora 31 on 2020-11-24. It is Fedora's policy to close all bug reports from releases that are no longer maintained. At that time this bug will be closed as EOL if it remains open with a Fedora 'version' of '31'. Package Maintainer: If you wish for this bug to remain open because you plan to fix it in a currently maintained version, simply change the 'version' to a later Fedora version. Thank you for reporting this issue and we are sorry that we were not able to fix it before Fedora 31 is end of life. If you would still like to see this bug fixed and are able to reproduce it against a later version of Fedora, you are encouraged change the 'version' to a later Fedora version prior this bug is closed as described in the policy above. Although we aim to fix as many bugs as possible during every release's lifetime, sometimes those efforts are overtaken by events. Often a more recent Fedora release includes newer upstream software that fixes bugs or makes them obsolete. In the mean time the patches fixing this have landed quite a while ago, closing. |