Hey, Gabriel Feceoru schreef op wo 11-11-2015 om 20:27 [+0200]: > > On 11.11.2015 16:21, Jani Nikula wrote: > > On Wed, 11 Nov 2015, Ander Conselvan De Oliveira <conselvan2@xxxxxxxxx> wrote: > >> On Tue, 2015-11-10 at 14:53 +0200, Jani Nikula wrote: > >>> Ander, Maarten, where are we with this? Is it horribly wrong to merge > >>> the original patch in this ever-growing and diverging thread? > >> > >> I think the patch as is will cause problems with DP, since we might clear the > >> pll selection made in hsw_dp_set_ddi_pll_sel(). I think the easy fix > >> disregarding the discussion in this thread is to drop another memset in > >> intel_crt_compute_config(). Like this > > > > > > Ander, please post this as a proper patch for review. > > > > BR, > > Jani. > > Hi, > I tested this patch on my system and I can confirm it fixes the original > issue. > However there are a few memset in *_ddi_pll_select functions which might > not be needed anymore. For instance I tried to remove the hsw one and > didn't see any regression. Could you test http://lists.freedesktop.org/archives/intel-gfx/2015-September/075964.html ? Should be a less duct-tape fix.. ~Maarten --------------------------------------------------------------------- Intel International B.V. Registered in The Netherlands under number 34098535 Statutory seat: Haarlemmermeer Registered address: Capronilaan 37, 1119NG Schiphol-Rijk This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
