[PATCH v2] power: regulator: tps65941: Enable FPWM bits

[Kumar, Udit]

Thanks for v2

On 4/21/2026 1:51 PM, Neha Malcom Francis wrote:
> Depending on the phase selection (single or multi), the FPWM bits
> configured forces the regulator to operate in PWM mode. In case of
> multi-phase selection, the FPWM_MP bits enforce the regulator to also
> operate in multi-phase. This fixes correct multi-phase operation.
> 
> While at this, correct incorrect macro alignment as well.
> 
> Fixes: 065a452ae6a1 ("power: regulator: tps65941: add regulator support")
> Link: https://www.ti.com/lit/ds/symlink/tps6594-q1.pdf
> Signed-off-by: Keerthy <j-keerthy at ti.com>
> Signed-off-by: Takuma Fujiwara <t-fujiwara1 at ti.com>
> Signed-off-by: Neha Malcom Francis <n-francis at ti.com>
> ---
> [..] 
> diff --git a/drivers/power/regulator/tps65941_regulator.c b/drivers/power/regulator/tps65941_regulator.c
> index 2561d6f4c6c..209968b5718 100644
> --- a/drivers/power/regulator/tps65941_regulator.c
> +++ b/drivers/power/regulator/tps65941_regulator.c
> @@ -63,13 +63,14 @@ static inline int tps65941_get_chip_id(struct udevice *dev)
>  
>  static int tps65941_buck_enable(struct udevice *dev, int op, bool *enable)
>  {
> -	int ret;
> +	int ret, idx;
>  	unsigned int adr;
>  	struct dm_regulator_uclass_plat *uc_pdata;
>  
>  	uc_pdata = dev_get_uclass_plat(dev);
>  	adr = uc_pdata->ctrl_reg;
>  
> +	idx = dev->driver_data;
>  	ret = pmic_reg_read(dev->parent, adr);

Reviewed-by: Udit Kumar <u-kumar1 at ti.com>

>  	if (ret < 0)
>  		return ret;
> @@ -84,10 +85,18 @@ static int tps65941_buck_enable(struct udevice *dev, int op, bool *enable)
>  
>  		return 0;
>  	} else if (op == PMIC_OP_SET) {
[..]