[PATCH] scripts/kernel-doc: fix fatal error caused by cfg80211.h

[PATCH] scripts/kernel-doc: fix fatal error caused by cfg80211.h

[Randy Dunlap]

From: Randy Dunlap <rdunlap@xenotime.net>

include/net/cfg80211.h uses __must_check in functions that
have kernel-doc notation.  This was confusing scripts/kernel-doc,
so have scripts/kernel-doc ignore "__must_check".

Error(include/net/cfg80211.h:2702): cannot understand prototype: 'struct cfg80211_bss * __must_check cfg80211_inform_bss(...)

Signed-off-by: Randy Dunlap <rdunlap@xenotime.net>
Cc:	Johannes Berg <johannes@sipsolutions.net>
---
 scripts/kernel-doc |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

--- lnx-33-rc1.orig/scripts/kernel-doc
+++ lnx-33-rc1/scripts/kernel-doc
@@ -5,7 +5,7 @@ use strict;
 ## Copyright (c) 1998 Michael Zucchi, All Rights Reserved        ##
 ## Copyright (C) 2000, 1  Tim Waugh <twaugh@redhat.com>          ##
 ## Copyright (C) 2001  Simon Huggins                             ##
-## Copyright (C) 2005-2010  Randy Dunlap                         ##
+## Copyright (C) 2005-2012  Randy Dunlap                         ##
 ## 								 ##
 ## #define enhancements by Armin Kuster <akuster@mvista.com>	 ##
 ## Copyright (c) 2000 MontaVista Software, Inc.			 ##
@@ -1785,6 +1785,7 @@ sub dump_function($$) {
     $prototype =~ s/__devinit +//;
     $prototype =~ s/__init +//;
     $prototype =~ s/__init_or_module +//;
+    $prototype =~ s/__must_check +//;
     $prototype =~ s/^#\s*define\s+//; #ak added
     $prototype =~ s/__attribute__\s*\(\([a-z,]*\)\)//;
 

Re: [PATCH] scripts/kernel-doc: fix fatal error caused by cfg80211.h

[Johannes Berg]

On Sat, 2012-01-21 at 10:31 -0800, Randy Dunlap wrote:
> From: Randy Dunlap <rdunlap@xenotime.net>
> 
> include/net/cfg80211.h uses __must_check in functions that
> have kernel-doc notation.  This was confusing scripts/kernel-doc,
> so have scripts/kernel-doc ignore "__must_check".
> 
> Error(include/net/cfg80211.h:2702): cannot understand prototype: 'struct cfg80211_bss * __must_check cfg80211_inform_bss(...)

Thanks Randy, I'd seen this a little while ago but not gotten around to
investigating.

johannes