[U-Boot] [PATCH] power: rk808: fix ldo register offset

[Jacob Chen]

Hi,

2017-05-07 3:21 GMT+08:00 Heiko Stuebner <heiko at sntech.de>:
> Till now get_ldo_reg did a return &rk808_ldo[num - 1]; to return
> the ldo register offset but didn't take into account that its
> calling functions already created the ldo as ldo = dev->driver_data - 1.
>
> This resulted in the setting for ldo8 writing to the register of ldo7
> and so on. So fix this and get the correct ldo register data.
>
> Signed-off-by: Heiko Stuebner <heiko at sntech.de>
> ---
>  drivers/power/regulator/rk8xx.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/power/regulator/rk8xx.c b/drivers/power/regulator/rk8xx.c
> index e655c2d91f..c1ece96b66 100644
> --- a/drivers/power/regulator/rk8xx.c
> +++ b/drivers/power/regulator/rk8xx.c
> @@ -92,9 +92,9 @@ static const struct rk8xx_reg_info *get_ldo_reg(struct udevice *pmic,
>         struct rk8xx_priv *priv = dev_get_priv(pmic);
>         switch (priv->variant) {
>         case RK818_ID:
> -               return &rk818_ldo[num - 1];
> +               return &rk818_ldo[num];
>         default:
> -               return &rk808_ldo[num - 1];
> +               return &rk808_ldo[num];
>         }
>  }
>
> --
> 2.11.0
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot


Sorry for untest patches, Reviewed-by: Jacob Chen <jacob2.chen at rock-chips.com>.