From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Bernhard Bender <Bernhard.Bender@bytecmed.com>
Cc: "linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
Jakob Mustafa <Jakob.Mustafa@bytecmed.com>
Subject: Re: [PATCH 1/1] Avoid unresponsive TSC-30 touch screen
Date: Thu, 23 Jul 2015 14:03:01 -0700 [thread overview]
Message-ID: <20150723210301.GD12733@dtor-ws> (raw)
In-Reply-To: <6B919AC79333344DB0660E3FADEB2165057496DA51FD@BYTECEXCHGSRV.bytecexchg.local>
On Thu, Jul 23, 2015 at 05:01:31PM +0200, Bernhard Bender wrote:
> From: Bernhard Bender <bernhard.bender@bytecmed.com>
>
> This patch fixes a problem in the usbtouchscreen driver for DMC TSC-30 touch screen.
> Due to a missing delay between the RESET and SET_RATE commands, the touch screen may
> become unresponsive during system startup or driver loading.
>
> According to the DMC documentation, a delay is needed after the RESET command to allow
> the chip to complete its internal initialization. As this delay is not guaranteed, we
> had a system where the touch screen occasionally did not send any touch data. There was
> no other indication of the problem.
>
> The patch fixes the problem by adding a 150ms delay between the RESET and SET_RATE commands.
>
> Signed-off-by: Bernhard Bender <bernhard.bender@bytecmed.com>
> Suggested-by: Jakob Mustafa <jakob.mustafa@bytecmed.com>
Applied, thank you.
>
> ---
> Bernhard Bender
> Entwicklung
>
> Bytec Medizintechnik GmbH
> Hermann-Hollerith-Str. 11
> 52249 Eschweiler, Germany
> Tel. +49 (2403) 7829-943
> www.bytecmed.com
>
> Handelsregister Aachen: HRB 11222
> Ust-ID: DE 121732719
> Geschäftsführer: Dipl.-Ing. Paul Willi Coenen
>
>
> diff --git a/drivers/input/touchscreen/usbtouchscreen.c b/drivers/input/touchscreen/usbtouchscreen.c
> old mode 100644
> new mode 100755
> index a096633..4ebff01
> --- a/drivers/input/touchscreen/usbtouchscreen.c
> +++ b/drivers/input/touchscreen/usbtouchscreen.c
> @@ -625,6 +625,9 @@ static int dmc_tsc10_init(struct usbtouch_usb *usbtouch)
> goto err_out;
> }
>
> + /* TSC-25 data sheet specifies a delay after the RESET command */
> + msleep(150);
> +
> /* set coordinate output rate */
> buf[0] = buf[1] = 0xFF;
> ret = usb_control_msg(dev, usb_rcvctrlpipe (dev, 0),
>
>
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten haben, informieren Sie bitte sofort den Absender unter info@bytecmed.com und löschen Sie diese Mail. Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail ist nicht gestattet.
>
> This message may contain privileged and confidential information intended only for the use of the addressee named above. If you are not the intended recipient of this message you are hereby notified that any use, dissemination, distribution of reproduction of this message is prohibited. If you have received this message in error please notify info@bytecmed.com immediately.
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
prev parent reply other threads:[~2015-07-23 21:03 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-23 15:01 [PATCH 1/1] Avoid unresponsive TSC-30 touch screen Bernhard Bender
2015-07-23 21:03 ` Dmitry Torokhov [this message]
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=20150723210301.GD12733@dtor-ws \
--to=dmitry.torokhov@gmail.com \
--cc=Bernhard.Bender@bytecmed.com \
--cc=Jakob.Mustafa@bytecmed.com \
--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 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).