From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrian Bunk Subject: Re: [2.6 patch] drivers/video/: possible cleanups Date: Sun, 6 Nov 2005 12:17:44 +0100 Message-ID: <20051106111743.GA3847@stusta.de> References: <20051106005026.GE3668@stusta.de> <436D9AF3.8040008@pol.net> Reply-To: linux-fbdev-devel@lists.sourceforge.net Mime-Version: 1.0 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 1EYiXm-0001OH-8X for linux-fbdev-devel@lists.sourceforge.net; Sun, 06 Nov 2005 03:18:38 -0800 Received: from emailhub.stusta.mhn.de ([141.84.69.5] helo=mailout.stusta.mhn.de) by mail.sourceforge.net with smtp (Exim 4.44) id 1EYiXk-0004NS-Ly for linux-fbdev-devel@lists.sourceforge.net; Sun, 06 Nov 2005 03:18:38 -0800 Content-Disposition: inline In-Reply-To: <436D9AF3.8040008@pol.net> 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="us-ascii" Content-Transfer-Encoding: 7bit To: "Antonino A. Daplas" Cc: linux-fbdev-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org On Sun, Nov 06, 2005 at 01:56:03PM +0800, Antonino A. Daplas wrote: > Adrian Bunk wrote: > > This patch contains the possible cleanups including the following: > > - every file should #include the headers containing the prototypes for > > it's global functions > > - make needlessly global functions static > > - kyro/STG4000Interface.h: #include video/kyro.h and linux/pci.h > > instead of a manual "struct pci_dev" > > - i810_main.{c,h}: prototypes for static functions belong to the > > C file > > > > > > Signed-off-by: Adrian Bunk > > > > --- > > > > drivers/video/arcfb.c | 8 +-- > > drivers/video/console/softcursor.c | 2 > > drivers/video/i810/i810-i2c.c | 1 > > drivers/video/i810/i810_accel.c | 1 > > drivers/video/i810/i810_gtf.c | 1 > > drivers/video/i810/i810_main.c | 51 ++++++++++++++++++-- > > drivers/video/i810/i810_main.h | 56 +--------------------- > > drivers/video/kyro/STG4000InitDevice.c | 1 > > drivers/video/kyro/STG4000Interface.h | 3 - > > drivers/video/kyro/STG4000OverlayDevice.c | 1 > > drivers/video/matrox/matroxfb_g450.c | 2 > > drivers/video/nvidia/nv_hw.c | 1 > > drivers/video/tdfxfb.c | 2 > > 13 files changed, 68 insertions(+), 62 deletions(-) > > > > --- linux-2.6.14-rc5-mm1-full/drivers/video/console/softcursor.c.old 2005-11-06 00:31:15.000000000 +0100 > > +++ linux-2.6.14-rc5-mm1-full/drivers/video/console/softcursor.c 2005-11-06 00:31:30.000000000 +0100 > > @@ -17,6 +17,8 @@ > > #include > > #include > > > > +#include "fbcon.h" > > I don't think softcursor needs anything in fbcon.h. The rest looks okay. fbcon.h contains the function prototype for soft_cursor(). > Tony cu Adrian -- "Is there not promise of rain?" Ling Tan asked suddenly out of the darkness. There had been need of rain for many days. "Only a promise," Lao Er said. Pearl S. Buck - Dragon Seed ------------------------------------------------------- SF.Net email is sponsored by: Tame your development challenges with Apache's Geronimo App Server. Download it for free - -and be entered to win a 42" plasma tv or your very own Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php