From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Antonino A. Daplas" Subject: [PATCH 5/8] atyfb: Honor the return value of pci_register_driver Date: Thu, 10 Aug 2006 20:30:53 +0800 Message-ID: <44DB26FD.7030205@gmail.com> References: <44DB1DA4.3000703@gmail.com> Reply-To: linux-fbdev-devel@lists.sourceforge.net Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from sc8-sf-mx2-b.sourceforge.net ([10.3.1.92] helo=mail.sourceforge.net) by sc8-sf-list1-new.sourceforge.net with esmtp (Exim 4.43) id 1GB9gk-0005wq-DK for linux-fbdev-devel@lists.sourceforge.net; Thu, 10 Aug 2006 05:31:02 -0700 Received: from nz-out-0102.google.com ([64.233.162.200]) by mail.sourceforge.net with esmtp (Exim 4.44) id 1GB9gh-0005Pk-Lw for linux-fbdev-devel@lists.sourceforge.net; Thu, 10 Aug 2006 05:31:02 -0700 Received: by nz-out-0102.google.com with SMTP id m22so119573nzf for ; Thu, 10 Aug 2006 05:30:58 -0700 (PDT) In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-fbdev-devel-bounces@lists.sourceforge.net Errors-To: linux-fbdev-devel-bounces@lists.sourceforge.net To: linux-fbdev-devel@lists.sourceforge.net Cc: Andrew Morton , Geert Uytterhoeven Check return value of pci_register_driver(). Signed-off-by: Antonino Daplas --- Geert Uytterhoeven wrote: > > Actually it's a bit more complicated, as some Ataris do have PCI (although > currently Atari Hades PCI support is broken). > How about this? drivers/video/aty/atyfb_base.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/video/aty/atyfb_base.c b/drivers/video/aty/atyfb_base.c index 053ff63..3f1bdcd 100644 --- a/drivers/video/aty/atyfb_base.c +++ b/drivers/video/aty/atyfb_base.c @@ -3859,6 +3859,7 @@ #endif /* MODULE */ static int __devinit atyfb_init(void) { + int err = 0; #ifndef MODULE char *option = NULL; @@ -3868,12 +3869,14 @@ #ifndef MODULE #endif #ifdef CONFIG_PCI - pci_register_driver(&atyfb_driver); + err = pci_register_driver(&atyfb_driver); #endif #ifdef CONFIG_ATARI - atyfb_atari_probe(); + if (!err) + err = atyfb_atari_probe(); #endif - return 0; + + return err; } static void __exit atyfb_exit(void) ------------------------------------------------------------------------- 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