All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Tirdea, Irina" <irina.tirdea@intel.com>
To: Brad Baker <bbaker6212@gmail.com>, bastien nocera <hadess@hadess.net>
Cc: "linux-input@vger.kernel.org" <linux-input@vger.kernel.org>
Subject: RE: Goodix touchscreen works intermittently, fails reading config info.
Date: Sun, 11 Sep 2016 18:54:48 +0000	[thread overview]
Message-ID: <1F3AC3675D538145B1661F571FE1805F2F32C5C6@irsmsx105.ger.corp.intel.com> (raw)
In-Reply-To: <CAG=1usatOHQ3sq5NTanbRQZ088gD53Xu3APVxe4p6Lf4mf5ahQ@mail.gmail.com>



> -----Original Message-----
> From: linux-input-owner@vger.kernel.org [mailto:linux-input-owner@vger.kernel.org] On Behalf Of Brad Baker
> Sent: 11 September, 2016 0:21
> To: bastien nocera
> Cc: linux-input@vger.kernel.org
> Subject: Goodix touchscreen works intermittently, fails reading config info.
> 
> Summary: Goodix touchscreen works intermittently, fails reading config info.
>  https://bugzilla.kernel.org/show_bug.cgi?id=156511
> 
> Description:  Chuwi Hi12 tablet with Intel cherry-trail Atom Z8300,
> the touchscreen does work after bootup most of the time, but sometimes
> it does not work without running modprobe.  When it fails the dmesg
> log shows the error "Invalid config, using defaults" and when it works
> this error is not present.
> 
> Module: goodix, goodix_backport
> Linux version 4.8.0-040800rc5-generic (kernel@tangerine) (gcc version
> 6.2.0 20160830 (Ubuntu 6.2.0-2ubuntu11) ) #201609041832 SMP Sun Sep 4
> 22:34:01 UTC 2016
> 
> dmesg log when it's failing:
> [ 10.012314] Goodix-TS i2c-GDIX1001:00: ID 9111, version: 1060
> [ 10.017711] Goodix-TS i2c-GDIX1001:00: Invalid config, using defaults
> [ 10.018021] input: Goodix Capacitive TouchScreen as
> /devices/pci0000:00/808622C1:05/i2c-13/i2c-GDIX1001:00/input/input6
> 
> details given in bugzilla #156511

Hi Brad,

I had similar problems with some Goodix devices until I enabled the
reset sequence [1].

Although the touch screen was initialized and I could read/write to/from
the device, I read only 0s from the configuration registers. The datasheet
recommends using the reset procedure at power on, so I could not find any
information on how to make the device work reliably without it. Resetting the
device at boot fixed the problem for me. For backwards compatibility, the
reset sequence is only available if the interrupt and reset gpio pins are
connected and configured through _DSD properties in the ACPI table or DT
properties.

One reason for the touchscreen not working in this case is that the interrupts
are not properly configured. I noticed in the Bugzilla details that you already
tried changing the default value for trigger_type to the one read from the device.
Did that work?

Regards,
Irina

[1] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=ec6e1b4082d9f5b0858ce33169a1c22a27a982f6

  reply	other threads:[~2016-09-11 18:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-10 21:21 Goodix touchscreen works intermittently, fails reading config info Brad Baker
2016-09-11 18:54 ` Tirdea, Irina [this message]
2016-09-11 19:28   ` Brad Baker
2016-09-12 17:32 Brad Baker
2016-09-12 17:43 ` Brad Baker
2016-09-13  3:04 ` Brad Baker
2016-09-14  9:29   ` Tirdea, Irina
2016-10-12 19:48     ` sergk sergk2mail
2016-10-03  0:53 Brad Baker
2016-10-12 20:37 Brad Baker
2016-10-17 20:31 ` Tirdea, Irina

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1F3AC3675D538145B1661F571FE1805F2F32C5C6@irsmsx105.ger.corp.intel.com \
    --to=irina.tirdea@intel.com \
    --cc=bbaker6212@gmail.com \
    --cc=hadess@hadess.net \
    --cc=linux-input@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.