[coreboot-gerrit] Change in coreboot[master]: soc/intel/apollolake: Save/restore GMA OpRegion address

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25416 )

Change subject: soc/intel/apollolake: Save/restore GMA OpRegion address
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/25416/2/src/soc/intel/apollolake/gr... 
File src/soc/intel/apollolake/graphics.c:

https://review.coreboot.org/c/coreboot/+/25416/2/src/soc/intel/apollolake/gr... 
PS2, Line 43: }
> Doesn't seem to be used. If all platforms implement it, we could call […]

I thought the whole point of using the gnvs was so that we had the memory address available to set the ASLB register on S3 resume, which would otherwise be unavailable