[PATCH] Fix sparse message rtl8192u

Dan Carpenter error27 at gmail.com
Sun Nov 28 21:19:55 UTC 2010


On Sat, Nov 27, 2010 at 11:37:48PM +0100, David Chosrova wrote:
> Hello,
> 
>  In this patch I have removed the #ifdef ENABLE_DOT11D. 
> 

Cool man.  Looks good.

There are several places where ENABLE_DOT11D is still there in the
comments.  Can do a: grep ENABLE_DOT11D drivers/staging/rtl8192u/ -R
and remove all those?

Then add an:
Acked-by: Dan Carpenter <error27 at gmail.com>

When people apply patches they try to automate the process.  They
have scripts that automatically take an email and creates a kernel
patch.  So choose a good subject and write a patch description.
I would normally write this:

Subject: [patch] Staging: rtl8192u: remove dead code to disable dot11d

This patch removes all the ENABLE_DOT11D ifdefs.  It's always enabled
for this driver.  DOT11D has to do with regulatory domains.

What prompted this patch was a warning message in Sparse:

drivers/staging/rtl8192u/r8192U_core.c:247:1: warning: "eqMacAddr"
redefined in file included from drivers/staging/rtl8192u/:81:81:
drivers/staging/rtl8192u/dot11d.h:35:1: warning: this is the location
of the previous definition                                                                                                   

Now there are no ifdefs around dot11d.h it made no sense to have this
second definition, so I removed that macro as well.

---
Edit that however you want.  That's just an idea.

Don't put the "Hello etc" into the message because Greg will have to
strip those bits out of the patch description by hand.

After the Signed-off-by: line put three dashes "---".  Things below that
line get stripped out of the patch description.  That's where you can
put the "Compile tested."  or "Feedback requested" stuff if you want.

Send your email to: gregkh at suse.de
CC these addresses: devel at driverdev.osuosl.org, error27 at gmail.com, 
andreamrl at tiscali.it, linux-wireless at vger.kernel.org,
kernel-janitors at vger.kernel.org

regards,
dan carpenter



More information about the devel mailing list