Message ID | Yxn1WpmUJnJpqq23@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [PULL] drm-intel-fixes | expand |
Hi Rodrigo, On Thu, Sep 08, 2022 at 09:59:54AM -0400, Rodrigo Vivi wrote: > Hi Dave and Daniel, > > A few fixes, but most targeting stable. > > [...] > > Ville Syrjälä (2): > drm/i915: Implement WaEdpLinkRateDataReload Don't you need to revert d5929835080a60f9119d024fa42f315913942f76 in order for "drm/i915: Implement WaEdpLinkRateDataReload" to actually be useful/interesting? Otherwise, what's the point? Regards, Jason
On Sun, 2022-09-11 at 19:22 +0200, Jason A. Donenfeld wrote: > Hi Rodrigo, > > On Thu, Sep 08, 2022 at 09:59:54AM -0400, Rodrigo Vivi wrote: > > Hi Dave and Daniel, > > > > A few fixes, but most targeting stable. > > > > [...] > > > > Ville Syrjälä (2): > > drm/i915: Implement WaEdpLinkRateDataReload > > Don't you need to revert d5929835080a60f9119d024fa42f315913942f76 in > order for "drm/i915: Implement WaEdpLinkRateDataReload" to actually > be > useful/interesting? Otherwise, what's the point? Unfortunately there was no clear indication on the revert patch for the tool to pick, and I hadn't followed that front myself closely. Should 483e3d87a37e ("Revert "drm/i915/display: Re-add check for low voltage sku for max dp source rate"") have a Fixes tag? Or should dim consider all reverts as Fixes? Anyway, I will cherry pick this to our fixes branch for propagation this week. Thanks for the heads up, Rodrigo. > > Regards, > Jason