From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f65.google.com ([74.125.82.65]:36622 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751452AbcFBNMb (ORCPT ); Thu, 2 Jun 2016 09:12:31 -0400 Received: by mail-wm0-f65.google.com with SMTP id a20so4171658wma.3 for ; Thu, 02 Jun 2016 06:12:31 -0700 (PDT) Date: Thu, 2 Jun 2016 15:12:26 +0200 From: Daniel Vetter To: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Cc: Chris Wilson , Radek =?iso-8859-1?Q?Dost=E1l?= , dri-devel@lists.freedesktop.org, Jesse Barnes , Daniel Vetter , Julia Lemire , Dave Airlie , stable@vger.kernel.org Subject: Re: [PATCH v4] drm: Only create a cmdline mode if no probed modes match Message-ID: <20160602131226.GF7231@phenom.ffwll.local> References: <20160601094353.GC4329@intel.com> <1464774651-20376-1-git-send-email-chris@chris-wilson.co.uk> <574FFE92.6060806@radekdostal.com> <20160602105217.GL10319@nuc-i3427.alporthouse.com> <20160602113040.GK4329@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160602113040.GK4329@intel.com> Sender: stable-owner@vger.kernel.org List-ID: On Thu, Jun 02, 2016 at 02:30:40PM +0300, Ville Syrj�l� wrote: > On Thu, Jun 02, 2016 at 11:52:17AM +0100, Chris Wilson wrote: > > On Thu, Jun 02, 2016 at 11:38:26AM +0200, Radek Dost�l wrote: > > > On 06/01/2016 11:50 AM, Chris Wilson wrote: > > > >Fixes regression from > > > > > > > >commit eaf99c749d43ae74ac7ffece5512f3c73f01dfd2 > > > >Author: Chris Wilson > > > >Date: Wed Aug 6 10:08:32 2014 +0200 > > > > > > > > drm: Perform cmdline mode parsing during connector initialisation > > > > > > > >that breaks HDMI output on BeagleBone Black with LG TV (model 19LS4R-ZA). > > > > > > please remove this from the commit message. The original bug is no > > > longer reproducible with 4.7-rc1 > > > > If there's no motivation for the patch anymore, it can just wither away > > in one of my old trees. > > > > Does anyone care about pruning the autogenerated video= mode if a probed > > one matches? Presumably, it is still visible to userspace and switching > > to it will cause the same issue as before? Or was it always a driver > > bug (failing to set the mode)? > > IMO the patch makes total sense even if it's not needed for this > particular bug. Feel free to add > > Reviewed-by: Ville Syrj�l� Agreed. I dropped the cc: stable and adjusted the commit message to explain the situation. Merged to drm-misc, thanks. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch