On Tue, Jan 19, 2021 at 05:10:45PM +0100, Mauro Carvalho Chehab wrote:

> +static int hi6421_spmi_regulator_get_voltage_sel(struct regulator_dev *rdev)
> +{

> +static int hi6421_spmi_regulator_set_voltage_sel(struct regulator_dev *rdev,
> +						 unsigned int selector)

Now that the driver has been converted to regmap these are just
duplicates of the regmap helpers.  You may also be able to use them for
the disable() and is_enabled() operations, I didn't confirm that that's
OK with the device using multi-bit enable controls for some reason IIRC.