Re: [PATCH i-g-t v5 1/4] lib/i915/i915_dp: Add UHBR helpers and const-correct set_link_params

["Joshi, Kunal1" <kunal1.joshi@intel.com>]

On 17-08-2026 13:56, Sowmiya S wrote:
> Add i915_dp_is_uhbr_rate() and i915_dp_get_next_lower_rate(), prep for
> the UHBR-to-HBR fallback subtest later in this series; no caller yet.
>
> i915_dp_get_next_lower_rate() parses intel_dp_allowed_link_configs
> (source & sink & current limits) rather than i915_dp_force_link_rate,
> since the latter is source-only and lets the kernel silently clamp a
> rate the sink never advertised.
>
> Also take link_rate/lane_count as const char * in
> i915_dp_set_link_params() so callers can pass string literals.
>
> v4: - name the UHBR10 threshold and make i915_dp_is_uhbr_rate() a
>        static inline in the header to avoid a cross-TU call (Kunal)
>      - parse intel_dp_allowed_link_configs instead of the source-only
>        i915_dp_force_link_rate list in i915_dp_get_next_lower_rate()
>        (Kunal)
>      - avoid the implicit long to int narrowing on the return value
>        (Kunal)
>
> Signed-off-by: Sowmiya S <sowmiya.s@intel.com>
Reviewed-by: Kunal Joshi <kunal1.joshi@intel.com>