[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <331ABD5ECB02734CA317220B2BBEABC13E9E8E74@DBDE01.ent.ti.com>
Date: Tue, 3 Jul 2012 07:45:52 +0000
From: "AnilKumar, Chimata" <anilkumar@...com>
To: Axel Lin <axel.lin@...il.com>,
Mark Brown <broonie@...nsource.wolfsonmicro.com>
CC: "Girdwood, Liam" <lrg@...com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"Nori, Sekhar" <nsekhar@...com>
Subject: RE: [PATCH v2] regulator: tps65217: Fix voltage boundary checking
in tps65217_pmic_map_voltage
Hi Axel,
Thanks for the patch
Acked-by: AnilKumar Ch <anilkumar@...com>
On Tue, Jul 03, 2012 at 12:49:33, Axel Lin wrote:
> It is ok to request voltage with min_uV < tps->info[rid]->min_uV and
> max_uV > tps->info[rid]->max_uV.
>
> The equation we used in uv_to_vsel() does not allow
> min_uV < tps->info[rid]->min_uV, otherwise it returns negative selector.
> So we need to set min_uV = tps->info[rid]->min_uV if
> min_uV < tps->info[rid]->min_uV.
>
> Signed-off-by: Axel Lin <axel.lin@...il.com>
> ---
> drivers/regulator/tps65217-regulator.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/regulator/tps65217-regulator.c b/drivers/regulator/tps65217-regulator.c
> index 0a3df5b..f7d0495 100644
> --- a/drivers/regulator/tps65217-regulator.c
> +++ b/drivers/regulator/tps65217-regulator.c
> @@ -206,10 +206,10 @@ static int tps65217_pmic_map_voltage(struct regulator_dev *dev,
> if (rid < TPS65217_DCDC_1 || rid > TPS65217_LDO_4)
> return -EINVAL;
>
> - if (min_uV < tps->info[rid]->min_uV || min_uV > tps->info[rid]->max_uV)
> - return -EINVAL;
> + if (min_uV < tps->info[rid]->min_uV)
> + min_uV = tps->info[rid]->min_uV;
>
> - if (max_uV < tps->info[rid]->min_uV || max_uV > tps->info[rid]->max_uV)
> + if (max_uV < tps->info[rid]->min_uV || min_uV > tps->info[rid]->max_uV)
> return -EINVAL;
>
> ret = tps->info[rid]->uv_to_vsel(min_uV, &sel);
> --
> 1.7.9.5
>
>
>
>
Powered by blists - more mailing lists