From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Richter Subject: Re: [PATCH 6/6] drm/i915: i830M has watermarks like i855 Date: Mon, 02 Dec 2013 15:27:51 +0100 Message-ID: <529C98E7.4010007@rus.uni-stuttgart.de> References: <1385990771-3877-1-git-send-email-rodrigo.vivi@gmail.com> <1385990771-3877-7-git-send-email-rodrigo.vivi@gmail.com> Reply-To: richter@rus.uni-stuttgart.de Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from medousa.rus.uni-stuttgart.de (medousa.rus.uni-stuttgart.de [129.69.192.4]) by gabe.freedesktop.org (Postfix) with ESMTP id 937BFFA543 for ; Mon, 2 Dec 2013 06:28:02 -0800 (PST) In-Reply-To: <1385990771-3877-7-git-send-email-rodrigo.vivi@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: Rodrigo Vivi Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On 12/02/2013 02:26 PM, Rodrigo Vivi wrote: > From: Daniel Vetter > > So shuffle the checks around a bit. Also give all the structs and > functions proper prefixes: i830_ for the dual-pipe mobile platforms > and i845_ for the two single-pipe desktop platforms. > > Note that the max fifo value isn't actually correct for the i830M, but > since we don't frob the fifo split we don't actually need it. This is > different for some gen3 devices where we need the full fifo for self > refresh mode. > Thanks, will try, possibly next Friday. I'm unfortunately booked until then. Greetings, Thomas