* Re: [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: Add imx8qxp support [not found] ` <20240422-channel-emission-d485dee6ae48@spud> @ 2024-04-23 6:26 ` Alexander Stein 2024-04-23 17:25 ` Conor Dooley 0 siblings, 1 reply; 2+ messages in thread From: Alexander Stein @ 2024-04-23 6:26 UTC (permalink / raw) To: Conor Dooley Cc: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Lucas Stach, linux-kernel, devicetree, imx Hi Conor, cc'ed imx@lists.linux.dev Am Montag, 22. April 2024, 17:48:20 CEST schrieb Conor Dooley: > On Mon, Apr 22, 2024 at 08:49:49AM +0200, Alexander Stein wrote: > > Some SoC like i.MX8QXP use a power-domain for this IP. Add a SoC-specific > > compatible, which also requires a power-domain. > > > > Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com> > > --- > > Thanks for the feedback. > > > > Changes in v3: > > * Move 'allOf' before 'additionalProperties' > > * Remove superfluous 'minItems: 1' > > > > .../interrupt-controller/fsl,irqsteer.yaml | 17 ++++++++++++++++- > > 1 file changed, 16 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml b/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml > > index 20ad4ad82ad64..3d33b75d6ecfa 100644 > > --- a/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml > > +++ b/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml > > @@ -14,7 +14,9 @@ properties: > > oneOf: > > - const: fsl,imx-irqsteer > > - items: > > - - const: fsl,imx8m-irqsteer > > + - enum: > > + - fsl,imx8m-irqsteer > > + - fsl,imx8qxp-irqsteer > > - const: fsl,imx-irqsteer > > > > reg: > > @@ -42,6 +44,9 @@ properties: > > clock-names: > > const: ipg > > > > + power-domains: > > + maxItems: 1 > > + > > interrupt-controller: true > > > > "#interrupt-cells": > > @@ -70,6 +75,16 @@ required: > > - fsl,channel > > - fsl,num-irqs > > > > +allOf: > > + - if: > > + properties: > > + compatible: > > + contains: > > + const: fsl,imx8qxp-irqsteer > > + then: > > + required: > > + - power-domains > > Provided the power domains are optional on the existing platforms, > Acked-by: Conor Dooley <conor.dooley@microchip.com> > otherwise, > else: > properties: > power-domains: false A power-domain is used on imx8mp, but imx8mq does not. Best regards, Alexander -- TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018 Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider http://www.tq-group.com/ ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: Add imx8qxp support 2024-04-23 6:26 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: Add imx8qxp support Alexander Stein @ 2024-04-23 17:25 ` Conor Dooley 0 siblings, 0 replies; 2+ messages in thread From: Conor Dooley @ 2024-04-23 17:25 UTC (permalink / raw) To: Alexander Stein Cc: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Lucas Stach, linux-kernel, devicetree, imx [-- Attachment #1: Type: text/plain, Size: 2449 bytes --] On Tue, Apr 23, 2024 at 08:26:55AM +0200, Alexander Stein wrote: > Hi Conor, > > cc'ed imx@lists.linux.dev > > Am Montag, 22. April 2024, 17:48:20 CEST schrieb Conor Dooley: > > On Mon, Apr 22, 2024 at 08:49:49AM +0200, Alexander Stein wrote: > > > Some SoC like i.MX8QXP use a power-domain for this IP. Add a SoC-specific > > > compatible, which also requires a power-domain. > > > > > > Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com> > > > --- > > > Thanks for the feedback. > > > > > > Changes in v3: > > > * Move 'allOf' before 'additionalProperties' > > > * Remove superfluous 'minItems: 1' > > > > > > .../interrupt-controller/fsl,irqsteer.yaml | 17 ++++++++++++++++- > > > 1 file changed, 16 insertions(+), 1 deletion(-) > > > > > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml b/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml > > > index 20ad4ad82ad64..3d33b75d6ecfa 100644 > > > --- a/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml > > > +++ b/Documentation/devicetree/bindings/interrupt-controller/fsl,irqsteer.yaml > > > @@ -14,7 +14,9 @@ properties: > > > oneOf: > > > - const: fsl,imx-irqsteer > > > - items: > > > - - const: fsl,imx8m-irqsteer > > > + - enum: > > > + - fsl,imx8m-irqsteer > > > + - fsl,imx8qxp-irqsteer > > > - const: fsl,imx-irqsteer > > > > > > reg: > > > @@ -42,6 +44,9 @@ properties: > > > clock-names: > > > const: ipg > > > > > > + power-domains: > > > + maxItems: 1 > > > + > > > interrupt-controller: true > > > > > > "#interrupt-cells": > > > @@ -70,6 +75,16 @@ required: > > > - fsl,channel > > > - fsl,num-irqs > > > > > > +allOf: > > > + - if: > > > + properties: > > > + compatible: > > > + contains: > > > + const: fsl,imx8qxp-irqsteer > > > + then: > > > + required: > > > + - power-domains > > > > Provided the power domains are optional on the existing platforms, > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > > otherwise, > > else: > > properties: > > power-domains: false > > A power-domain is used on imx8mp, but imx8mq does not. So then make the contains be an enum with both of the one supporting it & add the else please. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-04-23 17:25 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20240422064949.70778-1-alexander.stein@ew.tq-group.com>
[not found] ` <20240422-channel-emission-d485dee6ae48@spud>
2024-04-23 6:26 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: Add imx8qxp support Alexander Stein
2024-04-23 17:25 ` Conor Dooley
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox