.../bindings/clock/qcom,sm8250-videocc.yaml | 85 ++++++++++++++++++++++ .../devicetree/bindings/clock/qcom,videocc.yaml | 20 ----- .../bindings/media/qcom,sm8250-venus.yaml | 10 +-- arch/arm64/boot/dts/qcom/sm8250.dtsi | 42 +++++++---- .../media/platform/qcom/iris/iris_platform_gen1.c | 2 +- drivers/media/platform/qcom/iris/iris_probe.c | 7 ++ drivers/media/platform/qcom/venus/core.c | 7 +- drivers/media/platform/qcom/venus/core.h | 1 + drivers/media/platform/qcom/venus/pm_helpers.c | 8 +- include/linux/pm_domain.h | 4 +- 10 files changed, 140 insertions(+), 46 deletions(-)
As pointed out by Konrad during the review of SM8350 / SC8280XP
patchset, Iris aka Venus description has several flows. It doesn't scale
MMCX, the frequencies in the OPP table are wrong, etc.
Let's correct the Iris/Venus enablement for SM8250 (unfortunately also
stopping it from being overclocked).
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
Changes in v2:
- Fixed example in the new sm8250-videocc schema
- Link to v1: https://lore.kernel.org/r/20260131-iris-venus-fix-sm8250-v1-0-b635ee66284c@oss.qualcomm.com
---
Dmitry Baryshkov (8):
dt-bindings: clock: qcom,sm8250-videocc: account for the MX domain
pmdomain: de-constify fields struct dev_pm_domain_attach_data
media: dt-bindings: qcom,sm8250-venus: sort out power domains
media: iris: scale MMCX power domain on SM8250
media: venus: scale MMCX power domain on SM8250
arm64: dts: qcom: sm8250: add MX power domain to the video CC
arm64: dts: qcom: sort out Iris power domains
arm64: dts: qcom: sm8250: correct frequencies in the Iris OPP table
.../bindings/clock/qcom,sm8250-videocc.yaml | 85 ++++++++++++++++++++++
.../devicetree/bindings/clock/qcom,videocc.yaml | 20 -----
.../bindings/media/qcom,sm8250-venus.yaml | 10 +--
arch/arm64/boot/dts/qcom/sm8250.dtsi | 42 +++++++----
.../media/platform/qcom/iris/iris_platform_gen1.c | 2 +-
drivers/media/platform/qcom/iris/iris_probe.c | 7 ++
drivers/media/platform/qcom/venus/core.c | 7 +-
drivers/media/platform/qcom/venus/core.h | 1 +
drivers/media/platform/qcom/venus/pm_helpers.c | 8 +-
include/linux/pm_domain.h | 4 +-
10 files changed, 140 insertions(+), 46 deletions(-)
---
base-commit: 44ef70faf71468e0ae4bdb782a6d43f0614b8ffa
change-id: 20260131-iris-venus-fix-sm8250-f938e29e7497
Best regards,
--
With best wishes
Dmitry
On Sun, 1 Feb 2026 at 11:49, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > As pointed out by Konrad during the review of SM8350 / SC8280XP > patchset, Iris aka Venus description has several flows. It doesn't scale > MMCX, the frequencies in the OPP table are wrong, etc. > > Let's correct the Iris/Venus enablement for SM8250 (unfortunately also > stopping it from being overclocked). > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Do you want me to pick up patch2 for v7.0 - or what do you suggest at this point? Kind regards Uffe > --- > Changes in v2: > - Fixed example in the new sm8250-videocc schema > - Link to v1: https://lore.kernel.org/r/20260131-iris-venus-fix-sm8250-v1-0-b635ee66284c@oss.qualcomm.com > > --- > Dmitry Baryshkov (8): > dt-bindings: clock: qcom,sm8250-videocc: account for the MX domain > pmdomain: de-constify fields struct dev_pm_domain_attach_data > media: dt-bindings: qcom,sm8250-venus: sort out power domains > media: iris: scale MMCX power domain on SM8250 > media: venus: scale MMCX power domain on SM8250 > arm64: dts: qcom: sm8250: add MX power domain to the video CC > arm64: dts: qcom: sort out Iris power domains > arm64: dts: qcom: sm8250: correct frequencies in the Iris OPP table > > .../bindings/clock/qcom,sm8250-videocc.yaml | 85 ++++++++++++++++++++++ > .../devicetree/bindings/clock/qcom,videocc.yaml | 20 ----- > .../bindings/media/qcom,sm8250-venus.yaml | 10 +-- > arch/arm64/boot/dts/qcom/sm8250.dtsi | 42 +++++++---- > .../media/platform/qcom/iris/iris_platform_gen1.c | 2 +- > drivers/media/platform/qcom/iris/iris_probe.c | 7 ++ > drivers/media/platform/qcom/venus/core.c | 7 +- > drivers/media/platform/qcom/venus/core.h | 1 + > drivers/media/platform/qcom/venus/pm_helpers.c | 8 +- > include/linux/pm_domain.h | 4 +- > 10 files changed, 140 insertions(+), 46 deletions(-) > --- > base-commit: 44ef70faf71468e0ae4bdb782a6d43f0614b8ffa > change-id: 20260131-iris-venus-fix-sm8250-f938e29e7497 > > Best regards, > -- > With best wishes > Dmitry >
On Mon, 2 Feb 2026 at 16:46, Ulf Hansson <ulf.hansson@linaro.org> wrote: > > On Sun, 1 Feb 2026 at 11:49, Dmitry Baryshkov > <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > As pointed out by Konrad during the review of SM8350 / SC8280XP > > patchset, Iris aka Venus description has several flows. It doesn't scale > > MMCX, the frequencies in the OPP table are wrong, etc. > > > > Let's correct the Iris/Venus enablement for SM8250 (unfortunately also > > stopping it from being overclocked). > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > Do you want me to pick up patch2 for v7.0 - or what do you suggest at > this point? I'd prefer it if it is picked through the immutable tag/branch. > > Kind regards > Uffe > > > > --- > > Changes in v2: > > - Fixed example in the new sm8250-videocc schema > > - Link to v1: https://lore.kernel.org/r/20260131-iris-venus-fix-sm8250-v1-0-b635ee66284c@oss.qualcomm.com > > > > --- > > Dmitry Baryshkov (8): > > dt-bindings: clock: qcom,sm8250-videocc: account for the MX domain > > pmdomain: de-constify fields struct dev_pm_domain_attach_data > > media: dt-bindings: qcom,sm8250-venus: sort out power domains > > media: iris: scale MMCX power domain on SM8250 > > media: venus: scale MMCX power domain on SM8250 > > arm64: dts: qcom: sm8250: add MX power domain to the video CC > > arm64: dts: qcom: sort out Iris power domains > > arm64: dts: qcom: sm8250: correct frequencies in the Iris OPP table > > > > .../bindings/clock/qcom,sm8250-videocc.yaml | 85 ++++++++++++++++++++++ > > .../devicetree/bindings/clock/qcom,videocc.yaml | 20 ----- > > .../bindings/media/qcom,sm8250-venus.yaml | 10 +-- > > arch/arm64/boot/dts/qcom/sm8250.dtsi | 42 +++++++---- > > .../media/platform/qcom/iris/iris_platform_gen1.c | 2 +- > > drivers/media/platform/qcom/iris/iris_probe.c | 7 ++ > > drivers/media/platform/qcom/venus/core.c | 7 +- > > drivers/media/platform/qcom/venus/core.h | 1 + > > drivers/media/platform/qcom/venus/pm_helpers.c | 8 +- > > include/linux/pm_domain.h | 4 +- > > 10 files changed, 140 insertions(+), 46 deletions(-) > > --- > > base-commit: 44ef70faf71468e0ae4bdb782a6d43f0614b8ffa > > change-id: 20260131-iris-venus-fix-sm8250-f938e29e7497 > > > > Best regards, > > -- > > With best wishes > > Dmitry > > -- With best wishes Dmitry
On Mon, Feb 02, 2026 at 05:51:16PM +0200, Dmitry Baryshkov wrote: > On Mon, 2 Feb 2026 at 16:46, Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > On Sun, 1 Feb 2026 at 11:49, Dmitry Baryshkov > > <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > As pointed out by Konrad during the review of SM8350 / SC8280XP > > > patchset, Iris aka Venus description has several flows. It doesn't scale > > > MMCX, the frequencies in the OPP table are wrong, etc. > > > > > > Let's correct the Iris/Venus enablement for SM8250 (unfortunately also > > > stopping it from being overclocked). > > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > > > Do you want me to pick up patch2 for v7.0 - or what do you suggest at > > this point? > > I'd prefer it if it is picked through the immutable tag/branch. Another option would be to ack merging it through the media tree. -- With best wishes Dmitry
On Mon, 2 Feb 2026 at 20:22, Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > On Mon, Feb 02, 2026 at 05:51:16PM +0200, Dmitry Baryshkov wrote: > > On Mon, 2 Feb 2026 at 16:46, Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > On Sun, 1 Feb 2026 at 11:49, Dmitry Baryshkov > > > <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > As pointed out by Konrad during the review of SM8350 / SC8280XP > > > > patchset, Iris aka Venus description has several flows. It doesn't scale > > > > MMCX, the frequencies in the OPP table are wrong, etc. > > > > > > > > Let's correct the Iris/Venus enablement for SM8250 (unfortunately also > > > > stopping it from being overclocked). > > > > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > > > > > Do you want me to pick up patch2 for v7.0 - or what do you suggest at > > > this point? > > > > I'd prefer it if it is picked through the immutable tag/branch. > > Another option would be to ack merging it through the media tree. I didn't think the series was considered as material for v7.0 as it's getting late for that. Anyway, I have queued patch2 to v7.0 and the patch is also available to pull in via an immutable "core" branch in my pmdomain tree. Kind regards Uffe
On Tue, Feb 03, 2026 at 11:40:45AM +0100, Ulf Hansson wrote: > On Mon, 2 Feb 2026 at 20:22, Dmitry Baryshkov > <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > On Mon, Feb 02, 2026 at 05:51:16PM +0200, Dmitry Baryshkov wrote: > > > On Mon, 2 Feb 2026 at 16:46, Ulf Hansson <ulf.hansson@linaro.org> wrote: > > > > > > > > On Sun, 1 Feb 2026 at 11:49, Dmitry Baryshkov > > > > <dmitry.baryshkov@oss.qualcomm.com> wrote: > > > > > > > > > > As pointed out by Konrad during the review of SM8350 / SC8280XP > > > > > patchset, Iris aka Venus description has several flows. It doesn't scale > > > > > MMCX, the frequencies in the OPP table are wrong, etc. > > > > > > > > > > Let's correct the Iris/Venus enablement for SM8250 (unfortunately also > > > > > stopping it from being overclocked). > > > > > > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > > > > > > > Do you want me to pick up patch2 for v7.0 - or what do you suggest at > > > > this point? > > > > > > I'd prefer it if it is picked through the immutable tag/branch. > > > > Another option would be to ack merging it through the media tree. > > I didn't think the series was considered as material for v7.0 as it's > getting late for that. > > Anyway, I have queued patch2 to v7.0 and the patch is also available > to pull in via an immutable "core" branch in my pmdomain tree. I think I mixed 7.0 vs 6.20 vs anything. Thanks anyway! -- With best wishes Dmitry
© 2016 - 2026 Red Hat, Inc.