Re: [U-Boot] [PATCH 02/25] tpm: Drop two unused options

Hi Simon,

I am ok with this one. I proposed about the same changed in here: http://lists.denx.de/pipermail/u-boot/2015-August/222598.html

Acked-by: Christophe Ricard christophe-h.ricard@st.com

Best Regards Christophe On 11/08/2015 16:47, Simon Glass wrote:

The address of the I2C TPM is now defined in the device tree so there is no need for the CONFIG options.

Remove them from the README and board config to avoid confusion.

Signed-off-by: Simon Glass sjg@chromium.org

README | 6 ------ include/configs/exynos5-common.h | 2 -- 2 files changed, 8 deletions(-)

diff --git a/README b/README index 1bcb63c..b55175a 100644 --- a/README +++ b/README @@ -1496,12 +1496,6 @@ The following options need to be configured: Support for i2c bus TPM devices. Only one device per system is supported at this time.

• 	CONFIG_TPM_TIS_I2C_BUS_NUMBER
  

• 	Define the the i2c bus number for the TPM device
  

• 	CONFIG_TPM_TIS_I2C_SLAVE_ADDRESS
  

• 	Define the TPM's address on the i2c bus
  

• CONFIG_TPM_TIS_I2C_BURST_LIMITATION
  Define the burst count bytes upper limit
  
  

diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-common.h index e710f41..fb5ee0d 100644 --- a/include/configs/exynos5-common.h +++ b/include/configs/exynos5-common.h @@ -58,8 +58,6 @@ #define CONFIG_TPM #define CONFIG_CMD_TPM #define CONFIG_TPM_TIS_I2C -#define CONFIG_TPM_TIS_I2C_BUS_NUMBER 3 -#define CONFIG_TPM_TIS_I2C_SLAVE_ADDR 0x20

/* MMC SPL */ #define COPY_BL2_FNPTR_ADDR 0x02020030