<br><br><div class="gmail_quote">2012/6/13 Hiremath, Vaibhav <span dir="ltr"><<a href="mailto:hvaibhav@ti.com" target="_blank">hvaibhav@ti.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Jun 13, 2012 at 07:14:10, Zumeng Chen wrote:<br>
> From: Zumeng Chen <<a href="mailto:zumeng.chen@windriver.com">zumeng.chen@windriver.com</a>><br>
><br>
> If we don't set proper debouce time for ads7846, then there are<br>
> flooded interrupt counters of ads7846 responding to one time<br>
> touch on screen, so the driver couldn't work well.<br>
><br>
> And since most OMAP3 series boards pass NULL pointer of board_pdata<br>
> to omap_ads7846_init, so it's more proper to set it in driver level<br>
> after having gpio_request done.<br>
><br>
> This patch has been validated on 3530evm.<br>
><br>
> Signed-off-by: Zumeng Chen <<a href="mailto:zumeng.chen@windriver.com">zumeng.chen@windriver.com</a>><br>
> Signed-off-by: Syed Mohammed Khasim <<a href="mailto:khasim@ti.com">khasim@ti.com</a>><br>
> ---<br>
>  drivers/input/touchscreen/ads7846.c |    4 ++++<br>
>  1 files changed, 4 insertions(+), 0 deletions(-)<br>
><br>
> diff --git a/drivers/input/touchscreen/ads7846.c b/drivers/input/touchscreen/ads7846.c<br>
> index f02028e..459ff29 100644<br>
> --- a/drivers/input/touchscreen/ads7846.c<br>
> +++ b/drivers/input/touchscreen/ads7846.c<br>
> @@ -980,6 +980,10 @@ static int __devinit ads7846_setup_pendown(struct spi_device *spi, struct ads784<br>
>               }<br>
><br>
>               ts->gpio_pendown = pdata->gpio_pendown;<br>
> +#ifdef CONFIG_ARCH_OMAP3<br>
> +             /* 310 means about 10 microsecond for omap3 */<br>
> +             gpio_set_debounce(pdata->gpio_pendown, 310);<br>
> +#endif<br>
><br>
<br>
</div>Zumeng,<br>
<br>
With my sign-off you are changing the original code, that too<br>
without my sign-off and ack.<br>
I wouldn't mind you to submit patches from my tree, but the expectation is<br>
if you are changing the original code, it should be under your sign-off.<br></blockquote><div>Many thanks for your information, good to learn. I'll remove too. Sorry for<br>inconvenience.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
Coming to the patch, #ifdef in driver is not recommended, and this 10msec<br>
delay is specific to OMAP GPIO and driver should not be aware of this,<br>
that's where you will find the original patch handling it in board file.<br></blockquote><div>Yes, I just went through again the board related codes, there is a commit from<br>Mike, I think that's the basic thoughts for me to do this patch for all omap3 about<br>
ads7846 :-)<br><br>Regards,<br>Zumeng <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks,<br>
Vaibhav<br>
<div class="HOEnZb"><div class="h5">>       } else {<br>
>               dev_err(&spi->dev, "no get_pendown_state nor gpio_pendown?\n");<br>
> --<br>
> 1.7.5.4<br>
><br>
><br>
<br>
</div></div></blockquote></div><br>