On Fri, Mar 29, 2024 at 11:50:33PM +0000, Andre Przywara wrote:
> The AXP717 also contains an adjustable boost regulator, to provide the
> 5V USB VBUS rail when running on battery.
>
> Add the regulator description that states the voltage range this
> regulator can cover.
>
> Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx>
Hi,
I checked this against the AXP717 datasheet and it looks correct.
John.
Reviewed-by: John Watts <contact@xxxxxxxxxx>
> ---
> drivers/regulator/axp20x-regulator.c | 4 ++++
> include/linux/mfd/axp20x.h | 1 +
> 2 files changed, 5 insertions(+)
>
> diff --git a/drivers/regulator/axp20x-regulator.c b/drivers/regulator/axp20x-regulator.c
> index 3907606b091f6..71407c5810941 100644
> --- a/drivers/regulator/axp20x-regulator.c
> +++ b/drivers/regulator/axp20x-regulator.c
> @@ -143,6 +143,7 @@
> #define AXP717_DCDC3_NUM_VOLTAGES 103
> #define AXP717_DCDC_V_OUT_MASK GENMASK(6, 0)
> #define AXP717_LDO_V_OUT_MASK GENMASK(4, 0)
> +#define AXP717_BOOST_V_OUT_MASK GENMASK(7, 4)
>
> #define AXP803_PWR_OUT_DCDC1_MASK BIT_MASK(0)
> #define AXP803_PWR_OUT_DCDC2_MASK BIT_MASK(1)
> @@ -829,6 +830,9 @@ static const struct regulator_desc axp717_regulators[] = {
> AXP_DESC(AXP717, CPUSLDO, "cpusldo", "vin1", 500, 1400, 50,
> AXP717_CPUSLDO_CONTROL, AXP717_LDO_V_OUT_MASK,
> AXP717_LDO1_OUTPUT_CONTROL, BIT(4)),
> + AXP_DESC(AXP717, BOOST, "boost", "vin1", 4550, 5510, 64,
> + AXP717_BOOST_CONTROL, AXP717_BOOST_V_OUT_MASK,
> + AXP717_MODULE_EN_CONTROL, BIT(4)),
> };
>
> /* DCDC ranges shared with AXP813 */
> diff --git a/include/linux/mfd/axp20x.h b/include/linux/mfd/axp20x.h
> index 4dad54fdf67ee..5e86b976c4caa 100644
> --- a/include/linux/mfd/axp20x.h
> +++ b/include/linux/mfd/axp20x.h
> @@ -486,6 +486,7 @@ enum {
> AXP717_CLDO3,
> AXP717_CLDO4,
> AXP717_CPUSLDO,
> + AXP717_BOOST,
> AXP717_REG_ID_MAX,
> };
>
> --
> 2.35.8
>
