Re: [PATCH 3/6] Input: edt-ft5x06 - add support to disable the wakeup-source

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 17, 2019 at 06:46:39PM +0200, Marco Felsch wrote:
> On 19-09-17 19:32, Andy Shevchenko wrote:
> > On Tue, Sep 17, 2019 at 05:58:05PM +0200, Marco Felsch wrote:
> > > Since day one the touch controller acts as wakeup-source. This seems to
> > > be wrong since the device supports deep-sleep mechanism [1] which
> > > requires a reset to leave it. Also some designs won't use the
> > > touchscreen as wakeup-source.
> > > 
> > > Add a firmware property to address this. The common 'wakeup-source'
> > > property can't be used for that because the driver must be backward
> > > compatible with old firmwares which may assume the default on
> > > wakeup-source behaviour. So we need to go the other way by explicit
> > > disable the wakeup-source capability.
> > > 
> > 
> > > [1] https://www.newhavendisplay.com/appnotes/datasheets/touchpanel/ \
> > >     FT5x26.pdf
> > 
> > Please, don't split URLs
> 
> Hm.. then checkpatch complains.. If you prefer it, I can change it in
> the v2.

Checkpatch complains about valid things and it complains about insane
things. In this case simply ignore it.

Thanks.

-- 
Dmitry



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux