From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: staging: media: tegra-video: vi: improve logic of source requesting
Date: Wed, 04 Mar 2026 07:48:07 +1000
Message-ID: <review-patch6-20260303084239.15007-7-clamor95@gmail.com>
In-Reply-To: <20260303084239.15007-7-clamor95@gmail.com>
References: <20260303084239.15007-1-clamor95@gmail.com> <20260303084239.15007-7-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

Clean refactoring. The new `tegra_channel_get_remote_csi_subdev()` in `csi.c` validates that the remote device is actually a CSI subdev by comparing its ops pointer:

```c
return subdev->ops == &tegra_csi_ops ? subdev : NULL;
```

The old function is renamed to `tegra_channel_get_remote_bridge_subdev()` and made static in `vi.c` since VI doesn't need CSI-specific checks. This is a good separation of concerns.

**No issues.**

---
Generated by Claude Code Patch Reviewer