[coreboot-gerrit] [M] Change in coreboot[main]: drivers/amd/opensil/memmap.c: Factor out common memmap code to driver

[Nick Kochlowski (Code Review)]

Attention is currently required from: Felix Held, Fred Reitberger, Jason Glenesk, Martin L Roth, Matt DeVillier, Varshit Pandya.

Nick Kochlowski has posted comments on this change by Nick Kochlowski. ( https://review.coreboot.org/c/coreboot/+/85634?usp=email )

Change subject: drivers/amd/opensil/memmap.c: Factor out common memmap code to driver
......................................................................

Patch Set 20:

(1 comment)

File src/vendorcode/amd/opensil/genoa_poc/memmap.c:

https://review.coreboot.org/c/coreboot/+/85634/comment/49757531_078617a5?usp... :
PS19, Line 16:
> i'd add a check to make sure that the size of the type field is what we expect: […]

The first alternative works well!