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