Add DEBUG_TPM option to Debugging menu
instead of having to edit the source code of tpm.c
Change-Id: I519d9ada14dd383e668a2da4219e5373a24c7c3d
Signed-off-by: Stefan Reinauer <reinauer@google.com>
Reviewed-on: http://review.coreboot.org/757
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
diff --git a/src/Kconfig b/src/Kconfig
index f7180a9..0696feb 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -862,6 +862,13 @@
If unsure, say N.
+config DEBUG_TPM
+ bool "Output verbose TPM debug messages"
+ default n
+ depends on TPM
+ help
+ This option enables additional TPM related debug messages.
+
config LLSHELL
bool "Built-in low-level shell"
default n
diff --git a/src/pc80/tpm.c b/src/pc80/tpm.c
index 8e94303..17e1ed7 100644
--- a/src/pc80/tpm.c
+++ b/src/pc80/tpm.c
@@ -27,7 +27,6 @@
* Infineon slb9635), so this driver provides access to locality 0 only.
*/
-/* #define DEBUG */
#include <stdlib.h>
#include <string.h>
#include <delay.h>
@@ -37,17 +36,11 @@
#include <pc80/tpm.h>
#include <cpu/x86/car.h>
-#ifdef DEBUG
-#define TPM_DEBUG_ON 1
-#else
-#define TPM_DEBUG_ON 0
-#endif
-
#define PREFIX "lpc_tpm: "
/* coreboot wrapper for TPM driver (start) */
#define TPM_DEBUG(fmt, args...) \
- if (TPM_DEBUG_ON) { \
+ if (CONFIG_DEBUG_TPM) { \
printk(BIOS_DEBUG, PREFIX); \
printk(BIOS_DEBUG, fmt , ##args); \
}