Change in coreboot[master]: chromeos: Remove old MOCK_TPM references

[Julius Werner (Code Review)]

Julius Werner has submitted this change and it was merged. ( https://review.coreboot.org/18977 )

Change subject: chromeos: Remove old MOCK_TPM references
......................................................................

chromeos: Remove old MOCK_TPM references

The correct way to mock out vboot TPM accesses these days is the
CONFIG_VBOOT_MOCK_SECDATA Kconfig option. There are some remnants of
older TPM-mocking infrastructure in our codebase that are as far as I
can tell inert. Remove them.

Change-Id: I3e00c94b71d53676e6c796e0bec0f3db67c78e34
Signed-off-by: Julius Werner 
Reviewed-on: https://review.coreboot.org/18977
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin 
Reviewed-by: Paul Menzel 
---
M src/lib/Makefile.inc
D src/lib/mocked_tlcl.c
M src/vendorcode/google/chromeos/Makefile.inc
3 files changed, 0 insertions(+), 153 deletions(-)

Approvals:
  Aaron Durbin: Looks good to me, approved
  Paul Menzel: Looks good to me, but someone else must approve
  build bot (Jenkins): Verified

diff --git a/src/lib/Makefile.inc b/src/lib/Makefile.inc
index 977770f..1117076 100644
--- a/src/lib/Makefile.inc
+++ b/src/lib/Makefile.inc
@@ -50,10 +50,6 @@
 verstage-y += boot_device.c
 verstage-$(CONFIG_CONSOLE_CBMEM) += cbmem_console.c

-ifeq ($(MOCK_TPM),1)
-libverstage-y += mocked_tlcl.c
-romstage-$(CONFIG_SEPARATE_VERSTAGE) += mocked_tlcl.c
-else
 libverstage-$(CONFIG_TPM) += tlcl.c
 libverstage-$(CONFIG_TPM2) += tpm2_marshaling.c
 libverstage-$(CONFIG_TPM2) += tpm2_tlcl.c
@@ -63,8 +59,6 @@
 romstage-$(CONFIG_TPM2) += tpm2_marshaling.c
 romstage-$(CONFIG_TPM2) += tpm2_tlcl.c
 endif # CONFIG_SEPARATE_VERSTAGE
-
-endif

verstage-$(CONFIG_GENERIC_UDELAY) += timer.c
 verstage-$(CONFIG_GENERIC_GPIO_LIB) += gpio.c
diff --git a/src/lib/mocked_tlcl.c b/src/lib/mocked_tlcl.c
deleted file mode 100644
index 2eec78d..0000000
--- a/src/lib/mocked_tlcl.c
+++ /dev/null
@@ -1,141 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2015 The ChromiumOS Authors.  All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#include <tpm_lite/tlcl.h>
-
-#ifdef FOR_TEST
-#include <stdio.h>
-#define VBDEBUG(format, args...) printf(format, ## args)
-#else
-#include <console/console.h>
-#define VBDEBUG(format, args...) printk(BIOS_DEBUG, format, ## args)
-#endif
-
-uint32_t tlcl_lib_init(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_startup(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_resume(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_self_test_full(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_continue_self_test(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_define_space(uint32_t index, uint32_t perm, uint32_t size)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_write(uint32_t index, const void *data, uint32_t length)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_read(uint32_t index, void *data, uint32_t length)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-
-uint32_t tlcl_assert_physical_presence(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_physical_presence_cmd_enable(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_finalize_physical_presence(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_set_nv_locked(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_force_clear(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_set_enable(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_set_deactivated(uint8_t flag)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_get_permanent_flags(TPM_PERMANENT_FLAGS *pflags)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_get_flags(uint8_t *disable, uint8_t *deactivated,
-			uint8_t *nvlocked)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_set_global_lock(void)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
-
-uint32_t tlcl_extend(int pcr_num, const uint8_t *in_digest,
-		     uint8_t *out_digest)
-{
-	VBDEBUG("MOCK_TPM: %s\n", __func__);
-	return TPM_E_NO_DEVICE;
-}
diff --git a/src/vendorcode/google/chromeos/Makefile.inc b/src/vendorcode/google/chromeos/Makefile.inc
index af5e178..3326ced 100644
--- a/src/vendorcode/google/chromeos/Makefile.inc
+++ b/src/vendorcode/google/chromeos/Makefile.inc
@@ -32,12 +32,6 @@
 ramstage-y += watchdog.c
 endif

-ifeq ($(MOCK_TPM),1)
-CFLAGS_common += -DMOCK_TPM=1
-else
-CFLAGS_common += -DMOCK_TPM=0
-endif
-
 CONFIG_GBB_HWID := $(call strip_quotes,$(CONFIG_GBB_HWID))
 CONFIG_GBB_BMPFV_FILE := $(call strip_quotes,$(CONFIG_GBB_BMPFV_FILE))
 CONFIG_VBOOT_KEYBLOCK := $(call strip_quotes,$(CONFIG_VBOOT_KEYBLOCK))