From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Richter Subject: Re: [PATCH] Workaround for flicker with panning on the i830 Date: Tue, 12 Nov 2013 17:41:12 +0100 Message-ID: <52825A28.3040500@math.tu-berlin.de> References: <1383818746-1199-1-git-send-email-daniel.vetter@ffwll.ch> <20131107134516.GP5986@intel.com> <32493_1383921850_527CF8B9_32493_10045_1_20131108075743.GZ14082@phenom.ffwll.local> <527D024D.8020000@math.tu-berlin.de> <32493_1383928311_527D11F3_32493_10984_1_20131108163213.GC14082@phenom.ffwll.local> <5280F8DF.5020205@math.tu-berlin.de> <1565_1384184620_5280FB2C_1565_9181_1_CAKMK7uF2UmKJHvVPrzE7-7A9DQ5JrLHAFnDiuVUDHFU+DoOXww@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from charybdis.rus.uni-stuttgart.de (smtp.uni-stuttgart.de [129.69.192.2]) by gabe.freedesktop.org (Postfix) with ESMTP id AF14FFA9DE for ; Tue, 12 Nov 2013 08:50:29 -0800 (PST) In-Reply-To: <1565_1384184620_5280FB2C_1565_9181_1_CAKMK7uF2UmKJHvVPrzE7-7A9DQ5JrLHAFnDiuVUDHFU+DoOXww@mail.gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Daniel Vetter Cc: intel-gfx List-Id: intel-gfx@lists.freedesktop.org Am 11.11.2013 16:43, schrieb Daniel Vetter: > Oh, that's really interesting. gen2 has a unified display fifo on > machines that support 2 outputs. DSPARB tells the hw how to exactly > split this up between the two pipes. There are two bit ranges of > interest here: /* snip */ Hmm, why I understand *that* it does make a difference, I do not understand the details. By a unified display fifo do you mean that the display output has an internal buffer memory (the fifo) which basically feeds the the DVOs or the LVDS with memory, which comes via DMA into the fifo. Is that right? By "split", do you mean that a fixed amount of bytes (or rather, lines as in multiples of 16 bytes) are allocated for each participating pipe? Simply enlarging the fifo does not help (i.e. writing a larger value into the register). Just the positions where I get the flicker change, but the problem does not go away. So whatever needs to be done is to adjust this register according to the alignment of the base address of the corresponding DMA engine that feeds the pipe. > What we'd need to do here is to update this register when switching > the number of active display pipes in the ->modeset_global_resources > hook. We also need to make sure we have updated watermark values set > up already, before rewriting the value of DSPARB (since the watermarks > depend upon the size of the fifo). > > For I start I'd go with splitting the fifo according to the display > clock between plane A and B and giving nothing to plane C. We don't > have any code to use plane C so giving everything to just A and B is > better. By that you mean "BEND = maximum" and "AEND" in between? That does not seem to be sufficient. It needs to be modified according to the buffer alignment, and sometimes smaller values work, sometimes larger ones. Greetings, Thomas