On 2/4/2026 6:29 AM, Dmitry Baryshkov wrote:
> The OPP table for the Iris core is wrong, it copies the VDD table from
> the downstream kernel, but that table is written for the
> video_cc_mvs0_clk_src, while the upstream uses video_cc_mvs0_clk for OPP
> rate setting (which is clk_src divided by 3). Specify correct
> frequencies in the OPP table.
>
> Fixes: fa245b3f06cd ("arm64: dts: qcom: sm8250: Add venus DT node")
> Reported-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/sm8250.dtsi | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
> index d8be5c1add1b..7d11f1a28546 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
> @@ -4350,26 +4350,26 @@ venus: video-codec@aa00000 {
> venus_opp_table: opp-table {
> compatible = "operating-points-v2";
>
> - opp-720000000 {
> - opp-hz = /bits/ 64 <720000000>;
> + opp-240000000 {
> + opp-hz = /bits/ 64 <240000000>;
> required-opps = <&rpmhpd_opp_svs>,
> <&rpmhpd_opp_low_svs>;
> };
>
> - opp-1014000000 {
> - opp-hz = /bits/ 64 <1014000000>;
> + opp-338000000 {
> + opp-hz = /bits/ 64 <338000000>;
> required-opps = <&rpmhpd_opp_svs>,
> <&rpmhpd_opp_svs>;
> };
>
> - opp-1098000000 {
> - opp-hz = /bits/ 64 <1098000000>;
> + opp-366000000 {
> + opp-hz = /bits/ 64 <366000000>;
> required-opps = <&rpmhpd_opp_svs_l1>,
> <&rpmhpd_opp_svs_l1>;
> };
>
> - opp-1332000000 {
> - opp-hz = /bits/ 64 <1332000000>;
> + opp-444000000 {
> + opp-hz = /bits/ 64 <444000000>;
> required-opps = <&rpmhpd_opp_svs_l1>,
> <&rpmhpd_opp_nom>;
> };
>
Reviewed-by: Dikshita Agarwal <dikshita.agarwal@oss.qualcomm.com>
Thanks,
Dikshita