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

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

Patch Review

**Status: Clean**

The change replaces:
```c
ret = i2c_check_functionality(client->adapter,
                              I2C_FUNC_SMBUS_BYTE_DATA);
if (!ret) {
```
with:
```c
if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA)) {
```

This is correct because:

1. **`i2c_check_functionality()` returns boolean, not an error code.** Assigning its result to `ret` is misleading — it suggests an error-code pattern (check `ret < 0`), but the old code was already checking `!ret` which is the boolean pattern. Inlining makes the intent clearer.

2. **`ret` is still used later in the function** (line 1213: `ret = devm_regulator_bulk_get_enable(...)`) so there is no risk of an unused-variable warning from removing this one assignment.

3. **No functional change** — the behavior is identical before and after.

The commit message is accurate and concise. The patch already carries a Reviewed-by tag.

---
Generated by Claude Code Patch Reviewer