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

Message-ID: <cee197ea-7403-4c1a-beb2-dd1fc9404cd5@quicinc.com>
Date: Fri, 4 Jul 2025 15:50:41 +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 7/4/2025 3:21 PM, Renjiang Han wrote:
>
> 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.
>
This patch has already been picked up, please ignore last email.

-- 
Best Regards,
Renjiang