* [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: Add imx8qxp support @ 2024-04-22 6:49 Alexander Stein 2024-04-22 15:48 ` Conor Dooley 2024-04-22 16:38 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: " Frank Li 0 siblings, 2 replies; 6+ messages in thread From: Alexander Stein @ 2024-04-22 6:49 UTC (permalink / raw) To: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Lucas Stach Cc: Alexander Stein, linux-kernel, devicetree 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 + additionalProperties: false examples: -- 2.34.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: Add imx8qxp support 2024-04-22 6:49 [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: Add imx8qxp support Alexander Stein @ 2024-04-22 15:48 ` Conor Dooley 2024-04-23 6:26 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: " Alexander Stein 2024-04-22 16:38 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: " Frank Li 1 sibling, 1 reply; 6+ messages in thread From: Conor Dooley @ 2024-04-22 15:48 UTC (permalink / raw) To: Alexander Stein Cc: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Lucas Stach, linux-kernel, devicetree [-- Attachment #1: Type: text/plain, Size: 1964 bytes --] 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 Cheers, Conor. > + > additionalProperties: false > > examples: > -- > 2.34.1 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: Add imx8qxp support 2024-04-22 15:48 ` Conor Dooley @ 2024-04-23 6:26 ` Alexander Stein 2024-04-23 17:25 ` Conor Dooley 0 siblings, 1 reply; 6+ 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] 6+ 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: " Alexander Stein @ 2024-04-23 17:25 ` Conor Dooley 0 siblings, 0 replies; 6+ 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] 6+ messages in thread
* Re: [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: Add imx8qxp support 2024-04-22 6:49 [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: Add imx8qxp support Alexander Stein 2024-04-22 15:48 ` Conor Dooley @ 2024-04-22 16:38 ` Frank Li 2024-04-23 6:28 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: " Alexander Stein 1 sibling, 1 reply; 6+ messages in thread From: Frank Li @ 2024-04-22 16:38 UTC (permalink / raw) To: Alexander Stein Cc: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Lucas Stach, linux-kernel, devicetree 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> > --- Next time, please cc imx@lists.linux.dev Frank > 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 > + > additionalProperties: false > > examples: > -- > 2.34.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: Add imx8qxp support 2024-04-22 16:38 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: " Frank Li @ 2024-04-23 6:28 ` Alexander Stein 0 siblings, 0 replies; 6+ messages in thread From: Alexander Stein @ 2024-04-23 6:28 UTC (permalink / raw) To: Frank Li Cc: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Lucas Stach, linux-kernel, devicetree Hi Frank, Am Montag, 22. April 2024, 18:38:09 CEST schrieb Frank Li: > 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> > > --- > > Next time, please cc imx@lists.linux.dev I use the addresses from ./scripts/get_maintainer.pl, maybe you want to add that to MAINTAINERS. Thanks and best regards, Alexander > Frank > > > 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 > > + > > additionalProperties: false > > > > examples: > -- 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] 6+ messages in thread
end of thread, other threads:[~2024-04-23 17:25 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-04-22 6:49 [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: Add imx8qxp support Alexander Stein 2024-04-22 15:48 ` Conor Dooley 2024-04-23 6:26 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: " Alexander Stein 2024-04-23 17:25 ` Conor Dooley 2024-04-22 16:38 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl,irqsteer: " Frank Li 2024-04-23 6:28 ` [PATCH v3 1/1] dt-bindings: interrupt-controller: fsl, irqsteer: " Alexander Stein
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox