All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bernhard Bender <Bernhard.Bender@bytecmed.com>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: "linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
	Jakob Mustafa <Jakob.Mustafa@bytecmed.com>
Subject: [PATCH 1/1] Avoid unresponsive TSC-30 touch screen
Date: Thu, 23 Jul 2015 17:01:31 +0200	[thread overview]
Message-ID: <6B919AC79333344DB0660E3FADEB2165057496DA51FD@BYTECEXCHGSRV.bytecexchg.local> (raw)

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>

---
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.

             reply	other threads:[~2015-07-23 15:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-23 15:01 Bernhard Bender [this message]
2015-07-23 21:03 ` [PATCH 1/1] Avoid unresponsive TSC-30 touch screen Dmitry Torokhov

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=6B919AC79333344DB0660E3FADEB2165057496DA51FD@BYTECEXCHGSRV.bytecexchg.local \
    --to=bernhard.bender@bytecmed.com \
    --cc=Jakob.Mustafa@bytecmed.com \
    --cc=dmitry.torokhov@gmail.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 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.