On Fri, Jan 16, 2026 at 02:38:56PM +0100, Luca Weiss wrote: > Add Milos compatible for the CAMSS CCI interfaces. > > Signed-off-by: Luca Weiss <[email protected]> > --- > .../devicetree/bindings/i2c/qcom,i2c-cci.yaml | 18 > ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > index a3fe1eea6aec..c57d81258fba 100644 > --- a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > +++ b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > @@ -27,6 +27,7 @@ properties: > - items: > - enum: > - qcom,kaanapali-cci > + - qcom,milos-cci > - qcom,qcm2290-cci > - qcom,sa8775p-cci > - qcom,sc7280-cci > @@ -263,6 +264,23 @@ allOf: > - const: cpas_ahb > - const: cci > > + - if: > + properties: > + compatible: > + contains: > + enum: > + - qcom,milos-cci > + then: > + properties: > + clocks: > + minItems: 3 > + maxItems: 3 > + clock-names: > + items: > + - const: soc_ahb > + - const: cpas_ahb > + - const: cci
Same comments as other discussion these days - I guess that soc_ahb serves the same purpose as camnoc_axi, so this is just last entri in the if:then: blocks. I really find this binding terrible - around six names for AHB - so I do not want another combination... Best regards, Krzysztof
