Daniel Maslowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32473 )
Change subject: Documentation/northbridge/intel/haswell/mrc.bin.md: add SPD addresses ......................................................................
Patch Set 4:
(1 comment)
Patch Set 2: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/32473/3/Documentation/northbridge/intel/hasw... File Documentation/northbridge/intel/haswell/mrc.bin.md:
https://review.coreboot.org/#/c/32473/3/Documentation/northbridge/intel/hasw... PS3, Line 106:
Only this last paragraph is somewhat haswell mrc. […]
Yeah that's what I had initially, and when I chatted with Philipp Deppenwiese, he suggested I should add more details on obtaining the information. Do you have suggestions about the order? And where would you put this part then? Maybe here? https://doc.coreboot.org/arch/x86/index.html?highlight=porting#porting-other...