mikeb mikeb has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29272 )
Change subject: src/cpu/amd/pi/00730F01: Add microcode update infrastructure for fam16h PI
......................................................................
Patch Set 3:
I am able to patch the blob, I have done it successfully. But I wonder about license. If it does not violate the law, I can push the AGESA blob with patched microcode.
AMD has allowed the coreboot project to merge the new microcodes into coreboot, therefore CB:28273 (f15tn) and CB:28370 (f16kb) updates have been merged into the opensource AGESA. So it seems to me that this particular modification activity (microcode updating) against this blob - is de facto allowed, provided that you don't modify anything else not related to the microcodes. In any case, feel free to submit your patched blob, and either it will be accepted or - if the things aren't clear enough - could be used as a reference.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/29272
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic15cba06f3cd9cfbc538b6764b158fa699f0ecf6
Gerrit-Change-Number: 29272
Gerrit-PatchSet: 3
Gerrit-Owner: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Patrick Rudolph
siro@das-labor.org
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Philipp Deppenwiese
zaolin.daisuki@gmail.com
Gerrit-Reviewer: Piotr Król
piotr.krol@3mdeb.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: mikeb mikeb
mikebdp2@gmail.com
Gerrit-Comment-Date: Mon, 27 May 2019 15:14:12 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment