16 comments:
File Documentation/mainboard/amd/Padmelon.md:
Patch Set #1, Line 20: The picture was extracted from manual, however, the numbering on the padmelon board is misplaced.
Ok. Will fix, but also change the text as I'll take my own pictures.
Done
Patch Set #1, Line 40: no, use dediprog
I'm not sure, as I copied some of it from a board that looked like having very complete documentatio […]
Done
File Documentation/mainboard/amd/Padmelon.md:
Patch Set #3, Line 20: The picture was extracted from manual, however, the numbering on the padmelon board is misplaced.
Picture will be replaced by one I myself will take.
Done
Patch Set #3, Line 26: dediprog is protected from harm (even if still connected to the header) because the mux will float
Will do.
Done
Patch Set #3, Line 31: +---------------------+--------------------+
will do.
Done
Patch Set #3, Line 40: | In circuit flashing | no, use dediprog |
No, there's a header. The text (both sides) will be replaced with something more obvious.
Done
Patch Set #3, Line 59: padmelon
Will do.
Done
File Documentation/mainboard/amd/padmelon/padmelon.jpg:
Will take my own picture. Might need help labeling.
Done
File Documentation/mainboard/amd/padmelon/padmelon_components.jpg:
Will do.
Done
File Documentation/mainboard/amd/padmelon/padmelon_io.jpg:
Will do.
Done
File Documentation/mainboard/amd/padmelon/padmelon_io_description.jpg:
Will do.
Done
File Documentation/soc/amd/index.md:
Patch Set #1, Line 5: ## Technology
Sure
Done: 34561
File Documentation/soc/amd/merlinfalcon.md:
Patch Set #1, Line 14: <sup>1</sup>
Not anymore... I copied from family 177h and adapted, and did not noticed this reference... […]
Moved to 34561 and solved there.
Patch Set #1, Line 19: <sup>TM</sup>
Again, copied from family 17h (which uses AGESA v9). […]
Moved to 34561 and replaced with _**TM**_
Patch Set #1, Line 27: <sup>R</sup>
Again, I don't know. I would assume so from a legal stand point.
Moved to 34561, but still present there.
File Documentation/soc/amd/merlinfalcon.md:
Patch Set #3, Line 14: specification<sup>1</sup>. coreboot relies on util/amdfwtool to build
Will remove it.
Moved to 34561 and removed there.
To view, visit change 34493. To unsubscribe, or for help writing mail filters, visit settings.