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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <ea5da9ab-c691-4339-bb39-205ea5759220@altera.com>
Date: Sat, 27 Sep 2025 06:51:13 -0700
From: Matthew Gerlach <matthew.gerlach@...era.com>
To: "Rob Herring (Arm)" <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: arm: altera: Drop socfpga-sdram-edac.txt



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@...nel.org>

Acked-by: Matthew Gerlach <matthew.gerlach@...era.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
>