On Thu, 26 Sep 2024, Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> wrote: > On Wed, 18 Sep 2024, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: >> + /* >> + * Use DP_LINK_RATE_SET if DP_SUPPORTED_LINK_RATES are available, >> + * default to DP_MAX_LINK_RATE and DP_LINK_BW_SET otherwise. >> + */ >> + if (intel_dp->num_sink_rates) >> + intel_dp->use_rate_select = true; >> + else >> + intel_dp_set_sink_rates(intel_dp); > > Isn't this kind of in the wrong place, it's not eDP? -ENOCOFFEE, sorry for the noise. BR, Jani. -- Jani Nikula, Intel