summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAmaury Pouly <amaury.pouly@gmail.com>2017-05-16 12:14:43 +1000
committerAmaury Pouly <amaury.pouly@gmail.com>2017-05-16 12:14:43 +1000
commit3210457764e0828056e64bbddbb9273d0384657c (patch)
treea399bc53cc448bb5db6e92d486e1c4845d12f716
parent5dd443c03385393de7458a03097e0b40635e6e4e (diff)
downloadrockbox-3210457764e0828056e64bbddbb9273d0384657c.tar.gz
rockbox-3210457764e0828056e64bbddbb9273d0384657c.zip
imx233: fix touchscreen driver
One cannot call lradc_acquire in IRQ context. The solution is to reserve the channel once at init. There is an additional complication on STMP3600 where channel mapping is fixed. Change-Id: Idccbac634a4d9002703e2b1d57748beb9b245cbb
-rw-r--r--firmware/target/arm/imx233/kernel-imx233.h2
-rw-r--r--firmware/target/arm/imx233/touchscreen-imx233.c36
2 files changed, 31 insertions, 7 deletions
diff --git a/firmware/target/arm/imx233/kernel-imx233.h b/firmware/target/arm/imx233/kernel-imx233.h
index 960f3f2431..74a6f248f2 100644
--- a/firmware/target/arm/imx233/kernel-imx233.h
+++ b/firmware/target/arm/imx233/kernel-imx233.h
@@ -43,4 +43,4 @@ int arbiter_acquire(struct channel_arbiter_t *a, int timeout);
void arbiter_release(struct channel_arbiter_t *a, int channel);
bool arbiter_acquired(struct channel_arbiter_t *a, int channel);
-#endif /* __KERNEL_IMX233__ */ \ No newline at end of file
+#endif /* __KERNEL_IMX233__ */
diff --git a/firmware/target/arm/imx233/touchscreen-imx233.c b/firmware/target/arm/imx233/touchscreen-imx233.c
index 7a784db5a3..f98dc9b738 100644
--- a/firmware/target/arm/imx233/touchscreen-imx233.c
+++ b/firmware/target/arm/imx233/touchscreen-imx233.c
@@ -57,6 +57,10 @@ enum touch_state_t
static enum touch_state_t touch_state;
static int touch_chan = -1;
+#if IMX233_SUBTARGET < 3700
+/* See imx233_touchscreen_init() */
+static int touch_chan_xp = -1, touch_chan_yp = -1;
+#endif
static int touch_delay = -1;
static int touch_x, touch_y;
/* once a touch is confirmed, the parameters are copied to these value for
@@ -98,11 +102,15 @@ static void pulldown_setup(bool xminus_enable, bool yminus_enable,
static void kick_measure(bool pull_x, bool pull_y, bool detect, int src)
{
- if(touch_chan >= 0)
- imx233_lradc_release_channel(touch_chan);
- touch_chan = imx233_lradc_acquire_channel(src, TIMEOUT_NOBLOCK);
- if(touch_chan < 0)
- panicf("touchscreen: cannot get adc channel");
+#if IMX233_SUBTARGET < 3700
+ /* Select proper channel on STMP3600 */
+ if(src == LRADC_SRC_XPLUS)
+ touch_chan = touch_chan_xp;
+ else if(src == LRADC_SRC_XPLUS)
+ touch_chan = touch_chan_xp;
+ else
+ panicf("Unknown channel source %d in touchscreen", src);
+#endif
/* enable interrupt */
imx233_lradc_set_channel_irq_callback(touch_chan, &touch_channel_irq);
imx233_icoll_enable_interrupt(INT_SRC_LRADC_CHx(touch_chan), true);
@@ -192,7 +200,23 @@ void imx233_touchscreen_init(void)
{
touch_delay = imx233_lradc_acquire_delay(TIMEOUT_NOBLOCK);
if(touch_delay < 0)
- panicf("Cannot acquire channel and delays for touchscreen measurement");
+ panicf("Cannot acquire delay for touchscreen measurement");
+#if IMX233_SUBTARGET >= 3700
+ /* On STMP3700+, any channel can measure any source so one channel is enough
+ * for all operations */
+ touch_chan = imx233_lradc_acquire_channel(LRADC_SRC_XPLUS, TIMEOUT_NOBLOCK);
+ if(touch_chan < 0)
+ panicf("touchscreen: cannot get adc channel");
+#else
+ /* On STMP3600, channel mapping is fixed for touch screen so we need to reserve
+ * two channels */
+ touch_chan_xp = imx233_lradc_acquire_channel(LRADC_SRC_XPLUS, TIMEOUT_NOBLOCK);
+ if(touch_chan_xp < 0)
+ panicf("touchscreen: cannot get adc X+ channel");
+ touch_chan_yp = imx233_lradc_acquire_channel(LRADC_SRC_YPLUS, TIMEOUT_NOBLOCK);
+ if(touch_chan_yp < 0)
+ panicf("touchscreen: cannot get adc Y+ channel");
+#endif
imx233_touchscreen_enable(false);
}