From: "Kristina Martšenko" <kristina.martsenko@gmail.com>
To: Stefan Wahren <stefan.wahren@i2se.com>
Cc: Peter Meerwald <pmeerw@pmeerw.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Hartmut Knaack <knaack.h@gmx.de>,
Lars-Peter Clausen <lars@metafoo.de>,
devel@driverdev.osuosl.org,
Alexandre Belloni <alexandre.belloni@free-electrons.com>,
Juergen Beisert <jbe@pengutronix.de>,
Fabio Estevam <fabio.estevam@freescale.com>,
linux-iio@vger.kernel.org, Marek Vasut <marex@denx.de>,
Jonathan Cameron <jic23@kernel.org>
Subject: Re: [PATCH 0/4] iio: mxs-lradc: fix interactions between the touchscreen and the ADC
Date: Mon, 19 Jan 2015 22:30:47 +0200 [thread overview]
Message-ID: <54BD6977.4050906@gmail.com> (raw)
In-Reply-To: <496109820.289562.1421580077115.JavaMail.open-xchange@oxbaltgw03.schlund.de>
On 18/01/15 13:21, Stefan Wahren wrote:
> Hi Kristina,
Hi Stefan!
>> Kristina Martšenko <kristina.martsenko@gmail.com> hat am 17. Januar 2015 um
>> 01:22 geschrieben:
>> These patches fix some issues with using the touchscreen and reading
>> other ADC channels at the same time.
>>
>> The main issue is that the "virtual" channels the touchscreen uses can
>> overlap with the channels that the iio buffer is configured to read,
>> causing the touchscreen to read those channels instead, and report wrong
>> values. Patch #1 fixes that by separating the channels they use.
>>
>> Patches #2 and #3 stop buffered mode and sysfs reads from disabling the
>> touchscreen. Patch #4 keeps the buffer from reporting wrong values when
>> the touchscreen is in use.
>>
>> Note that I've only tested these patches on i.MX28. I checked the i.MX23
>> reference manual [1], and there don't seem to be any differences in the
>> parts I changed, but it would still be nice if someone could test on
>> i.MX23 as well.
>
> i have a iMX233-Olinuxino-MAXI, but no touchscreen. If it does make sense
> to you, could you please send me testing instructions and i'll give it a try?
Thanks a lot for offering! Unfortunately in this case all testing
involves using the touchscreen while reading other ADC channels, so
without a touchscreen there isn't much to test, sorry :(
Kristina
next prev parent reply other threads:[~2015-01-19 20:31 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-17 0:22 [PATCH 0/4] iio: mxs-lradc: fix interactions between the touchscreen and the ADC Kristina Martšenko
2015-01-17 0:22 ` [PATCH 1/4] iio: mxs-lradc: separate touchscreen and buffer virtual channels Kristina Martšenko
2015-01-18 0:19 ` Marek Vasut
2015-01-19 19:02 ` Kristina Martšenko
2015-01-20 1:29 ` Marek Vasut
2015-01-17 0:22 ` [PATCH 2/4] iio: mxs-lradc: make ADC reads not disable touchscreen interrupts Kristina Martšenko
2015-01-18 0:21 ` Marek Vasut
2015-01-19 19:03 ` Kristina Martšenko
2015-01-17 0:22 ` [PATCH 3/4] iio: mxs-lradc: make ADC reads not unschedule touchscreen conversions Kristina Martšenko
2015-01-18 0:22 ` Marek Vasut
2015-01-19 7:20 ` Juergen Borleis
2015-01-19 7:57 ` Marek Vasut
2015-01-19 20:03 ` Kristina Martšenko
2015-01-17 0:22 ` [PATCH 4/4] iio: mxs-lradc: only update the buffer when its conversions have finished Kristina Martšenko
2015-01-18 0:23 ` Marek Vasut
2015-01-18 11:21 ` [PATCH 0/4] iio: mxs-lradc: fix interactions between the touchscreen and the ADC Stefan Wahren
2015-01-19 20:30 ` Kristina Martšenko [this message]
2015-01-20 1:32 ` Marek Vasut
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=54BD6977.4050906@gmail.com \
--to=kristina.martsenko@gmail.com \
--cc=alexandre.belloni@free-electrons.com \
--cc=devel@driverdev.osuosl.org \
--cc=fabio.estevam@freescale.com \
--cc=gregkh@linuxfoundation.org \
--cc=jbe@pengutronix.de \
--cc=jic23@kernel.org \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=marex@denx.de \
--cc=pmeerw@pmeerw.net \
--cc=stefan.wahren@i2se.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 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.