linux-kernel - Re: [PATCH v2] media: venus: pm_helpers: use opp-table for the frequency

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <f87b6127-a416-45fe-8b17-bd32574e20a3@quicinc.com>
Date: Fri, 4 Jul 2025 15:21:25 +0800
From: Renjiang Han <quic_renjiang@...cinc.com>
To: Vikash Garodia <quic_vgarodia@...cinc.com>,
        Dikshita Agarwal
	<quic_dikshita@...cinc.com>,
        Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>
CC: <linux-media@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] media: venus: pm_helpers: use opp-table for the
 frequency


On 6/12/2025 10:23 AM, Renjiang Han wrote:
> Some platforms (such as qcs615 and sc7180) use the same core but have
> different frequency tables. Using the opp-table allows us to separate
> the core description from the frequency data and supports the use of
> fallback compatibles.
>
> Reviewed-by: Vikash Garodia<quic_vgarodia@...cinc.com>
> Reviewed-by: Bryan O'Donoghue<bryan.odonoghue@...aro.org>
> Signed-off-by: Renjiang Han<quic_renjiang@...cinc.com>
> ---
> video core in QCS615 is same as that in SC7180, while the clock frequency
> corners are different. In such cases, the clock frequencies are better
> represented in DT than static table in video driver. The patch make use
> of OPP table in DT to choose the frequency corner to scale.
>
> Note:
> Earlier discussion was concluded in [1] where it was agreed to rely on
> opp-table for QCS615.
>
> [1]https://lore.kernel.org/linux-arm-msm/c9b83c8b-68d1-43bc-99d6-d2d2b9e445f4@oss.qualcomm.com/
> ---
> Changes in v2:
> - 1. Update message based on the review comment.
> - Link to v1:https://lore.kernel.org/r/20250610-use_freq_with_opp_table-v1-1-7339d690fd1a@quicinc.com
> ---
>   drivers/media/platform/qcom/venus/pm_helpers.c | 58 +++++++++++++++-----------
>   1 file changed, 34 insertions(+), 24 deletions(-)
I’d appreciate any thoughts you might have on this.

-- 
Best Regards,
Renjiang