On Thu, Apr 21, 2016 at 03:12:37PM +0200, Wadim Egorov wrote:
> +static int rk808_set_suspend_voltage(struct regulator_dev *rdev, int uv)
> +{
> + unsigned int reg;
> + int sel = regulator_map_voltage_linear(rdev, uv, uv);
> +
> + if (sel < 0)
> + return -EINVAL;
> +
> + reg = rdev->desc->vsel_reg + RK808_SLP_REG_OFFSET;
> +
> + return regmap_update_bits(rdev->regmap, reg,
> + rdev->desc->vsel_mask,
> + sel);
> +}This is fine but is adding a new feature and not part of the refactoring that the changelog talked about so should be in a separate commit.
signature.asc
Description: PGP signature
