Claude review: drm/i915/dp: Split AS SDP computation between compute_config and compute_config_late

Claude Code Review Bot
2026-04-12T02:51:30Z

Patch Review

This is the key architectural patch. The AS SDP computation is split:
- Early (`intel_dp_compute_as_sdp()`): only sets the `infoframes.enable` bit
- Late (`intel_dp_as_sdp_compute_config_late()`): fills in all actual AS SDP fields

This is needed because the guardband calculation (which runs between these phases) needs to know whether AS SDP is enabled, but the AS SDP fields themselves depend on feature decisions made later.

The function `intel_dp_sdp_compute_config_late()` now takes `intel_dp` as a parameter to pass through to `intel_dp_as_sdp_compute_config_late()`.

Sound design. Missing R-b.