[PATCH] Staging: rtl8187se: fixed some checkpatch warnings and errors in r8180_wx.c

2012-11-20 Thread MAACHE Mehdi
RROR: that open brace { should be on the previous line Signed-off-by MAACHE Mehdi --- drivers/staging/rtl8187se/r8180_wx.c | 95 +++--- 1 files changed, 53 insertions(+), 42 deletions(-) diff --git a/drivers/staging/rtl8187se/r8180_wx.c b/drivers/staging/rtl8187se/

Re: [PATCH] Staging: rtl8187se: fixed some checkpatch warnings and errors in r8180_wx.c

2012-11-21 Thread MAACHE Mehdi
Le 20/11/2012 15:25, Dan Carpenter a écrit : > On Tue, Nov 20, 2012 at 02:26:42PM +0100, MAACHE Mehdi wrote: >> This is a patch to the r8180_wx.c file that fixes up some warnings and >> errors found by the checkpatch.pl tool >> - WARNING: line over 80 characters >>

[PATCH] Staging: rtl8187se: bugfix in r8180_wx.c

2012-11-21 Thread MAACHE Mehdi
This bug was introduced in de171bd6ff "Staging: rtl8187se: r8180_wx: fixed a lot of checkpatch.pl issues". Signed-off-by: Mehdi MAACHE --- drivers/staging/rtl8187se/r8180_wx.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8187se/r8180_wx.c b/drive

Re: [PATCH] Staging: rtl8187se: bugfix in r8180_wx.c

2012-11-21 Thread MAACHE Mehdi
carpenter > Thanks again. I wiil do this and next time, I will try a better description. regards, maache mehdi -- 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-info.html Please read the FAQ at http://www.tux.org/lkml/

[PATCH v2] Staging: rtl8187se: remove check for IW_ENCODE_DISABLED

2012-11-21 Thread MAACHE Mehdi
The original code had a no-op stub where it checked if IW_ENCODE_DISABLED and did nothing. Then in a cleanup patch we accidentally turned the check on. That was: de171bd6ff "Staging: rtl8187se: r8180_wx: fixed a lot of checkpatch.pl issues". The check should just be removed. Signed-off-by: Mehd