Attention is currently required from: Julius Werner, Moritz Fischer. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50886 )
Change subject: soc/rockchip/rk3399/sdram: Clear PI_175 IRQs in data training ......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/50886/comment/daa1206a_c240045c PS1, Line 10: calls.
I couldn't figure out how to add a 'cover letter'. […]
I have no experience with ARM, but I like RAM stuff. This change makes sense, but I'd like to know for sure that it fixes something (or matches what some vendor spec says).
I've heard about cover letters for mailing list patches. AFAIK, Gerrit doesn't have such a thing. Personally, I prefer that commit messages explain the reason to change something.