Re: [PATCH 3/3] regulator: qcom_smd: Avoid overlapping linear voltage ranges

From: Andy Gross
Date: Mon Jul 11 2016 - 18:40:09 EST


On Mon, Jul 11, 2016 at 02:50:09PM -0700, Stephen Boyd wrote:
> The pm8x41_hfsmps ranges overlap. The first range is from 375000
> to 1562500:
>
> 375000 + (95 * 12500) == 1562500
>
> and the second range starts at 1550000. Interestingly, the second
> range ends at the correct value when it's set to be the
> appropriate start value, 1575000:
>
> 1575000 + ((158 - 96) * 25000) == 3125000
>
> Cc: Andy Gross <andy.gross@xxxxxxxxxx>
> Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> Fixes: da65e367b67e ("regulator: Regulator driver for the Qualcomm RPM")
> Signed-off-by: Stephen Boyd <sboyd@xxxxxxxxxxxxxx>
> ---

Reviewed-by: Andy Gross <andy.gross@xxxxxxxxxx>