[coreboot-gerrit] Change in coreboot[master]: timestamps: COLLECT_TIMESTAMPS is mostly optional

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has submitted this change. ( https://review.coreboot.org/c/coreboot/+/36527 )

Change subject: timestamps: COLLECT_TIMESTAMPS is mostly optional
......................................................................

timestamps: COLLECT_TIMESTAMPS is mostly optional

It is a user-visible option and enabled by default for ARCH_X86,
some consider it as debugging aid only. Therefore platform design
should not depend on it.

It must remain selected with CHROMEOS and boards are allowed
to explicitly select it as well.

For siemens/mc_bdx1,mc_aplX boot time will be increased due
the use of get_us_since_boot() with COLLECT_TIMESTAMPS=n.
When unable to determine if N seconds has elapsed from boot,
this turns into a delay of N seconds.

Change-Id: I6ee4195d266440143344781d39db9578cd8bdcb3
Signed-off-by: Kyösti Mälkki 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36527
Tested-by: build bot (Jenkins) 
Reviewed-by: Aaron Durbin 
Reviewed-by: Marshall Dawson 
---
M src/soc/amd/picasso/Kconfig
M src/soc/amd/stoneyridge/Kconfig
M src/soc/intel/apollolake/Kconfig
M src/soc/intel/braswell/Kconfig
M src/soc/intel/skylake/Kconfig
5 files changed, 0 insertions(+), 5 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Aaron Durbin: Looks good to me, approved
  Marshall Dawson: Looks good to me, approved

diff --git a/src/soc/amd/picasso/Kconfig b/src/soc/amd/picasso/Kconfig
index 6f75e9e..d87d634 100644
--- a/src/soc/amd/picasso/Kconfig
+++ b/src/soc/amd/picasso/Kconfig
@@ -38,7 +38,6 @@
    select SOC_AMD_COMMON_BLOCK_SPI
    select TSC_SYNC_LFENCE
    select UDELAY_TSC
-	select COLLECT_TIMESTAMPS
    select SOC_AMD_PI
    select SOC_AMD_COMMON
    select SOC_AMD_COMMON_BLOCK
diff --git a/src/soc/amd/stoneyridge/Kconfig b/src/soc/amd/stoneyridge/Kconfig
index 0e55937..0d6f2ff 100644
--- a/src/soc/amd/stoneyridge/Kconfig
+++ b/src/soc/amd/stoneyridge/Kconfig
@@ -54,7 +54,6 @@
    select HAVE_USBDEBUG_OPTIONS
    select SOC_AMD_COMMON_BLOCK_SPI
    select TSC_SYNC_LFENCE
-	select COLLECT_TIMESTAMPS
    select SOC_AMD_PI
    select SOC_AMD_COMMON
    select SOC_AMD_COMMON_BLOCK
diff --git a/src/soc/intel/apollolake/Kconfig b/src/soc/intel/apollolake/Kconfig
index a1d3c07..b54528f 100644
--- a/src/soc/intel/apollolake/Kconfig
+++ b/src/soc/intel/apollolake/Kconfig
@@ -40,7 +40,6 @@
    # Misc options
    select C_ENVIRONMENT_BOOTBLOCK
    select CACHE_MRC_SETTINGS
-	select COLLECT_TIMESTAMPS
    select COMMON_FADT
    select FSP_PLATFORM_MEMORY_SETTINGS_VERSIONS
    select GENERIC_GPIO_LIB
diff --git a/src/soc/intel/braswell/Kconfig b/src/soc/intel/braswell/Kconfig
index 5d6438f..5053790 100644
--- a/src/soc/intel/braswell/Kconfig
+++ b/src/soc/intel/braswell/Kconfig
@@ -14,7 +14,6 @@
    select ARCH_VERSTAGE_X86_32
    select BOOT_DEVICE_SUPPORTS_WRITES
    select CACHE_MRC_SETTINGS
-	select COLLECT_TIMESTAMPS
    select SUPPORT_CPU_UCODE_IN_CBFS
    select MICROCODE_BLOB_NOT_IN_BLOB_REPO
    select CPU_INTEL_TURBO_NOT_PACKAGE_SCOPED
diff --git a/src/soc/intel/skylake/Kconfig b/src/soc/intel/skylake/Kconfig
index d47572b..f9f6f93 100644
--- a/src/soc/intel/skylake/Kconfig
+++ b/src/soc/intel/skylake/Kconfig
@@ -26,7 +26,6 @@
    select BOOT_DEVICE_SPI_FLASH_RW_NOMMAP_EARLY if BOOT_DEVICE_SPI_FLASH
    select BOOT_DEVICE_SUPPORTS_WRITES
    select CACHE_MRC_SETTINGS
-	select COLLECT_TIMESTAMPS
    select COMMON_FADT
    select CPU_INTEL_COMMON
    select CPU_INTEL_FIRMWARE_INTERFACE_TABLE