From patchwork Tue May 8 11:49:46 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Heiko Stuebner X-Patchwork-Id: 10386031 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 95954602D8 for ; Tue, 8 May 2018 11:57:22 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 81F9828D81 for ; Tue, 8 May 2018 11:57:22 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7470D28D83; Tue, 8 May 2018 11:57:22 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 0243428D81 for ; Tue, 8 May 2018 11:57:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: Content-Transfer-Encoding:Content-Type:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=uGIACJfHLvEyxFCPpEbSSms0nKnAhKUJxP3Ry6Kd9I4=; b=SEEJ4W9TBScFRH ckCVV9zvKRU7IYwwML2fl1kksmmGydoYXGj3fnX80v78aGrMlLbOjaB9wnniTyIru9tGDeTNiPDum jOa4E2Jvv9s7zWvjp/1MnFGVvHtHRxsA++Er1ea+OuT6P/MUpytOGzBotahtixhmN4c1yt08Ccz9B Fyx3oFGM3XL/BTedkzqP6HIHJXvwkXOIUEYGVa8d9VUfPh5XwC0bjUS5ZZ7ogq7aldlRfl3v+RlsB jhrNqcJEid0pGU354NdkwfVs1cReGH25u/FAxgOtVAddFMyAUslXXzn0WfRsWKniXJSGHN45k25jq vh/NZqOD7ls1+JsAAxeA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fG1F6-0006yd-5G; Tue, 08 May 2018 11:57:16 +0000 Received: from gloria.sntech.de ([95.129.55.99]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1fG18F-0000te-QO for linux-rockchip@lists.infradead.org; Tue, 08 May 2018 11:50:18 +0000 Received: from [46.183.103.8] (helo=phil.localnet) by gloria.sntech.de with esmtpsa (TLS1.1:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80) (envelope-from ) id 1fG17w-0002hJ-Cv; Tue, 08 May 2018 13:49:52 +0200 From: Heiko Stuebner To: linux-rockchip@lists.infradead.org Subject: Re: [PATCH v0 1/2] gpio: syscon: Add gpio-syscon for rk3328 Date: Tue, 08 May 2018 13:49:46 +0200 Message-ID: <2356128.gufDZceMBS@phil> In-Reply-To: <1525747704-8537-2-git-send-email-djw@t-chip.com.cn> References: <1525747704-8537-1-git-send-email-djw@t-chip.com.cn> <1525747704-8537-2-git-send-email-djw@t-chip.com.cn> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20180508_045012_063581_2EA5DE27 X-CRM114-Status: GOOD ( 23.03 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-gpio@vger.kernel.org, Wayne Chou , djw@t-chip.com.cn, Linus Walleij , linux-kernel@vger.kernel.org Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+patchwork-linux-rockchip=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Hi Levin, Am Dienstag, 8. Mai 2018, 04:48:23 CEST schrieb djw@t-chip.com.cn: > From: Levin Du > > In Rockchip RK3328 Soc, there's a output only gpio pin labeled > `gpiomut_pmuio_iout`, which can be set by bit[1] of GRF_SOC_CON10. > (bit[0] controls the enable state of the pin and defaults to enabled.) > > This pin is used by the roc-rk3328-cc board to switch sdmmc io signal > voltage between 1.8V and 3.3V, which is essential to the SD card UHS > support. > > Signed-off-by: Levin Du Thanks for investigating that special pin. Please also add a devicetree-binding document under Documentation/devicetree/bindings/gpio. And I do have some more suggestions below. > --- > > drivers/gpio/gpio-syscon.c | 32 ++++++++++++++++++++++++++++++++ > 1 file changed, 32 insertions(+) > > diff --git a/drivers/gpio/gpio-syscon.c b/drivers/gpio/gpio-syscon.c > index 537cec7..b69f65f 100644 > --- a/drivers/gpio/gpio-syscon.c > +++ b/drivers/gpio/gpio-syscon.c > @@ -135,6 +135,34 @@ static const struct syscon_gpio_data clps711x_mctrl_gpio = { > .dat_bit_offset = 0x40 * 8 + 8, > }; > > +static void rockchip_gpio_set(struct gpio_chip *chip, unsigned int offset, > + int val) > +{ > + struct syscon_gpio_priv *priv = gpiochip_get_data(chip); > + unsigned int offs; > + u8 bit; > + u32 data; > + int ret; > + > + offs = priv->dreg_offset + priv->data->dat_bit_offset + offset; > + bit = offs % SYSCON_REG_BITS; > + data = (val ? BIT(bit) : 0) | BIT(bit + 16); > + ret = regmap_write(priv->syscon, > + (offs / SYSCON_REG_BITS) * SYSCON_REG_SIZE, > + data); > + if (ret < 0) > + dev_err(chip->parent, "gpio write failed ret(%d)\n", ret); > +} > + > +static const struct syscon_gpio_data rk3328_gpio_syscon10 = { > + /* Rockchip RK3328 GRF_SOC_CON10 Bits 0-1 */ > + .compatible = "rockchip,rk3328-grf", please drop the compatible above, include the attached patch before this one and follow the things I'll will outline in the devicetree patch shortly :-) Patch for getting the syscon frome the parent is compile-tested only so please double-check that I didn't mess up anything. > + .flags = GPIO_SYSCON_FEAT_OUT, > + .bit_count = 2, > + .dat_bit_offset = 0x0428 * 8, > + .set = rockchip_gpio_set, > +}; > + > #define KEYSTONE_LOCK_BIT BIT(0) > > static void keystone_gpio_set(struct gpio_chip *chip, unsigned offset, int val) > @@ -175,6 +203,10 @@ static const struct of_device_id syscon_gpio_ids[] = { > .compatible = "ti,keystone-dsp-gpio", > .data = &keystone_dsp_gpio, > }, > + { > + .compatible = "rockchip,rk3328-gpio-syscon10", rockchip,rk3328-gpio-mute [the naming from the TRM] could be a more suitable naming? Heiko From 8894fdd9fc4ad90abec32336cc2e528d49abf887 Mon Sep 17 00:00:00 2001 From: Heiko Stuebner Date: Tue, 8 May 2018 13:33:37 +0200 Subject: [PATCH] gpio: syscon: allow fetching syscon from parent node Syscon nodes can be a simple-mfd and the syscon-users then be declared as children of this node. That way the parent-child structure can be better represented for devices that are fully embedded in the syscon. Therefore allow getting the syscon from the parent if neither a special compatible nor a gpio,syscon-dev property is defined. Signed-off-by: Heiko Stuebner --- drivers/gpio/gpio-syscon.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpio/gpio-syscon.c b/drivers/gpio/gpio-syscon.c index 537cec7583fc..7325b864f52a 100644 --- a/drivers/gpio/gpio-syscon.c +++ b/drivers/gpio/gpio-syscon.c @@ -205,6 +205,8 @@ static int syscon_gpio_probe(struct platform_device *pdev) } else { priv->syscon = syscon_regmap_lookup_by_phandle(np, "gpio,syscon-dev"); + if (IS_ERR(priv->syscon) && np->parent) + priv->syscon = syscon_node_to_regmap(np->parent); if (IS_ERR(priv->syscon)) return PTR_ERR(priv->syscon); -- 2.16.2