From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: dma: contiguous: Make dev_get_cma_area() a proper function Date: Wed, 04 Mar 2026 07:41:35 +1000 Message-ID: In-Reply-To: <20260303-dma-buf-heaps-as-modules-v3-2-24344812c707@kernel.org> References: <20260303-dma-buf-heaps-as-modules-v3-0-24344812c707@kernel.org> <20260303-dma-buf-heaps-as-modules-v3-2-24344812c707@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 Converts `dev_get_cma_area()` from an inline function in the header to a proper function in `contiguous.c`. This is necessary to avoid having to export `dma_contiguous_default_area` directly. This patch also correctly fixes the `dma_contiguous_get_reserved_region()` declaration guard from `#ifdef CONFIG_DMA_CMA` to `#if defined(CONFIG_DMA_CMA) && defined(CONFIG_OF_RESERVED_MEM)`, since the implementation lives inside `#ifdef CONFIG_OF_RESERVED_MEM`. This is a v3 fix for a compilation failure and is correct. No issues. --- Generated by Claude Code Patch Reviewer