linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mugunthan V N <mugunthanvnm@ti.com>
To: <linux-input@vger.kernel.org>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Jonathan Cameron <jic23@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Lee Jones <lee.jones@linaro.org>, Sekhar Nori <nsekhar@ti.com>,
	Vignesh R <vigneshr@ti.com>, <devicetree@vger.kernel.org>,
	<linux-omap@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	Mugunthan V N <mugunthanvnm@ti.com>
Subject: [PATCH v2 0/3] increase TSCADC clock to 24MHz and fix ti,charge-delay to represent in nS
Date: Thu, 10 Nov 2016 22:05:12 +0530	[thread overview]
Message-ID: <20161110163515.27598-1-mugunthanvnm@ti.com> (raw)

This patch series enables ADC to be clocked at 24MHz as the
TI AM335x ADC driver has already adopted to use DMA to transfer
ADC samples. Now ADC can generated upto 800K Samples per second
with the patch [1] on AM335x BBB and AM437x GP EVM.

when ADC ref clock is set at 24MHz, I am seeing some issue with
touch screen pointer as the pointer jumps to random locations
with free draw application. The issue is due to increase in ADC
clock and charge delay for the touchscreen ADC line duration
reduced.

So the notation of ti,charge-delay in terms of ADC clock is
wrong, it has to be represented in time and driver has to convert
the charge delay time to ADC clocks based on what ADC clock
frequency is set.

Measured the performance with the iio_generic_buffer with the
patch [2] applied

Verified the touch screen on AM335x GP EVM and AM335x BBB LCD7
cape with [3] dts for display and touch screen to work.

Changes from initial version:
* Removed modification to STEPCONFIG_OPENDLY defined as it
  doesn't affect/improve touchscreen performance.
* Changed ti,charge-delay to ti,charge-delay-ns

[1] - http://pastebin.ubuntu.com/23357935/
[2] - http://pastebin.ubuntu.com/23357939/
[3] - http://pastebin.ubuntu.com/23456616/

Mugunthan V N (3):
  dt/binding: ti-tsc-adc: deprecate ti,charge-delay and add binding doc
    for ti,charge-delay-ns
  Input: ti_am335x_tsc: Add support for ti,charge-delay-ns
  drivers: mfd: ti_am335x_tscadc: increase ADC ref clock to 24MHz

 .../bindings/input/touchscreen/ti-tsc-adc.txt      | 32 ++++++++++++++--------
 drivers/input/touchscreen/ti_am335x_tsc.c          | 31 +++++++++++++++------
 include/linux/mfd/ti_am335x_tscadc.h               |  2 +-
 3 files changed, 45 insertions(+), 20 deletions(-)

-- 
2.11.0.rc0.7.gbe5a750

             reply	other threads:[~2016-11-10 16:49 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-10 16:35 Mugunthan V N [this message]
2016-11-10 16:35 ` [PATCH v2 1/3] dt/binding: ti-tsc-adc: deprecate ti,charge-delay and add binding doc for ti,charge-delay-ns Mugunthan V N
2016-11-15  1:54   ` Rob Herring
2016-11-11  7:58 ` [PATCH v2 2/3] Input: ti_am335x_tsc: Add support " Mugunthan V N
2016-11-29 21:09   ` Dmitry Torokhov
2016-11-11  7:58 ` [PATCH v2 3/3] drivers: mfd: ti_am335x_tscadc: increase ADC ref clock to 24MHz Mugunthan V N
2016-11-22 13:04   ` Lee Jones
2016-11-25  4:57 ` [PATCH v2 0/3] increase TSCADC clock to 24MHz and fix ti,charge-delay to represent in nS Mugunthan V N
2016-11-25  9:59   ` Lee Jones
2016-11-29  5:41     ` Mugunthan V N
2016-11-29 21:09       ` Dmitry Torokhov
2016-12-05  3:54         ` Mugunthan V N

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=20161110163515.27598-1-mugunthanvnm@ti.com \
    --to=mugunthanvnm@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jic23@kernel.org \
    --cc=lee.jones@linaro.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=nsekhar@ti.com \
    --cc=robh+dt@kernel.org \
    --cc=vigneshr@ti.com \
    /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).