Re: [PATCH v5 5/7] media: qcom: camss: Add support for named power-domains

[Bryan O'Donoghue]

On 22/11/2023 19:55, Konrad Dybcio wrote:
> On 11/18/23 13:11, Bryan O'Donoghue wrote:
> > Right now we use fixed indexes to assign power-domains, with a
> > requirement for the TOP GDSC to come last in the list.
> >
> > Adding support for named power-domains means the declaration in the dtsi
> > can come in any order.
> >
> > After this change we continue to support the old indexing - if a SoC
> > resource declaration or the in-use dtb doesn't declare power-domain names
> > we fall back to the default legacy indexing.
> >
> >  From this point on though new SoC additions should contain named
> > power-domains, eventually we will drop support for legacy indexing.
> >
> > Tested-by: Matti Lehtimäki <matti.lehtimaki@xxxxxxxxx>
> > Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
> > ---
> So, this commit should be a NOP within this series?
>
> res->pd_name isn't defined anywhere afaics
>
> Konrad

This series is mergeable though the linux-media tree standalone, yes.

Once merged, the dtsi change given in the cover letter will be submitted.

The sm8250 change is posted here : 
https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v5+sm8250

---
bod