[v5] Input: ads7846 - fix support for ADS7845

[PATCH v5 0/3] Input: ads7846 - fix support for ADS7845

Posted by Luca Ellero 2 years, 7 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
 
Changes for v3:
 - send from the same email address of "Signed-off"

Changes for v4:
 - fix tag
 - fix comment in patch 0002

Changes for v5:
 - add Fixes: tag
 - fix comment in patch 0001

Luca Ellero (3):
  Input: ads7846 - don't report pressure for ads7845
  Input: ads7846 - always set last command to PWRDOWN
  Input: 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 v5 0/3] Input: ads7846 - fix support for ADS7845

Posted by Dmitry Torokhov 2 years, 7 months ago

On Thu, Jan 26, 2023 at 11:52:24AM +0100, Luca Ellero wrote:
> ADS7845 support is buggy in this driver.
> These patches fix various issues to get it work properly.

Applied the lot, thank you.

-- 
Dmitry

Re: [PATCH v5 0/3] Input: ads7846 - fix support for ADS7845

Posted by Andy Shevchenko 2 years, 7 months ago

On Thu, Jan 26, 2023 at 11:52:24AM +0100, Luca Ellero wrote:
> ADS7845 support is buggy in this driver.
> These patches fix various issues to get it work properly.

Entire series now looks good to me
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Thanks!

> Changes for v2:
>  - add missing period in patch 0001 message
>  - elaborate comment in patch 0002
>  
> Changes for v3:
>  - send from the same email address of "Signed-off"
> 
> Changes for v4:
>  - fix tag
>  - fix comment in patch 0002
> 
> Changes for v5:
>  - add Fixes: tag
>  - fix comment in patch 0001
> 
> Luca Ellero (3):
>   Input: ads7846 - don't report pressure for ads7845
>   Input: ads7846 - always set last command to PWRDOWN
>   Input: 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