From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 00/58] modeset-rework, the basic conversion Date: Thu, 6 Sep 2012 08:55:40 +0200 Message-ID: <20120906065539.GA5523@phenom.ffwll.local> References: <1345403595-9678-1-git-send-email-daniel.vetter@ffwll.ch> <20120905162355.3d7fe823@jbarnes-desktop> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f177.google.com (mail-wi0-f177.google.com [209.85.212.177]) by gabe.freedesktop.org (Postfix) with ESMTP id EA94C9E797 for ; Wed, 5 Sep 2012 23:55:12 -0700 (PDT) Received: by wibhn17 with SMTP id hn17so828610wib.12 for ; Wed, 05 Sep 2012 23:55:12 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20120905162355.3d7fe823@jbarnes-desktop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Jesse Barnes Cc: Daniel Vetter , Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Wed, Sep 05, 2012 at 04:23:55PM -0700, Jesse Barnes wrote: > On Sun, 19 Aug 2012 21:12:17 +0200 > Daniel Vetter wrote: > > > Hi all, > > > > Changes since last time around: > > - The prep patches are all merged now. > > - I've left out the actual DP fixes/cleanups, I think we should merge those in a > > separte step. > > - A few bugfixes (thanks to Paulo, Jani and Chris). > > - I've also applied a few bikesheds for naming that Paulo suggested (but I'm not > > sure whether I've sent those out already in a previous patchbomb). > > > > Essentially this is just the core rework, which addes the new get_hw_state code, > > refactors all the encoders to use the new functions and finally reworks the > > modeset logic to disable/enable entire pipes, always (and with a deterministic > > order). > > > > For merging to -next, I plan to pull in everything with a real merge commit. For > > that reason I've put up a modeset-rework-base branch onto my private fdo repo[1]. > > That way I can put a short documentation for the new modeset design into the > > merge commit (stichted together from the previous patchbomb cover letters), > > documenting my folly assumptions for eternity. > > > > I'll also plan to put tags for the entire series in the merge commit, so if you > > have tested this on a few machines, read through and agree with the new designs, > > please reply with your tested-by/acked-by/reviewed-by tags. > > > > Flames, comments and test reports highly welcome. > > Ok I've tested on Ironlake, Cantiga, Crestline, and Pineview so far and > things look good. I ran testdisplay both with and without VGA > attached (the ILK has a eDP panel), and tried S3 and S4 both with and > without VGA both in the console and in X. > > There was one issue on Pineview where the VGA seemed to get > "forgotten", but I haven't isolated it yet. I canceled testdisplay > part way through and that seemed to confuse fbcon about what was there. QA reported a similar issue where fbcon refuses to light up the display after testdisplay completed a while back: https://bugs.freedesktop.org/show_bug.cgi?id=42194 > I also see an issue with 1280x800 modes across all platforms, but that > may just be the monitor, I need to test more. > > I'm testing Montara (that's 855 for you youngsters) now, but it's slow > so not all my builds have completed there yet. > > Overall though: > Tested-by: Jesse Barnes Cool, thanks a lot for review&testing. I'll do the merge now (need to slap all your r-b's onto patches first). -Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48