From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: staging: media: tegra-video: expand VI and VIP support to Tegra30 Date: Wed, 04 Mar 2026 07:48:06 +1000 Message-ID: In-Reply-To: <20260303084239.15007-2-clamor95@gmail.com> References: <20260303084239.15007-1-clamor95@gmail.com> <20260303084239.15007-2-clamor95@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Straightforward patch enabling existing Tegra20 VI and VIP code for Tegra30 by extending the `#if defined()` guards. The approach of using `||` in the preprocessor guards works correctly. The Makefile addition of `tegra-video-$(CONFIG_ARCH_TEGRA_3x_SOC) += tegra20.o` could cause duplicate object files if both TEGRA_2x_SOC and TEGRA_3x_SOC are enabled simultaneously. However, kbuild handles this correctly by deduplicating, so this is fine. **No issues.** --- Generated by Claude Code Patch Reviewer