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@xxxxxxxxxx>

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

> ---
>   .../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
>