Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37862 )
Change subject: amdblocks/biosram: Force use of abstraction
......................................................................
Patch Set 2:
Patch Set 1:
From what are we trying to protect here? From the unintended change of reserved (for boot process use) BIOSRAM addresses?
Just the general concept of exposing as little as possible to keep it simple to maintain.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/37862
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I774b6640cdd9873f52e446c4ca41b7c537a87883
Gerrit-Change-Number: 37862
Gerrit-PatchSet: 2
Gerrit-Owner: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Reviewer: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-Reviewer: Richard Spiegel
richard.spiegel@silverbackltd.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Fri, 20 Dec 2019 16:00:47 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment