[PATCH] ARM64: tegra: Set fw_devlink=on for Jetson TX2

[PATCH] ARM64: tegra: Set fw_devlink=on for Jetson TX2

[Jon Hunter]

Commit 5d25c476f252 ("Revert "arm64: tegra: Disable the ACONNECT for
Jetson TX2"") re-enabled the Tegra ADMA and ACONNECT drivers to support
audio on Jetson TX2. However, this revert was dependent upon commit
e590474768f1 ("driver core: Set fw_devlink=on by default") and without
this commit, enabling the ACONNECT is causing resume from system suspend
to fail on Jetson TX2. Resume fails because the ACONNECT driver is being
resumed before the BPMP driver, and the ACONNECT driver is attempting to
power on a power-domain that is provided by the BPMP.

Commit e590474768f1 ("driver core: Set fw_devlink=on by default") has
since been temporarily reverted while some issues are being
investigated. This is causing resume from system suspend on Jetson TX2
to fail again. Rather than disable the ACONNECT driver again, fix this
by setting fw_devlink is set to 'on' for Jetson TX2 in the bootargs
specified in device-tree.

Fixes: 5d25c476f252 ("Revert arm64: tegra: Disable the ACONNECT for Jetson TX2")
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
---
 arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi
index fd9177447711..fcd71bfc6707 100644
--- a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi
@@ -23,7 +23,7 @@ aliases {
 	};
 
 	chosen {
-		bootargs = "earlycon console=ttyS0,115200n8";
+		bootargs = "earlycon console=ttyS0,115200n8 fw_devlink=on";
 		stdout-path = "serial0:115200n8";
 	};
 
-- 
2.25.1

Re: [PATCH] ARM64: tegra: Set fw_devlink=on for Jetson TX2

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 1312 bytes --]

On Fri, Mar 12, 2021 at 10:15:51AM +0000, Jon Hunter wrote:
> Commit 5d25c476f252 ("Revert "arm64: tegra: Disable the ACONNECT for
> Jetson TX2"") re-enabled the Tegra ADMA and ACONNECT drivers to support
> audio on Jetson TX2. However, this revert was dependent upon commit
> e590474768f1 ("driver core: Set fw_devlink=on by default") and without
> this commit, enabling the ACONNECT is causing resume from system suspend
> to fail on Jetson TX2. Resume fails because the ACONNECT driver is being
> resumed before the BPMP driver, and the ACONNECT driver is attempting to
> power on a power-domain that is provided by the BPMP.
> 
> Commit e590474768f1 ("driver core: Set fw_devlink=on by default") has
> since been temporarily reverted while some issues are being
> investigated. This is causing resume from system suspend on Jetson TX2
> to fail again. Rather than disable the ACONNECT driver again, fix this
> by setting fw_devlink is set to 'on' for Jetson TX2 in the bootargs
> specified in device-tree.
> 
> Fixes: 5d25c476f252 ("Revert arm64: tegra: Disable the ACONNECT for Jetson TX2")
> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
> ---
>  arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]