Pull Request / Patch: Enable Usage of ECC with DDR on AM654x

Tom Rini trini at konsulko.com
Fri May 5 03:59:50 CEST 2023


On Thu, May 04, 2023 at 12:47:29PM +0000, Roytburd, Benjamin wrote:
o
> Hello,
> 
> Recently, when working with the AM65x_GP_EVM development board, I found that the U-boot source code (specifically the SPL), does not properly initialize the DDRSS ram drivers for the AM6548. There are missing register writes that are required from DDR to function with ECC on.
> 
> I validated this by comparing what U-boot source code does to Texas Instrument DDR test scripts (known as GEL scripts), and these GEL scripts could properly initialize DDR with ECC while U-boot could not, this is due to the missing register writes. The changes are shown here in this pull request: https://github.com/u-boot/u-boot/pull/289 (made just to check CI).
> 
> Should I submit a patch for this? I have not contributed to U-boot before and would like to know if such a change would be accepted, or even reviewed, as a submission.

Yes, please submit a patch for it and cc the people that the
scripts/get_maintainer.pl script suggests, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230504/ea3fd1a0/attachment.sig>


More information about the U-Boot mailing list