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/imagination: Rename fence returned by pvr_queue_job_arm() Date: Tue, 31 Mar 2026 17:33:25 +1000 Message-ID: In-Reply-To: <20260330-job-submission-fixes-cleanup-v1-5-7de8c09cef8c@imgtec.com> References: <20260330-job-submission-fixes-cleanup-v1-0-7de8c09cef8c@imgtec.com> <20260330-job-submission-fixes-cleanup-v1-5-7de8c09cef8c@imgtec.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Status**: Good rename. Renaming `done_fence` to `finished_fence` in `prepare_job_syncs()` and `pvr_sync_signal_array_update_fences()` avoids confusion with `pvr_job::done_fence` (which is a different thing - the internal job fence vs. the scheduler's finished fence). **No issues.** --- Generated by Claude Code Patch Reviewer