Claude review: drm/bridge: sii902x: inline i2c_check_functionality check

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Verdict: Looks good.**

The patch removes the intermediate `ret` assignment:

```c
-	ret = i2c_check_functionality(client->adapter,
-				      I2C_FUNC_SMBUS_BYTE_DATA);
-	if (!ret) {
+	if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA)) {
```

This is a clean improvement:

1. `i2c_check_functionality()` returns a boolean (true if the adapter supports the requested functionality), not an error code. Storing it in `ret` and testing `!ret` follows an error-code pattern that doesn't match the actual semantics.

2. The inlined form reads more naturally as a boolean predicate: "if the adapter doesn't support this functionality, bail out."

3. `ret` is declared at the top of the function and is still used later for actual error-code returns, so removing this one assignment doesn't leave a dangling unused variable.

One minor note: the commit message says "the function returns a boolean status rather than an error code," which is accurate. The old code wasn't *buggy* — `!ret` on a boolean works the same as `!ret` on a zero/nonzero — but the refactoring improves readability.

**Reviewed-by: appropriate.**

---
Generated by Claude Code Patch Reviewer