Commit 16aced80 authored by Zumeng Chen's avatar Zumeng Chen Committed by Tony Lindgren

ARM: OMAP3530evm: set pendown_state and debounce time for ads7846

Currently most ads7846 config definitions for OMAP3 series boards have
been moved to common-board-devices.c, and it is transparent for init.
And it's no very proper to do gpio_request based on get_pendown_state
since omap_ads7846_init knows everything about ads7846_config.

So it's more fit to request gpio according to the right gpio_pendown
and set debounce time conditionally. If we don't set proper debouce
time, there are flooded interrupt counters of ads7846 responding to
one time touch on screen, then the driver couldn't work very well.

This patch has been validated on 3530evm.
Signed-off-by: default avatarZumeng Chen <zumeng.chen@windriver.com>
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
parent cb8ca589
...@@ -58,7 +58,6 @@ ...@@ -58,7 +58,6 @@
#include "hsmmc.h" #include "hsmmc.h"
#include "common-board-devices.h" #include "common-board-devices.h"
#define OMAP3_EVM_TS_GPIO 175
#define OMAP3_EVM_EHCI_VBUS 22 #define OMAP3_EVM_EHCI_VBUS 22
#define OMAP3_EVM_EHCI_SELECT 61 #define OMAP3_EVM_EHCI_SELECT 61
......
...@@ -35,6 +35,16 @@ static struct omap2_mcspi_device_config ads7846_mcspi_config = { ...@@ -35,6 +35,16 @@ static struct omap2_mcspi_device_config ads7846_mcspi_config = {
.turbo_mode = 0, .turbo_mode = 0,
}; };
/*
* ADS7846 driver maybe request a gpio according to the value
* of pdata->get_pendown_state, but we have done this. So set
* get_pendown_state to avoid twice gpio requesting.
*/
static int omap3_get_pendown_state(void)
{
return !gpio_get_value(OMAP3_EVM_TS_GPIO);
}
static struct ads7846_platform_data ads7846_config = { static struct ads7846_platform_data ads7846_config = {
.x_max = 0x0fff, .x_max = 0x0fff,
.y_max = 0x0fff, .y_max = 0x0fff,
...@@ -45,6 +55,7 @@ static struct ads7846_platform_data ads7846_config = { ...@@ -45,6 +55,7 @@ static struct ads7846_platform_data ads7846_config = {
.debounce_rep = 1, .debounce_rep = 1,
.gpio_pendown = -EINVAL, .gpio_pendown = -EINVAL,
.keep_vref_on = 1, .keep_vref_on = 1,
.get_pendown_state = &omap3_get_pendown_state,
}; };
static struct spi_board_info ads7846_spi_board_info __initdata = { static struct spi_board_info ads7846_spi_board_info __initdata = {
...@@ -63,14 +74,17 @@ void __init omap_ads7846_init(int bus_num, int gpio_pendown, int gpio_debounce, ...@@ -63,14 +74,17 @@ void __init omap_ads7846_init(int bus_num, int gpio_pendown, int gpio_debounce,
struct spi_board_info *spi_bi = &ads7846_spi_board_info; struct spi_board_info *spi_bi = &ads7846_spi_board_info;
int err; int err;
if (board_pdata && board_pdata->get_pendown_state) { if (gpio_pendown) {
err = gpio_request_one(gpio_pendown, GPIOF_IN, "TSPenDown"); err = gpio_request_one(gpio_pendown, GPIOF_IN, "TSPenDown");
if (err) { if (err) {
pr_err("Couldn't obtain gpio for TSPenDown: %d\n", err); pr_err("Couldn't obtain gpio for TSPenDown: %d\n", err);
return; return;
} }
gpio_export(gpio_pendown, 0);
/* TS GPIOPendown doesn't allow user to change the direction */
gpio_export(gpio_pendown, false);
/* Set proper debouce time for ads7846. */
if (gpio_debounce) if (gpio_debounce)
gpio_set_debounce(gpio_pendown, gpio_debounce); gpio_set_debounce(gpio_pendown, gpio_debounce);
} }
......
...@@ -4,6 +4,7 @@ ...@@ -4,6 +4,7 @@
#include "twl-common.h" #include "twl-common.h"
#define NAND_BLOCK_SIZE SZ_128K #define NAND_BLOCK_SIZE SZ_128K
#define OMAP3_EVM_TS_GPIO 175
struct mtd_partition; struct mtd_partition;
struct ads7846_platform_data; struct ads7846_platform_data;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment