[v2] ads7846: fix support for ADS7845

[PATCH v2 0/3] ads7846: fix support for ADS7845

Luca Ellero posted 3 patches 3 years, 9 months ago

Diff against v1 v1 v3
Download series mbox

There is a newer version of this series

drivers/input/touchscreen/ads7846.c | 23 +++++++++++++----------
1 file changed, 13 insertions(+), 10 deletions(-)

Expand all Fold all

[PATCH v2 0/3] ads7846: fix support for ADS7845

Posted by Luca Ellero 3 years, 9 months ago

ADS7845 support is buggy in this driver.
These patches fix various issues to get it work properly.

Changes for v2:
 - add missing period in patch 0001 message
 - elaborate comment in patch 0002
 
Luca Ellero (3):
  ads7846: don't report pressure for ads7845
  ads7846: always set last command to PWRDOWN
  ads7846: don't check penirq immediately for 7845

 drivers/input/touchscreen/ads7846.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

-- 
2.25.1

Re: [PATCH v2 0/3] ads7846: fix support for ADS7845

Posted by Andy Shevchenko 3 years, 8 months ago

On Thu, Jul 21, 2022 at 10:34:55AM +0200, Luca Ellero wrote:
> ADS7845 support is buggy in this driver.
> These patches fix various issues to get it work properly.
> 
> Changes for v2:
>  - add missing period in patch 0001 message
>  - elaborate comment in patch 0002

All three looks good to me,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> Luca Ellero (3):
>   ads7846: don't report pressure for ads7845
>   ads7846: always set last command to PWRDOWN
>   ads7846: don't check penirq immediately for 7845
> 
>  drivers/input/touchscreen/ads7846.c | 23 +++++++++++++----------
>  1 file changed, 13 insertions(+), 10 deletions(-)
> 
> -- 
> 2.25.1
> 

-- 
With Best Regards,
Andy Shevchenko