Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
IronPort-SDR: 2TaUWfGz159jBrIxLvkvbG003RsTsfUQdEWkDyCfQ3Quvp2igP1+epXJpHTxq5NeluR5o/Tf4U
 QDAoJYl9PeXp/pPycO9PSW0MQWyxDpT0BudQj/KT2MOT6cCqhAgrHWoCPbNLzp8Hg5Ud7JEl+/
 hijxypV/Z6O5Tvg1pAu3oVkqwRKoOYpupBOnaccBfTutVt3Sgl4ufwzA9VHr2K5v03Gc6k2vkw
 dao+oSKrhDMQ993IkSJs143LMHyTkVBmpTdGnAA6rmu6OaSPFDVCOBXN+8+AVA2mNvKWaAnt9j
 /cU=
IronPort-SDR: pQpryPK47i03139lAwDnAblCiPUOtB7ZZx0CRFiUUCN9ujhFnqVvZg7adoMOdRnBu06nFofL4Q
 1CzXqBTB//uCB/5PTkHxvAOg7P9bZQePAT3/bEZTcLM55HQTYYvwVASaVtFKxCzKjZK7TyYC2N
 qU1c6GCFkf2jqwOOrvvJ9tLMSanB3r6lLqyBkT3wu6p41V07N17k3LzGFghJsmlu7QslBjA+K/
 oBWNTssnX0pYrlHZOIfKLCaTwAwdVoWUbK49202YQw4UmgAbm5RJfmvZS9NVaIZE8LoUZpvLZF
 rak=
Subject: Re: [PATCH v6 24/48] Input: atmel_mxt_ts - make bootloader interrupt
 driven
To:     Bartosz Szczepanek <bsz@semihalf.com>, <nick.dyer@itdev.co.uk>
CC:     <Andrew_Gabbasov@mentor.com>, <benjamin.tissoires@redhat.com>,
        <dmitry.torokhov@gmail.com>, <erosca@de.adit-jv.com>,
        <jikos@kernel.org>, <linux-input@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <nick@shmanahar.org>,
        <rydberg@bitmath.org>, <levinale@google.com>, <rad@semihalf.com>
References: <20191122082402.18173-25-jiada_wang@mentor.com>
 <20200127174127.25522-1-bsz@semihalf.com>
From:   "Wang, Jiada" <jiada_wang@mentor.com>
Message-ID: <884a9ce0-b6db-6716-e366-e4c8ed7b4910@mentor.com>
Date:   Wed, 12 Feb 2020 17:39:07 +0900
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101
 Thunderbird/68.4.2
MIME-Version: 1.0
In-Reply-To: <20200127174127.25522-1-bsz@semihalf.com>
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hello Bartosz

On 2020/01/28 2:41, Bartosz Szczepanek wrote:
> Hi,
> 
> I've been testing this patchset on Chromebook equipped with Atmel touchscreen &
> touchpad. In my setup, this particular patch seems to introduce a regression
> on firmware update:
> 
>> localhost /sys/devices/pci0000:00/0000:00:15.0/i2c_designware.0/i2c-6/i2c-ATML0001:00 # echo maxtouch-ts.fw > update_fw
>> bash: echo: write error: Remote I/O error
> 
> Kernel logs show that the reason is failed I2C transfer:
> 
>> [ 111.632131] atmel_mxt_ts i2c-ATML0001:00: Found bootloader addr:27 ID:21 version:4
>> [ 111.637711] atmel_mxt_ts i2c-ATML0001:00: Unlocking bootloader
>> [ 129.155091] atmel_mxt_ts i2c-ATML0001:00: Sent 1356 frames, 212224 bytes
>> [ 129.263269] atmel_mxt_ts i2c-ATML0001:00: The firmware update succeeded
>> [ 129.263952] atmel_mxt_ts i2c-ATML0001:00: __mxt_read_chunk: i2c transfer failed (-121)
>> [ 129.265072] atmel_mxt_ts i2c-ATML0001:00: mxt_bootloader_read: i2c recv failed (-121)
>> [ 129.265588] atmel_mxt_ts i2c-ATML0001:00: Trying alternate bootloader address
>> [ 129.266375] atmel_mxt_ts i2c-ATML0001:00: mxt_bootloader_read: i2c recv failed (-121)
> 
> Surprisingly, only touchscreen device is affected. When I checked out to
> 119e1b7e8481 ("Input: atmel_mxt_ts - refactor code to enter bootloader into
> separate func") all worked fine. In between these commits I got some mixed
> results, including timeout while waiting for completion:
> 
>> [ 190.006174] atmel_mxt_ts i2c-ATML0001:00: Found bootloader addr:27 ID:21 version:4
>> [ 190.317819] atmel_mxt_ts i2c-ATML0001:00: Wait for completion timed out.
>> [ 190.318267] atmel_mxt_ts i2c-ATML0001:00: Update wait error -110
>> [ 190.319310] atmel_mxt_ts i2c-ATML0001:00: Unlocking bootloader
>> [ 208.369825] atmel_mxt_ts i2c-ATML0001:00: Sent 1356 frames, 212224 bytes
>> [ 208.536942] atmel_mxt_ts i2c-ATML0001:00: The firmware update succeeded
>> [ 208.544835] atmel_mxt_ts i2c-ATML0001:00: Family: 164 Variant: 14 Firmware V2.3.AA Objects: 40
>> [ 208.547623] atmel_mxt_ts i2c-ATML0001:00: Touchscreen size X4095Y2729
> 
> Some more details - the touchscreen device reports itself as:
> 
>> atmel_mxt_ts i2c-ATML0001:00: Family: 164 Variant: 14 Firmware V2.3.AA Objects: 40
> 
> Due to Chromebook limitations on kernel version, I'm running 4.19 kernel
> with patches backported from master (so that atmel_mxt_ts is aligned between
> master and 4.19). The platform is Samsung Chromebook Pro.
> 
I have found the root cause for the regression,
will submit v7 patch-set shortly,
if you could validate for the update,
it will be very helpful

thanks,
Jiada
> Best regards,
> Bartosz
>