There is no power domain defined on i.MX952, so make power-domains to be
dedicated to i.MX8QM.
Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com>
---
.../devicetree/bindings/sound/fsl,audmix.yaml | 16 ++++++++++++++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl,audmix.yaml b/Documentation/devicetree/bindings/sound/fsl,audmix.yaml
index 3ad197b3c82c..07b9a38761f2 100644
--- a/Documentation/devicetree/bindings/sound/fsl,audmix.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,audmix.yaml
@@ -34,7 +34,9 @@ description: |
properties:
compatible:
- const: fsl,imx8qm-audmix
+ enum:
+ - fsl,imx8qm-audmix
+ - fsl,imx952-audmix
reg:
maxItems: 1
@@ -80,7 +82,17 @@ required:
- reg
- clocks
- clock-names
- - power-domains
+
+allOf:
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - fsl,imx8qm-audmix
+ then:
+ required:
+ - power-domains
unevaluatedProperties: false
--
2.34.1
On Tue, Jan 20, 2026 at 11:52:07AM +0800, Shengjiu Wang wrote: > There is no power domain defined on i.MX952, so make power-domains to be > dedicated to i.MX8QM. > > Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> > --- > .../devicetree/bindings/sound/fsl,audmix.yaml | 16 ++++++++++++++-- > 1 file changed, 14 insertions(+), 2 deletions(-) > With changes to commit msg: Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> Best regards, Krzysztof
On Tue, Jan 20, 2026 at 11:52:07AM +0800, Shengjiu Wang wrote: > There is no power domain defined on i.MX952, so make power-domains to be There is no defined or there is no power domain? If the first, then this patch is incomplete. Please read writing bindings part about complete bindings. If the latter, then you miss constraints ":false" and commit msg phrasing is incorrect (and remember that in such case you won't be able to add power domains later because now you add complete binding). > dedicated to i.MX8QM. > > Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> > --- Best regards, Krzysztof
On Tue, Jan 20, 2026 at 6:31 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On Tue, Jan 20, 2026 at 11:52:07AM +0800, Shengjiu Wang wrote: > > There is no power domain defined on i.MX952, so make power-domains to be > > There is no defined or there is no power domain? If the first, then this > patch is incomplete. Please read writing bindings part about complete > bindings. If the latter, then you miss constraints ":false" and commit > msg phrasing is incorrect (and remember that in such case you won't be > able to add power domains later because now you add complete binding). Thanks for pointing this out. There is a power domain on i.MX952 for the mix system of AUDMIX. But it is enabled by default, AUDMIX device don't need to enable it. So can we make the power-domains to be optional for this case? Best regards Shengjiu Wang > > > dedicated to i.MX8QM. > > > > Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com> > > --- > > Best regards, > Krzysztof >
On Tue, Jan 20, 2026 at 07:07:48PM +0800, Shengjiu Wang wrote: > On Tue, Jan 20, 2026 at 6:31 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > On Tue, Jan 20, 2026 at 11:52:07AM +0800, Shengjiu Wang wrote: > > > There is no power domain defined on i.MX952, so make power-domains to be > > > > There is no defined or there is no power domain? If the first, then this > > patch is incomplete. Please read writing bindings part about complete > > bindings. If the latter, then you miss constraints ":false" and commit > > msg phrasing is incorrect (and remember that in such case you won't be > > able to add power domains later because now you add complete binding). > > Thanks for pointing this out. > > There is a power domain on i.MX952 for the mix system of AUDMIX. > But it is enabled by default, AUDMIX device don't need to enable it. This should be explained in the commit msg. Best regards, Krzysztof
© 2016 - 2026 Red Hat, Inc.