linux-kernel - Re: [BISECTED] input: ads7846: touchscreen broken on Nokia 770

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <YjgBzDzHuZMlI05j@google.com>
Date:   Sun, 20 Mar 2022 21:40:44 -0700
From:   Dmitry Torokhov <dmitry.torokhov@...il.com>
To:     Aaro Koskinen <aaro.koskinen@....fi>
Cc:     Oleksij Rempel <o.rempel@...gutronix.de>,
        linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [BISECTED] input: ads7846: touchscreen broken on Nokia 770

Hi Aaro,

On Sat, Mar 19, 2022 at 05:13:46PM +0200, Aaro Koskinen wrote:
> Hi,
> 
> On Sun, Feb 27, 2022 at 08:50:39PM +0200, Aaro Koskinen wrote:
> > I noticed touchscreen does not work anymore on Nokia 770:
> > 
> > [    2.131011] ads7846 spi2.0: touchscreen, irq 175
> > [    2.200622] input: ADS7846 Touchscreen as /devices/platform/omap_uwire/spi_master/spi2/spi2.0/input/input0
> > 
> > (then touching the screen:)
> > 
> > [   52.316894] ads7846 spi2.0: spi_sync --> -22
> > [   52.336944] ads7846 spi2.0: spi_sync --> -22
> > 
> > bisected this to:
> > 
> > commit 9c9509717b53e701469493a8d87ed42c7d782502
> > Author: Oleksij Rempel <o.rempel@...gutronix.de>
> > Date:   Wed Nov 11 16:39:05 2020 -0800
> > 
> >     Input: ads7846 - convert to full duplex
> > 
> > It seems it's failing on SPI_CONTROLLER_HALF_DUPLEX check in
> > __spi_validate(). The controller used on this board (spi-omap-uwire)
> > sets that flag. How ads7846 is now supposed to work in that case after
> > this conversion?
> 
> Any comments?
> 
> The touchscreen works after reverting the following patches:
> 
> 6965eece2a89 ("Input: ads7846 - convert to one message")
> 9c9509717b53 ("Input: ads7846 - convert to full duplex")

I do not believe we should simply revert the patches as that would
penalize systems with nicer hardware, so my preference would be to see
if we could support half duplex controllers within the new driver
structure.

Thanks.

-- 
Dmitry