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
signature.asc
Description: PGP signature
