Claude review: drm/msm/dp: add an API to initialize MST on sink side

Claude Code Review Bot
2026-04-12T00:16:18Z

Patch Review

**Hardcoded 100ms delay**:
```c
+ if (old_mstm_ctrl) {
+ drm_dbg_dp(dp->drm_dev, "wait %luus to set DP_MSTM_CTRL set 0\n",
+ clear_mstm_ctrl_timeout_us);
+ usleep_range(clear_mstm_ctrl_timeout_us,
+ clear_mstm_ctrl_timeout_us + 1000);
+ }
```
A 100ms unconditional sleep to wait for the sink to clear MST state is excessive. The DP spec doesn't mandate a specific delay here. This should either poll for completion or use a shorter delay. At minimum, a comment citing the rationale or spec section would help.

---
Generated by Claude Code Patch Reviewer