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 dma_contiguous_default_area static Date: Wed, 04 Mar 2026 07:41:36 +1000 Message-ID: In-Reply-To: <20260303-dma-buf-heaps-as-modules-v3-3-24344812c707@kernel.org> References: <20260303-dma-buf-heaps-as-modules-v3-0-24344812c707@kernel.org> <20260303-dma-buf-heaps-as-modules-v3-3-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 Now that `dev_get_cma_area()` is no longer inline, the `extern` declaration of `dma_contiguous_default_area` in the header can be removed and the variable made `static` in `contiguous.c`. Clean and correct. **Nit:** The commit message says "dma_contiguous_default_area()" with parentheses as if it were a function, but it's a variable. Very minor. --- Generated by Claude Code Patch Reviewer