linux-kernel - Re: [PATCH v3] dt-bindings: edac: Add Altera SOCFPGA SDRAM EDAC binding

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: <5a180c6b-aed6-4060-94c5-bdbd9ce06934@kernel.org>
Date: Sun, 4 Aug 2024 10:34:41 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Alessandro Zanni <alessandro.zanni87@...il.com>
Cc: robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
 skhan@...uxfoundation.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] dt-bindings: edac: Add Altera SOCFPGA SDRAM EDAC
 binding

On 02/08/2024 16:32, Alessandro Zanni wrote:
> Thank you for the review.
> 
> This patch aims to be a conversion from txt file
> "bindings/arm/altera/socfpga-sdram-edac.txt" to this
> "bindings/edac/altr,sdram-edac.yaml" file.
> That's why I also deleted the txt file in the patch.
> 
> Maybe I should have put a different title to make it clearer.
> 
> This is the original example in the txt file:
> sdramedac {
>     compatible = "altr,sdram-edac";
>     altr,sdr-syscon = <&sdr>;
>     interrupts = <0 39 4>;
> 
> and this is my conversion:
>> +  - |
>> +    memory-controller {
>> +      compatible = "altr,sdram-edac";
>> +      altr,sdr-syscon = <&sdr>;
>> +      interrupts = <0 39 4>;
> 
> Sorry for that but, to me, it's not clear what I should convert in
> this example to complete the yaml file.
> 

Don't top-post. I have no clue what you are referring to.

Best regards,
Krzysztof