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=-13.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_MUTT autolearn=unavailable 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 49B51C31E49 for ; Wed, 19 Jun 2019 08:33:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 18A00208CB for ; Wed, 19 Jun 2019 08:33:16 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="CMzGEE1l" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731319AbfFSIdO (ORCPT ); Wed, 19 Jun 2019 04:33:14 -0400 Received: from mail-wr1-f68.google.com ([209.85.221.68]:46257 "EHLO mail-wr1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731063AbfFSIdN (ORCPT ); Wed, 19 Jun 2019 04:33:13 -0400 Received: by mail-wr1-f68.google.com with SMTP id n4so2308093wrw.13; Wed, 19 Jun 2019 01:33:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=5jKn7IDzs6cCNKoPOLjaDiJN+tHr2DuoxatfoHUDWkQ=; b=CMzGEE1l0Mo2Ch9OjsAPz7PMVNdT1cTevOWXor/nelyFcqWhYcyTT9t/ho1TNlk4UN WZBR3upqEPfFjaXuUgkbiZsZctqAoUsrBfoGub9M4gGLk1cmLPXVFs5vjoMZkYZNTbTB cdHCWow4CEyItoe3r/zuuPXEaDFQ3Ya515pP5u7t55yGeUmlOk0jw+XCQ/IJTtWUjrZZ /LZi4AZSVKhErGR92RwBPLRZR21RzXyZX/8RXxXf2BbRVQSEAg1yd0lqc5yRHfVCf7o+ ON/b+eNQJUgY4cGL7TI26dqJatyJMemmZQWfOD0hvH8laK3BKrO2NjEjK3Jy8SPrjZUO W11Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=5jKn7IDzs6cCNKoPOLjaDiJN+tHr2DuoxatfoHUDWkQ=; b=ZbLK4gjW/PTa0WypNANP9DSXqZbCuWD3Wmn7GCyiFohTAOZi0M3yXC4xlDb0R/Ig6U IL+DW8xBRQe41t4rFJZwvqyi73ZDDP2it7iL6ohg9hBWFA57yRd8+6Nf2FLzOxPzu8ok HzsmPQqYAP5B6dr/3QuOvFFkN2pzvRq82DcGNInp1yuxPuBbOG9JyKVEI7PP25qnVw+S hu3cygCNmJwdTY1lMEtaaA90z+r7/vtT6e5I1LV5svGImTQdrOpNPVjWD8DmW2njX51H OEB8cOUnor+5b3E9XtyCn2/7vnxmneTQv3YzMvAuVDF9cF4tNZluC+MBu93p8Nv6kfjx o3QA== X-Gm-Message-State: APjAAAVRj0uTxc+Stne+VJUAQR8Qtwk6lqCmKuW7EB0veOXG4I6sZ+rl eCSVl3w/f/Wn/ukf1RxCRIsfR4MgjNg= X-Google-Smtp-Source: APXvYqx+Zywg2gJIQoDpj4jnpGmZvOXQHrYpWSW4oHCdl9oL1FbLXiBB6A6MZL1AK5TLoMUdpaB7GA== X-Received: by 2002:adf:b60f:: with SMTP id f15mr19615343wre.283.1560933190598; Wed, 19 Jun 2019 01:33:10 -0700 (PDT) Received: from localhost (p2E5BEF36.dip0.t-ipconnect.de. [46.91.239.54]) by smtp.gmail.com with ESMTPSA id t7sm12098689wrn.52.2019.06.19.01.33.09 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Wed, 19 Jun 2019 01:33:09 -0700 (PDT) Date: Wed, 19 Jun 2019 10:33:08 +0200 From: Thierry Reding To: Stephen Warren Cc: Dmitry Osipenko , Sowjanya Komatineni , jonathanh@nvidia.com, tglx@linutronix.de, jason@lakedaemon.net, marc.zyngier@arm.com, linus.walleij@linaro.org, stefan@agner.ch, mark.rutland@arm.com, pdeschrijver@nvidia.com, pgaikwad@nvidia.com, sboyd@kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, jckuo@nvidia.com, josephl@nvidia.com, talho@nvidia.com, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, mperttunen@nvidia.com, spatra@nvidia.com, robh+dt@kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH V3 02/17] pinctrl: tegra: add suspend and resume support Message-ID: <20190619083308.GM3187@ulmo> References: <1560843991-24123-1-git-send-email-skomatineni@nvidia.com> <1560843991-24123-3-git-send-email-skomatineni@nvidia.com> <7706a287-44b7-3ad6-37ff-47e97172a798@gmail.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="Nj4mAaUCx+wbOcQD" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.11.4 (2019-03-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Nj4mAaUCx+wbOcQD Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jun 18, 2019 at 09:41:03AM -0600, Stephen Warren wrote: > On 6/18/19 3:30 AM, Dmitry Osipenko wrote: > > 18.06.2019 12:22, Dmitry Osipenko =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > > > 18.06.2019 10:46, Sowjanya Komatineni =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > > > > This patch adds suspend and resume support for Tegra pinctrl driver > > > > and registers them to syscore so the pinmux settings are restored > > > > before the devices resume. > > > >=20 > > > > Signed-off-by: Sowjanya Komatineni > > > > --- > > > > drivers/pinctrl/tegra/pinctrl-tegra.c | 62 +++++++++++++++++++= +++++++++++++ > > > > drivers/pinctrl/tegra/pinctrl-tegra.h | 5 +++ > > > > drivers/pinctrl/tegra/pinctrl-tegra114.c | 1 + > > > > drivers/pinctrl/tegra/pinctrl-tegra124.c | 1 + > > > > drivers/pinctrl/tegra/pinctrl-tegra20.c | 1 + > > > > drivers/pinctrl/tegra/pinctrl-tegra210.c | 13 +++++++ > > > > drivers/pinctrl/tegra/pinctrl-tegra30.c | 1 + > > > > 7 files changed, 84 insertions(+) > > > >=20 > > > > diff --git a/drivers/pinctrl/tegra/pinctrl-tegra.c b/drivers/pinctr= l/tegra/pinctrl-tegra.c > > > > index 34596b246578..ceced30d8bd1 100644 > > > > --- a/drivers/pinctrl/tegra/pinctrl-tegra.c > > > > +++ b/drivers/pinctrl/tegra/pinctrl-tegra.c > > > > @@ -20,11 +20,16 @@ > > > > #include > > > > #include > > > > #include > > > > +#include > > > > #include "../core.h" > > > > #include "../pinctrl-utils.h" > > > > #include "pinctrl-tegra.h" > > > > +#define EMMC2_PAD_CFGPADCTRL_0 0x1c8 > > > > +#define EMMC4_PAD_CFGPADCTRL_0 0x1e0 > > > > +#define EMMC_DPD_PARKING (0x1fff << 14) > > > > + > > > > static inline u32 pmx_readl(struct tegra_pmx *pmx, u32 bank, u32 = reg) > > > > { > > > > return readl(pmx->regs[bank] + reg); > > > > @@ -619,6 +624,48 @@ static void tegra_pinctrl_clear_parked_bits(st= ruct tegra_pmx *pmx) > > > > pmx_writel(pmx, val, g->mux_bank, g->mux_reg); > > > > } > > > > } > > > > + > > > > + if (pmx->soc->has_park_padcfg) { > > > > + val =3D pmx_readl(pmx, 0, EMMC2_PAD_CFGPADCTRL_0); > > > > + val &=3D ~EMMC_DPD_PARKING; > > > > + pmx_writel(pmx, val, 0, EMMC2_PAD_CFGPADCTRL_0); > > > > + > > > > + val =3D pmx_readl(pmx, 0, EMMC4_PAD_CFGPADCTRL_0); > > > > + val &=3D ~EMMC_DPD_PARKING; > > > > + pmx_writel(pmx, val, 0, EMMC4_PAD_CFGPADCTRL_0); > > > > + } > > > > +} > > >=20 > > > Is there any reason why parked_bit can't be changed to parked_bitmask= like I was > > > asking in a comment to v2? > > >=20 > > > I suppose that it's more preferable to keep pinctrl-tegra.c platform-= agnostic for > > > consistency when possible, hence adding platform specifics here shoul= d be discouraged. > > > And then the parked_bitmask will also result in a proper hardware des= cription in the code. > > >=20 > >=20 > > I'm now also vaguely recalling that Stephen Warren had some kind of a "= code generator" > > for the pinctrl drivers. So I guess all those tables were auto-generate= d initially. > >=20 > > Stephen, maybe you could adjust the generator to take into account the = bitmask (of > > course if that's a part of the generated code) and then re-gen it all f= or Sowjanya? >=20 > https://github.com/NVIDIA/tegra-pinmux-scripts holds the scripts that > generate tegra-pinctrlNNN.c. See soc-to-kernel-pinctrl-driver.py. IIRC, > tegra-pinctrl.c (the core file) isn't auto-generated. Sowjanya is welcome= to > send a patch to that repo if the code needs to be updated. If we want to do that, we may need to start off by bringing the pinmux scripts up to date with the latest version of the generated files. There have been a number of changes in the meantime that cause the scripts to generate a bit of diff with regards to what's currently upstream. Sounds like something fairly trivial, though. Thierry --Nj4mAaUCx+wbOcQD Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl0J80QACgkQ3SOs138+ s6FBAg/7Bf4pKVuUgiXaL2ZpSW6kr69XzJ449U10mrw9x3AFaYkWRN+8naxPQgWS c338JFDAEDEOLmCY45MzLuD7UYyoSKTVqO5H5GHxnpCARKtkfRY0av6yXyf/C2qN FCdr7w4POoTfg9XlqddT8dTxYth3w8Y95shLpj6gSbc9gVqKNPVYyiB/YG8sPI6H G9c9dXFiRheXgLOkpk8AROj1iSh5F0DDDvcSBbds/KvYJKJGR62odzBBAzEd4Izq U+a+CBZDCN+x5UM3WEeh3/sXuDBnZUjAynKfsa/yPcdyKtnGENqjsVNfnWbk45r8 8cv601Pw110Co51JHS0slvvSCoTuzvq5cErKmIj52eFhstYNKek1evcNZMLSuF3R EEqh4ob3Rbt5kj2YJUT+H9WRzkzIZjx5AnS/CHAQVPTmqNC9FMPuFAL8FbVIYNFF SH0GVXmOdAGvHdQEkMe1BKb1nKv2U2o5KAGJgft4RBDVxu/KiuJNto1xEdbab3dK Lgz2WUuUoOwTwiZhT3vr2UWym78uwSk+D2hxT8o9aABXv8wGKlXg9gjIi7r6famJ rvLdWy6VjDsdJz9jeK89AfsWB40vix6AhLtZ7SqfZ6cXOIFXJXk5SUQZuWO3qM+r n8QVJCMaJWFCXSmnBpihGlICrr+/0SKA/axw7uLBypTj1EF1jk0= =sP7K -----END PGP SIGNATURE----- --Nj4mAaUCx+wbOcQD--