regulator: rohm-generic: remove unused dummies
authorMatti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Wed, 24 Nov 2021 05:57:49 +0000 (07:57 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 24 Nov 2021 12:57:31 +0000 (12:57 +0000)
Function rohm_regulator_set_voltage_sel_restricted() and
rohm_regulator_set_dvs_levels() had inlined dummy implementations for
cases when the real implementation was not configured in.

All of the drivers who issue the call to these functions do SELECT the
real implementation from the Kconfig. There should be no cases where the
real implementation was not selected by the drivers using these
functions - such a situation is likely to be an error which deserves to be
noticed at compile-time.

These dummies could in theory be used for compile-testing the drivers
only (without the generic rohm regulator pieces). However, for such
compile testing we should manually drop the selection from KConfig - and
I guess that if it does not work out-of-the-box, then it is not going to
happen. Especially when there should be no reason to omit
compile-testing the generic rohm_regulator part.

Crash test dummies.

Signed-off-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Link: https://lore.kernel.org/r/YZ3UXXrk/Efe7Scj@fedora
Signed-off-by: Mark Brown <broonie@kernel.org>
include/linux/mfd/rohm-generic.h

index 080d60a..5ed97a1 100644 (file)
@@ -82,20 +82,6 @@ int rohm_regulator_set_dvs_levels(const struct rohm_dvs_config *dvs,
 
 int rohm_regulator_set_voltage_sel_restricted(struct regulator_dev *rdev,
                                              unsigned int sel);
-#else
-static inline int rohm_regulator_set_dvs_levels(const struct rohm_dvs_config *dvs,
-                                               struct device_node *np,
-                                               const struct regulator_desc *desc,
-                                               struct regmap *regmap)
-{
-       return 0;
-}
-
-static inline int rohm_regulator_set_voltage_sel_restricted(struct regulator_dev *rdev,
-                                                    unsigned int sel)
-{
-       return 0;
-}
 #endif
 
 #endif