From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from layka.disroot.org (layka.disroot.org [178.21.23.139]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9DCC2342509; Thu, 30 Jul 2026 20:19:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=178.21.23.139 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785442755; cv=none; b=QjAX/vPSQ2ORyTEadqb27dS/8tolp2GLfpFqbVU9WfG/Ih/hQsbWVErQsCYIIpIBtGAWkHkHV9R61/GmqtZLDRGSmvkbfkvSJTb9ixqCmRRr/mI8l6E/HZBo3njL9QRuMPI3lb0qEViFq3skeA0VeBbshruqOUlkT+cgrhBLuGc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785442755; c=relaxed/simple; bh=5OosL7UAzAlw+AfOM4QcdD9qlKoevvkWJfZrS6zvBNQ=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=a5/HWJdIcNNapvoJsBrNx1/Y3MtJWfvU7lGiemTB9zcCOLT61r6FSm11cV4s41E+GmktotFNfvjtq1dB9SIZasHyl0gm3fVUBt7gpobMfEHqndMQcW5KciGdFb36stawcoLUHcVPXhqs5OtJblD0UPdLcPY6opKQb87v45fRo0o= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=disroot.org; spf=pass smtp.mailfrom=disroot.org; dkim=pass (2048-bit key) header.d=disroot.org header.i=@disroot.org header.b=Ewe99sfk; arc=none smtp.client-ip=178.21.23.139 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=disroot.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=disroot.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=disroot.org header.i=@disroot.org header.b="Ewe99sfk" Received: from mail01.layka.lan (localhost [127.0.0.1]) by disroot.org (Postfix) with ESMTP id 2AB1C41B26; Thu, 30 Jul 2026 22:19:13 +0200 (CEST) X-Virus-Scanned: SPAM Filter at disroot.org Received: from layka.disroot.org ([127.0.0.1]) by localhost (disroot.org [127.0.0.1]) (amavis, port 10024) with ESMTP id Q4j9SEw-Bkvx; Thu, 30 Jul 2026 22:19:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=disroot.org; s=mail; t=1785442752; bh=5OosL7UAzAlw+AfOM4QcdD9qlKoevvkWJfZrS6zvBNQ=; h=From:Date:Subject:References:In-Reply-To:To:Cc; b=Ewe99sfkvWSZXTNSu3ln2Uho22qPWxPzZK5pkAfcK4GjyMJm7ipEuSqnlTxwhnzTV EuWrjy1SVA8+z+sSyzlP4yDXvbphbPtU7T17aANDh6s0qHmx7ACGjQuQgQCdS/5cw2 pqxwUDrptkzRJnbW4D5ZAQ8STWhRT2lNqL+p1Tv8Je+ZLn6eQ69YrGesksfw/93D06 cnKC/0CO0SXqo101L1CY9b/YoL1xtQABdc0RHBwFBM8D0K+nABbiWSNGwpeFnElEpw fAH9nFvNLyzta2MwW97RlFPiV4GAkYL+920fXs189RJIM7xpODQJrQm9rUEoDH6dUV bxbtlQ5ZL5xog== From: Kaustabh Chakraborty Date: Fri, 31 Jul 2026 01:48:35 +0530 Subject: [PATCH v3 2/5] Input: zinitix - do not ignore non-moving fingers Precedence: bulk X-Mailing-List: linux-input@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260731-zinitix-modes-v3-2-2c8de712997a@disroot.org> References: <20260731-zinitix-modes-v3-0-2c8de712997a@disroot.org> In-Reply-To: <20260731-zinitix-modes-v3-0-2c8de712997a@disroot.org> To: Dmitry Torokhov , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michael Srba , Linus Walleij , Peter Griffin , Alim Akhtar Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, Kaustabh Chakraborty With the ZT7548 touchscreen present in the Galaxy J6, multitouch does not work reliably. This is due to the fact that the driver reports fingers only when their state is changed, so it's either placed against the screen, moved, or drawn away from the screen. The function which is responsible for this is zinitix_report_finger(). This function is called from the IRQ handler, under the following condition: if (p->sub_status & SUB_BIT_EXIST) zinitix_report_finger(bt541, i, p); This implies and ensures that every valid finger must have the SUB_BIT_EXIST flag. However, at the beginning of the function, it refuses to recognize any finger if it has none of SUB_BIT_UP | SUB_BIT_DOWN | SUB_BIT_MOVE. This excludes fingers in reports which do not move from the position since the previous interrupt. Add SUB_BIT_EXIST to the list of valid bits. Signed-off-by: Kaustabh Chakraborty --- drivers/input/touchscreen/zinitix.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/input/touchscreen/zinitix.c b/drivers/input/touchscreen/zinitix.c index 3421b8ffb19b..a0a89d25e25b 100644 --- a/drivers/input/touchscreen/zinitix.c +++ b/drivers/input/touchscreen/zinitix.c @@ -406,7 +406,7 @@ static void zinitix_report_finger(struct bt541_ts_data *bt541, int slot, u16 x, y; if (unlikely(!(p->sub_status & - (SUB_BIT_UP | SUB_BIT_DOWN | SUB_BIT_MOVE)))) { + (SUB_BIT_EXIST | SUB_BIT_UP | SUB_BIT_DOWN | SUB_BIT_MOVE)))) { dev_dbg(&bt541->client->dev, "unknown finger event %#02x\n", p->sub_status); return; @@ -473,7 +473,7 @@ static irqreturn_t zinitix_ts_irq_handler(int irq, void *bt541_handler) const struct point_coord *p = &touch_event.point_coord[i]; /* Only process contacts that are actually reported */ - if (p->sub_status & SUB_BIT_EXIST) + if (p->sub_status) zinitix_report_finger(bt541, i, p); } -- 2.54.0