Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34493 )
Change subject: Documentation/mainboard/amd: Add padmelon doucumentation and images ......................................................................
Patch Set 1:
(3 comments)
Will check the tool for reducing jpeg size.
https://review.coreboot.org/c/coreboot/+/34493/1/Documentation/soc/amd/merli... File Documentation/soc/amd/merlinfalcon.md:
https://review.coreboot.org/c/coreboot/+/34493/1/Documentation/soc/amd/merli... PS1, Line 14: <sup>1</sup>
Is there a reference?
Not anymore... I copied from family 177h and adapted, and did not noticed this reference... will remove.
https://review.coreboot.org/c/coreboot/+/34493/1/Documentation/soc/amd/merli... PS1, Line 19: <sup>TM</sup>
Is that needed in coreboot? If yes, I‘d prefer UTF-8 characters: ™
Again, copied from family 17h (which uses AGESA v9). I don't know if it's needed, and I don't have the proper editor/reader to know what <sup></sup> does, nor do I know if UFT-8 would work. That's one reason I copied from family 17h... I was sure that if <sup> was used there, than it should work here too. That said, if you want me to it, I will. I would wait for someone else opinion before I remove the <sup></sup>.
https://review.coreboot.org/c/coreboot/+/34493/1/Documentation/soc/amd/merli... PS1, Line 27: <sup>R</sup>
Are these symbols needed in the coreboot documentation? […]
Again, I don't know. I would assume so from a legal stand point.