From patchwork Mon Dec 5 13:40:29 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Quentin Schulz X-Patchwork-Id: 13064494 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B492FC4708D for ; Mon, 5 Dec 2022 13:41:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=0EXMCju8hva8iwRXpr9+KecuC2eHaNwdkJa+W+ki3I4=; b=xVP5K/1cwBpG7t jXraN8A4zhBXNjMZcpQMSGAy7MgWT1L50lF4d9WyRm13ipxn11M6GnfbpR3AvPloEAXlnJCWZi1KU ESnEAwWJ5vkyIvC1DDloDlS6jkvh44Z/EU+kK5xyDg2HivEXGHIXSSxVSCioWy3cqMEiqc9h5MM7B 6yaL38Y8U3qB84RNXHmYf0NRdAeDTu72H7XIfDog1XRw/+1Ej4g7/1QHc8s1Pn9ePKjj1zj/nmU/m n2eQgNGee1tSkUzAQFiNTSjAFgdpGxDeSMcs57HiZzfiBzaBGQeKRGMgWEYMPsAgkKneT+Pelv1Ne eby7EcPwPVW+f1RPJ9Ow==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p2Bih-003ZU9-5W; Mon, 05 Dec 2022 13:41:19 +0000 Received: from relay9-d.mail.gandi.net ([217.70.183.199]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p2BiU-003ZNb-Rp; Mon, 05 Dec 2022 13:41:09 +0000 Received: (Authenticated sender: foss@0leil.net) by mail.gandi.net (Postfix) with ESMTPSA id 0ED2BFF807; Mon, 5 Dec 2022 13:40:47 +0000 (UTC) From: Quentin Schulz To: Samuel Holland , Bastien Nocera , =?utf-8?q?Guido_G=C3=BCnther?= , Sascha Hauer , Pengutronix Kernel Team , Angus Ainslie , Ondrej Jirman , Icenowy Zheng , Andy Gross , Aleksei Mamlin , Fabio Estevam , David Jander , Frieder Schrempf , Bjorn Andersson , Konrad Dybcio , Peter Geis , Heiko Stuebner , Shawn Guo , Jernej Skrabec , Lukasz Majewski , AngeloGioacchino Del Regno , Chen-Yu Tsai , Michael Riesch , Rob Herring , NXP Linux Team , Dmitry Torokhov , Hans de Goede , Jagan Teki , Krzysztof Kozlowski Cc: Quentin Schulz , linux-input@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@lists.linux.dev, devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org Subject: [PATCH v3 0/9] fix reset line polarity for Goodix touchscreen controllers Date: Mon, 5 Dec 2022 14:40:29 +0100 Message-Id: <20221103-upstream-goodix-reset-v3-0-0975809eb183@theobroma-systems.com> X-Mailer: git-send-email 2.38.1 MIME-Version: 1.0 X-Mailer: b4 0.10.1 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221205_054107_224115_F67DB7BA X-CRM114-Status: GOOD ( 23.41 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org From: Quentin Schulz The Goodix touchscreen controller has a reset line active low. It happens to also be used to configure its i2c address at runtime. If the reset line is incorrectly asserted, the address will be wrongly configured. This cost me a few hours, trying to figure out why the touchscreen wouldn't work. The driver is "asserting" this reset GPIO by setting its output to 0, probably to reflect the physical state of the line. However, this relies on the fact that the Device Tree node setting the reset line polarity to active high, which is incorrect since the reset is active low in hardware. To fix this inconsistency, the polarity is inverted to not confuse the user about the reset line polarity. This obviously requires to fix the DT since most users had the "incorrect" value in there, it needs to be inverted. Note that the v2 highlighted that I was not the only one that got confused since PRT8MM board has the "correct" HW representation for this line in DT (which does not match what the driver was expecting). This is marked as RFC because I can neither test ACPI support nor boards I don't own. Please test on the boards you have that are impacted by this patchset and give your Tested-By. Do we also make this patch series only one patchset since the DT patches depend on the driver patch and vice-versa? In which tree would this go? Thanks, Quentin To: Bastien Nocera To: Hans de Goede To: Dmitry Torokhov To: Rob Herring To: Krzysztof Kozlowski To: Shawn Guo To: Sascha Hauer To: Pengutronix Kernel Team To: Fabio Estevam To: NXP Linux Team To: Chen-Yu Tsai To: Jernej Skrabec To: Samuel Holland To: Andy Gross To: Bjorn Andersson To: Konrad Dybcio To: Heiko Stuebner To: David Jander To: Angus Ainslie To: Peter Geis To: Michael Riesch To: Konrad Dybcio To: AngeloGioacchino Del Regno To: Guido Günther To: Jagan Teki To: Ondrej Jirman To: Icenowy Zheng To: Aleksei Mamlin To: Lukasz Majewski To: Frieder Schrempf Cc: linux-input@vger.kernel.org Cc: linux-kernel@vger.kernel.org Cc: devicetree@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-sunxi@lists.linux.dev Cc: linux-arm-msm@vger.kernel.org Cc: linux-rockchip@lists.infradead.org Signed-off-by: Quentin Schulz Tested-by: Hans de Goede Reviewed-by: Hans de Goede --- Changes in v3: - Cc'ing people who contributed to DTS of impacted boards, - removed PRT8MM DTS change since it's been reported the polarity is actually correct (goes through an inverter), keeping the appropriate folks in Cc though since it'd be a good idea to check this patch series anyways, - added ACPI_GPIO_QUIRK_NO_IO_RESTRICTION to acpi_gpio_mapping quirks to make gpiolib-acpi core respect GPIOD_ASIS flag in gpiod_get, - checked schematics of: - pinephone: https://files.pine64.org/doc/PinePhone/PinePhone%20v1.2%20Released%20Schematic.pdf - pinetab: https://files.pine64.org/doc/PineTab/PineTab%20Schematic%20v1.2-20191125.pdf - px30 evb: https://opensource.rock-chips.com/images/d/db/Px30_mini_evb_v10_20180528.pdf - rockpro64: https://files.pine64.org/doc/rockpro64/rockpro64_v21-SCH.pdf - librem5 devkit: https://source.puri.sm/Librem5/dvk-mx8m-bsb/blob/master/dvk-mx8m-bsb.pdf All seems to be directly connected to the GPIO on the SoC side, without an inverter on the line. - Link to v2: https://lore.kernel.org/r/20221103-upstream-goodix-reset-v2-0-2c38fb03a300@theobroma-systems.com Changes in v2: - implemented ACPI support as suggested by Hans, - removed Qcom SC7180 Trogdor-based devices changes as they are not using this Goodix driver, - added comment on how to read gpiod_request_output and the GPIO DT polarity, - Link to v1: https://lore.kernel.org/r/20221103-upstream-goodix-reset-v1-0-87b49ae589f1@theobroma-systems.com --- Quentin Schulz (9): Input: goodix - add macro for gpio mapping Input: goodix - make gpiod_get honor GPIOD_ASIS Input: goodix - fix reset polarity ARM: dts: imx: fix touchscreen reset GPIO polarity ARM: dts: sunxi: fix touchscreen reset GPIO polarity on Wexler TAB7200 tablet arm64: dts: allwinner: fix touchscreen reset GPIO polarity arm64: dts: librem5: fix touchscreen reset GPIO polarity arm64: dts: qcom: msm8998-fxtec: fix touchscreen reset GPIO polarity arm64: dts: rockchip: fix touchscreen reset GPIO polarity arch/arm/boot/dts/imx6q-kp.dtsi | 2 +- arch/arm/boot/dts/imx6ul-kontron-bl-43.dts | 2 +- arch/arm/boot/dts/sun7i-a20-wexler-tab7200.dts | 2 +- .../dts/allwinner/sun50i-a64-amarula-relic.dts | 2 +- .../allwinner/sun50i-a64-oceanic-5205-5inmfd.dts | 2 +- .../boot/dts/allwinner/sun50i-a64-pinephone.dtsi | 2 +- .../boot/dts/allwinner/sun50i-a64-pinetab.dts | 2 +- .../boot/dts/freescale/imx8mq-librem5-devkit.dts | 2 +- arch/arm64/boot/dts/qcom/msm8998-fxtec-pro1.dts | 2 +- arch/arm64/boot/dts/rockchip/px30-evb.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi | 2 +- arch/arm64/boot/dts/rockchip/rk3568-evb1-v10.dts | 2 +- drivers/input/touchscreen/goodix.c | 54 ++++++++++++++++++---- 13 files changed, 56 insertions(+), 22 deletions(-) --- base-commit: 76dcd734eca23168cb008912c0f69ff408905235 change-id: 20221103-upstream-goodix-reset-aa1c65994f57 Best regards,