From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com ([192.55.52.120]:10403 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750862AbcFBLbA (ORCPT ); Thu, 2 Jun 2016 07:31:00 -0400 Date: Thu, 2 Jun 2016 14:30:40 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: 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: <20160602113040.GK4329@intel.com> 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> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160602105217.GL10319@nuc-i3427.alporthouse.com> Sender: stable-owner@vger.kernel.org List-ID: 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� -- Ville Syrj�l� Intel OTC