linux-kernel - Re: [PATCH 3/7] soc: qcom: rpmpd: Hook up VDDMX as parent of SM6375 VDDGX

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <d3c362ce-48bd-696f-d7a0-45890695facd@linaro.org>
Date:   Wed, 25 Jan 2023 11:07:18 +0200
From:   Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To:     Konrad Dybcio <konrad.dybcio@...aro.org>,
        linux-arm-msm@...r.kernel.org, andersson@...nel.org,
        agross@...nel.org, krzysztof.kozlowski@...aro.org
Cc:     marijn.suijten@...ainline.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/7] soc: qcom: rpmpd: Hook up VDDMX as parent of SM6375
 VDDGX

On 25/01/2023 05:27, Konrad Dybcio wrote:
> The GPU core clock requires that both VDDGX and VDDMX domains are scaled
> at the same rate at the same time (well, MX just before GX but you get
> the idea). Set MX as parent of GX to take care of that.
> 
> Suggested-by: Bjorn Andersson <andersson@...nel.org>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
> ---
>   drivers/soc/qcom/rpmpd.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/soc/qcom/rpmpd.c b/drivers/soc/qcom/rpmpd.c
> index 9a7da7cead9e..a69182618b76 100644
> --- a/drivers/soc/qcom/rpmpd.c
> +++ b/drivers/soc/qcom/rpmpd.c
> @@ -1027,6 +1027,7 @@ static struct rpmpd sm6375_vddgx_ao;
>   static struct rpmpd sm6375_vddgx = {
>   	.pd = {	.name = "vddgx", },
>   	.peer = &sm6375_vddgx_ao,
> +	.parent = &sm6125_vddmx.pd,
>   	.res_type = RPMPD_RWGX,
>   	.res_id = 0,
>   	.key = KEY_LEVEL,
> @@ -1035,6 +1036,7 @@ static struct rpmpd sm6375_vddgx = {
>   static struct rpmpd sm6375_vddgx_ao = {
>   	.pd = {	.name = "vddgx_ao", },
>   	.peer = &sm6375_vddgx,
> +	.parent = &sm6125_vddmx_ao.pd,
>   	.active_only = true,
>   	.res_type = RPMPD_RWGX,
>   	.res_id = 0,

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>

-- 
With best wishes
Dmitry