Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-12 Thread Rodrigo Vivi
On Thu, Oct 12, 2017 at 09:30:37PM +, James Ausmus wrote: > Per BSpec, 400us is "BDW+ Do not use this setting." - not just PORT_A. > Set BDW to 600us unconditionally. > > v2: > -Split in to two patches (Rodrigo) > > Cc: Jani Nikula > Cc: Ville Syrjälä

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-12 Thread Ausmus, James
On Thu, Oct 12, 2017 at 1:59 PM, Rodrigo Vivi wrote: > On Thu, Oct 12, 2017 at 06:58:20PM +, Ausmus, James wrote: >> On Wed, Oct 4, 2017 at 3:54 PM, Runyan, Arthur J >> wrote: >> > I think the failure was with one particularly slow eDP

[Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-12 Thread James Ausmus
Per BSpec, 400us is "BDW+ Do not use this setting." - not just PORT_A. Set BDW to 600us unconditionally. v2: -Split in to two patches (Rodrigo) Cc: Jani Nikula Cc: Ville Syrjälä Signed-off-by: James Ausmus ---

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-12 Thread Rodrigo Vivi
On Thu, Oct 12, 2017 at 06:58:20PM +, Ausmus, James wrote: > On Wed, Oct 4, 2017 at 3:54 PM, Runyan, Arthur J > wrote: > > I think the failure was with one particularly slow eDP panel, but it is > > safer to apply this to all ports. > > > > Thanks Art. Anyone else

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-12 Thread Ausmus, James
On Wed, Oct 4, 2017 at 3:54 PM, Runyan, Arthur J wrote: > I think the failure was with one particularly slow eDP panel, but it is safer > to apply this to all ports. > Thanks Art. Anyone else have thoughts/comments on this, or should this be good for merge now?

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-04 Thread Runyan, Arthur J
I think the failure was with one particularly slow eDP panel, but it is safer to apply this to all ports. -Original Message- From: Vivi, Rodrigo Sent: Wednesday, 4 October, 2017 1:25 PM To: Ausmus, James Cc: intel-gfx@lists.freedesktop.org;

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-04 Thread Rodrigo Vivi
On Wed, Oct 04, 2017 at 08:09:22PM +, James Ausmus wrote: > Per BSpec, 400us is "BDW+ Do not use this setting." - not just PORT_A. > Set BDW to 600us unconditionally. Besides that statement I also found on BSpec: " Workaround Project BDW, EXCLUDE(CHV) Set the Timeout timer value to at least

[Intel-gfx] [PATCH v2 2/2] drm/i915/bdw: Fix DP_AUX_CH_CTL_TIME_OUT setting

2017-10-04 Thread James Ausmus
Per BSpec, 400us is "BDW+ Do not use this setting." - not just PORT_A. Set BDW to 600us unconditionally. v2: -Split in to two patches (Rodrigo) Cc: Jani Nikula Cc: Ville Syrjälä Signed-off-by: James Ausmus ---