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/xe: update outdated reference to xe_pt_commit_bind() Date: Sun, 22 Mar 2026 03:03:03 +1000 Message-ID: In-Reply-To: <20260321105748.6646-1-kexinsun@smail.nju.edu.cn> References: <20260321105748.6646-1-kexinsun@smail.nju.edu.cn> <20260321105748.6646-1-kexinsun@smail.nju.edu.cn> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Correctness**: The comment fix is accurate. The old function `xe_pt_commi= t_bind()` was indeed split, and the two functions referenced (`xe_pt_commit= _prepare_bind()` at line 1115 and `xe_pt_commit()` at line 1058) both exist= and are used in the codebase. **Nit on the "Assisted-by" tag**: The tag `Assisted-by: unnamed:deepseek-v3= .2 coccinelle` is non-standard. The kernel community has debated AI-assiste= d contribution tags, and this format doesn't follow any established convent= ion. It's unclear what "unnamed:deepseek-v3.2 coccinelle" means =E2=80=94 w= as Coccinelle used to find the stale reference and DeepSeek used to draft t= he fix? A maintainer may ask for clarification or removal of this tag. **Minor**: The commit message accurately references commit `a708f6501c69` a= s the source of the split, which is good practice for traceability. **Verdict**: The code change itself is correct and trivial. The only point = of discussion is the non-standard `Assisted-by` trailer. --- Generated by Claude Code Patch Reviewer