From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/prime: Limit scatter list size with dedicated DMA device Date: Thu, 12 Mar 2026 07:08:42 +1000 Message-ID: In-Reply-To: <20260311094929.3393338-2-wenst@chromium.org> References: <20260311094929.3393338-1-wenst@chromium.org> <20260311094929.3393338-2-wenst@chromium.org> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review A one-liner that changes `dev->dev` to `drm_dev_dma_dev(dev)` in `drm_prime_pages_to_sg()`: ```c - max_segment = dma_max_mapping_size(dev->dev); + max_segment = dma_max_mapping_size(drm_dev_dma_dev(dev)); ``` This is correct and straightforward. The existing code queries the DMA max mapping size from the parent device, but if a dedicated DMA device is set, the constraints of that device are the relevant ones. The `drm_dev_dma_dev()` helper gracefully falls back to `dev->dev` when no dedicated device is set, so this is safe for all existing users. No issues. --- Generated by Claude Code Patch Reviewer