From:   Colin Foster <colin.foster@in-advantage.com>
To:     netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org
Cc:     Russell King <linux@armlinux.org.uk>,
        Richard Cochran <richardcochran@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>, UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>,
        Paolo Abeni <pabeni@redhat.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>, Lee Jones <lee@kernel.org>
Subject: [PATCH v5 net-next 07/13] net: dsa: felix: add support for MFD configurations
Date:   Fri, 27 Jan 2023 11:35:53 -0800
Message-Id: <20230127193559.1001051-8-colin.foster@in-advantage.com>
In-Reply-To: <20230127193559.1001051-1-colin.foster@in-advantage.com>
References: <20230127193559.1001051-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?5VtjRdd+bPIX/xJewmjJoXJOAYXC8XMuCN0I6T6qU7r9DrjxpuDNWclRrlQW?=
 =?us-ascii?Q?dPKPlup19mwUrq/+NdNGgoxvWacZiW1qSx6Gl1JCG34cNlC65QmxTwYwI/3m?=
 =?us-ascii?Q?Az+VIxMGD0dV+qqsRO+AYGcwANMViRN5TgThCeiAgnrRMsELEylv2S92w9w5?=
 =?us-ascii?Q?Sqd4Gys6GrAuhNfenbrUwAbn417dlEIkG2IXEqYUi5BOSHOelbv5iktG7tiD?=
 =?us-ascii?Q?bsXwa1UZZTza7bjtbGaNgQaE4kfD33staDK65xPinQeJMq2lWpTo0QYc2gTA?=
 =?us-ascii?Q?Y9yqA3y/qEI/CF5N66SfdXGdoZ1prxgzA7ggPKeBKJGVYIPE1Q0bIWEDQB30?=
 =?us-ascii?Q?2CyDekv36u/YP9tCItVF8jx9mA2sSXLsRSQvGiZtsNAU/HpPiFE0OvFshtCN?=
 =?us-ascii?Q?01cAG7f+/9wPRJoOZeCZdGGsLAtgvua3WN9KEZdYHSGE+dve21AHSxGieEX5?=
 =?us-ascii?Q?zI7rf/v3V9Pm4yJab4RmH4ZBZSSIICm1xhAyxDCBDIOKZfWmkQuPUqW/hnCw?=
 =?us-ascii?Q?htShbK2ZYHnPWwk9Iq0gwCnJxyG6zZOlRcqfy+pXJ6+tR1D6Egn8JJsOycyf?=
 =?us-ascii?Q?UqTLFHqm8XG9AY4ztyCD/j5fvmnX44aMbeiQBgwCFbFuQC+l/VrJayttOu80?=
 =?us-ascii?Q?izy0biKUC4k5QI00pghTj/xAXAUkYavyzGl+M2NY5jGYn6s0tzlf/mZ8M9pd?=
 =?us-ascii?Q?Z3Y45RzbPIP7PReSaKkFOyKHb+/pKTfbQn+9wgnNkFVGJUaK2se1jJQkpkZg?=
 =?us-ascii?Q?mFdINZcKKUpl34v4Bzqx8EcY0qeUFX1MXQcmqFoRq5eO6wzWnocMCZVvarrV?=
 =?us-ascii?Q?xr5eUYTiCTsMdN5HEMWiRx4pqnjNkoimYvBLrJjuK078XEbW8/W0+kVQvZgJ?=
 =?us-ascii?Q?fSfs2FUniPa1mfcESp25zPcEBqZQe7CtLl3Raf23IEgotZT90w7Tcse+LGVz?=
 =?us-ascii?Q?TrITXK/xi+LDw+bIXF8WAH11WDkITXESWxxH0CV5HT2m+CczsQnViVyPL84l?=
 =?us-ascii?Q?sfmYsCN3o5K5zi9pXctU+SFld3tzIknu89xxJrXRzxhGqRF8+NhkJ5BArFhJ?=
 =?us-ascii?Q?4BOaeUx3IL9NLFLgzLO2cCT31XuLbcdKRsfeHJ00dljdwvH0mQhmkMuvheBH?=
 =?us-ascii?Q?1k98eszELcJYFyfmEJ/s/VN1G1aRiX0ArFp6OesfVb5P5FDNdb5PE77n1qbc?=
 =?us-ascii?Q?DFprNHbIF6WL8mqjIvSvU/piErKQF6DyqTu0Ld15jp345BYNm3oV8Yf9+KMe?=
 =?us-ascii?Q?CFu64P2RjUgwGLNRb9XdFrtZGtRQPwdT5iQw1APSxJ0+tUS/BW5YILXMqt9y?=
 =?us-ascii?Q?isFMzF9xF1Tg6Z29VusE3vR6PlhRnszDUG6z+2duq32Wwsj40YjWSEUIq9cm?=
 =?us-ascii?Q?I/Mj031bjRMqL+YFEBniC4Uphee6IMuQiB5dv5iFQBK43zTImV7MgSd0oSpw?=
 =?us-ascii?Q?pOk/Q1TDbT8XRru/gcMbznO02RyzwLI8yytQpNJIak3kx8+eYWbC2+lKE6xU?=
 =?us-ascii?Q?06tiZ/ZWsjHFMLt+Lx/p3drvLLsgILU8MwFzseMDoupz7jb/VWyzuChHG9cF?=
 =?us-ascii?Q?V3ar5QWjB45BwQzHzUeChVssaHUd7vmzMhnuzRWCn5E4YDqdFgEkbXXFJpZz?=
 =?us-ascii?Q?6OvwxmdmsbUbG62B3GCxALU=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3b5af373-871b-4e11-89f1-08db009dc5ce
X-MS-Exchange-CrossTenant-AuthSource: DM5PR1001MB2345.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jan 2023 19:36:23.3757
 (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: zR6qxdTyKLOfHm6FLnbF2pa6OduqDvtf52J1laoErvB22toZakPWpKyRe5smyGiSmV/URAonip295XGRD8EgKrexgwoXyjFhekPa7Wo03n0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4636
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The architecture around the VSC7512 differs from existing felix drivers. In
order to add support for all the chip's features (pinctrl, MDIO, gpio) the
device had to be laid out as a multi-function device (MFD).

One difference between an MFD and a standard platform device is that the
regmaps are allocated to the parent device before the child devices are
probed. As such, there is no need for felix to initialize new regmaps in
these configurations, they can simply be requested from the parent device.

Add support for MFD configurations by performing this request from the
parent device.

Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
---

v5
    * No changes

v4
    * New patch

---
 drivers/net/dsa/ocelot/felix.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/net/dsa/ocelot/felix.c b/drivers/net/dsa/ocelot/felix.c
index 462a1a683996..d3ff6e8a82e9 100644
--- a/drivers/net/dsa/ocelot/felix.c
+++ b/drivers/net/dsa/ocelot/felix.c
@@ -1315,6 +1315,13 @@ static struct regmap *felix_request_regmap_by_name(struct felix *felix,
 	struct resource res;
 	int i;
 
+	/* In an MFD configuration, regmaps are registered directly to the
+	 * parent device before the child devices are probed, so there is no
+	 * need to initialize a new one.
+	 */
+	if (!felix->info->resources)
+		return dev_get_regmap(ocelot->dev->parent, resource_name);
+
 	for (i = 0; i < felix->info->num_resources; i++) {
 		if (strcmp(resource_name, felix->info->resources[i].name))
 			continue;
-- 
2.25.1