GNUBoot/coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Remove dependency for HAS_PRECBMEM_TIMESTAMP_REGION

Browse source 
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@google.com>
Reviewed-on: http://review.coreboot.org/11338
Tested-by: build bot (Jenkins)
Reviewed-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
This commit is contained in:
Martin Roth 2015-08-24 16:42:31 -06:00
parent 8420ad4b41
commit 355dfda3f0
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/Kconfig
View file

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