[coreboot-gerrit] Patch set updated for coreboot: Remove dependency for HAS_PRECBMEM_TIMESTAMP_REGION

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Thu Nov 12 19:39:42 CET 2015


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11338

-gerrit

commit 1e287503a032ddb572ab9c9ecb8e7a3b2cc758fa
Author: Martin Roth <martinroth at google.com>
Date:   Mon Aug 24 16:42:31 2015 -0600

    Remove dependency for HAS_PRECBMEM_TIMESTAMP_REGION
    
    HAS_PRECBMEM_TIMESTAMP_REGION was dependent on COLLECT_TIMESTAMPS,
    but should be allowed to be selected independently.  My thought is that
    the code may only be used when collecting timestamps, the HAS prefix
    signifies that this is a platform configuration option.
    
    This fix could also be done by adding 'if COLLECT_TIMESTAMPS' everywhere
    that 'select HAS_PRECBMEM_TIMESTAMP_REGION' is used
    
    Change-Id: Iaf4895475c38a855a048dc9b82d4c97e5e3f4e5c
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 src/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/Kconfig b/src/Kconfig
index 3562887..f0aea2b 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -188,7 +188,6 @@ config COLLECT_TIMESTAMPS
 config HAS_PRECBMEM_TIMESTAMP_REGION
 	bool "Timestamp region exists for pre-cbmem timestamps"
 	default y if ARCH_ROMSTAGE_X86_32 && CACHE_AS_RAM
-	depends on COLLECT_TIMESTAMPS
 	help
 	  A separate region is maintained to allow storing of timestamps before
 	  cbmem comes up. This is useful for storing timestamps across different



More information about the coreboot-gerrit mailing list