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/panel: jadard-jd9365da-h3: Fix signedness bug Date: Tue, 05 May 2026 12:04:55 +1000 Message-ID: In-Reply-To: <20260429024954.697411-1-ethantidmore06@gmail.com> References: <20260429024954.697411-1-ethantidmore06@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm/panel: jadard-jd9365da-h3: Fix signedness bug Author: Ethan Tidmore Patches: 2 Reviewed: 2026-05-05T12:04:55.315030 --- This is a single-patch fix for a real signedness bug detected by Smatch. Th= e function `drm_of_get_data_lanes_count_remote()` returns `int` (including = negative error codes), but its return value was being stored directly into = `dsi->lanes`, which is `unsigned int`. This made the subsequent `dsi->lanes= < 0` check dead code =E2=80=94 it could never be true. The fix is correct,= minimal, and follows standard kernel patterns for handling this class of b= ug. The patch is suitable for merging. One minor nit below. --- Generated by Claude Code Patch Reviewer