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: 2604 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:f944:0:0:0:0:0 with SMTP id q4csp2261195wrr; Mon, 5 Dec 2022 05:45:11 -0800 (PST) X-Google-Smtp-Source: AA0mqf6Z6e7DnMuxxrahnq18EnaPNI+Lir3eZ5ZhyaLGgGYdpE5v0jcoo/mXEfY1MqSYyUtnalnA X-Received: by 2002:a17:906:95c3:b0:78e:975:5e8 with SMTP id n3-20020a17090695c300b0078e097505e8mr53170444ejy.82.1670247911172; Mon, 05 Dec 2022 05:45:11 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1670247911; cv=none; d=google.com; s=arc-20160816; b=o0ITSvWlHObg9ZSKgD2P3BO3vF/WTi9f2z9z7hTS/pzYUA1kmrW66DQz9b5OYbCHnQ cT9xLUR7hQGcnLQbK+OYk4znCFkSI9hnSOt6IjeTd85tqZZOOlrH9bEcuzdirvuC959D 3XmloBYzmer4m94lXlsQYUsdW2X+FW7Cg5ezejF23DpgJpfZ3EDcXetT0fH8Vaimj69w r0I9Djspv1DrTl8gK85XdI34lyp50CZz4vUfG3RjSv2GH0TIgCFw/fwdpWpVB8abcBDK SKVLzdeAQ7U0QUWY0Qky0AivmR+T74QQ/1LYDePQbdq4Uhuh9NitCzO5+EK+00Bd6ABp ea8Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=5VUdxmLUxpnjvFgDUlVLOLYl65qW54aCHBne/e4MVl8=; b=0WTlrJ6HqvW6EoirWr4tsQgm7rTXa6loAeAkW4OBkZZoDyyJs3Yk+NH+GyfIEAYbg5 bQYb2v1c1z5Ug7Vi+HN1qUv44jg/wYMaGV6xdtt+BQ8Xuv9e/u9jTeFHjAbzRf2e6Wn1 slo0QJqtzv3mQBsmNJcCZDMzQHVSdSqjtttbXW0v2dSq/FLtOM1DSO6FCvlz1x7DcvsK F8Y6RmzreLVugB9gyaUI78HK4rK8eR/WS/f3NlFOmiIs3DDzfheAOPVL1bHCh3akR3Ep a+yUDFN2lWlaAwiXg06uFtny8ke35gU6QaoEV0bDPgeDGgTdrLnwfsZXujiNsoaeYA2Q u4ZA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id e22-20020a17090658d600b007c0b0f9b97esi8466514ejs.775.2022.12.05.05.44.45; Mon, 05 Dec 2022 05:45:11 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232208AbiLENlI (ORCPT + 99 others); Mon, 5 Dec 2022 08:41:08 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34326 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229954AbiLENlG (ORCPT ); Mon, 5 Dec 2022 08:41:06 -0500 Received: from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net [217.70.183.199]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 53912DF5C; Mon, 5 Dec 2022 05:41:02 -0800 (PST) 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-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1751381873919086243?= X-GMAIL-MSGID: =?utf-8?q?1751381873919086243?= 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,