mbox series

[v3,0/3] ads7846: fix support for ADS7845

Message ID 20220812084248.9270-1-l.ellero@asem.it
Headers show
Series ads7846: fix support for ADS7845 | expand

Message

Luca Ellero Aug. 12, 2022, 8:42 a.m. UTC 
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"

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(-)

Comments

Andy Shevchenko Aug. 23, 2022, 2:26 p.m. UTC | #1 
On Fri, Aug 12, 2022 at 10:42:45AM +0200, Luca Ellero wrote:
> ADS7845 support is buggy in this driver.
> These patches fix various issues to get it work properly.

Don't remember if I given a tag already (note, it's contributor's
responsibility to carry tags if received), so here it is:

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> 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"
> 
> 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
>