On Wed, Oct 05, 2022 at 04:57:24PM +0200, Alexandre Mergnat wrote:
> +static int mt6357_set_voltage_sel(struct regulator_dev *rdev,
> + unsigned int selector)
> +{
> + int idx, ret;
> + const u32 *pvol;
> + struct mt6357_regulator_info *info = rdev_get_drvdata(rdev);
> +
> + pvol = info->index_table;
> +
> + idx = pvol[selector];
> + ret = regmap_update_bits(rdev->regmap, info->desc.vsel_reg,
> + info->desc.vsel_mask,
> + idx << 8);
> +
> + return ret;
> +}
There's a bunch of other Mediatek drivers doing this and multiple
instances in this driver, the helper should really be factored out.
Attachment:
signature.asc
Description: PGP signature
