* [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
@ 2023-07-25 10:29 Peng Fan (OSS)
2023-07-25 18:36 ` Conor Dooley
2023-07-26 7:44 ` Krzysztof Kozlowski
0 siblings, 2 replies; 6+ messages in thread
From: Peng Fan (OSS) @ 2023-07-25 10:29 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, conor+dt, ulf.hansson, shawnguo
Cc: devicetree, linux-kernel, linux-pm, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Add an entry dedicated for fsl,scu-pd which could serve i.MX8DXL
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
.../devicetree/bindings/power/fsl,scu-pd.yaml | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
index 407b7cfec783..3a92e4fef7ae 100644
--- a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
+++ b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
@@ -18,11 +18,14 @@ allOf:
properties:
compatible:
- items:
+ oneOf:
- enum:
- - fsl,imx8qm-scu-pd
- - fsl,imx8qxp-scu-pd
- - const: fsl,scu-pd
+ - fsl,scu-pd
+ - items:
+ - enum:
+ - fsl,imx8qm-scu-pd
+ - fsl,imx8qxp-scu-pd
+ - const: fsl,scu-pd
'#power-domain-cells':
const: 1
--
2.37.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
2023-07-25 10:29 [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd Peng Fan (OSS)
@ 2023-07-25 18:36 ` Conor Dooley
2023-07-26 0:21 ` Peng Fan
2023-07-26 7:44 ` Krzysztof Kozlowski
1 sibling, 1 reply; 6+ messages in thread
From: Conor Dooley @ 2023-07-25 18:36 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, ulf.hansson, shawnguo,
devicetree, linux-kernel, linux-pm, Peng Fan
[-- Attachment #1: Type: text/plain, Size: 1251 bytes --]
On Tue, Jul 25, 2023 at 06:29:00PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Add an entry dedicated for fsl,scu-pd which could serve i.MX8DXL
Why not just add a soc-specific compatible for the i.MX8DXL?
The current form of this does not look right to me.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> .../devicetree/bindings/power/fsl,scu-pd.yaml | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> index 407b7cfec783..3a92e4fef7ae 100644
> --- a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> +++ b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> @@ -18,11 +18,14 @@ allOf:
>
> properties:
> compatible:
> - items:
> + oneOf:
> - enum:
> - - fsl,imx8qm-scu-pd
> - - fsl,imx8qxp-scu-pd
> - - const: fsl,scu-pd
> + - fsl,scu-pd
> + - items:
> + - enum:
> + - fsl,imx8qm-scu-pd
> + - fsl,imx8qxp-scu-pd
> + - const: fsl,scu-pd
>
> '#power-domain-cells':
> const: 1
> --
> 2.37.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
2023-07-25 18:36 ` Conor Dooley
@ 2023-07-26 0:21 ` Peng Fan
2023-07-26 7:42 ` Conor Dooley
0 siblings, 1 reply; 6+ messages in thread
From: Peng Fan @ 2023-07-26 0:21 UTC (permalink / raw)
To: Conor Dooley, Peng Fan (OSS)
Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
conor+dt@kernel.org, ulf.hansson@linaro.org, shawnguo@kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pm@vger.kernel.org
> Subject: Re: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
>
> On Tue, Jul 25, 2023 at 06:29:00PM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > Add an entry dedicated for fsl,scu-pd which could serve i.MX8DXL
>
> Why not just add a soc-specific compatible for the i.MX8DXL?
> The current form of this does not look right to me.
I could add one dedicated for i.MX8DXL.
Thanks,
Peng.
>
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> > .../devicetree/bindings/power/fsl,scu-pd.yaml | 11 +++++++----
> > 1 file changed, 7 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> > index 407b7cfec783..3a92e4fef7ae 100644
> > --- a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> > +++ b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> > @@ -18,11 +18,14 @@ allOf:
> >
> > properties:
> > compatible:
> > - items:
> > + oneOf:
> > - enum:
> > - - fsl,imx8qm-scu-pd
> > - - fsl,imx8qxp-scu-pd
> > - - const: fsl,scu-pd
> > + - fsl,scu-pd
> > + - items:
> > + - enum:
> > + - fsl,imx8qm-scu-pd
> > + - fsl,imx8qxp-scu-pd
> > + - const: fsl,scu-pd
> >
> > '#power-domain-cells':
> > const: 1
> > --
> > 2.37.1
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
2023-07-26 0:21 ` Peng Fan
@ 2023-07-26 7:42 ` Conor Dooley
2023-07-26 7:45 ` Peng Fan
0 siblings, 1 reply; 6+ messages in thread
From: Conor Dooley @ 2023-07-26 7:42 UTC (permalink / raw)
To: Peng Fan
Cc: Conor Dooley, Peng Fan (OSS), robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
ulf.hansson@linaro.org, shawnguo@kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pm@vger.kernel.org
[-- Attachment #1: Type: text/plain, Size: 584 bytes --]
On Wed, Jul 26, 2023 at 12:21:10AM +0000, Peng Fan wrote:
> > Subject: Re: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
> >
> > On Tue, Jul 25, 2023 at 06:29:00PM +0800, Peng Fan (OSS) wrote:
> > > From: Peng Fan <peng.fan@nxp.com>
> > >
> > > Add an entry dedicated for fsl,scu-pd which could serve i.MX8DXL
> >
> > Why not just add a soc-specific compatible for the i.MX8DXL?
> > The current form of this does not look right to me.
>
> I could add one dedicated for i.MX8DXL.
Also, where is the user for this? I'd expect to see a dts patch too?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
2023-07-26 7:42 ` Conor Dooley
@ 2023-07-26 7:45 ` Peng Fan
0 siblings, 0 replies; 6+ messages in thread
From: Peng Fan @ 2023-07-26 7:45 UTC (permalink / raw)
To: Conor Dooley
Cc: Conor Dooley, Peng Fan (OSS), robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
ulf.hansson@linaro.org, shawnguo@kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pm@vger.kernel.org
> > > Subject: Re: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-
> pd
> > >
> > > On Tue, Jul 25, 2023 at 06:29:00PM +0800, Peng Fan (OSS) wrote:
> > > > From: Peng Fan <peng.fan@nxp.com>
> > > >
> > > > Add an entry dedicated for fsl,scu-pd which could serve i.MX8DXL
> > >
> > > Why not just add a soc-specific compatible for the i.MX8DXL?
> > > The current form of this does not look right to me.
> >
> > I could add one dedicated for i.MX8DXL.
>
> Also, where is the user for this? I'd expect to see a dts patch too?
arch/arm64/boot/dts/freescale/imx8dxl.dtsi scu node
not use fsl,imx8dxl-scxx. It's already in tree.I am just trying to address
the dtbs_check issue.
Regards,
Peng.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd
2023-07-25 10:29 [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd Peng Fan (OSS)
2023-07-25 18:36 ` Conor Dooley
@ 2023-07-26 7:44 ` Krzysztof Kozlowski
1 sibling, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-26 7:44 UTC (permalink / raw)
To: Peng Fan (OSS), robh+dt, krzysztof.kozlowski+dt, conor+dt,
ulf.hansson, shawnguo
Cc: devicetree, linux-kernel, linux-pm, Peng Fan
On 25/07/2023 12:29, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Add an entry dedicated for fsl,scu-pd which could serve i.MX8DXL
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> .../devicetree/bindings/power/fsl,scu-pd.yaml | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> index 407b7cfec783..3a92e4fef7ae 100644
> --- a/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> +++ b/Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
> @@ -18,11 +18,14 @@ allOf:
>
> properties:
> compatible:
> - items:
> + oneOf:
> - enum:
> - - fsl,imx8qm-scu-pd
> - - fsl,imx8qxp-scu-pd
> - - const: fsl,scu-pd
> + - fsl,scu-pd
No, sorry, you got in the past comments about style of compatibles.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-07-26 7:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-25 10:29 [PATCH] dt-bindings: power: fsl,scu-pd: separate out fsl,scu-pd Peng Fan (OSS)
2023-07-25 18:36 ` Conor Dooley
2023-07-26 0:21 ` Peng Fan
2023-07-26 7:42 ` Conor Dooley
2023-07-26 7:45 ` Peng Fan
2023-07-26 7:44 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).