Re: [PATCH v5 1/3] dt-bindings: media: qcom: Add JPEG encoder binding

[Gjorgji Rosikopulos (Consultant)]

Hi Atanas,

On 7/28/2026 12:28 PM, Atanas Filipov wrote:
> On 7/28/2026 12:12 PM, Bryan O'Donoghue wrote:
>> On 28/07/2026 09:25, Atanas Filipov wrote:
>>> +
>>> +        power-domains = <&camcc TITAN_TOP_GDSC>;
>>> +
>>
>> I'd expect to see MMCX and MXC in this list.
>>
>> ---
>> bod
> 
> The MMCX vote is already covered at the clock controller level.
> The camcc node in sm8250.dtsi declares:
> 
>     power-domains = <&rpmhpd RPMHPD_MMCX>;
> 
> TITAN_TOP_GDSC is a child power domain of camcc, so when the JPEG
> encoder enables it the framework walks the parent chain and votes
> for MMCX through camcc automatically.
> 
> This is consistent with cci0, cci1, and camss in sm8250.dtsi --
> all use only TITAN_TOP_GDSC with no direct MMCX/MX entry.

This was commented in v4. Bryan asks JPEG node to explicitly list its full power-domain dependencies.
Please add MMCX and MXC to the power domain list. I hope that will be enough.

Regards,
~Gjorgji