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: buddy: remove stale @p parameter from gpu_buddy_print kernel-doc Date: Thu, 04 Jun 2026 15:10:52 +1000 Message-ID: In-Reply-To: <20260530195736.5321-1-rosenp@gmail.com> References: <20260530195736.5321-1-rosenp@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm: buddy: remove stale @p parameter from gpu_buddy_print kernel-= doc Author: Rosen Penev Patches: 1 Reviewed: 2026-06-04T15:10:52.966866 --- This is a single-patch series that fixes a trivial kernel-doc warning in `d= rivers/gpu/buddy.c`. The `gpu_buddy_print()` function takes only one parame= ter (`struct gpu_buddy *mm`), but the kernel-doc comment documented a secon= d `@p` parameter that doesn't exist. The fix simply removes the stale `@p` = line. The patch is **correct and straightforward**. The function signature confir= ms there is no `@p` parameter =E2=80=94 it was likely left behind when the = function's signature was refactored (the `@p` name suggests it was once a `= struct drm_printer *p` parameter, a common pattern in DRM subsystem print f= unctions). One minor observation: the commit message uses the tag `Assisted-by: Openco= de:Big-pickle`, which is not a standard kernel tag. The recognized tag for = AI-assisted contributions is currently under discussion in the kernel commu= nity, but this non-standard tag is the submitter's/maintainer's concern, no= t a code correctness issue. **Recommendation:** Acceptable to merge as-is. --- Generated by Claude Code Patch Reviewer