aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAge
* staging: rtl8192u: make in r819xU_cmdpkt.c some local functions staticAna Rey2014-03-19
| | | | | | | | | | | Make some local functions static. Fix sparse warnings in r819xU_cmdpkt.c drivers/staging/rtl8192u/r819xU_cmdpkt.c:185:6: warning: symbol 'cmdpkt_beacontimerinterrupt_819xusb' was not declared. Should it be static? Signed-off-by: Ana Rey <anarey@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: make in ieee80211/ieee80211_wx.c a local variable staticAna Rey2014-03-19
| | | | | | | | | | Make some local variable eee80211_modes static. Fix sparse warnings: drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c:42:19: warning: symbol 'ieee80211_modes' was not declared. Should it be static? Signed-off-by: Ana Rey <anarey@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: Delete 'fwSendNullPacket' function in r819xU_firmware.cAna Rey2014-03-19
| | | | | | | | | | | Delete the fwSendNullPacket function that is not used in anywhere in the driver. Fix sparse warnings: drivers/staging/rtl8192u/r819xU_firmware.c:109:1: warning: symbol 'fwSendNullPacket' was not declared. Should it be static? Signed-off-by: Ana Rey <anarey@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: make in r819xU_firmware.c some local functions staticAna Rey2014-03-19
| | | | | | | | | | | | | | | | | Make some local functions static (firmware_init_param, fw_download_code, CPUcheck_maincodeok_turnonCPU and CPUcheck_firmware_ready) and fix coding style in these function declarations when It is necessary. Fixed the following sparse warnings in r819xU_firmware.c CHECK drivers/staging/rtl8192u/r819xU_firmware.c drivers/staging/rtl8192u/r819xU_firmware.c:20:6: warning: symbol 'firmware_init_param' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_firmware.c:32:6: warning: symbol 'fw_download_code' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_firmware.c:159:6: warning: symbol 'CPUcheck_maincodeok_turnonCPU' was not declared. Should it be static? drivers/staging/rtl8192u/r819xU_firmware.c:208:6: warning: symbol 'CPUcheck_firmware_ready' was not declared. Should it be static? Signed-off-by: Ana Rey <anarey@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: bcm: Fix line over 80 characters.Heena Sirwani2014-03-19
| | | | | | | Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani <heenasirwani@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: Delete duplicate function declarations and extern keywordAna Rey2014-03-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | Delete the duplicate function declarations in r8192U_dm.c. These function declarations are defined in r8192U_dm.h. Also, delete the 'extern' keyword of these function definitions. Fix the following sparse warnings: CHECK drivers/staging/rtl8192u/r8192U_dm.c drivers/staging/rtl8192u/r8192U_dm.c:159:1: warning: function 'init_hal_dm' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:179:13: warning: function 'deinit_hal_dm' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:245:17: warning: function 'hal_dm_watchdog' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:278:13: warning: function 'init_rate_adaptive' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:878:17: warning: function 'dm_txpower_trackingcallback' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:1609:13: warning: function 'dm_cck_txpower_adjust' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:1776:13: warning: function 'dm_change_dynamic_initgain_thresh' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:2543:13: warning: function 'dm_init_edca_turbo' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:2782:17: warning: function 'dm_rf_pathcheck_workitemcallback' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:3142:13: warning: function 'dm_fsync_timer_callback' with external linkage has definition drivers/staging/rtl8192u/r8192U_dm.c:3481:13: warning: function 'dm_shadow_init' with external linkage has definition Signed-off-by: Ana Rey <anarey@gmail.com> Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: Delete 'DM_CTSToSelfSetting' function in r8192U_dm.cAna Rey2014-03-19
| | | | | | | | | | | | Delete the DM_CTSToSelfSetting function that is not used in anywhere in the driver. Fix sparse warnings: drivers/staging/rtl8192u/r8192U_dm.c:2623:6: warning: symbol 'DM_CTSToSelfSetting' was not declared. Should it be static? Signed-off-by: Ana Rey <anarey@gmail.com> Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: Delete 'dm_change_fsync_setting' function in r8192U_dm.cAna Rey2014-03-19
| | | | | | | | | | | | | Delete the 'dm_change_fsync_setting' function that is not used in anywhere in the driver. Fix sparse warnings: drivers/staging/rtl8192u/r8192U_dm.c:1812:6: warning: symbol 'dm_change_fsync_setting' was not declared. Should it be static? Signed-off-by: Ana Rey <anarey@gmail.com> Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8192u: Fix whitespaces in function declarations in r8192U_dm.hAna Rey2014-03-19
| | | | | | | | Fix whitespaces in function declarations in r8192U_dm.h Signed-off-by: Ana Rey <anarey@gmail.com> Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/tidspbridge/rmgr/mgr.c Fix quoted string split across linesAybuke Ozdemir2014-03-19
| | | | | | | | This patch fixes "quoted string split across lines warning" warning in mgr.c Signed-off-by: Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: winbond: Fix line over 80 characters.Gulsah Kose2014-03-19
| | | | | | | Fix checkpatch.pl issues with line over 80 characters in localpara.h Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: cxt1e1: Fix externs should be avoided in .c files in comet.cEbru Akagunduz2014-03-19
| | | | | | | | Fix checkpatch.pl issues with externs should be avoided in comet.c Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: cxt1e1: Fix use of volatile is usually wrong in comet.cEbru Akagunduz2014-03-19
| | | | | | | | Fix checkpatch.pl issues with use of volatile is usually wrong in comet.c Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8821ae: Fix braces {} are not necessary.Gulsah Kose2014-03-19
| | | | | | | Fix checkpatch.pl issues with braces {} are not necessary in base.c Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8821ae: Fix suspect code indent for conditional statementsGulsah Kose2014-03-19
| | | | | | | | Fix checkpatch.pl issues with suspect code indent for conditional statements (16, 16) in base.c Signed-off-by: Gulsah Kose <gulsah.1004@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: comedi: Fix line over 80 characters in s626.cEbru Akagunduz2014-03-19
| | | | | | | | Fix checkpatch.pl issues line over 80 characters in s626.c Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: comedi: Fix missing space after return type in s626.cEbru Akagunduz2014-03-19
| | | | | | | | Fix checkpatch.pl issues with missing space after return type in s626.c Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warning of braces are not necessary.Elena Oat2014-03-19
| | | | | | | | This patch fixes the warning of "braces are not necessary for single statement blocks" in file rtw_ieee80211.c. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warning of space prohibited.Elena Oat2014-03-19
| | | | | | | | This patch fixes the warning of "space prohibited before semicolon" in file rtw_efuse.c. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warning of braces not necessary.Elena Oat2014-03-19
| | | | | | | | This small patch fixes the warning of "braces are necessary for any arm of this statement" in file rtw_efuse.c. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warnings of no space before tabs.Elena Oat2014-03-19
| | | | | | | | | This patch fixes the warning of "please, no space before tabs" in file rtw_efuse.c. I have also modified the style of the comments for readability. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warning of space prohibited.Elena Oat2014-03-19
| | | | | | | | This small patch fixes the warning of "space prohibited before semicolon" in file rtw_debug.c. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: rtl8188eu: Reduced nesting and improved coding style.Chi Pham2014-03-19
| | | | | Signed-off-by: Chi Pham <fempsci@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging:lustre:lnet: lib-types.h Fix open brace '{' err.Aybuke Ozdemir2014-03-19
| | | | | | | | | Fix checkpatch.pl issues with open brace '{' following struct go on the same line in lib-types.h Signed-off-by: Aybuke Ozdemir <aybuke.147@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging:lustre:lnet: lib-types.h Fix unnecessary space func. pointer name.Aybuke Ozdemir2014-03-19
| | | | | | | | Fix checkpatch.pl issues with unnecessary space after function pointer name in lib-types.h Signed-off-by: Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: lustre: add an extra lineGeorgiana Rodica Chelu2014-03-19
| | | | | | | | | Add an extra line break between the variable declaration and the rest of the code Signed-off-by: Georgiana Rodica Chelu <georgiana.chelu93@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warnings of no space before tabs.Elena Oat2014-03-19
| | | | | | | | | This patch fixes the warnings of "please, no space before tabs" in the file rtw_mlme.c. I have also modified the comment style and removed some unnecessary tabs. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warnings of space prohibited.Elena Oat2014-03-19
| | | | | | | | | | | | This patch fixes the warnings of "space prohibited between function name and open parenthesis" in file rtw_mlme.c. While at it, I have also removed unneccessary tabs. Also one comment that didn't make sense was removed. Signed-off-by: Elena Oat <oat.elena@gmail.com> Acked-by: Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: rtl8188eu: Fix warning of braces {} are not necessary.Elena Oat2014-03-19
| | | | | | | | | This patch fixes the warning of "braces {} are not necessary for single statement blocks" in file rtw_mlme.c. I have removed the else statement in one case, as it didn't have any value. Signed-off-by: Elena Oat <oat.elena@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_uGetRTSCTSRsvTime fix return.Malcolm Priestley2014-03-19
| | | | | | | Always returns to __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: vnt_rxtx_rsvtime_le16 fix return.Malcolm Priestley2014-03-19
| | | | | | | Always returns to __le16 type. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_uGetDataDuration use __le16.Malcolm Priestley2014-03-19
| | | | | | | s_uGetDataDuration always returns to __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Correct return type s_uGetRTSCTSDuration.Malcolm Priestley2014-03-19
| | | | | | | s_uGetRTSCTSDuration always returns in __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx endian correct return durationMalcolm Priestley2014-03-19
| | | | | | | All duration base types are __le16 correct back to cpu Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: vnt_time_stamp_off fix return to __le16Malcolm Priestley2014-03-19
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: struct vnt_tx_short_buf_head correct base typesMalcolm Priestley2014-03-19
| | | | | | | Both variables duration and time_stamp_off are __le16 types Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx Replace wTimeStampOff* variablesMalcolm Priestley2014-03-19
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wTimeStampOff* -> time_stamp_off* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx Replace wCTSTxRrvTime_baMalcolm Priestley2014-03-19
| | | | | | | Replace with cts_rrv_time_ba with __le16 base. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wCTSDuration*Malcolm Priestley2014-03-19
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wCTSDuration* -> cts_duration* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wTxRrvTime*Malcolm Priestley2014-03-19
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wTxRrvTime* -> rrv_time* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wRTSTxRrvTime*Malcolm Priestley2014-03-19
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wRTSTxRrvTime* -> rts_rrv_time* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wRTSDuration* variablesMalcolm Priestley2014-03-19
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wRTSDuration* -> rts_duration* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx Replace wDuration* variables.Malcolm Priestley2014-03-19
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wDuration* -> duration* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: re-factor some more code for 80+ linesMark Hounschell2014-03-19
| | | | | | | | This patch fixes more 80+ char lines as reported by checkpatch Signed-off-by: Mark Hounschell <markh@compro.net> Tested-by: Mark Hounschell <markh@compro.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: get rid of DGAP_LOCK and DGAP_UNLOCK macrosMark Hounschell2014-03-19
| | | | | | | | This patch replaces the DGAP_LOCK and DGAP_UNLOCK macros with spin_lock_irqsave spin_unlock_irqrestore. Signed-off-by: Mark Hounschell <markh@compro.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgap: re-factor some code for 80+ lines as reported by checkpatchMark Hounschell2014-03-19
| | | | | | | | | | | This patch re-factors some code to fix some 80+ lines as reported by checkpatch. This patch was suggested and authored by Dan Carpenter. A test case for this patch has been written and used to show the patch to be correct. Suggested-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Mark Hounschell <markh@compro.net> Tested-by: Mark Hounschell <markh@compro.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wlags49_h2: Correct comment style in wl_netdev.cMathieu Maret2014-03-19
| | | | | | | Code is using C99 code style, so change them to kernel style Signed-off-by: Mathieu Maret <mathieu.maret@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wlags49_h2: remove commented code in wl_netdev.cMathieu Maret2014-03-19
| | | | | | | Remove unused code under C99 comment style Signed-off-by: Mathieu Maret <mathieu.maret@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: wlags49_h2: reindent wl_netdev.cMathieu Maret2014-03-19
| | | | | | | | As the file is using a mix between space and tab, and space instead of tab, use Lindent to fix the file Signed-off-by: Mathieu Maret <mathieu.maret@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: comedi: proc: title block updateMichael Welling2014-03-19
| | | | | | | | Follow up patch for "STAGING: comedi: style and checkpatch fixes". Removes the file name in the comment block per suggestions. Signed-off-by: Michael Welling <mwelling@ieee.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
generated by cgit v1.2.2 (git 2.25.0) at 2025-08-13 18:00:23 -0400