Claude review: drm/bridge: tc358768: Separate video format config

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

Patch Review

Moves `TC358768_DATAFMT` and `TC358768_DSITX_DT` register writes from `pre_enable()` to a new `tc358768_config_video_format()` function called from `enable()`. This is a prerequisite for patch 7, which needs to write these same registers with different values for long command transmission during the pre_enable/enable window.

The `default` case in the new function calls `return` without setting `priv->error`, unlike the original code path in `pre_enable()` which had a goto to handle the error. However, the function does call `dev_err()`, and the missing format would already have been caught during the `hact` computation in `pre_enable()` (which retains the `default` case with the `goto err` path). So the video format function's default case is effectively unreachable. Acceptable.

---
Generated by Claude Code Patch Reviewer