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;
MIME-Version: 1.0
References: <20210216060128.126938-1-axel.lin@ingics.com>
In-Reply-To: <20210216060128.126938-1-axel.lin@ingics.com>
From:   Adrien Grassein <adrien.grassein@gmail.com>
Date:   Wed, 17 Feb 2021 00:56:27 +0100
Message-ID: <CABkfQAFO6Xog=Eej1k6iT8BpJWLO9=n6o_YEkBxp4d6uiUv0uQ@mail.gmail.com>
Subject: Re: [PATCH] regulator: pf8x00: Use regulator_map_voltage_ascend for pf8x00_buck7_ops
To:     Axel Lin <axel.lin@ingics.com>
Cc:     Mark Brown <broonie@kernel.org>,
        Jagan Teki <jagan@amarulasolutions.com>,
        Liam Girdwood <lgirdwood@gmail.com>,
        linux-kernel <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Le mar. 16 f=C3=A9vr. 2021 =C3=A0 07:01, Axel Lin <axel.lin@ingics.com> a =
=C3=A9crit :
>
> The voltages in pf8x00_sw7_voltages are in ascendant order, so use
> regulator_map_voltage_ascend.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
>  drivers/regulator/pf8x00-regulator.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/regulator/pf8x00-regulator.c b/drivers/regulator/pf8=
x00-regulator.c
> index 9b28bd63208d..5d319fb81288 100644
> --- a/drivers/regulator/pf8x00-regulator.c
> +++ b/drivers/regulator/pf8x00-regulator.c
> @@ -359,6 +359,7 @@ static const struct regulator_ops pf8x00_buck7_ops =
=3D {
>         .disable =3D regulator_disable_regmap,
>         .is_enabled =3D regulator_is_enabled_regmap,
>         .list_voltage =3D regulator_list_voltage_table,
> +       .map_voltage =3D regulator_map_voltage_ascend,
>         .set_voltage_sel =3D regulator_set_voltage_sel_regmap,
>         .get_voltage_sel =3D regulator_get_voltage_sel_regmap,
>         .get_current_limit =3D regulator_get_current_limit_regmap,
> --
> 2.25.1
>

Reviewed-by: Adrien Grassein <adrien.grassein@gmail.com>