diff mbox

[OPW,kernel] staging:rtl8187se: Shift open brace of function declarations to next line

Message ID 20131105220611.GA15993@himangi-Inspiron-N5110gmail.com
State New, archived
Headers show

Commit Message

HIMANGI SARAOGI Nov. 5, 2013, 10:06 p.m. UTC
This patch fixes the checkpatch.pl error "open brace '{' following
function declarations go on the next line" in ieee80211_softmac.c.

Signed-off-by: Himangi Saraogi <himangi774@gmail.com>
---
 .../rtl8187se/ieee80211/ieee80211_softmac.c        |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Waskiewicz Jr, Peter P Nov. 5, 2013, 10:14 p.m. UTC | #1
On Wed, 2013-11-06 at 03:36 +0530, Himangi Saraogi wrote:
> This patch fixes the checkpatch.pl error "open brace '{' following
> function declarations go on the next line" in ieee80211_softmac.c.
> 
> Signed-off-by: Himangi Saraogi <himangi774@gmail.com>

Looks fine to me.

Reviewed-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>

> ---
>  .../rtl8187se/ieee80211/ieee80211_softmac.c        |    6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
> index 7b6a1c6..a973872 100644
> --- a/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
> +++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
> @@ -106,7 +106,8 @@ void ieee80211_MFIE_Grate(struct ieee80211_device *ieee, u8 **tag_p)
>  }
>  
> 
> -void ieee80211_WMM_Info(struct ieee80211_device *ieee, u8 **tag_p) {
> +void ieee80211_WMM_Info(struct ieee80211_device *ieee, u8 **tag_p)
> +{
>  	u8 *tag = *tag_p;
>  
>  	*tag++ = MFIE_TYPE_GENERIC; //0
> @@ -129,7 +130,8 @@ void ieee80211_WMM_Info(struct ieee80211_device *ieee, u8 **tag_p) {
>  	*tag_p = tag;
>  }
>  
> -void ieee80211_TURBO_Info(struct ieee80211_device *ieee, u8 **tag_p) {
> +void ieee80211_TURBO_Info(struct ieee80211_device *ieee, u8 **tag_p)
> +{
>  	u8 *tag = *tag_p;
>  
>          *tag++ = MFIE_TYPE_GENERIC; //0
> -- 
> 1.7.9.5
>
gregkh@linuxfoundation.org Nov. 10, 2013, 5:07 p.m. UTC | #2
On Wed, Nov 06, 2013 at 03:36:11AM +0530, Himangi Saraogi wrote:
> This patch fixes the checkpatch.pl error "open brace '{' following
> function declarations go on the next line" in ieee80211_softmac.c.
> 
> Signed-off-by: Himangi Saraogi <himangi774@gmail.com>

Applied, thanks.

greg k-h
diff mbox

Patch

diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
index 7b6a1c6..a973872 100644
--- a/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
+++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
@@ -106,7 +106,8 @@  void ieee80211_MFIE_Grate(struct ieee80211_device *ieee, u8 **tag_p)
 }
 
 
-void ieee80211_WMM_Info(struct ieee80211_device *ieee, u8 **tag_p) {
+void ieee80211_WMM_Info(struct ieee80211_device *ieee, u8 **tag_p)
+{
 	u8 *tag = *tag_p;
 
 	*tag++ = MFIE_TYPE_GENERIC; //0
@@ -129,7 +130,8 @@  void ieee80211_WMM_Info(struct ieee80211_device *ieee, u8 **tag_p) {
 	*tag_p = tag;
 }
 
-void ieee80211_TURBO_Info(struct ieee80211_device *ieee, u8 **tag_p) {
+void ieee80211_TURBO_Info(struct ieee80211_device *ieee, u8 **tag_p)
+{
 	u8 *tag = *tag_p;
 
         *tag++ = MFIE_TYPE_GENERIC; //0