Re: [PATCH v1 2/4] arm64: dts: qcom: sm8550: Add JPEG encoder node

[Bryan O'Donoghue <bod@kernel.org>]

On 13/06/2026 17:05, Atanas Filipov wrote:
> Checked.  The MMCX/MXC dependency is covered transitively — camcc
> itself declares:
> 
>     power-domains = <&rpmhpd RPMHPD_MMCX>, <&rpmhpd RPMHPD_MXC>;
> 
> CAM_CC_TITAN_TOP_GDSC is a subdomain of camcc, so the RPMh votes are
> enforced before any consumer is powered on.

Powered on yes, scaling the voltage correctly no.

> This matches the pattern used by every other single-GDSC camera node
> in the upstream tree (cci0/1/2 on sm8250, sm8550, sm8650) — none of
> them list MMCX/MXC directly.  No change needed.

CCI doesn't have any operating points. Please check the clock frequency 
plans.

> afilipov
> 
> On 6/13/2026 2:52 AM, Bryan O'Donoghue wrote:

Honestly. Stop top posting !

Documentation/process/submitting-patches.rst:.. _interleaved_replies:

sm8650 which you have cited, has operating points for the JPEG encoder 
and votes, must vote, on MXC. I believe the same as Hamoa for the JPEG 
core clock.

This is melting my head a bit now TBH.

---
bod