Re: [PATCH v10 1/7] dt-bindings: media: qcom,x1e80100-camss: Add simple-mfd compatible

[Krzysztof Kozlowski <krzk@kernel.org>]

On 17/03/2026 17:21, Bryan O'Donoghue wrote:
> On 17/03/2026 16:00, Krzysztof Kozlowski wrote:
>>>> I actually got the idea from:
>>>>
>>>> commit fe6a952b567f6a771d087d2e969914f31574d6ab
>>>> Author: Krzysztof Kozlowski<krzk@kernel.org>
>>>> Date:   Fri Jan 27 20:40:50 2023 +0100
>>> If you refer to my commits, at least read them. Do you see children
>>> there? Yes. You have none.
>>>
>>> Plus trying to get 13 year old legacy device without dtbs_check warnings
>>> is different thing than correcting a recently introduced bindings.
>>> Terrible example to take... And if you find example from 20 years ago,
>>> it will be even worse.
>>>
>> One more thing, that commit was still correct because it fulfilled the
>> simple-mfd requirement of lack of dependency on the parent. I do not
>> believe your code fulfills it. I am pretty sure it does not.
> 
> Yeah I take the logic of that point:
> 
> CSIPHY here depends on the GDSC, power-domains and clocks we've listed, 
> its not being supplied by any of the other blocks that live inside of 
> the rest.
> 
> They don't require any resources from the parent and the parent consumes 
> the child phys via phandles.
> 
> To my mind that fits the criteria you've set of !depends on parent.

OK, understood, although I have doubts though, because you have also
interconnects and iommus only in the parent.

Regardless of that, even if this fits simple-mfd there is simply no
benefits of doing that way and your driver should just populate children.

Best regards,
Krzysztof