Since the GPU on the Jetson TX1 was enabled in its device tree, nouveau often fails to bring up the GM20B: BOOT_0 reads back garbage, PRI reads time out, or the first register reads stall until an SError. The GPU needs the fuse clock running while it is powered, but nothing keeps that clock enabled: the Tegra fuse driver only turns it on around its own fuse reads.
This series describes the fuse clock in the GPU device tree nodes and has nouveau keep it enabled from before the GPU rail is turned on until after it is turned off. Patch 1 makes the fuse clock required by the binding and lists the exact clocks for each GPU. Patches 2-5 add it to the Tegra124, Tegra132, Tegra186 and Tegra210 device trees; Tegra194 already has it. Patch 6 makes nouveau use it. nouveau keeps treating the fuse clock as optional, so device trees that lack it continue to work as before. On eight Jetson TX1 devkits, the GPU failed to come up in 27 of 109 boots without the fix and in 11 of 291 with it; patch 6 has the details. Aaron Kling reported that a Jetson Nano that almost never loaded nouveau did so on 3 of 3 boots with v1. The Tegra124, Tegra132 and Tegra186 device tree changes have only been checked with dtbs_check. Changes in v2: - 1/6: List the exact clocks for each GPU, so that "fuse" cannot appear twice, and make the fuse clock required (Thierry). - 2/6, 3/6, 4/6: New. Add the fuse clock to the Tegra124, Tegra132 and Tegra186 GPUs as well (Thierry). - 5/6: Add a Fixes: tag, as the nouveau fix depends on it. - 6/6: Disable the fuse clock on power down even if disabling the regulator fails (sashiko). v1: https://lore.kernel.org/r/[email protected] Sasha Levin (6): dt-bindings: gpu: nvidia,gk20a: Require the fuse clock ARM: tegra: Add the fuse clock to the Tegra124 GPU arm64: tegra: Add the fuse clock to the Tegra132 GPU arm64: tegra: Add the fuse clock to the Tegra186 GPU arm64: tegra: Add the fuse clock to the Tegra210 GPU drm/nouveau/tegra: Keep the fuse clock enabled while the GPU is powered .../devicetree/bindings/gpu/nvidia,gk20a.yaml | 52 ++++++++++++++----- arch/arm/boot/dts/nvidia/tegra124.dtsi | 5 +- arch/arm64/boot/dts/nvidia/tegra132.dtsi | 5 +- arch/arm64/boot/dts/nvidia/tegra186.dtsi | 5 +- arch/arm64/boot/dts/nvidia/tegra210.dtsi | 5 +- .../gpu/drm/nouveau/include/nvkm/core/tegra.h | 1 + .../drm/nouveau/nvkm/engine/device/tegra.c | 22 +++++--- 7 files changed, 69 insertions(+), 26 deletions(-) base-commit: 70dd4ee72eb2d5793c62e71c14e3aba78e5a4e56 -- 2.53.0
