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;
From:   Colin Foster <colin.foster@in-advantage.com>
To:     linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org
Cc:     Terry Bowman <terry.bowman@amd.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Wolfram Sang <wsa@kernel.org>,
        Andy Shevchenko <andy.shevchenko@gmail.com>,
        UNGLinuxDriver@microchip.com,
        Steen Hegelund <Steen.Hegelund@microchip.com>,
        Lars Povlsen <lars.povlsen@microchip.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Russell King <linux@armlinux.org.uk>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Lee Jones <lee.jones@linaro.org>,
        katie.morris@in-advantage.com,
        Jonathan Cameron <Jonathan.Cameron@huawei.com>,
        Dan Williams <dan.j.williams@intel.com>
Subject: [PATCH v16 mfd 3/8] pinctrl: ocelot: add ability to be used in a non-mmio configuration
Date:   Sun, 14 Aug 2022 17:55:48 -0700
Message-Id: <20220815005553.1450359-4-colin.foster@in-advantage.com>
In-Reply-To: <20220815005553.1450359-1-colin.foster@in-advantage.com>
References: <20220815005553.1450359-1-colin.foster@in-advantage.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?NvFJ86bZyvzLEreErLjuyH/UsBN4BD+TR2Y+hAXXhz6LaFGyh1PdHVkTM/yV?=
 =?us-ascii?Q?rq6dFlcaEqnf9J+1Lw+k1IW99y6nMnyLHrYnc8sAm8EJyzkiEWUsz+pQcdoF?=
 =?us-ascii?Q?0AiTWYA2+fvMa0l+4Xz18C4MFS7R6u+I0ljpSfS1ODj5Woiw3QG5viZmsmBZ?=
 =?us-ascii?Q?m/6VXgsRmf/S6QlQqcHO3gp45hdN9JLllUWk22iVnL7wl2vUmlMsss65d50g?=
 =?us-ascii?Q?Y4SVJC9AOL9x0WbTSSrOSZt3k6PHlRSgLfJBrxpa8NEeJhdmtbOphCl+/BWU?=
 =?us-ascii?Q?MTwMVEmRpwuOYy2fZ30aGKVqPumlbPEfjK9Jjp6mk3c7DNg1JSHgvEtCdBOe?=
 =?us-ascii?Q?4n+KKA1poRMEF64ogEaHkqHEWev38kDvr/fQ3Mpdh0akuGUoq5GaxleW7mi9?=
 =?us-ascii?Q?VhHEcN21FGP2Cqk7wzPI5lhK/VYhlpo62yp+V/jwmQOSv25I1WUqSbFU0Kdr?=
 =?us-ascii?Q?aLnzAf03gQZZ1Qs02VScqpnhBctSWpVRkyvt+Gy1R6U57V/S5dedns0MsFqt?=
 =?us-ascii?Q?ho+mxNBaWOX6/EA9XbofPlGcLb5L1lQ0sgskeTPc4XAMr0TnAb3nJvkQEJR8?=
 =?us-ascii?Q?jrcPgmN/lp2F7iYulVjmQ8weN/PgXEN71Ximze/As25fTl+ebV0WEiDWoIde?=
 =?us-ascii?Q?/soOUqwW1ozJhlvfzrxdBGA6062Hv4xSsHHrK7PHZy4caH9HWE+v4o9gWg2Q?=
 =?us-ascii?Q?JMumw5rBjZecUUMO6G1SRgRgTjSi5TKNsKfGd6x5dlnwihsVfXtEn5sGloAL?=
 =?us-ascii?Q?Du95CpXeuIAeeHxA5Tm/0BLJJKN+0+C9tlVk0VnXmjh8wz4KpFEWMIu0voNZ?=
 =?us-ascii?Q?MAR5B/n13AvhaZ0CTg3wFtM+g+a2dMod7qab+J+n65rictyilYOAVUt6/gbK?=
 =?us-ascii?Q?HqXIbX+TRiNBWfIG1priZha8b9jnpHnhcRgiw8PAfb6fr+RMFZQClZRJDsin?=
 =?us-ascii?Q?ne1OA4xfDVAGUBb3Wqzyo+uhD4/00rTAw1U2Td+l+EI9bGFxUiXlkF1ZObRg?=
 =?us-ascii?Q?j45T6yaybe9Dr2a2hFdYCeZgjoHu+wZ9GqvWIos8POOI3kuiDSVDr4b/uaVN?=
 =?us-ascii?Q?eScZZRuaqyeIsmxAw54nEm9WhtI3w8rewPfD9BSZXC+7wrlW5va9HCc1pRta?=
 =?us-ascii?Q?0+fZZmi3KuvLIiNxTv/kBy6WaS45FR+0C3DhHDY+a1Ur8gyHmou+VHJGz78X?=
 =?us-ascii?Q?keJ6As6DjeRHZN8ZNrcjfY+/6eaQsYVMOO46CYh0TlEMFvxvcj27ImzoNQxR?=
 =?us-ascii?Q?eukL/QeNj0LWKmtNpR92zvOkhb/KrJH4PuHzFmoEHuGL6ObLonoqzoFsc/0j?=
 =?us-ascii?Q?C3kc8YgvyWfD5K/qK4Ko711fJG7HWIf7hL7eypwn/iEJHJDzYvYUlj6pDOJt?=
 =?us-ascii?Q?0CtCQ5ccuxg2nGoBN7Zp69Y8+Ms4GT/bd9HvvNSjE+Qln44aeekWKXNkVHay?=
 =?us-ascii?Q?Mx3ez/ug7ral2dBxyJrFzSDGJO45qK84braixcMSUPJ7pyS4r8vu5aaHSBUa?=
 =?us-ascii?Q?wtn7ytpVkY/Ch2J+PHjgNtBbZG5vhi6i4Tc4d3e/yZ2aI6/6glrR0PFTmIQl?=
 =?us-ascii?Q?mOfojRurCqFSYuQRGXiP751BHknTmhNfMRPUDR6bUOVjgiliK2rlPRPzcpsH?=
 =?us-ascii?Q?qA=3D=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fae45f93-82d9-45f2-937c-08da7e58f15e
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Aug 2022 00:56:10.0106
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: W8E+Xakqg5aoXqTbeIXh+7mOJCuhnde8++MUKMMV7XeqaOKYFNCZcpDECwT/w+vZcIjUcnApqh/kwXG0N2hD2o+UHFooaW6cENuud/MSYjg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR10MB3862
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

There are a few Ocelot chips that contain pinctrl logic, but can be
controlled externally. Specifically the VSC7511, 7512, 7513 and 7514. In
the externally controlled configurations these registers are not
memory-mapped.

Add support for these non-memory-mapped configurations.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
---

v16
    * Add Andy Reviewed-by tag

v15
    * No changes

v14
    * Add Reviewed and Acked tags

---
 drivers/pinctrl/pinctrl-ocelot.c | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/drivers/pinctrl/pinctrl-ocelot.c b/drivers/pinctrl/pinctrl-ocelot.c
index c5fd154990c8..340ca2373429 100644
--- a/drivers/pinctrl/pinctrl-ocelot.c
+++ b/drivers/pinctrl/pinctrl-ocelot.c
@@ -10,6 +10,7 @@
 #include <linux/gpio/driver.h>
 #include <linux/interrupt.h>
 #include <linux/io.h>
+#include <linux/mfd/ocelot.h>
 #include <linux/of_device.h>
 #include <linux/of_irq.h>
 #include <linux/of_platform.h>
@@ -1975,7 +1976,6 @@ static int ocelot_pinctrl_probe(struct platform_device *pdev)
 	struct ocelot_pinctrl *info;
 	struct reset_control *reset;
 	struct regmap *pincfg;
-	void __iomem *base;
 	int ret;
 	struct regmap_config regmap_config = {
 		.reg_bits = 32,
@@ -2004,20 +2004,14 @@ static int ocelot_pinctrl_probe(struct platform_device *pdev)
 				     "Failed to get reset\n");
 	reset_control_reset(reset);
 
-	base = devm_ioremap_resource(dev,
-			platform_get_resource(pdev, IORESOURCE_MEM, 0));
-	if (IS_ERR(base))
-		return PTR_ERR(base);
-
 	info->stride = 1 + (info->desc->npins - 1) / 32;
 
 	regmap_config.max_register = OCELOT_GPIO_SD_MAP * info->stride + 15 * 4;
 
-	info->map = devm_regmap_init_mmio(dev, base, &regmap_config);
-	if (IS_ERR(info->map)) {
-		dev_err(dev, "Failed to create regmap\n");
-		return PTR_ERR(info->map);
-	}
+	info->map = ocelot_regmap_from_resource(pdev, 0, &regmap_config);
+	if (IS_ERR(info->map))
+		return dev_err_probe(dev, PTR_ERR(info->map),
+				     "Failed to create regmap\n");
 	dev_set_drvdata(dev, info->map);
 	info->dev = dev;
 
-- 
2.25.1