On 28/03/2024 04:59, Manivannan Sadhasivam wrote: >> - if: >> properties: >> compatible: >> @@ -250,7 +276,7 @@ allOf: >> reg: >> maxItems: 1 >> clocks: >> - minItems: 8 >> + minItems: 7 >> maxItems: 8 >> else: >> properties: >> @@ -258,7 +284,7 @@ allOf: >> minItems: 1 >> maxItems: 2 >> clocks: >> - minItems: 8 >> + minItems: 7 > > I'm getting confused by the clock requirements for qcom,ice. Why does specifying > the qcom,ice phandle require these clocks? These are the UFSHC clocks and > already defined above. I am also confused, but I did not change that logic. I don't think that it is anyway useful, but that separate topic from this patch. Best regards, Krzysztof