Re: [PATCH v1 net-next 3/6] net: dsa: ocelot: felix: add interface for custom regmaps
From: Colin Foster <colin.foster@in-advantage.com>
Date: 2021-12-04 00:11:52
Also in:
lkml
On Mon, Nov 22, 2021 at 08:45:56AM -0800, Colin Foster wrote:
On Sun, Nov 21, 2021 at 05:19:02PM +0000, Vladimir Oltean wrote:quoted
On Fri, Nov 19, 2021 at 02:43:10PM -0800, Colin Foster wrote:quoted
Add an interface so that non-mmio regmaps can be used Signed-off-by: Colin Foster <colin.foster@in-advantage.com> ---What is your plan with treating the vsc7514 spi chip as a multi function device, of which the DSA driver would probe only on the Ethernet switch portion of it? Would this patch still be needed in its current form?I don't have this fully mapped out, so I'm not positive. I think it would be needed though. Felix and Ocelot need regmaps and will need to get them from somewhere. The VSC7512 switch driver will need to provide the regmap directly (current form) or indirectly (by requesting it from the MFD parent). I'll be looking more into MFD devices as well. The madera driver seems like one I'd use to model the VSC751X MFD after - just from a brief look around the drivers/mfd directory.
As you can infer from my RFC today - I've looked more into what the MFD implementation will be. I believe that will have no effect on this patch. Felix needs "ANA", so if it gets that regmap from the resource (felix / seville), by "devm_regmap_init" (current implementation) or "dev_get_regmap(dev->parent, res->name);" should make no difference from the Felix driver standpoint. That said, I'm fine holding this one off until that's proven out. I'd like to get feedback on my general RFC before skinking a couple days into that restructure.