On Mon, Mar 25, 2024 at 9:27 AM Tim Harvey <thar...@gateworks.com> wrote: > > Enable support for TPM2 devices. As the ATTPM20P TPM2 used on the > Gateworks Venice boards hangs off the SPI bus we enable SPI support > as well. > > Signed-off-by: Tim Harvey <thar...@gateworks.com> > --- > configs/imx8mm_venice_defconfig | 10 ++++++++++ > configs/imx8mn_venice_defconfig | 10 ++++++++++ > configs/imx8mp_venice_defconfig | 10 ++++++++++ > 3 files changed, 30 insertions(+) > > diff --git a/configs/imx8mm_venice_defconfig b/configs/imx8mm_venice_defconfig > index cb6b97d097c1..86928687df2d 100644 > --- a/configs/imx8mm_venice_defconfig > +++ b/configs/imx8mm_venice_defconfig > @@ -68,6 +68,7 @@ CONFIG_CMD_TIME=y > CONFIG_CMD_UUID=y > CONFIG_CMD_PMIC=y > CONFIG_CMD_REGULATOR=y > +CONFIG_CMD_TPM=y > CONFIG_CMD_EXT4_WRITE=y > # CONFIG_ISO_PARTITION is not set > # CONFIG_SPL_EFI_PARTITION is not set > @@ -127,14 +128,20 @@ CONFIG_SPL_DM_PMIC_MP5416=y > CONFIG_DM_REGULATOR=y > CONFIG_DM_REGULATOR_FIXED=y > CONFIG_DM_REGULATOR_GPIO=y > +# CONFIG_DM_RNG is not set > CONFIG_DM_SERIAL=y > CONFIG_MXC_UART=y > +CONFIG_SPI=y > +CONFIG_DM_SPI=y > +CONFIG_MXC_SPI=y > CONFIG_SYSRESET=y > CONFIG_SPL_SYSRESET=y > CONFIG_SYSRESET_PSCI=y > CONFIG_SYSRESET_WATCHDOG=y > CONFIG_DM_THERMAL=y > CONFIG_IMX_TMU=y > +# CONFIG_TPM_V1 is not set > +CONFIG_TPM2_TIS_SPI=y > CONFIG_USB=y > CONFIG_USB_EHCI_HCD=y > CONFIG_USB_HOST_ETHER=y > @@ -152,4 +159,7 @@ CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 > CONFIG_CI_UDC=y > CONFIG_USB_GADGET_DOWNLOAD=y > CONFIG_IMX_WATCHDOG=y > +CONFIG_TPM=y > +# CONFIG_SPL_SHA512 is not set > +# CONFIG_SPL_SHA384 is not set > CONFIG_HEXDUMP=y > diff --git a/configs/imx8mn_venice_defconfig b/configs/imx8mn_venice_defconfig > index 0a4fba5f1ca8..6bb0f015a9bf 100644 > --- a/configs/imx8mn_venice_defconfig > +++ b/configs/imx8mn_venice_defconfig > @@ -71,6 +71,7 @@ CONFIG_CMD_TIME=y > CONFIG_CMD_UUID=y > CONFIG_CMD_PMIC=y > CONFIG_CMD_REGULATOR=y > +CONFIG_CMD_TPM=y > CONFIG_CMD_EXT4_WRITE=y > # CONFIG_ISO_PARTITION is not set > # CONFIG_SPL_EFI_PARTITION is not set > @@ -128,14 +129,20 @@ CONFIG_SPL_DM_PMIC_MP5416=y > CONFIG_DM_REGULATOR=y > CONFIG_DM_REGULATOR_FIXED=y > CONFIG_DM_REGULATOR_GPIO=y > +# CONFIG_DM_RNG is not set > CONFIG_DM_SERIAL=y > CONFIG_MXC_UART=y > +CONFIG_SPI=y > +CONFIG_DM_SPI=y > +CONFIG_MXC_SPI=y > CONFIG_SYSRESET=y > CONFIG_SPL_SYSRESET=y > CONFIG_SYSRESET_PSCI=y > CONFIG_SYSRESET_WATCHDOG=y > CONFIG_DM_THERMAL=y > CONFIG_IMX_TMU=y > +# CONFIG_TPM_V1 is not set > +CONFIG_TPM2_TIS_SPI=y > CONFIG_USB=y > CONFIG_USB_EHCI_HCD=y > CONFIG_USB_HOST_ETHER=y > @@ -153,4 +160,7 @@ CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 > CONFIG_CI_UDC=y > CONFIG_USB_GADGET_DOWNLOAD=y > CONFIG_IMX_WATCHDOG=y > +CONFIG_TPM=y > +# CONFIG_SPL_SHA512 is not set > +# CONFIG_SPL_SHA384 is not set > CONFIG_HEXDUMP=y > diff --git a/configs/imx8mp_venice_defconfig b/configs/imx8mp_venice_defconfig > index 6e4addc7728f..b1055ef2f066 100644 > --- a/configs/imx8mp_venice_defconfig > +++ b/configs/imx8mp_venice_defconfig > @@ -73,6 +73,7 @@ CONFIG_CMD_TIME=y > CONFIG_CMD_UUID=y > CONFIG_CMD_PMIC=y > CONFIG_CMD_REGULATOR=y > +CONFIG_CMD_TPM=y > CONFIG_CMD_EXT4_WRITE=y > # CONFIG_ISO_PARTITION is not set > # CONFIG_SPL_EFI_PARTITION is not set > @@ -135,14 +136,20 @@ CONFIG_SPL_DM_PMIC_MP5416=y > CONFIG_DM_REGULATOR=y > CONFIG_DM_REGULATOR_FIXED=y > CONFIG_DM_REGULATOR_GPIO=y > +# CONFIG_DM_RNG is not set > CONFIG_DM_SERIAL=y > CONFIG_MXC_UART=y > +CONFIG_SPI=y > +CONFIG_DM_SPI=y > +CONFIG_MXC_SPI=y > CONFIG_SYSRESET=y > CONFIG_SPL_SYSRESET=y > CONFIG_SYSRESET_PSCI=y > CONFIG_SYSRESET_WATCHDOG=y > CONFIG_DM_THERMAL=y > CONFIG_IMX_TMU=y > +# CONFIG_TPM_V1 is not set > +CONFIG_TPM2_TIS_SPI=y > CONFIG_USB=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_DWC3=y > @@ -160,4 +167,7 @@ CONFIG_USB_ETHER_MCS7830=y > CONFIG_USB_ETHER_RTL8152=y > CONFIG_USB_ETHER_SMSC95XX=y > CONFIG_IMX_WATCHDOG=y > +CONFIG_TPM=y > +# CONFIG_SPL_SHA512 is not set > +# CONFIG_SPL_SHA384 is not set > CONFIG_HEXDUMP=y > -- > 2.25.1 >
Fabio, This one got tagged as 'changes requested' in patchwork but I'm not clear why. Are there any issues with this? Best Regards, Tim