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

Claude Code Review Bot
2026-03-11T20:55:29Z

Patch Review

Removes the `assume_all_enabled && HAS_AS_SDP(display)` check and only accounts for AS SDP guardband when it's actually enabled in `crtc_state->infoframes.enable`.

**Concern:** The `assume_all_enabled` parameter was presumably there for a reason — to conservatively compute guardbands during early configuration when individual SDP enables aren't yet known. By removing the AS SDP case from the `assume_all_enabled` path, this could under-count guardbands in scenarios where `assume_all_enabled=true` is passed. However, patch 19 later always enables AS SDP when supported, which may compensate. The ordering here means there's a window between patches 7 and 19 where guardbands could be under-counted.

---
Generated by Claude Code Patch Reviewer