Re: [PM-WIP/voltdm_c][PATCH 08/11] OMAP4: PM: TWL6030: fix uv to voltage for >0x39

[Kevin Hilman <khilman@ti.com>]

Nishanth Menon <nm@ti.com> 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@ti.com>

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;