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: Add missing rogue context reset reasons Date: Fri, 13 Mar 2026 14:16:58 +1000 Message-ID: In-Reply-To: <20260312-b4-firmware-context-reset-notification-handling-v2-1-aec5a64cb06f@imgtec.com> References: <20260312-b4-firmware-context-reset-notification-handling-v2-0-aec5a64cb06f@imgtec.com> <20260312-b4-firmware-context-reset-notification-handling-v2-1-aec5a64cb06f@imgtec.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review This patch fills in the gap (values 6=E2=80=9311) in `enum rogue_context_re= set_reason`. The values and comments look correct and consistent with the e= xisting entries. **Looks good.** No issues found. The Co-authored-by tag for Sarah Walker is= present. One minor note: this patch does not carry a `Fixes:` tag, though it could a= rguably have one since the enum was incomplete from the initial FWIF header= s commit. However, since these values weren't consumed by any code until pa= tch 3, it's reasonable to leave it without one. --- --- Generated by Claude Code Patch Reviewer