[coreboot-gerrit] [M] Change in coreboot[master]: soc/common/smbus: Add support for reading spd data via smbus for DDR5

[Paul Menzel (Code Review)]

Attention is currently required from: Angel Pons, Balaji Manigandan, Kapil Porwal, Karthik Ramasubramanian, Maulik Vaghela, Meera Ravindranath, Nick Vaccaro, Patrick Rudolph, Raj Astekar, Reka Norman, Subrata Banik, Subrata Banik, Tarun Tuli.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52731?usp=email )

Change subject: soc/common/smbus: Add support for reading spd data via smbus for DDR5
......................................................................

Patch Set 13:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/52731/comment/31ca7796_ed71b762 :
PS13, Line 7: Add support for reading spd data via smbus for DDR5
“Add support for” can be substituted by “Support”:

Support reading SPD data via SMBus for DDR5

https://review.coreboot.org/c/coreboot/+/52731/comment/0b2f5aea_c6de5750 :
PS13, Line 11: This CL adds support to read the spd5 hub device via smbus.
Please add a blank line above to separate the paragraphs.

https://review.coreboot.org/c/coreboot/+/52731/comment/1ead3db1_6c6fb4f1 :
PS13, Line 12: 
What datasheet did you use?

https://review.coreboot.org/c/coreboot/+/52731/comment/ff18ab13_ff5ed2c4 :
PS13, Line 14: TEST=Boot adlrvp DDR5 board to kernel
Before it didn’t work? What RAM modules did you test with?