Hi Michael,
On 1/14/26 16:11, Michael Walle wrote:
Hi,
On Thu Jan 8, 2026 at 9:51 AM CET, Krzysztof Kozlowski wrote:
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: ti,am62p-dss
+ then:
+ properties:
+ power-domains:
+ minItems: 1
+ maxItems: 3
This is still not constrained enough. You need to define the items
instead. I still do not understand why number of power domains is
flexible.
So looking at the downstream devicetree, there is one power domain
for each OLDI and for the DSS itself. Thus, in the am62p case, there
are two DSS as described above, so DSS0 has a power domain for dss0
and two power domains for the OLDI transmitters. The same for dss1
but with just one OLDI transmitter.
So I don't know why there is minItems: 1 because it's either 2 or 3.
One can use DSS power-domain only, if they don't want to use OLDI0 or OLDI1.
What about the following:
..
- if:
properties:
compatible:
contains:
const: ti,am62p-dss
then:
properties:
power-domains:
minItems: 2
items:
- description: DSS controller
- description: OLDI0 transmitter
- description: OLDI1 transmitter
else:
properties:
power-domains:
maxItems: 1
-michael
Using the following:
..
power-domains:
minItems: 1
description:
phandle to the associated power domain(s).
items:
- description: DSS controller power domain
- description: OLDI0 power domain
- description: OLDI1 power domain
And using constraints for "ti,am62p-dss" and other compatibles
separately. Please check v4[1].
[1]: https://lore.kernel.org/all/[email protected]/
Regards,
Swamil.
