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-buf: heaps: use max3() in dma_heap_ioctl Date: Tue, 28 Apr 2026 14:15:58 +1000 Message-ID: In-Reply-To: <20260427172519.417333-3-thorsten.blum@linux.dev> References: <20260427172519.417333-3-thorsten.blum@linux.dev> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: dma-buf: heaps: use max3() in dma_heap_ioctl Author: Thorsten Blum Patches: 2 Reviewed: 2026-04-28T14:15:58.686759 --- This is a single-patch trivial cleanup that replaces nested `max(max(a, b), c)` with the equivalent `max3(a, b, c)` macro. The change is correct, improves readability marginally, and has no functional impact. The `max3` macro is already available through the existing include chain (via `linux/minmax.h`, included transitively through `linux/device.h` -> `linux/kernel.h`). **Verdict:** Harmless cleanup. No concerns. --- Generated by Claude Code Patch Reviewer