Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp637699pxu; Fri, 11 Dec 2020 10:31:07 -0800 (PST) X-Google-Smtp-Source: ABdhPJzXpGYVgOOv4C8I3ZrDuHGnq2wL9/9ErE9EPzVyhwwhBuC5zkj4UM696rFlN6M8WxOtJ3w+ X-Received: by 2002:aa7:db01:: with SMTP id t1mr13151686eds.185.1607711467483; Fri, 11 Dec 2020 10:31:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607711467; cv=none; d=google.com; s=arc-20160816; b=M8wa7HBE5sIDvxza33rEt3wke4j+2AeBrldGsbjgy2vCYcfEwQsv2CZCMaixYwylMs x0SLyOpBK53cg3n/W+s/uo/pHJA+UVj0x1etcMIj3WHMMsYZ1GJUZgPMK8M4/yJGCOXZ YpePYPKew06K2Yu3BX54FhQ+a3ExnanPTeLzncbI5iDbWuMb8dKyqV2eiHiqF5hsfQYZ wN1ABi6+uKpmRCpxO+2JewyeJxLrtNHMAiaE9E6QVbxAGDAebKemNEcBA0m/vHX3uLxe DWeQtYYNvDjfwJGMNkybRj5kSoui43WmhDBsYWCPszgTQ5P066FRyFzS0xjZMXv/77f/ dxnQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=I/KFNKGST+1Ht5tnND0rl1MivSO+GhBdUVdNNPWR9v4=; b=HE72j6v7kwKeG7rCoCRrC0C/yfYv1QAGRcSvK/ouvNj3/3Rxd5iI24HT6ONHsXmYz4 iA7pB1Skk13y08+9QXHUvaKzVefxp6Yt1UqGco4BtdDAmd1OHyC7ITGRDbiGFBbL/4i6 yAvkqvaolrEzFfYjiSssN5bHzcoBCK/Gdd9byeijRX1OIKEzUXydmAP2OzOE6Id/uBLm YBY7AxsXqCEvfNAvST4pOaPdvTaz5qS4TfpIU4HgcZYX2XuWFZJg4+IyWxHEZXs+b+pZ eyVlODahGSYU8OYEXU4JqwM074ZSVBOg87uR7lLSPs8RFSX0paYPcZOzCq3C1VWuhlaa PdHA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@rere.qmqm.pl header.s=1 header.b=FB+54ThX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id go36si4884496ejc.560.2020.12.11.10.30.42; Fri, 11 Dec 2020 10:31:07 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@rere.qmqm.pl header.s=1 header.b=FB+54ThX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2394841AbgLKQKV (ORCPT + 99 others); Fri, 11 Dec 2020 11:10:21 -0500 Received: from rere.qmqm.pl ([91.227.64.183]:25199 "EHLO rere.qmqm.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404259AbgLKQJ4 (ORCPT ); Fri, 11 Dec 2020 11:09:56 -0500 Received: from remote.user (localhost [127.0.0.1]) by rere.qmqm.pl (Postfix) with ESMTPSA id 4CswhV34YMz4y; Fri, 11 Dec 2020 17:09:14 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=rere.qmqm.pl; s=1; t=1607702954; bh=JA0EdqTBGYeoKbAC0O4t5RZsbxp09DvSH5E83admmgo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FB+54ThX+KIVulK3J+dtBnWEhUPjaXdPSuk1c/MkLuQDlQr2D7Sd0LGWJU2JnVlUA 1Rsxkbrd9ekWRbnEonutMh0b6M2YIOr6vdXuCf4/NT9SGyB/aFay3PCnBnrh5/j8BB vi4a0l7G2k1/97LeteE3vfB467CvVvg6q+C54mFUg3UFp6w3XAQUNIxeQCJDcm0EW4 gVCMNC85wwuH/rfFDmSxrTtd4NNv1azHyCqWAaBTvMRvTAKMxkBbsbq8KEhQK0wMz1 JhtE/7xjG23Lw4OsVTLDXUzZ3mDLvg1TLqIrwA1zXTI5hY28gK3v15YfTeseLlM5HY pviz4QCIjaN5g== X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.102.4 at mail Date: Fri, 11 Dec 2020 17:09:17 +0100 From: =?iso-8859-2?Q?Micha=B3_Miros=B3aw?= To: Dmitry Torokhov Cc: Dmitry Osipenko , Johnny Chuang , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RESEND v8 2/4] input: elants: support old touch report format Message-ID: <20201211160917.GA23095@qmqm.qmqm.pl> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 10, 2020 at 11:29:40PM -0800, Dmitry Torokhov wrote: > Hi Micha?, > On Fri, Dec 11, 2020 at 07:53:56AM +0100, Micha? Miros?aw wrote: > > @@ -998,17 +1011,18 @@ static irqreturn_t elants_i2c_irq(int irq, void *_dev) > > } > > > > report_len = ts->buf[FW_HDR_LENGTH] / report_count; > > - if (report_len != PACKET_SIZE) { > > + if (report_len != PACKET_SIZE && > > + report_len != PACKET_SIZE_OLD) { > > dev_err(&client->dev, > > - "mismatching report length: %*ph\n", > > + "unsupported report length: %*ph\n", > > HEADER_SIZE, ts->buf); > Do I understand this correctly that the old packets are only observed on > EKTF3624? If so can we expand the check so that we only accept packets > with "old" size when we know we are dealing with this device? We only have EKTF3624 and can't be sure there are no other chips needing this. Best Regards Micha? Miros?aw