Re: [PATCH 1/6] media: dt-bindings: media: camss: Add qcom,sc7180-camss binding

Bryan O'Donoghue
2024-06-23T11:10:09Z

On 22/06/2024 16:31, george chan wrote:
> On Fri, Jun 21, 2024 at 7:24 PM Bryan O'Donoghue
> <bryan.odonoghue@linaro.org> wrote:
>>
>> On 21/06/2024 10:40, George Chan via B4 Relay wrote:
>>> + power-domains:
>>> + items:
>>> + - description: IFE0 GDSC - Image Front End, Global Distributed Switch Controller.
>>> + - description: IFE1 GDSC - Image Front End, Global Distributed Switch Controller.
>>> + - description: Titan GDSC - Titan ISP Block, Global Distributed Switch Controller.
>>
>> Please name these power-domains and require them in your yaml,
>> remembering to add them into the DTS.
>
> I can borrow the idea from sdm845-venus-v2
>
> @@ -191,6 +190,7 @@ required:
> - interrupts
> - iommus
> - power-domains
> + - power-domain-names
> - reg
> ...
> + power-domain-names = "ife0",
> + "ife1",
> + "top";

Something like this

https://git.codelinaro.org/bryan.odonoghue/kernel/-/commit/7626cd2a2a509832c214e538827b91c5dbf1bf09

---
bod