Claude review: drm/bridge: tc358762: Use mipi_dsi_generic_write_multi()

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

Patch Review

**Correctness:**

The conversion is correct. The old pattern:
```c
static void tc358762_write(struct tc358762 *ctx, u16 addr, u32 val)
{
    ...
    if (ctx->error)
        return;
    ...
    ret = mipi_dsi_generic_write(dsi, data, sizeof(data));
    if (ret < 0)
        ctx->error = ret;
}
```
is functionally equivalent to the new pattern:
```c
static void tc358762_write(struct mipi_dsi_multi_context *dsi_ctx, u16 addr, u32 val)
{
    ...
    mipi_dsi_generic_write_multi(dsi_ctx, data, sizeof(data));
}
```
because `mipi_dsi_generic_write_multi()` internally checks and sets `ctx->accum_err`, providing the same early-return-on-error and error-capture semantics.

The removal of the `int error` field from `struct tc358762` is correct — no other code references it outside the removed functions.

The return value change from `tc358762_clear_error(ctx)` to `dsi_ctx.accum_err` at the end of `tc358762_init()` is correct. The old `tc358762_clear_error()` returned the accumulated error and reset it to zero; since `dsi_ctx` is a local variable that goes out of scope immediately after the return, clearing is unnecessary.

The `struct mipi_dsi_device *dsi` lookup moved from `tc358762_write()` into `tc358762_init()` where it's used to initialize `dsi_ctx` — this is appropriate since the multi-context carries the device pointer.

**Style:** The removal of the extra blank line before the final `return` (noted in the v2 changelog) is correct; the result matches typical kernel style.

**No issues found.** 

Reviewed-by worthy as-is.

---
Generated by Claude Code Patch Reviewer