Change in coreboot[master]: soc/amd/cezanne: call bootblock_main_with_basetime in bootblock_c_entry
Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/48473 )
Change subject: soc/amd/cezanne: call bootblock_main_with_basetime in bootblock_c_entry ......................................................................
soc/amd/cezanne: call bootblock_main_with_basetime in bootblock_c_entry
Change-Id: Iaac661fcb7581236ace4b5bf057b3e70289f1c8b Signed-off-by: Felix Held felix-coreboot@felixheld.de --- M src/soc/amd/cezanne/bootblock.c 1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/48473/1
diff --git a/src/soc/amd/cezanne/bootblock.c b/src/soc/amd/cezanne/bootblock.c index 3fd5677..7fe9f81 100644 --- a/src/soc/amd/cezanne/bootblock.c +++ b/src/soc/amd/cezanne/bootblock.c @@ -2,11 +2,20 @@
#include <amdblocks/amd_pci_mmconf.h> #include <bootblock_common.h> +#include <cpu/x86/tsc.h> #include <stdint.h>
asmlinkage void bootblock_c_entry(uint64_t base_timestamp) { enable_pci_mmconf(); + + /* + * base_timestamp is raw tsc value. We need to divide by tsc_freq_mhz + * to get micro-seconds granularity. + */ + base_timestamp /= tsc_freq_mhz(); + + bootblock_main_with_basetime(base_timestamp); }
void bootblock_soc_early_init(void)
Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48473 )
Change subject: soc/amd/cezanne: call bootblock_main_with_basetime in bootblock_c_entry ......................................................................
Patch Set 1: Code-Review+2
Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/48473 )
Change subject: soc/amd/cezanne: call bootblock_main_with_basetime in bootblock_c_entry ......................................................................
soc/amd/cezanne: call bootblock_main_with_basetime in bootblock_c_entry
Change-Id: Iaac661fcb7581236ace4b5bf057b3e70289f1c8b Signed-off-by: Felix Held felix-coreboot@felixheld.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/48473 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/soc/amd/cezanne/bootblock.c 1 file changed, 9 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Marshall Dawson: Looks good to me, approved
diff --git a/src/soc/amd/cezanne/bootblock.c b/src/soc/amd/cezanne/bootblock.c index 3fd5677..7fe9f81 100644 --- a/src/soc/amd/cezanne/bootblock.c +++ b/src/soc/amd/cezanne/bootblock.c @@ -2,11 +2,20 @@
#include <amdblocks/amd_pci_mmconf.h> #include <bootblock_common.h> +#include <cpu/x86/tsc.h> #include <stdint.h>
asmlinkage void bootblock_c_entry(uint64_t base_timestamp) { enable_pci_mmconf(); + + /* + * base_timestamp is raw tsc value. We need to divide by tsc_freq_mhz + * to get micro-seconds granularity. + */ + base_timestamp /= tsc_freq_mhz(); + + bootblock_main_with_basetime(base_timestamp); }
void bootblock_soc_early_init(void)
-
Felix Held (Code Review) -
Marshall Dawson (Code Review)