Re: [PATCH v2 2/2] soc: qcom: rpmh-rsc: Skip read requests on unsupported platforms

[Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>]

On 9/11/26 11:08 AM, Maulik Shah wrote:
> On some platforms AOSS do not respond to read requests. Read request in
> such cases will consume the ACTIVE TCS but forever waits for a response
> blocking the subsequent write requests.
> 
> Skip issuing read commands on sm8150 and sc8180x platforms. For such
> platforms return success to the caller with the resource level at zero
> to avoid the caller taking any action on error code.
> 
> Fixes: edbafe65eef2 ("soc: qcom: rpmh: Add support to read back resource settings")
> Signed-off-by: Maulik Shah <maulik.shah@oss.qualcomm.com>
> ---

This works as an immediate fix, but I'd like to get some
comment on this:

https://lore.kernel.org/all/3058d570-aeb9-4f73-831c-c30626296006@oss.qualcomm.com/


[...]

>  	DECLARE_COMPLETION_ONSTACK(compl);
>  	DEFINE_RPMH_MSG_ONSTACK(dev, RPMH_ACTIVE_ONLY_STATE, &compl, rpm_msg);
> +	struct rpmh_ctrlr *ctrlr = get_rpmh_ctrlr(dev);
>  	int ret;
>  
>  	ret = __fill_rpmh_msg(&rpm_msg, RPMH_ACTIVE_ONLY_STATE, cmd, 1, true);
>  	if (ret)
>  		return ret;
>  
> +	if (ctrlr->no_rpmh_read) {
> +		cmd[0].data = 0;
> +		return 0;
> +	}

This should be -EOPNOTSUPP and consumers should handle it
appropriately

Konrad