Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1637130 > unrolled thread

[PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue

Started byRiccardo Marotti <riccardo.marotti@gmail.com>
First post2017-05-08 01:00 +0200
Last post2017-05-12 11:50 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue Riccardo Marotti <riccardo.marotti@gmail.com> - 2017-05-08 01:00 +0200
    Re: [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix  style issue Greg KH <gregkh@linuxfoundation.org> - 2017-05-12 11:10 +0200
      Re: [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix  style issue Riccardo Marotti <riccardo.marotti@gmail.com> - 2017-05-12 11:50 +0200

#1637130 — [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue

FromRiccardo Marotti <riccardo.marotti@gmail.com>
Date2017-05-08 01:00 +0200
Subject[PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue
Message-ID<tEDjH-3LC-5@gated-at.bofh.it>
Fixed a brace coding style issue, found via checkpatch.

Signed-off-by: Riccardo Marotti <riccardo.marotti@gmail.com>
---
 drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
index a791175b86f5..8f236b332a47 100644
--- a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
+++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
@@ -157,8 +157,7 @@ struct net_device *alloc_ieee80211(int sizeof_priv)
 	ieee80211_softmac_init(ieee);

 	ieee->pHTInfo = kzalloc(sizeof(RT_HIGH_THROUGHPUT), GFP_KERNEL);
-	if (ieee->pHTInfo == NULL)
-	{
+	if (ieee->pHTInfo == NULL) {
 		IEEE80211_DEBUG(IEEE80211_DL_ERR, "can't alloc memory for HTInfo\n");
 		goto failed;
 	}
--
2.12.2

[toc] | [next] | [standalone]


#1640327 — Re: [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue

FromGreg KH <gregkh@linuxfoundation.org>
Date2017-05-12 11:10 +0200
SubjectRe: [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue
Message-ID<tGeKd-2eI-5@gated-at.bofh.it>
In reply to#1637130
On Sun, May 07, 2017 at 11:01:07AM +0200, Riccardo Marotti wrote:
> Fixed a brace coding style issue, found via checkpatch.
> 
> Signed-off-by: Riccardo Marotti <riccardo.marotti@gmail.com>
> ---
>  drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

What changed from v1?  Always put that info below the --- line.

Please fix and resend v3.

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]


#1640365 — Re: [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue

FromRiccardo Marotti <riccardo.marotti@gmail.com>
Date2017-05-12 11:50 +0200
SubjectRe: [PATCH v2] Staging: rtl8192u: ieee80211: ieee80211_module.c: fix style issue
Message-ID<tGfmV-2vB-5@gated-at.bofh.it>
In reply to#1640327
On 05/12/2017 11:07 AM, Greg KH wrote:
> On Sun, May 07, 2017 at 11:01:07AM +0200, Riccardo Marotti wrote:
>> Fixed a brace coding style issue, found via checkpatch.
>>
>> Signed-off-by: Riccardo Marotti <riccardo.marotti@gmail.com>
>> ---
>>  drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> What changed from v1?  Always put that info below the --- line.
> 
> Please fix and resend v3.

Done!

Thanks for your patience,
Riccardo
 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web