From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com ([134.134.136.65]:52167 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753168AbcERSJz (ORCPT ); Wed, 18 May 2016 14:09:55 -0400 Date: Wed, 18 May 2016 21:09:49 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Daniel Vetter Cc: Intel Graphics Development , Daniel Vetter , stable , "Pandiyan, Dhinakaran" , Rodrigo Vivi Subject: Re: [Intel-gfx] [PATCH 2/7] drm/i915/psr: Try to program link training times correctly Message-ID: <20160518180949.GC4329@intel.com> References: <1463590036-17824-1-git-send-email-daniel.vetter@ffwll.ch> <1463590036-17824-2-git-send-email-daniel.vetter@ffwll.ch> <20160518173922.GX4329@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Wed, May 18, 2016 at 08:04:02PM +0200, Daniel Vetter wrote: > On Wed, May 18, 2016 at 7:39 PM, Ville Syrj�l� > wrote: > > On Wed, May 18, 2016 at 06:47:11PM +0200, Daniel Vetter wrote: > >> Oops. Hw default for programming these fields to 0 is "skip link > >> training". Display won't take that too well usually. > > > > s/skip/500 usec/ > > Yeah, my reading skills have reached an all time low ;-) But we > confirmed on irc that the hardcoded 500usec was indeed wrong, since > the fixed machines now run on 2.5ms of link training time. I'll update > the commit message when merging to reflect that. Also > > Tested-by: Ville Syrj�l� That can be slapped onto the entire series. > Tested-by: fritsch@kodi.tv > -- > Daniel Vetter > Software Engineer, Intel Corporation > +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- Ville Syrj�l� Intel OTC