[PATCH] dt-bindings: arm: altera: Drop socfpga-sdram-edac.txt

[PATCH] dt-bindings: arm: altera: Drop socfpga-sdram-edac.txt

[Rob Herring (Arm)]

The contents of arm/altera/socfpga-sdram-edac.txt are already covered by
edac/altr,socfpga-ecc-manager.yaml except for the "altr,sdram-edac"
compatible string. Add the compatible and drop the old .txt binding doc.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../bindings/arm/altera/socfpga-sdram-edac.txt    | 15 ---------------
 .../bindings/edac/altr,socfpga-ecc-manager.yaml   |  1 +
 2 files changed, 1 insertion(+), 15 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt

diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt b/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
deleted file mode 100644
index f5ad0ff69fae..000000000000
--- a/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
+++ /dev/null
@@ -1,15 +0,0 @@
-Altera SOCFPGA SDRAM Error Detection & Correction [EDAC]
-The EDAC accesses a range of registers in the SDRAM controller.
-
-Required properties:
-- compatible : should contain "altr,sdram-edac" or "altr,sdram-edac-a10"
-- altr,sdr-syscon : phandle of the sdr module
-- interrupts : Should contain the SDRAM ECC IRQ in the
-	appropriate format for the IRQ controller.
-
-Example:
-	sdramedac {
-		compatible = "altr,sdram-edac";
-		altr,sdr-syscon = <&sdr>;
-		interrupts = <0 39 4>;
-	};
diff --git a/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml b/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml
index ec4634c5fa89..3d787dea0f14 100644
--- a/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml
+++ b/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml
@@ -53,6 +53,7 @@ properties:
     properties:
       compatible:
         enum:
+          - altr,sdram-edac
           - altr,sdram-edac-a10
           - altr,sdram-edac-s10
 
-- 
2.51.0

Re: [PATCH] dt-bindings: arm: altera: Drop socfpga-sdram-edac.txt

[Matthew Gerlach]

On 9/26/25 2:47 PM, Rob Herring (Arm) wrote:
> The contents of arm/altera/socfpga-sdram-edac.txt are already covered by
> edac/altr,socfpga-ecc-manager.yaml except for the "altr,sdram-edac"
> compatible string. Add the compatible and drop the old .txt binding doc.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Acked-by: Matthew Gerlach <matthew.gerlach@altera.com>

> ---
>   .../bindings/arm/altera/socfpga-sdram-edac.txt    | 15 ---------------
>   .../bindings/edac/altr,socfpga-ecc-manager.yaml   |  1 +
>   2 files changed, 1 insertion(+), 15 deletions(-)
>   delete mode 100644 Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
>
> diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt b/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
> deleted file mode 100644
> index f5ad0ff69fae..000000000000
> --- a/Documentation/devicetree/bindings/arm/altera/socfpga-sdram-edac.txt
> +++ /dev/null
> @@ -1,15 +0,0 @@
> -Altera SOCFPGA SDRAM Error Detection & Correction [EDAC]
> -The EDAC accesses a range of registers in the SDRAM controller.
> -
> -Required properties:
> -- compatible : should contain "altr,sdram-edac" or "altr,sdram-edac-a10"
> -- altr,sdr-syscon : phandle of the sdr module
> -- interrupts : Should contain the SDRAM ECC IRQ in the
> -	appropriate format for the IRQ controller.
> -
> -Example:
> -	sdramedac {
> -		compatible = "altr,sdram-edac";
> -		altr,sdr-syscon = <&sdr>;
> -		interrupts = <0 39 4>;
> -	};
> diff --git a/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml b/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml
> index ec4634c5fa89..3d787dea0f14 100644
> --- a/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml
> +++ b/Documentation/devicetree/bindings/edac/altr,socfpga-ecc-manager.yaml
> @@ -53,6 +53,7 @@ properties:
>       properties:
>         compatible:
>           enum:
> +          - altr,sdram-edac
>             - altr,sdram-edac-a10
>             - altr,sdram-edac-s10
>