On 28/10/2025 09:46, Krzysztof Kozlowski wrote:
+ power-domain-names:
+ items:
+ - const: ife0
+ - const: ife1
+ - const: ife2
+ - const: top
Uh, not your fault, but who came with this list in previous generations?
Instead of simple and obvious "top+ifeX" which allows growing/shrinking,
someone put "top" at the end which means this cannot follow same order
as X1E for example... Heh, it follows at least sm8550.
Shall we put top as first power-domain? I don't think it's an issue to
change the order.
Well, it matches sm8550, so I am just grumpy complaining. It's fine.
The provenance here is "top" was required to be added last because the
code depended on magic indexing in dtb to know which was the TOP GDSC.
But since power-domain names are now required, you can have any order
you want.
+
+ vdda-0.9-supply:
There are no dots in property names. Are you sure these are called
VDDA_0.9 in the device datasheet (not schematics)? Please look at other
bindings how this is being named, depending whether this is PHY or PLL
supply (or only PHY).
The following power supplies are mentioned:
* VDD_CAMSS_PLL_0P9 - Camera SS PLL 0.9 V circuits
(not referenced in downstream kernel, connected to vreg_s5a in
schematics)
So that's vdda-pll-supply
* VDD_A_CSI_x_0P9 - MIPI CSIx 0.9 V circuits
With pad names VDD_A_CSI_0_0P9 to VDD_A_CSI_3_0P9
vdd-csiphy-0p9-supply
* VDD_A_CSI_x_1P25 - MIPI CSIx 1.25 V circuits
With pad names VDD_A_CSI_0_1P25 to VDD_A_CSI_3_1P25
vdd-csiphy-1p25-supply
@Luca
VDD_A_CSI_X_0P9
VDD_A_CSI_X_1P25
Agree see this on schematics.
and look indeed there it is VDD_CAMSS_PLL_0P9
but if you look at where that rail comes from its SM_VDD_MX
So I believe the MX power-domain should cover this instead of having a
new separate rail defined in CAMSS for this.
---
bod
