Re: [PATCH v1] regulator: core: Add stubs for driver API

From: Dmitry Osipenko
Date: Mon Jun 21 2021 - 08:05:47 EST


21.06.2021 14:46, Mark Brown пишет:
> On Sat, Jun 19, 2021 at 03:40:52PM +0300, Dmitry Osipenko wrote:
>
>> +static inline struct regulator_dev *
>> +regulator_register(const struct regulator_desc *regulator_desc,
>> + const struct regulator_config *config)
>
> Why would something be registering a regulator device without a
> dependency on the regulator API?
>

For example if regulator device registration is optional in the code. We
don't have such cases in the kernel today, but I decided that it will be
cleaner to add stubs for the whole API.

The point of these stubs is to fix the new compile-testing problem of
Tegra regulator coupler drivers. The drivers indeed don't depend on the
regulator-driver API since the regulator-consumer API provides all the
necessary stubs, but now drivers use the regulator-driver API too. The
dependency should reduce the compile-testing scope, IIUC.

If you think that it will be better to add the dependency to the coupler
drivers, then let's do it.