imx8m*-venice: enable TPM support

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 <tharvey@gateworks.com>
---
 configs/imx8mm_venice_defconfig | 10 ++++++++++
 configs/imx8mn_venice_defconfig | 10 ++++++++++
 configs/imx8mp_venice_defconfig | 10 ++++++++++
 3 files changed, 30 insertions(+)

On Mon, Mar 25, 2024 at 9:27 AM Tim Harvey <tharvey@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 <tharvey@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

Hi Tim,

On Wed, Apr 17, 2024 at 2:00 PM Tim Harvey <tharvey@gateworks.com> wrote:

> Fabio,
>
> This one got tagged as 'changes requested' in patchwork but I'm not
> clear why. Are there any issues with this?

Sorry, this was a mistake. I will apply it in the next batch of patches.

On Mon, Mar 25, 2024 at 1:27 PM Tim Harvey <tharvey@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 <tharvey@gateworks.com>

Applied, thanks.