* [PATCH 2/4] arm64: dts: imx8mp-venice-gw72xx: fix TPM SPI frequency
2025-05-23 17:37 [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: fix TPM SPI frequency Tim Harvey
@ 2025-05-23 17:37 ` Tim Harvey
2025-05-23 17:37 ` [PATCH 3/4] arm64: dts: imx8mp-venice-gw73xx: " Tim Harvey
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Tim Harvey @ 2025-05-23 17:37 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel, devicetree, linux-stable
Cc: imx, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, Tim Harvey
The IMX8MPDS Table 37 [1] shows that the max SPI master read frequency
depends on the pins the interface is muxed behind with ECSPI2
muxed behind ECSPI2 supporting up to 25MHz.
Adjust the spi-max-frequency based on these findings.
[1] https://www.nxp.com/webapp/Download?colCode=IMX8MPIEC
Fixes: 5016f22028e4 ("arm64: dts: imx8mp-venice-gw72xx: add TPM device")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi
index 5ab3ffe9931d..cf747ec6fa16 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw72xx.dtsi
@@ -110,7 +110,7 @@ &ecspi2 {
tpm@1 {
compatible = "atmel,attpm20p", "tcg,tpm_tis-spi";
reg = <0x1>;
- spi-max-frequency = <36000000>;
+ spi-max-frequency = <25000000>;
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 3/4] arm64: dts: imx8mp-venice-gw73xx: fix TPM SPI frequency
2025-05-23 17:37 [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: fix TPM SPI frequency Tim Harvey
2025-05-23 17:37 ` [PATCH 2/4] arm64: dts: imx8mp-venice-gw72xx: " Tim Harvey
@ 2025-05-23 17:37 ` Tim Harvey
2025-05-23 17:37 ` [PATCH 4/4] arm64: dts: imx8mp-venice-gw74xx: " Tim Harvey
2025-06-19 7:49 ` [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: " Shawn Guo
3 siblings, 0 replies; 5+ messages in thread
From: Tim Harvey @ 2025-05-23 17:37 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel, devicetree, linux-stable
Cc: imx, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, Tim Harvey
The IMX8MPDS Table 37 [1] shows that the max SPI master read frequency
depends on the pins the interface is muxed behind with ECSPI2
muxed behind ECSPI2 supporting up to 25MHz.
Adjust the spi-max-frequency based on these findings.
[1] https://www.nxp.com/webapp/Download?colCode=IMX8MPIEC
Fixes: 2b3ab9d81ab4 ("arm64: dts: imx8mp-venice-gw73xx: add TPM device")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi | 2 +-
1 file changed, 1 insertion(+), 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 e2b5e7ac3e46..5eb114d2360a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw73xx.dtsi
@@ -122,7 +122,7 @@ &ecspi2 {
tpm@1 {
compatible = "atmel,attpm20p", "tcg,tpm_tis-spi";
reg = <0x1>;
- spi-max-frequency = <36000000>;
+ spi-max-frequency = <25000000>;
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 4/4] arm64: dts: imx8mp-venice-gw74xx: fix TPM SPI frequency
2025-05-23 17:37 [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: fix TPM SPI frequency Tim Harvey
2025-05-23 17:37 ` [PATCH 2/4] arm64: dts: imx8mp-venice-gw72xx: " Tim Harvey
2025-05-23 17:37 ` [PATCH 3/4] arm64: dts: imx8mp-venice-gw73xx: " Tim Harvey
@ 2025-05-23 17:37 ` Tim Harvey
2025-06-19 7:49 ` [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: " Shawn Guo
3 siblings, 0 replies; 5+ messages in thread
From: Tim Harvey @ 2025-05-23 17:37 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel, devicetree, linux-stable
Cc: imx, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, Tim Harvey
The IMX8MPDS Table 37 [1] shows that the max SPI master read frequency
depends on the pins the interface is muxed behind with ECSPI2
muxed behind ECSPI2 supporting up to 25MHz.
Adjust the spi-max-frequency based on these findings.
Fixes: 531936b218d8 ("arm64: dts: imx8mp-venice-gw74xx: update to revB PCB")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts
index f00099f0cd4e..12de7cf1e853 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw74xx.dts
@@ -201,7 +201,7 @@ &ecspi1 {
tpm@0 {
compatible = "atmel,attpm20p", "tcg,tpm_tis-spi";
reg = <0x0>;
- spi-max-frequency = <36000000>;
+ spi-max-frequency = <25000000>;
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: fix TPM SPI frequency
2025-05-23 17:37 [PATCH 1/4] arm64: dts: imx8mp-venice-gw71xx: fix TPM SPI frequency Tim Harvey
` (2 preceding siblings ...)
2025-05-23 17:37 ` [PATCH 4/4] arm64: dts: imx8mp-venice-gw74xx: " Tim Harvey
@ 2025-06-19 7:49 ` Shawn Guo
3 siblings, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2025-06-19 7:49 UTC (permalink / raw)
To: Tim Harvey
Cc: linux-arm-kernel, linux-kernel, devicetree, linux-stable, imx,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
On Fri, May 23, 2025 at 10:37:20AM -0700, Tim Harvey wrote:
> The IMX8MPDS Table 37 [1] shows that the max SPI master read frequency
> depends on the pins the interface is muxed behind with ECSPI2
> muxed behind ECSPI2 supporting up to 25MHz.
>
> Adjust the spi-max-frequency based on these findings.
>
> [1] https://www.nxp.com/webapp/Download?colCode=IMX8MPIEC
>
> Fixes: 1a8f6ff6a291 ("arm64: dts: imx8mp-venice-gw71xx: add TPM device")
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Applied all, thanks!
^ permalink raw reply [flat|nested] 5+ messages in thread