Re: elan_i2c: failed to read report data: -71

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

 



On Thu, Mar 04, 2021 at 11:49:59AM +0300, Nikolai Kostrigin wrote:
> Hi,
> 
> 04.03.2021 09:59, Uwe Kleine-König пишет:
> > Hello,
> > 
> > On Wed, Mar 03, 2021 at 05:53:37PM -0800, 'Dmitry Torokhov' wrote:
> >> On Wed, Mar 03, 2021 at 09:03:30PM +0100, Uwe Kleine-König wrote:
> >>> On Wed, Mar 03, 2021 at 07:32:23PM +0100, Uwe Kleine-König wrote:
> >>>> On Wed, Mar 03, 2021 at 11:13:21AM +0800, jingle wrote:
> >>>>> Please updates this patchs.
> >>>>>
> >>>>> https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/commit/?h=next&id=056115daede8d01f71732bc7d778fb85acee8eb6
> >>>>>
> >>>>> https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/commit/?h=next&id=e4c9062717feda88900b566463228d1c4910af6d
> >>>>
> >>>> The first was one of the two patches I already tried, but the latter
> >>>> indeed fixes my problem \o/.
> >>>>
> >>>> @Dmitry: If you don't consider your tree stable, feel free to add a
> >>>>
> >>>> 	Tested-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> >>>>
> >>>> to e4c9062717feda88900b566463228d1c4910af6d.
> >>>
> >>> Do you consider this patch for stable? I'd like to see it in Debian's
> >>> 5.10 kernel and I guess I'm not the only one who would benefit from such
> >>> a backport.
> >>
> >> When I was applying the patches I did not realize that there was already
> >> hardware in the wild that needed it. The patches are now in mainline, so
> >> I can no longer adjust the tags, but I will not object if you propose
> >> them for stable.
> > 
> > I want to propose to backport commit
> > 
> > e4c9062717fe ("Input: elantech - fix protocol errors for some trackpoints in SMBus mode")
> > 
> > to the active stable kernels. This commit repairs the track point and
> > the touch pad buttons on a Lenovo Thinkpad E15 here. Without this change
> > I don't get any events apart from an error message for each button press
> > or move of the track point in the kernel log. (Also the error message is
> > the same for all buttons and the track point, so I cannot create a new
> > input event driver in userspace that emulates the right event depending
> > on the error message :-)
> > 
> > At least to 5.10.x it applies cleanly, I didn't try the older stable
> > branches.
> > 
> > Best regards and thanks
> > Uwe
> > 
> 
> I'd like to propose to backport [1] also as it was checked along with
> previously proposed patch and fixes Elan Trackpoint operation on
> Thinkpad L13.
> 
> Both patches apply cleanly to 5.10.17 in my case.
> 
> I also tried to apply to 5.4.x, but failed.
> 
> [1] 056115daede8 Input: elan_i2c - add new trackpoint report type 0x5F
> 
> Additional info is available here:
> 
> https://lore.kernel.org/linux-input/fe31f6f8-6e38-2ed6-8548-6fa271bf36e9@xxxxxxxxxx/T/#m514047f2c5e7e2ec4ed9658782f14221ed7598fc

FTR: I tested 5.10 + e4c9062717fe ("Input: elantech - fix protocol
errors for some trackpoints in SMBus mode") now and in this setup the
touchpad is still broken. I assume that in combination with 056115daede8
it will work. The working setup I tested was 5.10 + c7f0169e3bd2 +
056115daede8 + e4c9062717fe and I assume c7f0169e3bd2 isn't relevant.

Best regards
Uwe


-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux