From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753407AbbIESx3 (ORCPT ); Sat, 5 Sep 2015 14:53:29 -0400 Received: from mail-pa0-f54.google.com ([209.85.220.54]:33960 "EHLO mail-pa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751815AbbIESxT (ORCPT ); Sat, 5 Sep 2015 14:53:19 -0400 Date: Sat, 5 Sep 2015 11:53:08 -0700 From: Dmitry Torokhov To: Haibo Chen Cc: shawnguo@kernel.org, kernel@pengutronix.de, linux@arm.linux.org.uk, ijc+devicetree@hellion.org.uk, mark.rutland@arm.com, pawel.moll@arm.com, robh+dt@kernel.org, mpa@pengutronix.de, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-input@vger.kernel.org Subject: Re: [PATCH v3 1/5] input: touchscreen: add imx6ul_tsc driver support Message-ID: <20150905185308.GC18006@dtor-ws> References: <1440752976-9094-1-git-send-email-haibo.chen@freescale.com> <1440752976-9094-2-git-send-email-haibo.chen@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1440752976-9094-2-git-send-email-haibo.chen@freescale.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 28, 2015 at 05:09:32PM +0800, Haibo Chen wrote: > Freescale i.MX6UL contains a internal touchscreen controller, > this patch add a driver to support this controller. > > Signed-off-by: Haibo Chen Merged #1 and #2 and applied, thank you. Also: > + /* > + * Delay some time(max 2ms), wait the pre-charge done. > + * After the pre-change mode, TSC go into detect mode. > + * And in detect mode, we can get the xnur gpio value. > + * If xnur is high, this means the touch screen still > + * be touched. If xnur is low, this means finger leave > + * the touch screen. > + */ > + timeout = jiffies + HZ/500; I do not think that would quite work with HZ < 500. I changed this to timeout = jiffies + msecs_to_jiffies(2); and also moved this block into a separate function. Thanks. -- Dmitry