Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 6 Jul 2021 09:24:13 +0100
From:   Lee Jones <lee.jones@linaro.org>
To:     cy_huang <u0084500@gmail.com>
Cc:     broonie@kernel.org, linux-kernel@vger.kernel.org,
        lgirdwood@gmail.com, axel.lin@ingics.com, cy_huang@richtek.com
Subject: Re: [PATCH v2] regulator: rt5033: Use linear ranges to map all
 voltage selection
Message-ID: <YOQTLWsXpTOK58dr@dell>
References: <1625553939-9109-1-git-send-email-u0084500@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <1625553939-9109-1-git-send-email-u0084500@gmail.com>
Precedence: bulk

On Tue, 06 Jul 2021, cy_huang wrote:

> From: ChiYuan Huang <cy_huang@richtek.com>
> 
> Instead of linear mapping, Use linear range to map all voltage selection.
> 
> Signed-off-by: ChiYuan Huang <cy_huang@richtek.com>
> ---
> Since v2
> - recover the volt number back in rt5033-private.h
> - Add the n_voltages from v1 back.
> 
> Even though commit 6549c46af855 ("regulator: rt5033: Fix n_voltages settings for BUCK and LDO")
> can fix the linear mapping to the correct min/max voltage
> But there're still non-step ranges for the reserved value.
> 
> To use the linear range can fix it for mapping all voltage selection.
> ---
>  drivers/regulator/rt5033-regulator.c | 21 +++++++++++++++------

>  include/linux/mfd/rt5033-private.h   |  4 ++--

Acked-by: Lee Jones <lee.jones@linaro.org>

>  2 files changed, 17 insertions(+), 8 deletions(-)

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog