[PATCH v10 net-next 1/7] mfd: ocelot: add helper to get regmap from a resource

Andy Shevchenko andy.shevchenko at gmail.com
Sat Jun 11 03:37:26 PDT 2022


On Fri, Jun 10, 2022 at 10:23 PM Colin Foster
<colin.foster at in-advantage.com> wrote:
>
> Several ocelot-related modules are designed for MMIO / regmaps. As such,
> they often use a combination of devm_platform_get_and_ioremap_resource and
> devm_regmap_init_mmio.
>
> Operating in an MFD might be different, in that it could be memory mapped,
> or it could be SPI, I2C... In these cases a fallback to use IORESOURCE_REG
> instead of IORESOURCE_MEM becomes necessary.
>
> When this happens, there's redundant logic that needs to be implemented in
> every driver. In order to avoid this redundancy, utilize a single function
> that, if the MFD scenario is enabled, will perform this fallback logic.

v10 has the same issues I have pointed out in v9.

Please, take your time and instead of bombing mailing lists with new
versions try to look how other (most recent) drivers have been done.

Also pay attention to the API design.

-- 
With Best Regards,
Andy Shevchenko



More information about the linux-arm-kernel mailing list