[PATCH] spmi: msm: correct mac_channels for v5 controllers

Neil Armstrong neil.armstrong at linaro.org
Mon Jun 24 10:18:49 CEST 2024


On 24/06/2024 04:23, Caleb Connolly wrote:
> Commit ee1d8aa5ecf7 ("spmi: msm: support controller version 7") broke
> support for channels > 128 on v5 controllers, resulting in some
> peripherals (like the power button / pon) working but others (like
> gpios) reading bogus data.
> 
> Correct max_channels for v5 controllers.
> 
> Fixes: ee1d8aa5ecf7 ("spmi: msm: support controller version 7")
> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
> ---
>   drivers/spmi/spmi-msm.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/spmi/spmi-msm.c b/drivers/spmi/spmi-msm.c
> index 58c676423f40..49aad439648f 100644
> --- a/drivers/spmi/spmi-msm.c
> +++ b/drivers/spmi/spmi-msm.c
> @@ -270,9 +270,9 @@ static int msm_spmi_probe(struct udevice *dev)
>   		priv->max_channels = SPMI_MAX_CHANNELS;
>   	} else if (hw_ver < PMIC_ARB_VERSION_V7_MIN) {
>   		priv->arb_ver = V5;
>   		priv->arb_chnl = core_addr + APID_MAP_OFFSET_V5;
> -		priv->max_channels = SPMI_MAX_CHANNELS;
> +		priv->max_channels = SPMI_MAX_CHANNELS_V5;
>   		priv->spmi_cnfg = dev_read_addr_name(dev, "cnfg");
>   	} else {
>   		/* TOFIX: handle second bus */
>   		priv->arb_ver = V7;

Thanks for spotting this...

Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>

Neil


More information about the U-Boot mailing list