* [2.6 patch] orinoco.h: "extern inline" -> "static __always_inline"
@ 2006-08-15 0:40 Adrian Bunk
2006-08-15 19:26 ` Pavel Roskin
0 siblings, 1 reply; 2+ messages in thread
From: Adrian Bunk @ 2006-08-15 0:40 UTC (permalink / raw)
To: proski, hermes; +Cc: orinoco-devel, linux-kernel, netdev, linville, jgarzik
"extern inline" generates a warning with -Wmissing-prototypes and I'm
currently working on getting the kernel cleaned up for adding this to
the CFLAGS since it will help us to avoid a nasty class of runtime
errors.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
drivers/net/wireless/orinoco.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
--- linux-2.6.18-rc4-mm1/drivers/net/wireless/orinoco.h.old 2006-08-13 23:14:05.000000000 +0200
+++ linux-2.6.18-rc4-mm1/drivers/net/wireless/orinoco.h 2006-08-13 23:14:39.000000000 +0200
@@ -138,8 +138,8 @@
* SPARC, due to its weird semantics for save/restore flags. extern
* inline should prevent the kernel from linking or module from
* loading if they are not inlined. */
-extern inline int orinoco_lock(struct orinoco_private *priv,
- unsigned long *flags)
+static __always_inline int orinoco_lock(struct orinoco_private *priv,
+ unsigned long *flags)
{
spin_lock_irqsave(&priv->lock, *flags);
if (priv->hw_unavailable) {
@@ -151,8 +151,8 @@
return 0;
}
-extern inline void orinoco_unlock(struct orinoco_private *priv,
- unsigned long *flags)
+static __always_inline void orinoco_unlock(struct orinoco_private *priv,
+ unsigned long *flags)
{
spin_unlock_irqrestore(&priv->lock, *flags);
}
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [2.6 patch] orinoco.h: "extern inline" -> "static __always_inline"
2006-08-15 0:40 [2.6 patch] orinoco.h: "extern inline" -> "static __always_inline" Adrian Bunk
@ 2006-08-15 19:26 ` Pavel Roskin
0 siblings, 0 replies; 2+ messages in thread
From: Pavel Roskin @ 2006-08-15 19:26 UTC (permalink / raw)
To: Adrian Bunk
Cc: hermes, orinoco-devel, linux-kernel, netdev, linville, jgarzik
On Tue, 2006-08-15 at 02:40 +0200, Adrian Bunk wrote:
> "extern inline" generates a warning with -Wmissing-prototypes and I'm
> currently working on getting the kernel cleaned up for adding this to
> the CFLAGS since it will help us to avoid a nasty class of runtime
> errors.
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
Thanks for the patch! I'm removing the last sentence in the comment as
irrelevant and fixing the comment. The patch is being sent to netdev by
StGIT, cc: author.
--
Regards,
Pavel Roskin
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2006-08-15 19:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-08-15 0:40 [2.6 patch] orinoco.h: "extern inline" -> "static __always_inline" Adrian Bunk
2006-08-15 19:26 ` Pavel Roskin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).