Nishanth Menon <nm@xxxxxx> writes:
> using 1.35V as a check is not correct, we know that beyond 0x39,
> voltages are non linear - hence use the conversion iff uV greater
> than that for 0x39. For example, with 709mV as the smps offset,
> the max linear is actually 1.41V(0x39vsel)!
>
> Signed-off-by: Nishanth Menon <nm@xxxxxx>
Thanks, adding to voltdm queue.
Kevin
> ---
> arch/arm/mach-omap2/omap_twl.c | 7 ++++++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/omap_twl.c b/arch/arm/mach-omap2/omap_twl.c
> index 78fd985..fce7e6d 100644
> --- a/arch/arm/mach-omap2/omap_twl.c
> +++ b/arch/arm/mach-omap2/omap_twl.c
> @@ -134,8 +134,13 @@ static u8 twl6030_uv_to_vsel(unsigned long uv)
> * hardcoding only for 1.35 V which is used for 1GH OPP for
> * OMAP4430.
> */
> - if (uv == 1350000)
> + if (uv > twl6030_vsel_to_uv(0x39)) {
> + if (uv == 1350000)
> + return 0x3A;
> + pr_err("%s:OUT OF RANGE! non mapped vsel for %ld Vs max %ld\n",
> + __func__, uv, twl6030_vsel_to_uv(0x39));
> return 0x3A;
> + }
>
> if (smps_offset & 0x8)
> return DIV_ROUND_UP(uv - 709000, 12660) + 1;
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
