From mboxrd@z Thu Jan 1 00:00:00 1970 From: christophe.ricard Date: Tue, 11 Aug 2015 23:44:46 +0200 Subject: [U-Boot] [PATCH 02/25] tpm: Drop two unused options In-Reply-To: <1439304497-10081-3-git-send-email-sjg@chromium.org> References: <1439304497-10081-1-git-send-email-sjg@chromium.org> <1439304497-10081-3-git-send-email-sjg@chromium.org> Message-ID: <55CA6CCE.2090709@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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 > --- > > 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