linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Sebastian Reichel <sre@kernel.org>,
	kernel list <linux-kernel@vger.kernel.org>,
	maxime.ripard@free-electrons.com, dmitry.torokhov@gmail.com
Cc: "Pali Rohár" <pali.rohar@gmail.com>,
	"Ivaylo Dimitrov" <ivo.g.dimitrov.75@gmail.com>
Subject: 4.1 touchscreen regression on n900 -- pinpointed [was Re: linux-n900 v4.1-rc4]
Date: Fri, 29 May 2015 21:08:16 +0200	[thread overview]
Message-ID: <20150529190816.GA20232@amd> (raw)
In-Reply-To: <20150527145837.GA13223@earth>

Hi!

> mh I remember having problems with tsc2005 before. It helped to
> reset the controller (should actually happen automatically when it
> hangs, but I'm not sure, that it actually works).

Ok, I did some more testing, and found out rather bogus values in
evtest:

Input device name: "TSC2005 touchscreen"
Supported events:
  Event type 0 (EV_SYN)
  Event type 1 (EV_KEY)
  Event code 330 (BTN_TOUCH)
  Event type 3 (EV_ABS)
  Event code 0 (ABS_X)
  Value   2514
 Min        0
 Max        0
 Fuzz       4

Which made me go through the git logs, and these patches looked
suspicious. After a revert... yes, touchscreen works as well as it
worked before.

0a363a380954e10fece7cd9931b66056eeb07d56
3eea8b5d68c801fec788b411582b803463834752

(It is impossible to revert just 3eea..)

Its funny... changelog complains about duplicated code.. then inserts
30 lines of code. Along with check that was not there before.

Maxime, can you suggest a fix?
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

       reply	other threads:[~2015-05-29 19:08 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <201505241444.41039@pali>
     [not found] ` <20150527132545.GA23434@amd>
     [not found]   ` <20150527133311.GJ30798@pali>
     [not found]     ` <20150527143722.GA28108@amd>
     [not found]       ` <20150527145837.GA13223@earth>
2015-05-29 19:08         ` Pavel Machek [this message]
2015-05-29 19:25           ` 4.1 touchscreen regression on n900 -- pinpointed [was Re: linux-n900 v4.1-rc4] Pavel Machek
2015-05-29 19:32             ` [PATCH] fix n900 dts file to work around 4.1 touchscreen regression on n900 Pavel Machek
2015-05-29 19:49               ` Felipe Balbi
2015-05-29 19:56                 ` Pavel Machek
2015-05-29 20:17                   ` Maxime Ripard
2015-05-29 20:21                     ` Felipe Balbi
2015-05-29 20:29                       ` Dmitry Torokhov
2015-05-29 20:34                         ` Pavel Machek
2015-05-29 20:48                           ` Dmitry Torokhov
2015-05-29 21:02                             ` Pavel Machek
2015-05-29 21:38                               ` Dmitry Torokhov
2015-06-01  9:55                           ` Maxime Ripard
2015-06-01 14:06                             ` Please revert 3eea8b5d68c801fec788b411582b803463834752 as it breaks touchscreen " Pavel Machek
2015-06-01 14:58                               ` Maxime Ripard
2015-06-01 15:21                                 ` Pavel Machek
2015-06-01 17:47                                   ` Dmitry Torokhov
2015-06-01 20:27                                     ` Pavel Machek
2015-06-01 20:45                                       ` Dmitry Torokhov
2015-06-01 20:54                                         ` Tony Lindgren
2015-06-01 21:22                                     ` Maxime Ripard
2015-06-01 21:32                                       ` Dmitry Torokhov
2015-06-02  8:25                                         ` Pavel Machek
2015-06-02  9:44                                         ` Maxime Ripard
2015-06-02 17:58                                           ` Dmitry Torokhov
2015-06-02 18:08                                             ` Dmitry Torokhov
2015-06-25 20:25                                               ` Pavel Machek
2015-05-29 20:22                   ` [PATCH] fix n900 dts file to work around 4.1 touchscreen regression " Dmitry Torokhov
2015-05-29 20:03                 ` Maxime Ripard
2015-05-29 20:18                   ` Tony Lindgren
2015-05-29 20:30                     ` Pavel Machek
2015-05-30 10:14                   ` Pavel Machek
2015-06-01  9:49                     ` Maxime Ripard
2015-06-01  9:54                       ` Pavel Machek
2015-05-30  2:21               ` Sebastian Reichel
2015-05-30  2:24                 ` [PATCH] Input: of_touchscreen - remove interdependence of max/fuzz values Sebastian Reichel
2015-05-30  6:26                   ` Pavel Machek
2015-05-29 19:57             ` 4.1 touchscreen regression on n900 -- pinpointed [was Re: linux-n900 v4.1-rc4] Maxime Ripard
2015-05-29 20:29               ` Pavel Machek
2015-05-29 21:17               ` Pavel Machek
2015-05-29 21:36                 ` Dmitry Torokhov
2015-05-29 21:58                   ` Pavel Machek

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=20150529190816.GA20232@amd \
    --to=pavel@ucw.cz \
    --cc=dmitry.torokhov@gmail.com \
    --cc=ivo.g.dimitrov.75@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maxime.ripard@free-electrons.com \
    --cc=pali.rohar@gmail.com \
    --cc=sre@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).