Re: [PATCH v6 2/5] dt-bindings: media: Add qcom,x1e80100-camss

[Vladimir Zapolskiy]

Krzysztof,

On 4/24/25 13:07, Krzysztof Kozlowski wrote:
> On 24/04/2025 11:34, Bryan O'Donoghue wrote:
> > On 24/04/2025 07:40, Krzysztof Kozlowski wrote:
> > > > +  vdd-csiphy-0p8-supply:
> > > Same comment as other series on the lists - this is wrong name. There
> > > are no pins named like this and all existing bindings use different name.
> >
> > The existing bindings are unfortunately not granular enough.
> >
> > I'll post s series to capture pin-names per the SoC pinout shortly.
> How are the pins/supplies actually called?

concerning it I would appreciate if you can review/comment the v2 of SM8650
CAMSS dt bindings I've just sent recently [1], the equivalent property names on
that platform were named "vdda-csiXY-0p9-supply" for VDD_A_CSI_X_Y_0P9 pin.

I believe both these platforms and likely the following ones should provide
similar properties, thus it makes sense to discuss it at the same time.

[1] https://lore.kernel.org/linux-arm-msm/20250423221954.1926453-2-vladimir.zapolskiy@xxxxxxxxxx/

--
Best wishes,
Vladimir