Messages in this thread | | | Subject | Re: [PATCH v1] regulator: core: Add stubs for driver API | From | Dmitry Osipenko <> | Date | Mon, 21 Jun 2021 15:04:30 +0300 |
| |
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.
| |