On 26/09/2023 21:43, Tim Harvey wrote: > Add the TPM device found on the GW73xx revision F PCB. > > Signed-off-by: Tim Harvey <tharvey@xxxxxxxxxxxxx> > --- > .../boot/dts/freescale/imx8mp-venice-gw73xx.dtsi | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi > index 48a284478468..43e5e838cefa 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi > @@ -95,8 +95,17 @@ reg_usdhc2_vmmc: regulator-usdhc2-vmmc { > &ecspi2 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_spi2>; > - cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>; > + cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>, > + <&gpio1 10 GPIO_ACTIVE_LOW>; > status = "okay"; > + > + tpm@1 { > + compatible = "tcg,tpm_tis-spi"; > + #address-cells = <0x1>; > + #size-cells = <0x1>; Why do you need these? > + reg = <0x1>; reg is always after compatible > + spi-max-frequency = <36000000>; > + }; > }; > > &gpio4 { Best regards, Krzysztof