linux-kernel - Re: [GIT PATCH] More USB patches for 2.6.18

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20060928165951.2c5bd4c7.akpm@osdl.org>
Date:	Thu, 28 Sep 2006 16:59:51 -0700
From:	Andrew Morton <akpm@...l.org>
To:	Linus Torvalds <torvalds@...l.org>
Cc:	Greg KH <gregkh@...e.de>, linux-kernel@...r.kernel.org,
	linux-usb-devel@...ts.sourceforge.net
Subject: Re: [GIT PATCH] More USB patches for 2.6.18

On Thu, 28 Sep 2006 16:40:23 -0700 (PDT)
Linus Torvalds <torvalds@...l.org> wrote:

> 
> 
> On Thu, 28 Sep 2006, Greg KH wrote:
> >
> > Here are some more USB bugfixes and device ids 2.6.18.  They should all
> > fix the reported problems in your current tree (if not, please let me
> > know.)
> > 
> > All of these changes have been in the -mm tree for a while.
> 
> Maybe I shouldn't have hurried you.
> 
> 	In file included from drivers/usb/host/ohci-hcd.c:140:
> 	drivers/usb/host/ohci-hub.c: In function 'ohci_rh_resume':
> 	drivers/usb/host/ohci-hub.c:184: error: invalid storage class for function 'ohci_restart'
> 	drivers/usb/host/ohci-hub.c:188: warning: implicit declaration of function 'ohci_restart'
> 	drivers/usb/host/ohci-hcd.c: At top level:
> 	drivers/usb/host/ohci-hcd.c:815: error: static declaration of 'ohci_restart' follows non-static declaration
> 	drivers/usb/host/ohci-hub.c:188: error: previous implicit declaration of 'ohci_restart' was here
> 	make[3]: *** [drivers/usb/host/ohci-hcd.o] Error 1
> 	make[2]: *** [drivers/usb/host] Error 2
> 	make[1]: *** [drivers/usb] Error 2
> 	make: *** [drivers] Error 2
> 

That's the "some gccs dont like static function decls in that scope" thing.

I fixed it (unpleasantly) like this:


diff -puN drivers/usb/host/ohci-hub.c~ohci-add-auto-stop-support-hack-hack drivers/usb/host/ohci-hub.c
--- a/drivers/usb/host/ohci-hub.c~ohci-add-auto-stop-support-hack-hack
+++ a/drivers/usb/host/ohci-hub.c
@@ -132,6 +132,10 @@ static inline struct ed *find_head (stru
 	return ed;
 }
 
+#ifdef CONFIG_PM
+static int ohci_restart(struct ohci_hcd *ohci);
+#endif
+
 /* caller has locked the root hub */
 static int ohci_rh_resume (struct ohci_hcd *ohci)
 __releases(ohci->lock)
@@ -181,8 +185,6 @@ __acquires(ohci->lock)
 #ifdef	CONFIG_PM
 	if (status == -EBUSY) {
 		if (!autostopped) {
-			static int ohci_restart (struct ohci_hcd *ohci);
-
 			spin_unlock_irq (&ohci->lock);
 			(void) ohci_init (ohci);
 			status = ohci_restart (ohci);
_

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/