From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: arm64: defconfig: Enable dma-buf heaps Date: Tue, 28 Apr 2026 14:45:01 +1000 Message-ID: In-Reply-To: <20260427-dma-buf-heaps-as-modules-v5-4-b6f5678feefc@kernel.org> References: <20260427-dma-buf-heaps-as-modules-v5-0-b6f5678feefc@kernel.org> <20260427-dma-buf-heaps-as-modules-v5-4-b6f5678feefc@kernel.org> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Change:** Adds three lines to `arch/arm64/configs/defconfig`: ``` CONFIG_DMABUF_HEAPS=y CONFIG_DMABUF_HEAPS_SYSTEM=m CONFIG_DMABUF_HEAPS_CMA=m ``` The placement is between `CONFIG_STM32_DMA3=m` and `CONFIG_VFIO=y` (confirmed at lines 1413-1417 in the applied tree). The base framework (`DMABUF_HEAPS`) is `=y` which is required since the heaps depend on it and it's a `bool`. Both heaps are `=m` which exercises the new tristate functionality. **Minor observation:** The defconfig insertion point is in the DMA engine section (between STM32_DMA3 and VFIO). This is arguably fine since defconfigs aren't strictly section-organized by subsystem, but one could argue these belong near other dmabuf/graphics config entries. This is purely cosmetic and not worth blocking on. No issues. --- Generated by Claude Code Patch Reviewer