From nobody Thu Apr 2 22:57:35 2026 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 89AC5C54EE9 for ; Tue, 20 Sep 2022 10:31:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230520AbiITKb4 (ORCPT ); Tue, 20 Sep 2022 06:31:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44762 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230473AbiITKbV (ORCPT ); Tue, 20 Sep 2022 06:31:21 -0400 Received: from mail-m11875.qiye.163.com (mail-m11875.qiye.163.com [115.236.118.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5C5CC6F264 for ; Tue, 20 Sep 2022 03:31:19 -0700 (PDT) Received: from localhost.localdomain (unknown [58.22.7.114]) by mail-m11875.qiye.163.com (Hmail) with ESMTPA id 3A561280BAF; Tue, 20 Sep 2022 18:31:17 +0800 (CST) From: Jianqun Xu To: linus.walleij@linaro.org, heiko@sntech.de, brgl@bgdev.pl, andriy.shevchenko@linux.intel.com Cc: robert.moore@intel.com, robh@kernel.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, lenb@kernel.org, rafael@kernel.org, Jianqun Xu Subject: [PATCH 10/20] gpio/rockchip: revert deferred output settings on probe Date: Tue, 20 Sep 2022 18:30:58 +0800 Message-Id: <20220920103108.23074-11-jay.xu@rock-chips.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220920103108.23074-1-jay.xu@rock-chips.com> References: <20220920103108.23074-1-jay.xu@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFJSktLSjdXWS1ZQUlXWQ8JGhUIEh9ZQVkZT0pIVkIaTxpIH0odSktDGFUTARMWGhIXJB QOD1lXWRgSC1lBWU5DVUlJVUxVSkpPWVdZFhoPEhUdFFlBWU9LSFVKSktISkxVSktLVUtZBg++ X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6PRQ6Lzo5Tj0sLhw2FwkoLxka CA4aCj1VSlVKTU1ITU1CQ0xMQkNCVTMWGhIXVREaAlUDDjsJFBgQVhgTEgsIVRgUFkVZV1kSC1lB WU5DVUlJVUxVSkpPWVdZCAFZQUNPTEg3Bg++ X-HM-Tid: 0a835a73baf02eb1kusn3a561280baf Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" This patch revert 3 patches: Revert "gpio/rockchip: handle deferring input-enable pinconfs" This reverts commit 7ff11357810fd124825fdd9aaf0df90262b77844. Revert "pinctrl/rockchip: support deferring other gpio params" This reverts commit 8ce5ef64546850294b021497046588a7abcebe96. Revert "gpio/rockchip: fetch deferred output settings on probe" This reverts commit 59dd178e1d7cb6cac03b32aba7ed9bbce6761b6f. A patch for pinctrl-rockchip must be applied to make sure the gpiochips have been registered before the pinctrl to register. --- drivers/gpio/gpio-rockchip.c | 35 ------------------- drivers/pinctrl/pinctrl-rockchip.c | 54 ++++++++++++++---------------- drivers/pinctrl/pinctrl-rockchip.h | 7 ++-- 3 files changed, 27 insertions(+), 69 deletions(-) diff --git a/drivers/gpio/gpio-rockchip.c b/drivers/gpio/gpio-rockchip.c index dafcc8be1687..b294ef009daf 100644 --- a/drivers/gpio/gpio-rockchip.c +++ b/drivers/gpio/gpio-rockchip.c @@ -19,7 +19,6 @@ #include #include #include -#include #include =20 #include "../pinctrl/core.h" @@ -708,7 +707,6 @@ static int rockchip_gpio_probe(struct platform_device *= pdev) struct device_node *pctlnp =3D of_get_parent(np); struct pinctrl_dev *pctldev =3D NULL; struct rockchip_pin_bank *bank =3D NULL; - struct rockchip_pin_deferred *cfg; static int gpio; int id, ret; =20 @@ -736,45 +734,12 @@ static int rockchip_gpio_probe(struct platform_device= *pdev) if (ret) return ret; =20 - /* - * Prevent clashes with a deferred output setting - * being added right at this moment. - */ - mutex_lock(&bank->deferred_lock); - ret =3D rockchip_gpiolib_register(bank); if (ret) { clk_disable_unprepare(bank->clk); - mutex_unlock(&bank->deferred_lock); return ret; } =20 - while (!list_empty(&bank->deferred_pins)) { - cfg =3D list_first_entry(&bank->deferred_pins, - struct rockchip_pin_deferred, head); - list_del(&cfg->head); - - switch (cfg->param) { - case PIN_CONFIG_OUTPUT: - ret =3D rockchip_gpio_direction_output(&bank->gpio_chip, cfg->pin, cfg-= >arg); - if (ret) - dev_warn(dev, "setting output pin %u to %u failed\n", cfg->pin, - cfg->arg); - break; - case PIN_CONFIG_INPUT_ENABLE: - ret =3D rockchip_gpio_direction_input(&bank->gpio_chip, cfg->pin); - if (ret) - dev_warn(dev, "setting input pin %u failed\n", cfg->pin); - break; - default: - dev_warn(dev, "unknown deferred config param %d\n", cfg->param); - break; - } - kfree(cfg); - } - - mutex_unlock(&bank->deferred_lock); - platform_set_drvdata(pdev, bank); dev_info(dev, "probed %pOF\n", np); =20 diff --git a/drivers/pinctrl/pinctrl-rockchip.c b/drivers/pinctrl/pinctrl-r= ockchip.c index 801a1f6ca4db..97af52bb5481 100644 --- a/drivers/pinctrl/pinctrl-rockchip.c +++ b/drivers/pinctrl/pinctrl-rockchip.c @@ -2702,20 +2702,19 @@ static bool rockchip_pinconf_pull_valid(struct rock= chip_pin_ctrl *ctrl, return false; } =20 -static int rockchip_pinconf_defer_pin(struct rockchip_pin_bank *bank, - unsigned int pin, u32 param, u32 arg) +static int rockchip_pinconf_defer_output(struct rockchip_pin_bank *bank, + unsigned int pin, u32 arg) { - struct rockchip_pin_deferred *cfg; + struct rockchip_pin_output_deferred *cfg; =20 cfg =3D kzalloc(sizeof(*cfg), GFP_KERNEL); if (!cfg) return -ENOMEM; =20 cfg->pin =3D pin; - cfg->param =3D param; cfg->arg =3D arg; =20 - list_add_tail(&cfg->head, &bank->deferred_pins); + list_add_tail(&cfg->head, &bank->deferred_output); =20 return 0; } @@ -2736,25 +2735,6 @@ static int rockchip_pinconf_set(struct pinctrl_dev *= pctldev, unsigned int pin, param =3D pinconf_to_config_param(configs[i]); arg =3D pinconf_to_config_argument(configs[i]); =20 - if (param =3D=3D PIN_CONFIG_OUTPUT || param =3D=3D PIN_CONFIG_INPUT_ENAB= LE) { - /* - * Check for gpio driver not being probed yet. - * The lock makes sure that either gpio-probe has completed - * or the gpio driver hasn't probed yet. - */ - mutex_lock(&bank->deferred_lock); - if (!gpio || !gpio->direction_output) { - rc =3D rockchip_pinconf_defer_pin(bank, pin - bank->pin_base, param, - arg); - mutex_unlock(&bank->deferred_lock); - if (rc) - return rc; - - break; - } - mutex_unlock(&bank->deferred_lock); - } - switch (param) { case PIN_CONFIG_BIAS_DISABLE: rc =3D rockchip_set_pull(bank, pin - bank->pin_base, @@ -2783,6 +2763,22 @@ static int rockchip_pinconf_set(struct pinctrl_dev *= pctldev, unsigned int pin, if (rc !=3D RK_FUNC_GPIO) return -EINVAL; =20 + /* + * Check for gpio driver not being probed yet. + * The lock makes sure that either gpio-probe has completed + * or the gpio driver hasn't probed yet. + */ + mutex_lock(&bank->deferred_lock); + if (!gpio || !gpio->direction_output) { + rc =3D rockchip_pinconf_defer_output(bank, pin - bank->pin_base, arg); + mutex_unlock(&bank->deferred_lock); + if (rc) + return rc; + + break; + } + mutex_unlock(&bank->deferred_lock); + rc =3D gpio->direction_output(gpio, pin - bank->pin_base, arg); if (rc) @@ -3109,7 +3105,7 @@ static int rockchip_pinctrl_register(struct platform_= device *pdev, pdesc++; } =20 - INIT_LIST_HEAD(&pin_bank->deferred_pins); + INIT_LIST_HEAD(&pin_bank->deferred_output); mutex_init(&pin_bank->deferred_lock); } =20 @@ -3371,7 +3367,7 @@ static int rockchip_pinctrl_remove(struct platform_de= vice *pdev) { struct rockchip_pinctrl *info =3D platform_get_drvdata(pdev); struct rockchip_pin_bank *bank; - struct rockchip_pin_deferred *cfg; + struct rockchip_pin_output_deferred *cfg; int i; =20 of_platform_depopulate(&pdev->dev); @@ -3380,9 +3376,9 @@ static int rockchip_pinctrl_remove(struct platform_de= vice *pdev) bank =3D &info->ctrl->pin_banks[i]; =20 mutex_lock(&bank->deferred_lock); - while (!list_empty(&bank->deferred_pins)) { - cfg =3D list_first_entry(&bank->deferred_pins, - struct rockchip_pin_deferred, head); + while (!list_empty(&bank->deferred_output)) { + cfg =3D list_first_entry(&bank->deferred_output, + struct rockchip_pin_output_deferred, head); list_del(&cfg->head); kfree(cfg); } diff --git a/drivers/pinctrl/pinctrl-rockchip.h b/drivers/pinctrl/pinctrl-r= ockchip.h index 4759f336941e..3531633c0397 100644 --- a/drivers/pinctrl/pinctrl-rockchip.h +++ b/drivers/pinctrl/pinctrl-rockchip.h @@ -338,7 +338,7 @@ struct rockchip_pin_bank { u32 toggle_edge_mode; u32 recalced_mask; u32 route_mask; - struct list_head deferred_pins; + struct list_head deferred_output; struct mutex deferred_lock; }; =20 @@ -414,12 +414,9 @@ struct rockchip_pin_config { unsigned int nconfigs; }; =20 -enum pin_config_param; - -struct rockchip_pin_deferred { +struct rockchip_pin_output_deferred { struct list_head head; unsigned int pin; - enum pin_config_param param; u32 arg; }; =20 --=20 2.25.1