Re: [PATCH v2 1/4] arm64: dts: imx93-var-som-symphony: add TPM support

[Frank Li]

On Fri, May 29, 2026 at 12:52:44PM +0200, Stefano Radaelli wrote:
> From: Stefano Radaelli <stefano.r@xxxxxxxxxxxxx>
>
> Add the ST33KTPM2XI2C TPM device on the Symphony carrier board.
>
> The TPM enable line is a board-level signal driven through the PCAL6408
> GPIO expander. Keep it asserted with a GPIO hog so the TPM is enabled on
> this board.
>
> The TPM binding does not provide a dedicated enable GPIO or supply
> property, so this cannot currently be expressed as a direct TPM
> dependency.

there are reset/reset-gpio, can you use it?

Frank
>
> Signed-off-by: Stefano Radaelli <stefano.r@xxxxxxxxxxxxx>
> ---
> v1->v2:
>  - Rename pca6408 label to pcal6408 to match the compatible
>  - Clarify that the TPM enable line is a board-level GPIO hog
>
>  .../dts/freescale/imx93-var-som-symphony.dts  | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx93-var-som-symphony.dts b/arch/arm64/boot/dts/freescale/imx93-var-som-symphony.dts
> index c0842fb3cfa3..f541308ebf6b 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-var-som-symphony.dts
> +++ b/arch/arm64/boot/dts/freescale/imx93-var-som-symphony.dts
> @@ -208,6 +208,26 @@ &lpi2c1 {
>  	sda-gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>;
>  	status = "okay";
>
> +	pcal6408: gpio@21 {
> +		compatible = "nxp,pcal6408";
> +		reg = <0x21>;
> +		gpio-controller;
> +		#gpio-cells = <2>;
> +		status = "okay";
> +
> +		tpm-enable-hog {
> +			gpio-hog;
> +			gpios = <4 GPIO_ACTIVE_HIGH>;
> +			output-high;
> +			line-name = "tpm_en";
> +		};
> +	};
> +
> +	st33ktpm2xi2c: tpm@2e {
> +		compatible = "st,st33ktpm2xi2c", "tcg,tpm-tis-i2c";
> +		reg = <0x2e>;
> +	};
> +
>  	/* Capacitive touch controller */
>  	ft5x06_ts: touchscreen@38 {
>  		compatible = "edt,edt-ft5206";
> --
> 2.47.3
>