Re: [PATCH v4 0/5] Add Qualcomm JPEG V4L2 encoder for SM8250

[Vladimir Zapolskiy]

On 7/6/26 13:12, Bryan O'Donoghue wrote:
> On 06/07/2026 08:11, Atanas Filipov wrote:
> > Note: The handling of shared camera subsystem resources (power domains,
> > interconnects) for child IP blocks is still an open design question.
>
> Why ?
>
> A device needs to vote on its own interconnect and power-domains on any
> bus. A sub-device of another device may wish to ramp a clock for
> whatever reason.

Certainly a CAMSS device will vote on all needed to it resources, some of
which are shared and got their description under CAMSS device tree node.

> There is no "master" device in this block of devices - save perhaps for
> the CSID mux / wrappers on some of these parts.
>
> We have shared resources like camera noc, system noc and external clocks.
>
> Please include power-domains and interconnects.

Why? The common power domain and interconnects have already been
described as resources of the parent CAMSS device, there is no need
to duplicate descriptions in every child device tree node of CAMSS.

--
Best wishes,
Vladimir