From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 4/12] atyfb: Remove unneeded calls to wait_for_idle Date: Mon, 1 May 2006 13:20:23 +0300 Message-ID: <20060501102023.GA9104@sci.fi> References: <4452B422.2080907@gmail.com> Reply-To: linux-fbdev-devel@lists.sourceforge.net Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Return-path: Received: from sc8-sf-mx1-b.sourceforge.net ([10.3.1.91] helo=mail.sourceforge.net) by sc8-sf-list1.sourceforge.net with esmtp (Exim 4.30) id 1FaVW8-0003n4-Ss for linux-fbdev-devel@lists.sourceforge.net; Mon, 01 May 2006 03:20:36 -0700 Received: from gw02.mail.saunalahti.fi ([195.197.172.116]) by mail.sourceforge.net with esmtp (Exim 4.44) id 1FaVW7-0005Hb-ER for linux-fbdev-devel@lists.sourceforge.net; Mon, 01 May 2006 03:20:36 -0700 Content-Disposition: inline In-Reply-To: <4452B422.2080907@gmail.com> Sender: linux-fbdev-devel-admin@lists.sourceforge.net Errors-To: linux-fbdev-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Id: List-Post: List-Help: List-Subscribe: , List-Archive: Content-Type: text/plain; charset="iso-8859-1" To: "Antonino A. Daplas" Cc: Andrew Morton , Linux Fbdev development list , Alexander Kern On Sat, Apr 29, 2006 at 08:32:34AM +0800, Antonino A. Daplas wrote: > The drawing functions of atyfb is unecessary syncing the GPU which is a= ffecting > performance. Remove the calls, any direct access by fbcon to the frame= buffer > will always be preceeded by a call to atyfb_sync(). Still works on my Mobility :) I'll try to test with older chips later=20 this week, but I don't expect any problems. AFAIK the only case when an=20 engine sync is necessary is 3D/scaler <-> 2D transition on pre-Rage Pro=20 chips. --=20 Ville Syrj=E4l=E4 syrjala@sci.fi http://www.sci.fi/~syrjala/ ------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642