Claude review: drm/i915/dp: Account for AS_SDP guardband only when enabled

Claude Code Review Bot
2026-03-31T07:39:46Z

Patch Review

**Concern:** This patch removes the `assume_all_enabled` path for AS SDP guardband:
```c
- if ((assume_all_enabled && HAS_AS_SDP(display)) ||
- crtc_state->infoframes.enable & intel_hdmi_infoframe_enable(DP_SDP_ADAPTIVE_SYNC))
+ if (crtc_state->infoframes.enable &
+ intel_hdmi_infoframe_enable(DP_SDP_ADAPTIVE_SYNC))
```

The `assume_all_enabled` parameter is used by callers that need worst-case guardband estimates. By removing the AS SDP case from this path, those callers may underestimate the guardband. This is presumably intentional because patch 19 will always enable AS SDP when supported, but between patches 7 and 19 there could be a regression window. No R-b tag.

Also removes the `display` variable but doesn't note this is a side effect — minor style nit.

---
Generated by Claude Code Patch Reviewer