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/bridge: analogix_dp: Convert status check functions to bool Date: Mon, 25 May 2026 20:22:40 +1000 Message-ID: In-Reply-To: <20260521102716.1373519-1-damon.ding@rock-chips.com> References: <20260521102716.1373519-1-damon.ding@rock-chips.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/bridge: analogix_dp: Convert status check functions to bool Author: Damon Ding Patches: 2 Reviewed: 2026-05-25T20:22:40.229679 --- This is a single-patch cleanup that converts two status-check functions (`a= nalogix_dp_is_slave_video_stream_clock_on` and `analogix_dp_is_video_stream= _on`) from `int` (returning 0/-EINVAL) to `bool` (returning true/false), an= d updates their two callers accordingly. The conversion is **logically correct** =E2=80=94 the semantic inversion be= tween the old `=3D=3D 0` check and the new truthy check is properly handled= . Both functions are internal to this driver (no out-of-tree callers possib= le), and the grep confirms exactly one call site each. The patch is straigh= tforward and low-risk. One minor style observation below, but no correctness issues. **Verdict: Looks good. Suitable for merge.** --- Generated by Claude Code Patch Reviewer