> > +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 Thanks to your pointing out. I checked the history and found the legacy "fsl,codec-mqs" is not used anymore. I will drop it and also "fsl,imx8qxp-mqs" is completed which uses reg. Regards, Chancel Liu