On 20/11/2023 09:49, Chancel Liu wrote: > +allOf: > + - if: > + properties: > + compatible: > + contains: > + const: fsl,imx8qm-mqs > + then: > + properties: > + clocks: > + items: > + - description: Master clock > + - description: Clock for register access > + clock-names: > + items: > + - const: mclk > + - const: core > + required: > + - reg > + - power-domains > + else: > + properties: > + clocks: > + items: > + - description: Master clock > + clock-names: > + items: > + - const: mclk > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - fsl,imx6sx-mqs > + - fsl,imx93-mqs > + then: > + required: > + - gpr This is a bit confusing. Only "fsl,imx8qm-mqs" requires (uses?) reg. imx6sx and imx93 use gpr. Then what do the other variants use? Best regards, Krzysztof