- 17 Mar, 2014 40 commits
-
-
Ana Rey authored
Delete spaces prohibited before semicolon, before ':' and after '!' and fix coding style in these lines when It is necessary. Fix checkpàtch.pl error and warning messages: WARNING: space prohibited before semicolon ERROR: space prohibited before that ':' ERROR: space prohibited after that '!' Signed-off-by:
Ana Rey <anarey@gmail.com> Signed-off-by:
Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
-
Ana Rey authored
Delete space prohibited after open parenthesis and fix coding style in the lines changed when It is necessary Fix checkpatch.pl errors: ERROR: space prohibited after that open parenthesis '(' Signed-off-by: -
Ana Rey authored
Delete space prohibited between function name and open parenthesis and fix coding style in the lines changed when It is necessary Fix checkpatch.pl warning message: WARNING: space prohibited between function name and open parenthesis '('. Signed-off-by: -
Ana Rey authored
Add a space required before the open parenthesis in while and for structures as the codingStyle of the kernel recommends. Signed-off-by:
-
Ana Rey authored
Add a space required before the open parenthesis in conditional statements as the codingStyle of the kernel recommends. Signed-off-by:
-
Ana Rey authored
Delete braces are not necessary for single statement blocks and fix coding style in these lines when It is necessary. Fix checkpatch.pl warning: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: -
Ana Rey authored
Fix whitespaces in do-while statements and while statements as the codingStyle of the kernel recommends in ieee80211/ieee80211_softmac.c Signed-off-by:
-
Ana Rey authored
Fix whitespaces around else-statements and fix coding style in the lines when It is necessary as the codingStyle of the kernel recommends in ieee80211/ieee80211_softmac.c Signed-off-by:
-
Ana Rey authored
Fix whitespaces around braces in if-statement as the codingStyle of the kernel recommends in ieee80211/ieee80211_softmac.c Also, fix coding style in these lines when It is necessary. Fix checkpatch.pl errors: ERROR: space required before the open brace '{' Signed-off-by: -
Ana Rey authored
staging: rtl8187se: Remove unused functions 'rsn_authen_cipher_suite' and 'ext_ieee80211_send_beacon_wq' Remove unused functions rsn_authen_cipher_suite() and ext_ieee80211_send_beacon_wq() from ieee80211_softmac.c. Fix sparse warnings: drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:27:11: warning: ‘rsn_authen_cipher_suite’ defined but not used [-Wunused-variable] drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:327:13: warning: ‘ext_ieee80211_send_beacon_wq’ defined but not used [-Wunused-function] Signed-off-by:
-
Ana Rey authored
Move comments from inside if conditionals to the lines above the if-statement. Also delete unnecessary whitespace in these lines. Signed-off-by:
-
Ana Rey authored
Convert style of comments from C99-style to C89 and fix alignment in these lines as the CodingStyle recommends. Signed-off-by:
-
Ana Rey authored
Remove unnecessary blank lines in ieee80211/ieee80211_softmac.c (and add some necessary blank lines). Signed-off-by:
-
Ana Rey authored
Delete all commented code in ieee80211/ieee80211_softmac.c that are not necessary. Also, remove the unused variable referenced in the commented code. Signed-off-by:
-
Uma Sharma authored
This patch removes the following checkpatch.pl warning in tcm825x.h Warning : missing space after return type Signed-off-by:
Uma Sharma <uma.sharma523@gmail.com> Signed-off-by:
-
Uma Sharma authored
This patch removes the following checkpatch.pl warning in sdr-msi3101.c Warning : Unnecessary space after function pointer name Signed-off-by:
-
Gulsah Kose authored
This patch fixes no spaces before tabs warning in crystalhd_misc.h Signed-off-by:
Gulsah Kose <gulsah.1004@gmail.com> Acked-by:
Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issues with line over 80 characters in go7007-v4l2.c Signed-off-by:
Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issue with line over 80 characters in ethernet-mdio.c Signed-off-by:
-
Ebru Akagunduz authored
This patch removes unreachable code line and unnecessary braces in Bcmchar.c Signed-off-by:
Ebru Akagunduz <ebru.akagunduz@gmail.com> Acked-by:
Bob Copeland <me@bobcopeland.com> Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with line over 80 characters in Bcmchar.c Signed-off-by:
-
Uma Sharma authored
This patch removes the following checkpatch.pl warning in gdm_lte.h Warning : space after return function Signed-off-by:
-
Gulsah Kose authored
This patch fixes "quoted string split across lines warning" warning in nokia_core.c Signed-off-by:
-
Gulsah Kose authored
This patch fixes "quoted string split across lines warning" warning in alphatrack.c Signed-off-by:
-
Ashley Smith authored
A function prototype is always a declaration and thus has "extern" prepended by default. Signed-off-by:
Ashley Smith <ashley@eclipso.ch> Acked-by:
-
Gulsah Kose authored
Fix checkpatch.pl issues with line over 80 characters in bp_mod.h Signed-off-by:
-
Kristina Martšenko authored
Remove an item from the TODO file since it appears to have been completed by the following commits: * 1839c7eb "staging/gdm72xx: usb_boot: replace firmware upgrade API" * 3afcb91c "staging/gdm72xx: usb_boot: replace firmware upgrade API in em_download" * 9e412a0a "staging/gdm72xx: sdio_boot: replace firmware upgrade API" Signed-off-by:
Kristina Martšenko <kristina.martsenko@gmail.com> Signed-off-by:
-
Kristina Martšenko authored
Replace printk(KERN_DEBUG ...) with netdev_dbg and dev_dbg. Remove debug macros which become unnecessary. This removes the following types of checkpatch warnings from the driver: drivers/staging/gdm72xx/gdm_sdio.c:461: WARNING: Prefer netdev_dbg(netdev, ... then dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG ... Signed-off-by:
-
Kristina Martšenko authored
Since the kernel already has a function for hex dumps, use that instead of the driver's own versions. The function supports dynamic debugging, so also remove some unnecessary debug macros. Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issues with line over 80 characters in gdm_usb.c Signed-off-by:
-
Gulsah Kose authored
Fix checkpatch.pl issues with line over 80 characters in 80211mgr.c Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issues with line over 80 characters in crystalhd_lnx.c Signed-off-by:
-
Jelena Bjelja authored
This patch fixes the checkpatch warnings: WARNING: braces {} are not necessary for single statement blocks Signed-off-by:Jelena Bjelja <jelena.bjelja.ing@gmail.com> Acked-by:
-
Himangi Saraogi authored
This patch fixes the warning: drivers/staging/keucr/smilsub.c:603:6: warning: symbol '_Check_D_DevCode' was not declared. Should it be static? by making function static as it is used within the file. Signed-off-by:
Himangi Saraogi <himangi774@gmail.com> Acked-by:
Pablo Neira Ayuso <pablo@gnumonks.org> Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issues with line over 80 characters in tiomap3430.c Signed-off-by:
-
Ana Rey authored
Make some local functions static and fix coding style in these function declarations when It is necessary These are the local functions that were made static: rtl8192_CalculateBitShift, rtl8192_phy_RFSerialRead, rtl8192_phy_RFSerialWrite, rtl8192_InitBBRFRegDef, rtl8192_BB_Config_ParaFile,rtl8192_SetTxPowerLevel, rtl8192_phy_SetSwChnlCmdArray, rtl8192_phy_SwChnlStepByStep, rtl8192_phy_FinishSwChnlNow Fixed the following sparse warnings in r819xU_phy.c drivers/staging/rtl8192u/r819xU_phy.c:47:5: warning: symbol 'rtl8192_CalculateBitShift' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:147:5: warning: symbol 'rtl8192_phy_RFSerialRead' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:232:6: warning: symbol 'rtl8192_phy_RFSerialWrite' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:574:6: warning: symbol 'rtl8192_InitBBRFRegDef' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:783:6: warning: symbol 'rtl8192_BB_Config_ParaFile' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:1073:6: warning: symbol 'rtl8192_SetTxPowerLevel' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:1242:4: warning: symbol 'rtl8192_phy_SetSwChnlCmdArray' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:1279:4: warning: symbol 'rtl8192_phy_SwChnlStepByStep' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_phy.c:1436:6: warning: symbol 'rtl8192_phy_FinishSwChnlNow' was not declared. Should it be static? Signed-off-by:
-
Gulsah Kose authored
Fix checkpatch.pl issues with line over 80 characters in gdm_lte.c Signed-off-by:
-
Jelena Bjelja authored
This patch fixes the following checkpatch error in aes_ccmp.c: ERROR: that open brace { should be on the previous line Signed-off-by: -
Kristina Martšenko authored
Since q_ops is only referenced in dt3155v4l.c, mark it as static. This removes the following sparse warning: drivers/staging/media/dt3155v4l//dt3155v4l.c:302:22: warning: symbol 'q_ops' was not declared. Should it be static? Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following error reported by sparse: drivers/staging/dgap/dgap.c:4428:13: error: symbol 'dgap_do_fep_load' redeclared with different type (originally declared at drivers/staging/dgap/dgap.c:215) - incompatible argument 2 (different address spaces) Signed-off-by:
-
