Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42463 )
Change subject: [WIP] ACPI: Allocate GNVS early in ramstage
......................................................................
Patch Set 3:
How do you want to approach with the implementations for
size_t gnvs_size_of_array(void);
uint32_t *gnvs_cbmc_ptr(void);
void *gnvs_chromeos_ptr(void);
We could mostly do with one common implementation with <soc/nvs.h>.
Then have a KConfig to opt-out for individual <sb/intel/xxx/nvs.h> cases.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/42463
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I45b4b5c0c5bf8bee258a465d1e364bfe98190e44
Gerrit-Change-Number: 42463
Gerrit-PatchSet: 3
Gerrit-Owner: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Thu, 02 Jul 2020 01:33:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment