diff options
| author | Peter Delevoryas <peter@pjd.dev> | 2022-07-28 00:33:54 +0300 |
|---|---|---|
| committer | Patrick Williams <patrick@stwcx.xyz> | 2022-08-10 15:33:26 +0300 |
| commit | 3bcb9ad90853b70933e7d103cf910f9b8c9fbb61 (patch) | |
| tree | 9df49b608c1fc62e91c74c2effaf76d60bf6722b /meta-facebook | |
| parent | d8a64f0710af47a84f898da31dcf19456721e5dc (diff) | |
| download | openbmc-3bcb9ad90853b70933e7d103cf910f9b8c9fbb61.tar.xz | |
fby35: Disable tpm_tis_i2c
fby35 uses a SPI TPM interface, not the I2C interface, and the I2C module produces compilation
warnings, and build failures with -Werror, so we should disable it
until it's fixed in the kernel.
Building fby35:
```
. setup fby35 build
bitbake obmc-phosphor-image
```
Before:
```
| /home/pdel/lfopenbmc/build/tmp/work-shared/fby35/kernel-source/drivers/char/tpm/tpm_tis_i2c.c: In function 'tpm_tis_i2c_write_bytes':
| /home/pdel/lfopenbmc/build/tmp/work-shared/fby35/kernel-source/drivers/char/tpm/tpm_tis_i2c.c:114:17: error: ISO C90 forbids mixed declarati
ons and code [-Werror=declaration-after-statement]
| 114 | struct i2c_msg msgs[] = {
| | ^~~~~~
```
After:
Compiles successfully.
Change-Id: Ib57bcad51837956d8a277d2be1ff7c887ade9606
Signed-off-by: Peter Delevoryas <peter@pjd.dev>
Diffstat (limited to 'meta-facebook')
| -rw-r--r-- | meta-facebook/meta-fby35/recipes-kernel/linux/linux-aspeed/fby35.cfg | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/meta-facebook/meta-fby35/recipes-kernel/linux/linux-aspeed/fby35.cfg b/meta-facebook/meta-fby35/recipes-kernel/linux/linux-aspeed/fby35.cfg index 70d5a553ab..f9653bc1a2 100644 --- a/meta-facebook/meta-fby35/recipes-kernel/linux/linux-aspeed/fby35.cfg +++ b/meta-facebook/meta-fby35/recipes-kernel/linux/linux-aspeed/fby35.cfg @@ -13,6 +13,9 @@ CONFIG_ARCH_ASPEED=y CONFIG_PWM_ASPEED_AST2600=y CONFIG_SENSORS_TACH_ASPEED_G6=y +# Keep this disabled until the compilation warnings are fixed. +CONFIG_TCG_TIS_I2C=n + # IPMB I2C CONFIG CONFIG_I2C_SLAVE=y CONFIG_IPMB_DEVICE_INTERFACE=y |