mbox series

[v2,0/5] EDT-FT5x06 improvements

Message ID 20191127120948.22251-1-m.felsch@pengutronix.de (mailing list archive)
Headers show
Series EDT-FT5x06 improvements | expand

Message

Marco Felsch Nov. 27, 2019, 12:09 p.m. UTC
Hi,

this v2 address all comments made on [1]. See the patch based changelog
for further information.

During the discussion we all agreed to use the common wakeup-src device
property but I added all users of this driver again. So they can provide
their ack-tag.

Regards,
  Marco

[1] https://patchwork.kernel.org/cover/11149039/

Marco Felsch (4):
  Input: edt-ft5x06 - alphabetical include reorder
  dt-bindings: Input: edt-ft5x06 - document wakeup-source capability
  Input: edt-ft5x06 - make wakeup-source switchable
  Input: edt-ft5x06 - improve power management operations

Philipp Zabel (1):
  Input: edt-ft5x06: work around first register access error

 .../bindings/input/touchscreen/edt-ft5x06.txt |  2 +
 drivers/input/touchscreen/edt-ft5x06.c        | 85 ++++++++++++++++---
 2 files changed, 73 insertions(+), 14 deletions(-)

Comments

Andy Shevchenko Nov. 27, 2019, 5:10 p.m. UTC | #1
On Wed, Nov 27, 2019 at 01:09:43PM +0100, Marco Felsch wrote:
> Hi,
> 
> this v2 address all comments made on [1]. See the patch based changelog
> for further information.
> 
> During the discussion we all agreed to use the common wakeup-src device
> property but I added all users of this driver again. So they can provide
> their ack-tag.

It doesn't break my case (Adafruit 2.8" TFT + cap/ts).

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

> 
> Regards,
>   Marco
> 
> [1] https://patchwork.kernel.org/cover/11149039/
> 
> Marco Felsch (4):
>   Input: edt-ft5x06 - alphabetical include reorder
>   dt-bindings: Input: edt-ft5x06 - document wakeup-source capability
>   Input: edt-ft5x06 - make wakeup-source switchable
>   Input: edt-ft5x06 - improve power management operations
> 
> Philipp Zabel (1):
>   Input: edt-ft5x06: work around first register access error
> 
>  .../bindings/input/touchscreen/edt-ft5x06.txt |  2 +
>  drivers/input/touchscreen/edt-ft5x06.c        | 85 ++++++++++++++++---
>  2 files changed, 73 insertions(+), 14 deletions(-)
> 
> -- 
> 2.20.1
>
Andy Shevchenko Nov. 27, 2019, 5:14 p.m. UTC | #2
On Wed, Nov 27, 2019 at 01:09:43PM +0100, Marco Felsch wrote:
> Hi,
> 
> this v2 address all comments made on [1]. See the patch based changelog
> for further information.
> 
> During the discussion we all agreed to use the common wakeup-src device
> property but I added all users of this driver again. So they can provide
> their ack-tag.
> 

Independently on the decision on patch 5, FWIW,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> Regards,
>   Marco
> 
> [1] https://patchwork.kernel.org/cover/11149039/
> 
> Marco Felsch (4):
>   Input: edt-ft5x06 - alphabetical include reorder
>   dt-bindings: Input: edt-ft5x06 - document wakeup-source capability
>   Input: edt-ft5x06 - make wakeup-source switchable
>   Input: edt-ft5x06 - improve power management operations
> 
> Philipp Zabel (1):
>   Input: edt-ft5x06: work around first register access error
> 
>  .../bindings/input/touchscreen/edt-ft5x06.txt |  2 +
>  drivers/input/touchscreen/edt-ft5x06.c        | 85 ++++++++++++++++---
>  2 files changed, 73 insertions(+), 14 deletions(-)
> 
> -- 
> 2.20.1
>
Marco Felsch Nov. 27, 2019, 5:25 p.m. UTC | #3
Hi Andy,

On 19-11-27 19:14, Andy Shevchenko wrote:
> On Wed, Nov 27, 2019 at 01:09:43PM +0100, Marco Felsch wrote:
> > Hi,
> > 
> > this v2 address all comments made on [1]. See the patch based changelog
> > for further information.
> > 
> > During the discussion we all agreed to use the common wakeup-src device
> > property but I added all users of this driver again. So they can provide
> > their ack-tag.
> > 
> 
> Independently on the decision on patch 5, FWIW,
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Thanks for testing and the review =)

Regards,
  Marco

> > Regards,
> >   Marco
> > 
> > [1] https://patchwork.kernel.org/cover/11149039/
> > 
> > Marco Felsch (4):
> >   Input: edt-ft5x06 - alphabetical include reorder
> >   dt-bindings: Input: edt-ft5x06 - document wakeup-source capability
> >   Input: edt-ft5x06 - make wakeup-source switchable
> >   Input: edt-ft5x06 - improve power management operations
> > 
> > Philipp Zabel (1):
> >   Input: edt-ft5x06: work around first register access error
> > 
> >  .../bindings/input/touchscreen/edt-ft5x06.txt |  2 +
> >  drivers/input/touchscreen/edt-ft5x06.c        | 85 ++++++++++++++++---
> >  2 files changed, 73 insertions(+), 14 deletions(-)
> > 
> > -- 
> > 2.20.1
> > 
> 
> -- 
> With Best Regards,
> Andy Shevchenko
> 
> 
>