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/panthor: correct firmware related messages Date: Wed, 25 Mar 2026 08:01:15 +1000 Message-ID: In-Reply-To: <20260323081132.3217646-1-christianshewitt@gmail.com> References: <20260323081132.3217646-1-christianshewitt@gmail.com> <20260323081132.3217646-1-christianshewitt@gmail.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 **Fix 1** (line 110): Changes `"Firmware protected mode entry not be suppor= ted, ignoring"` to `"Firmware protected mode entry is not supported, ignori= ng"` =E2=80=94 clearly a grammar fix for a broken sentence ("not be support= ed" =E2=86=92 "is not supported"). Correct. **Fix 2** (line 120): Changes `"Firmware entry isn't 32 bit aligned"` to `"= Firmware entry is not 32-bit aligned"` =E2=80=94 two changes here: - `isn't` =E2=86=92 `is not`: stylistic preference for formal language in k= ernel messages. Reasonable, though the kernel has no strict rule against co= ntractions. - `32 bit` =E2=86=92 `32-bit`: correct hyphenation when used as a compound = adjective. This is the more substantive language fix. Both changes confirmed against the current drm-next tree. The patch is stra= ightforward, low-risk, and improves message clarity. **Reviewed-by: appropriate** =E2=80=94 no issues found. --- Generated by Claude Code Patch Reviewer