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: Mon, 18 May 2026 16:02:26 +1000 Message-ID: In-Reply-To: <20260517172136.3321-2-thorsten.blum@linux.dev> References: <20260517172136.3321-2-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-05-18T16:02:26.028511 --- This is a single, trivial cleanup patch. The change is correct: `i2c_check_= functionality()` returns `bool` (via `__i2c_check_functionality()` which re= turns `u32` used as a boolean), not an error code. Storing the boolean resu= lt in `int ret` and then testing `!ret` works but is misleading =E2=80=94 i= t makes it look like an error-code check pattern. Inlining the call removes= the unnecessary variable assignment and makes the control flow clearer. No functional change. No risk. --- Generated by Claude Code Patch Reviewer