Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34662 )
Change subject: Documentation/binaries: Add AMD FSP documentation ......................................................................
Patch Set 8:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34662/8/Documentation/binaries/amd/... File Documentation/binaries/amd/AMD_FSP_family_17h.md:
https://review.coreboot.org/c/coreboot/+/34662/8/Documentation/binaries/amd/... PS8, Line 5: Starting with Family 17h, AMD is AGESA version 9. However, it is currently \
no need for \ at end of line
I'm using it to force a line break. However, this is mainly when you want to keep the next line with the same formatting as the previous. I believe there are sections in here, this line included, where you are correct. Will remove it where I can.
https://review.coreboot.org/c/coreboot/+/34662/8/Documentation/binaries/amd/... PS8, Line 34: As a consequence, there’s no need of FSP-T. Very little of original \
add 3 spaces at the beginning of each line to place the block inside the option list
Yes, but it would re-arrange the line formatting for the whole frame, thus causing difference with other lines above. Using < > was the only option that I found to work as I needed. I'm trying to keep it around 76 characters once framed. Unless you present another solution that does work for the 76 characters formatting, I'll keep it this way. Three spaces before the line does not work as I need. By the way, this is how an add-on to google docs converted it to mark down.