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: sii902x: inline i2c_check_functionality check Date: Thu, 04 Jun 2026 12:14:08 +1000 Message-ID: In-Reply-To: <20260602223102.1087000-3-thorsten.blum@linux.dev> References: <20260602223102.1087000-3-thorsten.blum@linux.dev> 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: sii902x: inline i2c_check_functionality check Author: Thorsten Blum Patches: 2 Reviewed: 2026-06-04T12:14:08.715137 --- This is a single, trivial cleanup patch. It inlines the `i2c_check_function= ality()` call directly into the `if` condition, removing the unnecessary in= termediate `ret` variable assignment. The change is correct: `i2c_check_fun= ctionality()` returns a `bool` (actually `u32` interpreted as boolean), not= an error code, so storing it in `int ret` was misleading =E2=80=94 it was = never used as an error code, only checked for zero/non-zero. The patch is safe and already has a Reviewed-by tag from Dmitry Baryshkov. **Verdict: No issues. Good to merge.** --- Generated by Claude Code Patch Reviewer