[PATCH 0/3] dt-bindings: mfd: syscon: Sync compatible lists

[PATCH 0/3] dt-bindings: mfd: syscon: Sync compatible lists

[Andrew Davis]

Hello all,

Couple simple fixes for the two compatible lists being out of sync.

This is a resend of this series but this time based on latest next
(next-20241220), last sending won't apply, sorry for the noise.

Thanks,
Andrew

Andrew Davis (3):
  dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible
  dt-bindings: mfd: syscon: Fix ti,j721e-acspcie-proxy-ctrl compatible
  dt-bindings: mfd: syscon: Fix al,alpine-sysfabric-service compatible

 Documentation/devicetree/bindings/mfd/syscon.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

-- 
2.39.2

[PATCH 1/3] dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible

[Andrew Davis]

This compatible was only added to the list for compatibility with older
dtschema (<2024.02). Add it to the other list also so both new and old
tools work.

Fixes: 0d078d47cd3e ("dt-bindings: mfd: syscon: Add ti,j784s4-acspcie-proxy-ctrl compatible")
Signed-off-by: Andrew Davis <afd@ti.com>
---
 Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
index 032cdd30d95f1..717b616349e3d 100644
--- a/Documentation/devicetree/bindings/mfd/syscon.yaml
+++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
@@ -214,6 +214,7 @@ properties:
           - ti,am625-dss-oldi-io-ctrl
           - ti,am62p-cpsw-mac-efuse
           - ti,am654-dss-oldi-io-ctrl
+          - ti,j784s4-acspcie-proxy-ctrl
           - ti,j784s4-pcie-ctrl
           - ti,keystone-pllctrl
       - const: syscon
-- 
2.39.2

Re: [PATCH 1/3] dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible

[Siddharth Vadapalli]

On Fri, Jan 03, 2025 at 11:45:22AM -0600, Andrew Davis wrote:
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old
> tools work.
> 
> Fixes: 0d078d47cd3e ("dt-bindings: mfd: syscon: Add ti,j784s4-acspcie-proxy-ctrl compatible")
> Signed-off-by: Andrew Davis <afd@ti.com>

Thank you for the fix.

Reviewed-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Regards,
Siddharth.

> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index 032cdd30d95f1..717b616349e3d 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -214,6 +214,7 @@ properties:
>            - ti,am625-dss-oldi-io-ctrl
>            - ti,am62p-cpsw-mac-efuse
>            - ti,am654-dss-oldi-io-ctrl
> +          - ti,j784s4-acspcie-proxy-ctrl
>            - ti,j784s4-pcie-ctrl
>            - ti,keystone-pllctrl
>        - const: syscon
> -- 
> 2.39.2
> 

Re: [PATCH 1/3] dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible

[Krzysztof Kozlowski]

On Fri, Jan 03, 2025 at 11:45:22AM -0600, Andrew Davis wrote:
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old

That's not a list of compatibles, but just select to apply the schema on
older dtschema.

> tools work.

This compatible was simply never documented and never really worked.
Original patch was probably never tested. :/

> 
> Fixes: 0d078d47cd3e ("dt-bindings: mfd: syscon: Add ti,j784s4-acspcie-proxy-ctrl compatible")
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

Re: [PATCH 1/3] dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible

[Romain Naour]

Hello Andrew, All,

Le 03/01/2025 à 18:45, Andrew Davis a écrit :
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old
> tools work.
> 
> Fixes: 0d078d47cd3e ("dt-bindings: mfd: syscon: Add ti,j784s4-acspcie-proxy-ctrl compatible")
> Signed-off-by: Andrew Davis <afd@ti.com>

Reviewed-by: Romain Naour <romain.naour@smile.fr>

Best regards,
Romain


> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index 032cdd30d95f1..717b616349e3d 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -214,6 +214,7 @@ properties:
>            - ti,am625-dss-oldi-io-ctrl
>            - ti,am62p-cpsw-mac-efuse
>            - ti,am654-dss-oldi-io-ctrl
> +          - ti,j784s4-acspcie-proxy-ctrl
>            - ti,j784s4-pcie-ctrl
>            - ti,keystone-pllctrl
>        - const: syscon

Re: (subset) [PATCH 1/3] dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible

[Lee Jones]

On Fri, 03 Jan 2025 11:45:22 -0600, Andrew Davis wrote:
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old
> tools work.
> 
> 

Applied, thanks!

[1/3] dt-bindings: mfd: syscon: Fix ti,j784s4-acspcie-proxy-ctrl compatible
      commit: 756d4b7a873c1170b65ea1e4cf0312f0a3f9976f

--
Lee Jones [李琼斯]

[PATCH 2/3] dt-bindings: mfd: syscon: Fix ti,j721e-acspcie-proxy-ctrl compatible

[Andrew Davis]

This compatible was only added to the list for compatibility with older
dtschema (<2024.02). Add it to the other list also so both new and old
tools work.

Fixes: 8dfc4a014086 ("dt-bindings: mfd: syscon: Add ti,j721e-acspcie-proxy-ctrl compatible")
Signed-off-by: Andrew Davis <afd@ti.com>
---
 Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
index 717b616349e3d..1f3e67f432e7b 100644
--- a/Documentation/devicetree/bindings/mfd/syscon.yaml
+++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
@@ -214,6 +214,7 @@ properties:
           - ti,am625-dss-oldi-io-ctrl
           - ti,am62p-cpsw-mac-efuse
           - ti,am654-dss-oldi-io-ctrl
+          - ti,j721e-acspcie-proxy-ctrl
           - ti,j784s4-acspcie-proxy-ctrl
           - ti,j784s4-pcie-ctrl
           - ti,keystone-pllctrl
-- 
2.39.2

Re: [PATCH 2/3] dt-bindings: mfd: syscon: Fix ti,j721e-acspcie-proxy-ctrl compatible

[Siddharth Vadapalli]

On Fri, Jan 03, 2025 at 11:45:23AM -0600, Andrew Davis wrote:
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old
> tools work.
> 
> Fixes: 8dfc4a014086 ("dt-bindings: mfd: syscon: Add ti,j721e-acspcie-proxy-ctrl compatible")
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---

Thank you for the fix.

Reviewed-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Regards,
Siddharth.

>  Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index 717b616349e3d..1f3e67f432e7b 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -214,6 +214,7 @@ properties:
>            - ti,am625-dss-oldi-io-ctrl
>            - ti,am62p-cpsw-mac-efuse
>            - ti,am654-dss-oldi-io-ctrl
> +          - ti,j721e-acspcie-proxy-ctrl
>            - ti,j784s4-acspcie-proxy-ctrl
>            - ti,j784s4-pcie-ctrl
>            - ti,keystone-pllctrl
> -- 
> 2.39.2
> 

Re: [PATCH 2/3] dt-bindings: mfd: syscon: Fix ti,j721e-acspcie-proxy-ctrl compatible

[Krzysztof Kozlowski]

On Fri, Jan 03, 2025 at 11:45:23AM -0600, Andrew Davis wrote:
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old
> tools work.
> 
> Fixes: 8dfc4a014086 ("dt-bindings: mfd: syscon: Add ti,j721e-acspcie-proxy-ctrl compatible")
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

Re: [PATCH 2/3] dt-bindings: mfd: syscon: Fix ti,j721e-acspcie-proxy-ctrl compatible

[Romain Naour]

Hello Andrew,

Le 03/01/2025 à 18:45, Andrew Davis a écrit :
> This compatible was only added to the list for compatibility with older
> dtschema (<2024.02). Add it to the other list also so both new and old
> tools work.
> 
> Fixes: 8dfc4a014086 ("dt-bindings: mfd: syscon: Add ti,j721e-acspcie-proxy-ctrl compatible")

This commit is no longer part of the kernel git tree, I'll resend the series for
the j721e with your fix applied.

Thanks!

Best regards,
Romain


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index 717b616349e3d..1f3e67f432e7b 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -214,6 +214,7 @@ properties:
>            - ti,am625-dss-oldi-io-ctrl
>            - ti,am62p-cpsw-mac-efuse
>            - ti,am654-dss-oldi-io-ctrl
> +          - ti,j721e-acspcie-proxy-ctrl
>            - ti,j784s4-acspcie-proxy-ctrl
>            - ti,j784s4-pcie-ctrl
>            - ti,keystone-pllctrl

Re: [PATCH 2/3] dt-bindings: mfd: syscon: Fix ti,j721e-acspcie-proxy-ctrl compatible

[Lee Jones]

On Thu, 09 Jan 2025, Romain Naour wrote:

> Hello Andrew,
> 
> Le 03/01/2025 à 18:45, Andrew Davis a écrit :
> > This compatible was only added to the list for compatibility with older
> > dtschema (<2024.02). Add it to the other list also so both new and old
> > tools work.
> > 
> > Fixes: 8dfc4a014086 ("dt-bindings: mfd: syscon: Add ti,j721e-acspcie-proxy-ctrl compatible")
> 
> This commit is no longer part of the kernel git tree, I'll resend the series for
> the j721e with your fix applied.

So drop this patch and take the other 2?

-- 
Lee Jones [李琼斯]

[PATCH 3/3] dt-bindings: mfd: syscon: Fix al,alpine-sysfabric-service compatible

[Andrew Davis]

This compatible seems to be missing the last 'e', looks to be a typo
when creating this file. Noticed this when diff'ing the two compatible
lists (which should stay in sync).

Fixes: f97b0435c857 ("dt-bindings: mfd: syscon: Split and enforce documenting MFD children")
Signed-off-by: Andrew Davis <afd@ti.com>
---
 Documentation/devicetree/bindings/mfd/syscon.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
index 1f3e67f432e7b..5eb0cfe1f0b89 100644
--- a/Documentation/devicetree/bindings/mfd/syscon.yaml
+++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
@@ -27,7 +27,7 @@ select:
     compatible:
       contains:
         enum:
-          - al,alpine-sysfabric-servic
+          - al,alpine-sysfabric-service
           - allwinner,sun8i-a83t-system-controller
           - allwinner,sun8i-h3-system-controller
           - allwinner,sun8i-v3s-system-controller
-- 
2.39.2

Re: [PATCH 3/3] dt-bindings: mfd: syscon: Fix al,alpine-sysfabric-service compatible

[Siddharth Vadapalli]

On Fri, Jan 03, 2025 at 11:45:24AM -0600, Andrew Davis wrote:
> This compatible seems to be missing the last 'e', looks to be a typo
> when creating this file. Noticed this when diff'ing the two compatible
> lists (which should stay in sync).
> 
> Fixes: f97b0435c857 ("dt-bindings: mfd: syscon: Split and enforce documenting MFD children")
> Signed-off-by: Andrew Davis <afd@ti.com>

Reviewed-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Regards,
Siddharth.

> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml
> index 1f3e67f432e7b..5eb0cfe1f0b89 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.yaml
> +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml
> @@ -27,7 +27,7 @@ select:
>      compatible:
>        contains:
>          enum:
> -          - al,alpine-sysfabric-servic
> +          - al,alpine-sysfabric-service
>            - allwinner,sun8i-a83t-system-controller
>            - allwinner,sun8i-h3-system-controller
>            - allwinner,sun8i-v3s-system-controller
> -- 
> 2.39.2
> 

Re: [PATCH 3/3] dt-bindings: mfd: syscon: Fix al,alpine-sysfabric-service compatible

[Krzysztof Kozlowski]

On Fri, Jan 03, 2025 at 11:45:24AM -0600, Andrew Davis wrote:
> This compatible seems to be missing the last 'e', looks to be a typo
> when creating this file. Noticed this when diff'ing the two compatible
> lists (which should stay in sync).
> 
> Fixes: f97b0435c857 ("dt-bindings: mfd: syscon: Split and enforce documenting MFD children")
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  Documentation/devicetree/bindings/mfd/syscon.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

Re: (subset) [PATCH 3/3] dt-bindings: mfd: syscon: Fix al,alpine-sysfabric-service compatible

[Lee Jones]

On Fri, 03 Jan 2025 11:45:24 -0600, Andrew Davis wrote:
> This compatible seems to be missing the last 'e', looks to be a typo
> when creating this file. Noticed this when diff'ing the two compatible
> lists (which should stay in sync).
> 
> 

Applied, thanks!

[3/3] dt-bindings: mfd: syscon: Fix al,alpine-sysfabric-service compatible
      commit: 476f519d1ba1671f97f657027b33d5833b9e691d

--
Lee Jones [李琼斯]