Claude review: gpu/drm: tegra: add DSI support for Tegra20/Tegra30

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: gpu/drm: tegra: add DSI support for Tegra20/Tegra30
Author: Svyatoslav Ryhel <clamor95@gmail.com>
Patches: 3
Reviewed: 2026-05-16T15:48:58.584162

---

This is a well-structured 2-patch series adding DSI support for Tegra20/Tegra30 to the existing Tegra DSI driver. The approach is sound: introduce a per-SoC config struct with boolean flags to gate the hardware differences (pad control registers and clock muxing), then make `clk_lp` optional since Tegra20/Tegra30 don't have it. The second patch re-introduces a bootloader-DSI workaround with better placement. Both patches have a Reviewed-by from Mikko Perttunen and the author has tested on real hardware across multiple SoC generations.

**Overall assessment: Series looks good with minor issues.**

The primary concern is in patch 2 where `tegra_dsi_disable()` is called unconditionally — including on ganged-mode setups where `dsi->slave` may not be initialized yet — and the comment style doesn't match kernel conventions. Patch 1 is clean and well-factored.

---

---
Generated by Claude Code Patch Reviewer