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

From: Krzysztof Kozlowski
Date: Sun Aug 04 2024 - 04:34:53 EST


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