[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/apollolake: Fix value stored to gnvs is never read

[John Zhao (Code Review)]

John Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32903 )

Change subject: soc/intel/apollolake: Fix value stored to gnvs is never read
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/#/c/32903/2/src/soc/intel/apollolake/chip.c 
File src/soc/intel/apollolake/chip.c:

https://review.coreboot.org/#/c/32903/2/src/soc/intel/apollolake/chip.c@a421 
PS2, Line 421:
> gnvs is used by other blocks in common code as well e.g. uart. […]

The order appears soc_init->uart->southbridge_inject_dsdt. While enabling uart resources, uart refers to GNVS in CBMEM. Since *gnvs is never used here, just changing sizeof(*gnvs) to sizeof(global_nvs_t).