1. 17 Mar, 2014 34 commits
    • Ana Rey's avatar
      staging: rtl8192u: make in r819xU_phy.c some local functions static · c92f473d
      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: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      c92f473d
    • Gulsah Kose's avatar
      staging: gdm724x: Fix line over 80 characters. · 097b4d8c
      Gulsah Kose authored
      Fix checkpatch.pl issues with line over 80 characters in gdm_lte.c
      Signed-off-by: default avatarGulsah Kose <gulsah.1004@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      097b4d8c
    • Jelena Bjelja's avatar
      staging:vt6655: Fix open brace placement related error · 9c45c42a
      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: default avatarJelena Bjelja <jelena.bjelja.ing@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      9c45c42a
    • Kristina Martšenko's avatar
      staging: dt3155v4l: make q_ops static · 5ae7437e
      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: default avatarKristina Martšenko <kristina.martsenko@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      5ae7437e
    • Himangi Saraogi's avatar
      staging: dgap: Fix mismatch in function argument types · e054e2a7
      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: default avatarHimangi Saraogi <himangi774@gmail.com>
      Acked-by: default avatarPablo Neira Ayuso <pablo@gnumonks.org>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      e054e2a7
    • Andreea-Cristina Bernat's avatar
      Staging: unisys: Replace kmalloc/memset with kzalloc · 97a84f12
      Andreea-Cristina Bernat authored
      This patch solves the Coccinelle warning: "kzalloc should be used instead of
      kmalloc/memset".
      Signed-off-by: default avatarAndreea-Cristina Bernat <bernat.ada@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      97a84f12
    • Ana Rey's avatar
      staging: rtl8192u: Deleted 'HTUseDefaultSetting' function in ieee80211/rtl819x_HTProc.c · ac0ba3f8
      Ana Rey authored
      This patch delete the 'HTUseDefaultSetting' function that is not used in anywhere in the driver.
      
      Fix the following sparse warnings in ieee80211/rtl819x_HTProc.c
      
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:1323:6: warning: symbol 'HTUseDefaultSetting' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      ac0ba3f8
    • Ana Rey's avatar
      staging: rtl8192u: make in ieee80211/ieee80211_rx.c some local functions static · 4b10d50c
      Ana Rey authored
      Make some local functions (AddReorderEntry, RxReorderIndicate and
      parse_subframe) static and fix coding style in this function
      declarations when It is necessary.
      
      Fix the following sparse warnings in ieee80211/ieee80211_rx.c
      
      drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:521:1: warning: symbol 'AddReorderEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:604:6: warning: symbol 'RxReorderIndicatePacket' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:774:4: warning: symbol 'parse_subframe' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      4b10d50c
    • Ana Rey's avatar
      staging: rtl8192u: make in ieee80211/ieee80211_softmac.c some local functions static · fabdbdb2
      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:
      
      ieee80211_MFIE_rate_len, ieee80211_MFIE_Brate, ieee80211_MFIE_Grate,
      ieee80211_WMM_Info, enqueue_mgmt, dequeue_mgmt, init_mgmt_queue,
      MgntQuery_MgntFrameTxRate, ieee80211_send_beacon,
      ieee80211_send_beacon_cb, ieee80211_softmac_scan_wq,
      ieee80211_beacons_start, ieee80211_beacons_stop,
      ieee80211_softmac_stop_scan, ieee80211_auth_resp, ieee80211_null_func,
      ieee80211_resp_to_assoc_rq, ieee80211_resp_to_auth,
      ieee80211_resp_to_probe, ieee80211_associate_abort_cb,
      ieee80211_associate_step1, ieee80211_auth_challenge,
      ieee80211_associate_step2, ieee80211_associate_complete_wq,
      ieee80211_associate_complete, ieee80211_associate_procedure_wq,
      auth_rq_parse, assoc_rq_parse, ieee80211_sta_ps_send_null_frame,
      ieee80211_sta_ps_sleep, ieee80211_process_action, ieee80211_resume_tx,
      ieee80211_start_monitor_mode, ieee80211_start_ibss_wq,
      ieee80211_associate_retry_wq, ieee80211_wpa_assoc_frame,
      ieee80211_sta_ps, ieee80211_send_probe, ieee80211_start_scan
      and ieee80211_assoc_resp
      
      Fix the following sparse warnings in ieee80211/ieee80211_softmac.c
      
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:41:14: warning: symbol 'ieee80211_MFIE_rate_len' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:59:6: warning: symbol 'ieee80211_MFIE_Brate' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:76:6: warning: symbol 'ieee80211_MFIE_Grate' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:100:6: warning: symbol 'ieee80211_WMM_Info' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:142:6: warning: symbol 'enqueue_mgmt' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:160:16: warning: symbol 'dequeue_mgmt' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:175:6: warning: symbol 'init_mgmt_queue' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:180:4: warning: symbol 'MgntQuery_MgntFrameTxRate' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:359:6: warning: symbol 'ieee80211_send_beacon' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:385:6: warning: symbol 'ieee80211_send_beacon_cb' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:488:6: warning: symbol 'ieee80211_softmac_scan_wq' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:532:6: warning: symbol 'ieee80211_beacons_start' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:543:6: warning: symbol 'ieee80211_beacons_stop' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:575:6: warning: symbol 'ieee80211_softmac_stop_scan' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:890:16: warning: symbol 'ieee80211_auth_resp' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:918:16: warning: symbol 'ieee80211_null_func' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:944:6: warning: symbol 'ieee80211_resp_to_assoc_rq' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:953:6: warning: symbol 'ieee80211_resp_to_auth' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:962:6: warning: symbol 'ieee80211_resp_to_probe' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1244:6: warning: symbol 'ieee80211_associate_abort_cb' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1250:6: warning: symbol 'ieee80211_associate_step1' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1276:6: warning: symbol 'ieee80211_auth_challenge' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1306:6: warning: symbol 'ieee80211_associate_step2' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1325:6: warning: symbol 'ieee80211_associate_complete_wq' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1372:6: warning: symbol 'ieee80211_associate_complete' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1383:6: warning: symbol 'ieee80211_associate_procedure_wq' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1556:5: warning: symbol 'auth_rq_parse' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1612:5: warning: symbol 'assoc_rq_parse' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1706:6: warning: symbol 'ieee80211_sta_ps_send_null_frame' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1717:7: warning: symbol 'ieee80211_sta_ps_sleep' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1882:6: warning: symbol 'ieee80211_process_action' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:2185:6: warning: symbol 'ieee80211_resume_tx' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:2314:6: warning: symbol 'ieee80211_start_monitor_mode' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:2324:6: warning: symbol 'ieee80211_start_ibss_wq' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:2497:6: warning: symbol 'ieee80211_associate_retry_wq' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:2768:6: warning: symbol 'ieee80211_wpa_assoc_frame' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1765:13: warning: symbol 'ieee80211_sta_ps' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:398:6: warning: symbol 'ieee80211_send_probe' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:604:6: warning: symbol 'ieee80211_start_scan' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:836:16: warning: symbol 'ieee80211_assoc_resp' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      fabdbdb2
    • Ana Rey's avatar
      staging: rtl8192u: make in ieee80211/ieee80211_tx.c some local functions static · a5ac48ac
      Ana Rey authored
      Make some local functions (ieee80211_qurey_ShortPreambleMode and
      ieee80211_query_HTCapShortGI) static and fix coding style in these
      function declarations when It is necessary.
      
      Fix the following sparse warnings in ieee80211/ieee80211_tx.c
      
      drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:381:6: warning: symbol 'ieee80211_qurey_ShortPreambleMode' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c:395:1: warning: symbol 'ieee80211_query_HTCapShortGI' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      a5ac48ac
    • Ana Rey's avatar
      staging: rtl8192u: Added in r819xU_firmware_img.c an include to fix a sparse warning · 627877bd
      Ana Rey authored
      Fix the following warnings from sparse in r819xU_firmware_img.c
      due to the missing include of r819xU_firmware_img.h
      
        CHECK   drivers/staging/rtl8192u/r819xU_firmware_img.c
      drivers/staging/rtl8192u/r819xU_firmware_img.c:4:5: warning: symbol 'Rtl8192UsbPHY_REGArray' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:7:5: warning: symbol 'Rtl8192UsbPHY_REG_1T2RArray' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:158:5: warning: symbol 'Rtl8192UsbRadioA_Array' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:284:5: warning: symbol 'Rtl8192UsbRadioB_Array' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:326:5: warning: symbol 'Rtl8192UsbRadioC_Array' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:329:5: warning: symbol 'Rtl8192UsbRadioD_Array' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:332:5: warning: symbol 'Rtl8192UsbMACPHY_Array' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:341:5: warning: symbol 'Rtl8192UsbMACPHY_Array_PG' was not declared. Should it be static?
      drivers/staging/rtl8192u/r819xU_firmware_img.c:354:5: warning: symbol 'Rtl8192UsbAGCTAB_Array' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      627877bd
    • Ashley Smith's avatar
      staging: bcm: Replace <asm/uaccess.h> by <linux/uaccess.h> · a7e4be9d
      Ashley Smith authored
      This patch fixes the checkpatch warning to include <linux/uaccess.h> instead
      of <asm/uaccess.h>.
      Signed-off-by: default avatarAshley Smith <ashley@eclipso.ch>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      a7e4be9d
    • Andreea-Cristina Bernat's avatar
      Staging: solo6x10: Replace expressions that don't use ALIGN macro · 0cb6dfd7
      Andreea-Cristina Bernat authored
      There are some expressions that compute the roundup of a number, but don't use
      the existing macro defined in /include/kernel.h. This patch uses the following
      Coccinelle semantic patch:
      @ haskernel @
      @@
      
      @ depends on haskernel @
      expression E1, E2;
      @@
      
      - (E1 + (E2 - 1)) & ~(E2 - 1)
      + ALIGN(E1, E2)
      Signed-off-by: default avatarAndreea-Cristina Bernat <bernat.ada@gmail.com>
      Acked-by: default avatarBob Copeland <me@bobcopeland.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      0cb6dfd7
    • Tugce Sirin's avatar
      Staging: bcm: Fix sparse non-static symbol warning · 8fa7fdeb
      Tugce Sirin authored
      Fix sparse non-static symbol warning in bcm driver.
      Signed-off-by: default avatarTugce Sirin <ztugcesirin@gmail.com>
      Acked-by: default avatarSarah Sharp <sarah.a.sharp@linux.intel.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      8fa7fdeb
    • Tugce Sirin's avatar
      Staging: wlan-ng: Fix sparse non-static symbol warning · 36d9c250
      Tugce Sirin authored
      Fix sparse non-static symbol warning in wlan-ng driver.
      Signed-off-by: default avatarTugce Sirin <ztugcesirin@gmail.com>
      Acked-by: default avatarSarah Sharp <sarah.a.sharp@linux.intel.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      36d9c250
    • Andreea-Cristina Bernat's avatar
      staging: sep: Add fallthrough comment · 98e2dda3
      Andreea-Cristina Bernat authored
      This patch solves the warning "possible switch/case default not preceded by
      break or fallthrough comment".
      Signed-off-by: default avatarAndreea-Cristina Bernat <bernat.ada@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      98e2dda3
    • Himangi Saraogi's avatar
      staging:lustre: Fix sparse warning of plain integer as NULL · 87355c1b
      Himangi Saraogi authored
      This patch fixes the following sparse warning:
      drivers/staging/lustre/lustre/lov/lov_request.c:53:28: warning: Using plain integer as NULL pointer
      Signed-off-by: default avatarHimangi Saraogi <himangi774@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      87355c1b
    • Ana Rey's avatar
      staging: rtl8192u: Deleted 'rtl8192_beacon_stop' function in r8192U_core.c · feb600e5
      Ana Rey authored
      This patch delete the 'rtl8192_beacon_stop' function that is not used
      in anywhere in the driver.
      
      Fix the following sparse warning in r8192U_core.c
      
      drivers/staging/rtl8192u/r8192U_core.c:1332:6: warning: symbol 'rtl8192_beacon_stop' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      feb600e5
    • Ana Rey's avatar
      staging: rtl8192u: Deleted 'rtl8192_proc_module_remove' function in r8192U_core.c · 4f59e193
      Ana Rey authored
      This patch delete the 'rtl8192_proc_module_remove' function that is not
      used in anywhere in the driver.
      
      Fix the following sparse warnings in r8192U_core.c
      
      drivers/staging/rtl8192u/r8192U_core.c:595:6: warning: symbol 'rtl8192_proc_module_remove' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      4f59e193
    • Ana Rey's avatar
      staging: rtl8192u: Deleted 'rtl8192_set_mode' function in r8192U_core.c · a5cb9b39
      Ana Rey authored
      This patch delete the 'rtl8192_set_mode' function that is not used in
      anywhere in the driver.
      
      Fix the following sparse warning in r8192U_core.c
      drivers/staging/rtl8192u/r8192U_core.c:692:6: warning: symbol 'rtl8192_set_mode' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      a5cb9b39
    • Ana Rey's avatar
      staging: rtl8192u: Deleted 'rtl819xU_cmd_isr' function in r8192U_core.c · 6a1bf3a5
      Ana Rey authored
      This patch delete the 'rtl819xU_cmd_isr' function that is not used in
      anywhere in the driver.
      
      Fix the following sparse warning in r8192U_core.c
      
      drivers/staging/rtl8192u/r8192U_core.c:1516:6: warning: symbol 'rtl819xU_cmd_isr' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      6a1bf3a5
    • Ana Rey's avatar
      staging: rtl8192u: make in ieee80211/rtl819x_BAProc.c some local functions static · 51296cdf
      Ana Rey authored
      Make some local functions (ActivateBAEntry, DeActivateBAEntry,
      TxTsDeleteBA ,RxTsDeleteBA and ieee80211_send_DELBA) static and fix
      coding style in this function declarations when It is necessary
      
      Fix the following sparse warnings in ieee80211/rtl819x_BAProc.c
      
      drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c:16:6: warning: symbol 'ActivateBAEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c:28:6: warning: symbol 'DeActivateBAEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c:40:4: warning: symbol 'TxTsDeleteBA' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c:70:4: warning: symbol 'RxTsDeleteBA' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_BAProc.c:310:6: warning: symbol 'ieee80211_send_DELBA' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      51296cdf
    • Ana Rey's avatar
      staging: rtl8192u: make in ieee80211/rtl819x_HTProc.c some local functions static · 140cdd73
      Ana Rey authored
      Make some local functions static.
      
      These are the local functions that were made static:
      
      IsHTHalfNmode40Bandwidth, IsHTHalfNmodeSGI, HTIOTPeerDetermine,
      HTIOTActIsDisableMCS14, HTIOTActIsDisableMCS15,
      HTIOTActIsDisableMCSTwoSpatialStream, HTIOTActIsDisableEDCATurbo,
      HTIOTActIsMgntUseCCK6M, HTIOTActIsCCDFsync, HT_PickMCSRate,
      HTFilterMCSRate, HTUseDefaultSetting
      
      Fix the following sparse warnings in ieee80211/rtl819x_HTProc.c
      
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:222:6: warning: symbol 'IsHTHalfNmode40Bandwidth' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:241:6: warning: symbol 'IsHTHalfNmodeSGI' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:379:6: warning: symbol 'HTIOTPeerDetermine' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:416:4: warning: symbol 'HTIOTActIsDisableMCS14' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:435:6: warning: symbol 'HTIOTActIsDisableMCS15' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:472:6: warning: symbol 'HTIOTActIsDisableMCSTwoSpatialStream' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:489:4: warning: symbol 'HTIOTActIsDisableEDCATurbo' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:503:4: warning: symbol 'HTIOTActIsMgntUseCCK6M' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:518:4: warning: symbol 'HTIOTActIsCCDFsync' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:795:4: warning: symbol 'HT_PickMCSRate' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:910:4: warning: symbol 'HTFilterMCSRate' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c:1320:6: warning: symbol 'HTUseDefaultSetting' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      140cdd73
    • Ana Rey's avatar
      staging: rtl8192u: make in ieee80211/rtl819x_TSProc.c some local functions static · 1d6fa895
      Ana Rey authored
      Make some local functions static and fix coding style in this function
      declarations when It is necessary.
      
      These are the local functions that were made static:
      TsSetupTimeOut, TsInactTimeout RxPktPendingTimeout, TsAddBaProcess,
      ResetTsCommonInfo, ResetTxTsEntry, ResetRxTsEntry, AdmitTS,
      SearchAdmitTRStream, MakeTSEntry, RemoveTsEntry.
      
      Fix the following sparse warnings in ieee80211/rtl819x_TSProc.c
      
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:6:6: warning: symbol 'TsSetupTimeOut' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:12:6: warning: symbol 'TsInactTimeout' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:25:6: warning: symbol 'RxPktPendingTimeout' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:102:6: warning: symbol 'TsAddBaProcess' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:113:6: warning: symbol 'ResetTsCommonInfo' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:122:6: warning: symbol 'ResetTxTsEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:133:6: warning: symbol 'ResetRxTsEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:227:6: warning: symbol 'AdmitTS' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:237:17: warning: symbol 'SearchAdmitTRStream' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:312:6: warning: symbol 'MakeTSEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c:475:6: warning: symbol 'RemoveTsEntry' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      1d6fa895
    • Ana Rey's avatar
      staging: rtl8192u: Deleted 'alloc_tx_beacon_desc_ring' function in r8192U_core.c · a72eea3b
      Ana Rey authored
      This patch delete the 'alloc_tx_beacon_desc_ring' function that is not
      used in anywhere in the driver.
      
      Fix the following sparse warning in r8192U_core.c
      
      drivers/staging/rtl8192u/r8192U_core.c:883:5: warning: symbol 'alloc_tx_beacon_desc_ring' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      a72eea3b
    • Monam Agarwal's avatar
      Staging: ft1000-usb: Fix Smatch Warning in ft1000_hw.c · 1633dc5e
      Monam Agarwal authored
      This patch fixes following smatch warning:
      /drivers/staging/ft1000/ft1000-usb/ft1000_hw.c:1148 ft1000_proc_drvmsg() warn: returning -1 instead of -ENOMEM is sloppy
      Signed-off-by: default avatarMonam Agarwal <monamagarwal123@gmail.com>
      Acked-by: default avatarNick Kossifidis <mickflemm@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      1633dc5e
    • Ana Rey's avatar
      staging: rts5139: Added in sd_cprm.c an include to fix a sparse warning · 2eeaa73c
      Ana Rey authored
      Fix the following warnings from sparse in drivers/staging/rts5139/sd_cprm.c
      due to the missing include of sd_cprm.h
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      2eeaa73c
    • Ana Rey's avatar
      staging: rts5139: make thrts5139_usb_idse local variable static in rts51x.c · d2a27670
      Ana Rey authored
      Make the local variable rts5139_usb_ids static
      
      Fix the following sparse warnings in rts51x.c
      
      CHECK   drivers/staging/rts5139/rts51x.c
      drivers/staging/rts5139/rts51x.c:835:22: warning: symbol 'rts5139_usb_ids' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      d2a27670
    • Ana Rey's avatar
      staging: rts5139: Added in ms_mg.c an include to fix a sparse warning · 884715f3
      Ana Rey authored
      Fix the following warnings from sparse in drivers/staging/rts5139/ms_mg.c
      due to the missing include of ms_mg.h
      
      drivers/staging/rts5139/ms_mg.c:81:5: warning: symbol 'mg_set_tpc_para_sub' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:122:5: warning: symbol 'rts51x_mg_set_leaf_id' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:173:5: warning: symbol 'rts51x_mg_get_local_EKB' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:232:5: warning: symbol 'rts51x_mg_chg' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:323:5: warning: symbol 'rts51x_mg_get_rsp_chg' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:392:5: warning: symbol 'rts51x_mg_rsp' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:450:5: warning: symbol 'rts51x_mg_get_ICV' was not declared. Should it be static?
      drivers/staging/rts5139/ms_mg.c:514:5: warning: symbol 'rts51x_mg_set_ICV' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      884715f3
    • Ana Rey's avatar
      staging: rts5139: make the local variable static · cf883f9b
      Ana Rey authored
      Make the local variable (media_not_present and invalid_cmd_field) static
      
      Fix the following sparse warnings in rts51x_transport.c
      
      drivers/staging/rts5139/rts51x_transport.c:649:4: warning: symbol 'media_not_present' was not declared. Should it be static?
      drivers/staging/rts5139/rts51x_transport.c:651:4: warning: symbol 'invalid_cmd_field' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      cf883f9b
    • Ana Rey's avatar
      staging: rtl8192u: make in r8180_93cx6.c some local functions static · f6bd19e0
      Ana Rey authored
      Make some local functions (eprom_cs, eprom_ck_cycle, eprom_w, eprom_r
      and eprom_send_bits_string) static
      
      Fix the following sparse warnings in r8180_93cx6.c
      
      drivers/staging/rtl8192u/r8180_93cx6.c:23:6: warning: symbol 'eprom_cs' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8180_93cx6.c:40:6: warning: symbol 'eprom_ck_cycle' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8180_93cx6.c:56:6: warning: symbol 'eprom_w' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8180_93cx6.c:71:7: warning: symbol 'eprom_r' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8180_93cx6.c:85:6: warning: symbol 'eprom_send_bits_string' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      f6bd19e0
    • Ana Rey's avatar
      staging: rtl8192u: make in r8192U_core.c the local functions static · f4c6074a
      Ana Rey authored
      Make some local functions static.
      
      The local functions are CamResetAllEntry, tx_timeout,
      rtl8192_data_hard_stop, rtl8192_data_hard_resume, rtl8192_hard_data_xmit,
      rtl8192_hard_start_xmit, rtl8192_config_rate, rtl8192_update_cap,
      rtl8192_net_update, MRateToHwRate8190Pci, tl8192_link_change,
      rtl8192_update_beacon, WDCAPARA_ADD, rtl8192_qos_activate,
      GetHalfNmodeSupportByAPs819xUsb, rtl8192_refresh_supportrate,
      rtl8192_getSupportedWireleeMode, rtl8192_SetWirelessMode,
      rtl8192_get_channel_map, rtl8192_init,
      rtl8192_hwconfig, rtl8192_adapter_start, TxCheckStuck,
      rtl819x_ifcheck_resetornot, CamRestoreAllEntry, rtl819x_ifsilentreset,
      rtl819x_update_rxcounts, rtl8192_open, r8192_set_mac_adr, rtl8192_ioctl,
      HwRateToMRate90, rtl819x_translate_todbm, rtl8192_process_phyinfo,
      rtl8192_record_rxdesc_forlateruse, rtl8192_rx_nomal,
      rtl819xusb_process_received_packet, rtl8192_rx_cmd.
      
      Fix the following sparse warnings in r8192U_core.c
      
      drivers/staging/rtl8192u/r8192U_core.c:241:6: warning: symbol 'CamResetAllEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:676:6: warning: symbol 'tx_timeout' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1014:6: warning: symbol 'rtl8192_data_hard_stop' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1020:6: warning: symbol 'rtl8192_data_hard_resume' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1028:6: warning: symbol 'rtl8192_hard_data_xmit' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1056:5: warning: symbol 'rtl8192_hard_start_xmit' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1380:6: warning: symbol 'rtl8192_config_rate' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1427:6: warning: symbol 'rtl8192_update_cap' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1449:6: warning: symbol 'rtl8192_net_update' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:1677:4: warning: symbol 'MRateToHwRate8190Pci' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2043:6: warning: symbol 'rtl8192_link_change' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2075:6: warning: symbol 'rtl8192_update_beacon' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2090:5: warning: symbol 'WDCAPARA_ADD' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2091:6: warning: symbol 'rtl8192_qos_activate' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2319:6: warning: symbol 'GetHalfNmodeSupportByAPs819xUsb' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2333:6: warning: symbol 'rtl8192_refresh_supportrate' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2344:4: warning: symbol 'rtl8192_getSupportedWireleeMode' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2363:6: warning: symbol 'rtl8192_SetWirelessMode' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2783:7: warning: symbol 'rtl8192_get_channel_map' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2796:7: warning: symbol 'rtl8192_init' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2844:6: warning: symbol 'rtl8192_hwconfig' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:2927:6: warning: symbol 'rtl8192_adapter_start' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3183:12: warning: symbol 'TxCheckStuck' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3286:12: warning: symbol 'rtl819x_ifcheck_resetornot' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3325:6: warning: symbol 'CamRestoreAllEntry' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3401:6: warning: symbol 'rtl819x_ifsilentreset' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3521:6: warning: symbol 'rtl819x_update_rxcounts' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3638:5: warning: symbol 'rtl8192_open' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3758:5: warning: symbol 'r8192_set_mac_adr' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3774:5: warning: symbol 'rtl8192_ioctl' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3865:4: warning: symbol 'HwRateToMRate90' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3951:6: warning: symbol 'rtl819x_translate_todbm' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:3967:6: warning: symbol 'rtl8192_process_phyinfo' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:4453:6: warning: symbol 'rtl8192_record_rxdesc_forlateruse' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:4725:6: warning: symbol 'rtl8192_rx_nomal' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:4875:6: warning: symbol 'rtl819xusb_process_received_packet' was not declared. Should it be static?
      drivers/staging/rtl8192u/r8192U_core.c:4936:6: warning: symbol 'rtl8192_rx_cmd' was not declared. Should it be static?
      Signed-off-by: default avatarAna Rey <anarey@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      f4c6074a
    • Aybuke Ozdemir's avatar
      staging: solo6x10: solo6x10-v4l2-enc.c: Fix line over 80 characters. · fa91783e
      Aybuke Ozdemir authored
      Fix checkpatch.pl issues with line over 80 characters in solo6x10-v4l2-enc.c
      Signed-off-by: default avatarAybuke Ozdemir <aybuke.147@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      fa91783e
    • Iulia Manda's avatar
      staging: vt6656: Use __releases/__acquires annotations · 9009dd16
      Iulia Manda authored
      This patch fixes the following sparse warnings:
      
      drivers/staging/vt6656/usbpipe.c:147:24: warning: context imbalance in 'PIPEnsControlOut' - unexpected unlock
      drivers/staging/vt6656/usbpipe.c:209:24: warning: context imbalance in 'PIPEnsControlIn' - unexpected unlock
      Signed-off-by: default avatarIulia Manda <iulia.manda21@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      9009dd16
  2. 16 Mar, 2014 1 commit
  3. 14 Mar, 2014 5 commits
    • Chi Pham's avatar
      staging: cxt1e1: Removed assignments from if statements. · 24e7d799
      Chi Pham authored
      Assignments removed from if statements.
      Fixed checkpatch warning such as indentation and negative error returns in
      adjacent code.
      
      Coccinelle was used for this patch. The following script found the match:
      @simple@
      expression E1, E2;
      statement S1, S2;
      @@
      
      + E1 = E2;
        if (
      -     (E1 = E2)
      +     E1
           )
        S1 else S2
      
      @left@
      expression E0, E1, E2;
      statement S1, S2;
      @@
      
      + E1 = E2;
        if (
      -     (E1 = E2)
      +     E1
               == E0
           )
        S1 else S2
      Signed-off-by: default avatarChi Pham <fempsci@gmail.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      24e7d799
    • Andreea-Cristina Bernat's avatar
      staging: lirc: Add fallthrough comment · 5a9e30ee
      Andreea-Cristina Bernat authored
      This patch adds fallthrough comments for the cases not preceded by
      break or fallthrough comment
      Signed-off-by: default avatarAndreea-Cristina Bernat <bernat.ada@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      5a9e30ee
    • Himangi Saraogi's avatar
      staging:lustre: Fix Sparse Warning for Static Declarations in console.c · 18ce30c2
      Himangi Saraogi authored
      This patch fixes the following Sparse warnings in console.c :
      drivers/staging/lustre/lnet/selftest/console.c:65:1: warning: symbol 'lstcon_node_get' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:118:1: warning: symbol 'lstcon_node_put' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:348:1: warning: symbol 'lstcon_sesrpc_condition' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:377:1: warning: symbol 'lstcon_sesrpc_readent' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:834:1: warning: symbol 'lstcon_batch_find' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1002:1: warning: symbol 'lstcon_batrpc_condition' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1145:1: warning: symbol 'lstcon_testrpc_condition' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1374:1: warning: symbol 'lstcon_test_find' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1389:1: warning: symbol 'lstcon_tsbrpc_readent' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1468:1: warning: symbol 'lstcon_statrpc_readent' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1492:1: warning: symbol 'lstcon_ndlist_stat' was not declared. Should it be static?
      drivers/staging/lustre/lnet/selftest/console.c:1581:1: warning: symbol 'lstcon_debug_ndlist' was not declared. Should it be static?
      Signed-off-by: default avatarHimangi Saraogi <himangi774@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      18ce30c2
    • Gulsah Kose's avatar
      staging: rts5208: Remove unnecessary parentheses. · 0dedbf9f
      Gulsah Kose authored
      Return is not a method and doesn't need parentheses. Removed unnecaasary
      parentheses.
      Signed-off-by: default avatarGulsah Kose <gulsah.1004@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      0dedbf9f
    • Himangi Saraogi's avatar
      staging:comedi: Fix sparse warnings of incorrect type in assignment · cb90e78d
      Himangi Saraogi authored
      This patch fixes the following sparse warnings:
      drivers/staging/comedi/drivers/mite.c:343:44: warning: incorrect type in assignment (different base types)
      drivers/staging/comedi/drivers/mite.c:343:44:    expected unsigned int [unsigned] [usertype] count
      drivers/staging/comedi/drivers/mite.c:343:44:    got restricted __le32 [usertype] <noident>
      drivers/staging/comedi/drivers/mite.c:344:43: warning: incorrect type in assignment (different base types)
      drivers/staging/comedi/drivers/mite.c:344:43:    expected unsigned int [unsigned] [usertype] addr
      drivers/staging/comedi/drivers/mite.c:344:43:    got restricted __le32 [usertype] <noident>
      drivers/staging/comedi/drivers/mite.c:346:43: warning: incorrect type in assignment (different base types)
      drivers/staging/comedi/drivers/mite.c:346:43:    expected unsigned int [unsigned] [usertype] next
      drivers/staging/comedi/drivers/mite.c:346:43:    got restricted __le32 [usertype] <noident>
      drivers/staging/comedi/drivers/mite.c:351:45: warning: incorrect type in assignment (different base types)
      drivers/staging/comedi/drivers/mite.c:351:45:    expected unsigned int [unsigned] [usertype] next
      drivers/staging/comedi/drivers/mite.c:351:45:    got restricted __le32 [usertype] <noident>
      Signed-off-by: default avatarHimangi Saraogi <himangi774@gmail.com>
      Acked-by: default avatarPaul E. McKenney <paulmck@linux.vnet.ibm.com>
      Signed-off-by: default avatarPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
      cb90e78d