From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 261ACC3A59F for ; Fri, 30 Aug 2019 00:02:01 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E64DB22CEA for ; Fri, 30 Aug 2019 00:02:00 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=typeblog-net.20150623.gappssmtp.com header.i=@typeblog-net.20150623.gappssmtp.com header.b="X0Az8igV" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727171AbfH3ACA (ORCPT ); Thu, 29 Aug 2019 20:02:00 -0400 Received: from mail-pf1-f194.google.com ([209.85.210.194]:45864 "EHLO mail-pf1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726416AbfH3ACA (ORCPT ); Thu, 29 Aug 2019 20:02:00 -0400 Received: by mail-pf1-f194.google.com with SMTP id w26so3228581pfq.12 for ; Thu, 29 Aug 2019 17:02:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=typeblog-net.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=IldQxyej46YU/SkHEEsnjl/LzmCcMggIbONwXFCSCKE=; b=X0Az8igVrc/4gMnl5ADWTxShGaQFNupE1xbuCezN4YHWmi1aCOQeQyNrfkUh91vEC/ Jnjy2Zvmwn316YjWnNmofhEJ1/MO3Q+vKCkvxCw4RdMjeiegd9ABp5z9r3LQzHhoF/Sr JSHn7bApaXUDGs0DvnqQgDtSNIdofrDOPfJe1tm2UV2HV6lp4kMRyBP4i7nCJag/dJvp r25V44aCqJvk25M6ytooHD9n+xecEEW0Jpn8seM5m4nvbLouIRt3wmhUH4TZ7QbiueqE nYJykgKYmcPoVqu7s78+gPF/ldglPJAbnZ/daCR/4Db90NrT3UrhNC6HvqaEF5zCzZTb zT4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=IldQxyej46YU/SkHEEsnjl/LzmCcMggIbONwXFCSCKE=; b=k93fiP+th2ACcKZgyoR3CaBEAsqCHb0uCnNhuielEC80stKxE126XEMo1bjv22Lxi/ wisVc70NrtopSzZAKgmA7NWQvgqaNnJe2Rx53PcSG/gm0wLQB71PmRG+Xjx2BFL0yvRr 3uVndRZWXUWeWVGjzFTwlQefPExI4mudomYZCbNQoSNkOVRZIlLTim86KwtskjB3hQx/ cnd7qZwMeBQcFJgl+CpGBOXzl55lx4hzoRc34yrkUrMlZXhJn4oZl0Py/IpjR5mdAk0w JP7Nzn6kdkbKDJwpRzlOjTM/dV7FJjM9FBklgnYoeSe0q2uVvH/skFzoDvPpUaEocvzD 5S0w== X-Gm-Message-State: APjAAAX4zN3CeaUcHKZoefHNgCELXC0swZSReboe1U1p1Tkk2q9Eg9KM UY5s9LItGGbGsphLM8jAW/LPOg== X-Google-Smtp-Source: APXvYqwjKLpuXVkSR1nIUL4L0503S/Jm0MrmR+DtmeNp33276pQCiT6zEVNp+Gbr2ogFQbHDmXmnfg== X-Received: by 2002:a63:4c5c:: with SMTP id m28mr11229487pgl.333.1567123319576; Thu, 29 Aug 2019 17:01:59 -0700 (PDT) Received: from peter-pc.home ([91.207.174.229]) by smtp.gmail.com with ESMTPSA id g2sm4369819pfm.32.2019.08.29.17.01.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Aug 2019 17:01:58 -0700 (PDT) From: Peter Cai Cc: Peter Cai , Mika Westerberg , Andy Shevchenko , Linus Walleij , Bartosz Golaszewski , Bastien Nocera , Dmitry Torokhov , "Rafael J. Wysocki" , Len Brown , linux-gpio@vger.kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org Subject: [PATCH 2/2] touchscreen: goodix: define GPIO mapping for GPD P2 Max Date: Fri, 30 Aug 2019 08:00:24 +0800 Message-Id: <20190830000024.20384-2-peter@typeblog.net> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20190830000024.20384-1-peter@typeblog.net> References: <20190830000024.20384-1-peter@typeblog.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit To: unlisted-recipients:; (no To-header on input) Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org The firmware of GPD P2 Max could not handle panel resets although code is present in DSDT. The kernel needs to take on this job instead, but the DSDT does not provide _DSD, rendering kernel helpless when trying to find the respective GPIO pins. Fortunately, this time GPD has proper DMI vendor / product strings that we could match against. We simply apply an acpi_gpio_mapping table when GPD P2 Max is matched. Additionally, the DSDT definition of the irq pin specifies a wrong polarity. The new quirk introduced in the previous patch (ACPI_GPIO_QUIRK_OVERRIDE_POLARITY) is applied to correct this. Signed-off-by: Peter Cai --- drivers/input/touchscreen/goodix.c | 37 ++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c index 5178ea8b5f30..65b8d04b6dcf 100644 --- a/drivers/input/touchscreen/goodix.c +++ b/drivers/input/touchscreen/goodix.c @@ -144,6 +144,34 @@ static const struct dmi_system_id rotated_screen[] = { {} }; +#ifdef CONFIG_ACPI +static const struct acpi_gpio_params irq_gpios_default = { 0, 0, false }; +static const struct acpi_gpio_params reset_gpios_default = { 1, 0, false }; +static const struct acpi_gpio_mapping gpio_mapping_force_irq_active_high[] = { + { "irq-gpios", &irq_gpios_default, 1, + ACPI_GPIO_QUIRK_OVERRIDE_POLARITY }, + { "reset-gpios", &reset_gpios_default, 1 }, + {} +}; + +/* + * Devices that need acpi_gpio_mapping to function correctly + */ +static const struct dmi_system_id need_gpio_mapping[] = { +#if defined(CONFIG_DMI) && defined(CONFIG_X86) + { + .ident = "GPD P2 Max", + .matches = { + DMI_MATCH(DMI_SYS_VENDOR, "GPD"), + DMI_MATCH(DMI_PRODUCT_NAME, "P2 MAX") + }, + .driver_data = &gpio_mapping_force_irq_active_high + }, +#endif + {} +}; +#endif + /** * goodix_i2c_read - read data from a register of the i2c slave device. * @@ -796,6 +824,15 @@ static int goodix_ts_probe(struct i2c_client *client, struct goodix_ts_data *ts; int error; +#ifdef CONFIG_ACPI + struct dmi_system_id *dmi_match; + + dmi_match = dmi_first_match(need_gpio_mapping); + if (dmi_match) + devm_acpi_dev_add_driver_gpios(&client->dev, + dmi_match->driver_data); +#endif + dev_dbg(&client->dev, "I2C Address: 0x%02x\n", client->addr); if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) { -- 2.23.0