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/amdgpu: update outdated comment for renamed amdgpu_fence_driver_init() Date: Sun, 22 Mar 2026 03:05:09 +1000 Message-ID: In-Reply-To: <20260321105728.6304-1-kexinsun@smail.nju.edu.cn> References: <20260321105728.6304-1-kexinsun@smail.nju.edu.cn> <20260321105728.6304-1-kexinsun@smail.nju.edu.cn> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Correctness:** The change is correct. The function `amdgpu_fence_driver_sw_init()` exists at `amdgpu_fence.c:484`, confirming the rename happened. The kdoc for `amdgpu_fence_driver_init_ring()` was referencing the stale name `amdgpu_fence_driver_init()`. ```diff - * Helper function for amdgpu_fence_driver_init(). + * Helper function for amdgpu_fence_driver_sw_init(). ``` **Minor nit on the commit message:** The `Assisted-by: unnamed:deepseek-v3.2 coccinelle` tag is non-standard. The kernel community has been discussing AI-assisted contributions, but this tag format isn't established. The maintainer may want to adjust this or ask for it to be dropped/reformatted. **Overall:** Trivially correct comment-only fix. No functional impact. Fine to merge as-is (modulo the tag discussion). --- Generated by Claude Code Patch Reviewer