Re: [PATCH] Fixed coding style issues at ieee80211_crypt_wep.c

2015-05-18 Thread pmarzo
On Mon, 2015-05-18 at 11:33 +0300, Dan Carpenter wrote: > Also the use a subsystem prefix in the subject: > > [PATCH] Staging: rtl8192u: whatever... > > > regards, > dan carpenter > Thanku for your comments, I will send an v2 patch version regards, Pedro -- To unsubscribe from this list: send

Re: [PATCH] Fixed coding style issues at ieee80211_crypt_wep.c

2015-05-18 Thread Dan Carpenter
Also the use a subsystem prefix in the subject: [PATCH] Staging: rtl8192u: whatever... regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo

Re: [PATCH] Fixed coding style issues at ieee80211_crypt_wep.c

2015-05-15 Thread Joe Perches
On Sat, 2015-05-16 at 10:40 +0200, Pedro Marzo Perez wrote: > This patch just fixes some warnings and style errors reported by > checkpatch.pl script > - Braces should not start in a new line > - C99 comments are not allowed (never use //) > - Literal strings should never be parted in

[PATCH] Fixed coding style issues at ieee80211_crypt_wep.c

2015-05-15 Thread Pedro Marzo Perez
This patch just fixes some warnings and style errors reported by checkpatch.pl script - Braces should not start in a new line - C99 comments are not allowed (never use //) - Literal strings should never be parted in several lines - There was a useless return on a void function Sig