OSDN Git Service
Jes Sorensen [Sun, 25 May 2014 20:43:23 +0000 (22:43 +0200)]
staging: rtl8723au: Create rtl8723au_bt_intf.h
Use this header for declaring functions that are currently #ifdef'ed
out in the general code.
Start by moving BTDM_1Ant8723A() there and renaming it appropriately.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:22 +0000 (22:43 +0200)]
staging: rtl8723au: PS_RDY_CHECK() return bools in 'bool'
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:21 +0000 (22:43 +0200)]
staging: rtl8723au: _rtw_pwr_wakeup23a(): Do not mix _FAIL/false as return value
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:20 +0000 (22:43 +0200)]
staging: rtl8723au: bInternalAutoSuspend is always false
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:19 +0000 (22:43 +0200)]
staging: rtl8723au: Remove unused prototype rtw_cfg80211_issue_p2p_provision_request23a()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:18 +0000 (22:43 +0200)]
staging: rtl8723au: rtw_wdev_alloc(): Don't register wiphy before kmalloc() completed
There is no point starting to register the wiphy and then having to
unregister it again if the followon kmalloc() fails.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:17 +0000 (22:43 +0200)]
staging: rtl8723au: Remove some pointless cfg80211_* wrapper macros
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:16 +0000 (22:43 +0200)]
staging: rtl8723au: struct rtw_wdev_priv: Remove some never 'true' variables
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:15 +0000 (22:43 +0200)]
staging: rtl8723au: Get rid of unused struct rtw_wdev_invit_info
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:14 +0000 (22:43 +0200)]
staging: rtl8723au: _cfg80211_rtw_mgmt_tx() clean up checking function return values
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:13 +0000 (22:43 +0200)]
staging: rtl8723au: Delete unused rtw_cfg80211_set_mgnt_wpsp2pie()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:12 +0000 (22:43 +0200)]
staging: rtl8723au: ioctl_cfg80211.c: Use ether_addr_equal()
... and in one case use is_zero_ether_addr()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:11 +0000 (22:43 +0200)]
staging: rtl8723au: ioctl_cfg80211.c: Use ether_addr_copy()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:10 +0000 (22:43 +0200)]
staging: rtl8723au: Do not call netdev_open23a() from cfg80211_rtw_change_iface()
Calling open on the device when changing the interface type is wrong.
Let the stack call open at the right time.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:09 +0000 (22:43 +0200)]
staging: rtl8723au: Remove unused struct hostapd_priv
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:08 +0000 (22:43 +0200)]
staging: rtl8723au: rtw_mlme.h: Remove a bunch of unused structs
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:07 +0000 (22:43 +0200)]
staging: rtl8723au: Remove unused enum SCAN_RESULT_TYPE
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:06 +0000 (22:43 +0200)]
staging: rtl8723au: Don't check check_fwstate() == false
In addition, remove a followon check for false just after having
checked for true.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:05 +0000 (22:43 +0200)]
staging: rtl8723au: Make check_fwstate() return bool
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:04 +0000 (22:43 +0200)]
staging: rtl8723au: Don't explicitly check check_fwstate() == true
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:03 +0000 (22:43 +0200)]
staging: rtl8723au: rtw_xmit.c: Use ether_addr_copy()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:02 +0000 (22:43 +0200)]
staging: rtl8723au: Use ieee80211.h defines for IV/ICV len values
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 25 May 2014 20:43:01 +0000 (22:43 +0200)]
staging: rtl8723au: ieee80211.h: Remove some unused #defines
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:29 +0000 (21:36 +0100)]
staging: vt6656: MACvSetKeyEntry create structure to write key
Create structure for wKeyCtl , pbyAddr and pbyKey
wKeyCtl(key_ctl) and pbyAddr(addr) form an union of 64 bits with swap
of two 32 bits. pbyKey(key) has a length of WLAN_KEY_LEN_CCMP(16)
bytes.
swap is needed because high order 32 bits needs to written out first.
pbyKey is memcpy on to key.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:28 +0000 (21:36 +0100)]
staging: vt6656: s_vFillTxKey fix indentation of memcpy
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:27 +0000 (21:36 +0100)]
staging: vt6656: s_uGetRTSCTSRsvTime fix word lenght
Correct to length
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:26 +0000 (21:36 +0100)]
staging: vt6656: BBbVT3184Init fix word lenght
Correct to length
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:25 +0000 (21:36 +0100)]
staging: vt6656: baseband.c replace DBG_PRT debug messages
Replace with dev_dbg
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:24 +0000 (21:36 +0100)]
staging: vt6656: card.c replace DBG_PRT debug messages.
Replace with dev_dbg
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:23 +0000 (21:36 +0100)]
staging; vt6656: swGetCCKControlRate remove camel case
camel case changes
pDevice -> priv
wRateIdx -> rate_idx
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 25 May 2014 20:36:22 +0000 (21:36 +0100)]
staging: vt6656: swGetOFDMControlRate remove camel case
camel case changes
pDevice -> priv
wRateIdx -> rate_idx
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Manuel Schölling [Sun, 25 May 2014 18:07:02 +0000 (20:07 +0200)]
staging: dgnc: Use time_after_eq()
To be future-proof and for better readability the time comparisons are
modified to use time_after_eq() instead of plain, error-prone math.
Signed-off-by: Manuel Schölling <manuel.schoelling@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Chaitanya Hazarey [Sun, 25 May 2014 16:15:06 +0000 (09:15 -0700)]
Staging: rtl8192u: r819xU_firmware_img.c Fixed checkpatch.pl ERRORs
Fixed a lot of errors of the type "ERROR: space required after that ',' (ctx:VxV)"
Added tabs at the beginning of line.
Signed-off-by: Chaitanya Hazarey <c@24.io>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Manuel Schölling [Sun, 25 May 2014 13:05:24 +0000 (15:05 +0200)]
staging: rtl8188eu: use time_before/time_after and change type of ips_deny_time
To be future-proof and for better readability the time comparisons are
modified to use time_before/_after() instead of plain, error-prone math.
To suppress compiler warnings the type of ips_deny_time was changed.
Signed-off-by: Manuel Schölling <manuel.schoelling@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:12 +0000 (17:23 +0200)]
staging: rtl8821ae: add static for local symbols in rtl_btc.c
This commit fixes the sparse warning:
"warning: symbol 'X' was not declared. Should it be static?"
Where the X is one of the following
rtl_btc_operation
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:11 +0000 (17:23 +0200)]
staging: rtl8821ae: deal with unused functions in halbtcoutsrc.c
This commit fixes the compiler -Wunused-function warning. Several
functions had to be removed.
Removed symbols
halbtc_is_hw_mailbox_exist
halbtc_set_macreg
halbtc_get_macreg
halbtc_under_ips
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:10 +0000 (17:23 +0200)]
staging: rtl8821ae: add static for local symbols in halbtcoutsrc.c
This commit fixes the sparse warning:
"warning: symbol 'X' was not declared. Should it be static?"
Where the X is one of the following:
bt_profile_string
bt_spec_string
bt_link_role_string
h2c_state_string
io_state_string
btc_dbg_buf
halbtc_is_bt_coexist_available
halbtc_is_wifi_busy
halbtc_dbg_init
halbtc_is_hw_mailbox_exist
halbtc_is_bt40
halbtc_legacy
halbtc_get_wifi_bw
halbtc_get_wifi_central_chnl
halbtc_leave_lps
halbtc_enter_lps
halbtc_normal_lps
halbtc_leave_low_power
halbtc_nomal_low_power
halbtc_disable_low_power
halbtc_aggregation_check
halbtc_get_bt_patch_version
halbtc_get_wifi_rssi
halbtc_get
halbtc_set
halbtc_display_coex_statistics
halbtc_display_bt_link_info
halbtc_display_bt_fw_info
halbtc_display_fw_pwr_mode_cmd
halbtc_read_1byte
halbtc_read_2byte
halbtc_read_4byte
halbtc_write_1byte
halbtc_bitmask_write_1byte
halbtc_write_2byte
halbtc_write_4byte
halbtc_set_macreg
halbtc_get_macreg
halbtc_set_bbreg
halbtc_get_bbreg
halbtc_set_rfreg
halbtc_get_rfreg
halbtc_fill_h2c_cmd
halbtc_display_dbg_msg
halbtc_under_ips
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:09 +0000 (17:23 +0200)]
staging: rtl8821ae: deal with unused functions in halbtc8723b2ant.c
This commit fixes the compiler -Wunused-function warning. It was enough
to put #define around them to supress this warning. One was a special
case because it's usage was commented-out therefore, as I did not want
to remove it completely, I disabled it with #if 0
Conditionally compiled symbols
halbtc8723b2ant_monitor_bt_enable_disable
halbtc8723b2ant_set_bt_auto_report
halbtc8723b2ant_bt_auto_report
Disabled symbols
halbtc8723b2ant_adc_backoff
halbtc8723b2ant_set_adc_backoff
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:08 +0000 (17:23 +0200)]
staging: rtl8821ae: add static for local symbols in halbtc8723b2ant.c
This commit fixes the sparse warning:
"warning: symbol 'X' was not declared. Should it be static?"
Where the X is one of the following:
glbt_info_src_8723b_2ant
glcoex_ver_date_8723b_2ant
glcoex_ver_8723b_2ant
halbtc8723b2ant_bt_rssi_state
halbtc8723b2ant_wifi_rssi_state
halbtc8723b2ant_monitor_bt_enable_disable
halbtc8723b2ant_monitor_bt_ctr
halbtc8723b2ant_query_bt_info
halbtc8723b2ant_is_wifi_status_changed
halbtc8723b2ant_update_bt_link_info
halbtc8723b2ant_action_algorithm
halbtc8723b2ant_need_to_dec_bt_pwr
halbtc8723b2ant_set_fw_dac_swing_level
halbtc8723b2ant_set_fw_dec_bt_pwr
halbtc8723b2ant_dec_bt_pwr
halbtc8723b2ant_set_bt_auto_report
halbtc8723b2ant_bt_auto_report
halbtc8723b2ant_fw_dac_swing_lvl
halbtc8723b2ant_set_sw_rf_rx_lpf_corner
halbtc8723b2ant_rf_shrink
halbtc8723b2ant_set_sw_penalty_txrate_adaptive
halbtc8723b2ant_low_penalty_ra
halbtc8723b2ant_set_dac_swing_reg
halbtc8723b2ant_set_sw_fulltime_dac_swing
halbtc8723b2ant_dac_swing
halbtc8723b2ant_set_adc_backoff
halbtc8723b2ant_adc_backoff
halbtc8723b2ant_set_agc_table
halbtc8723b2ant_agc_table
halbtc8723b2ant_set_coex_table
halbtc8723b2ant_coex_table
halbtc8723b2ant_coex_table_with_type
halbtc8723b2ant_set_fw_ignore_wlan_act
halbtc8723b2ant_ignore_wlan_act
halbtc8723b2ant_set_fw_ps_tdma
halbtc8723b2ant_sw_mechanism1
halbtc8723b2ant_sw_mechanism2
halbtc8723b2ant_set_ant_path
halbtc8723b2ant_ps_tdma
halbtc8723b2ant_coex_alloff
halbtc8723b2ant_init_coex_dm
halbtc8723b2ant_action_bt_inquiry
halbtc8723b2ant_is_common_action
halbtc8723b2ant_tdma_duration_adjust
halbtc8723b2ant_action_sco
halbtc8723b2ant_action_hid
halbtc8723b2ant_action_a2dp
halbtc8723b2ant_action_a2dp_pan_hs
halbtc8723b2ant_action_pan_edr
halbtc8723b2ant_action_pan_hs
halbtc8723b2ant_action_pan_edr_a2dp
halbtc8723b2ant_action_pan_edr_hid
halbtc8723b2ant_action_hid_a2dp_pan_edr
halbtc8723b2ant_action_hid_a2dp
halbtc8723b2ant_run_coexist_mechanism
halbtc8723b2ant_wifioff_hwcfg
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:07 +0000 (17:23 +0200)]
staging: rtl8821ae: deal with unused functions in halbtc8723b1ant.c
This commit fixes the compiler -Wunused-function warning. Most of the
unused functions were removed however for a few of them it was enough
to put #define around them.
Removed symbols:
halbtc8723b1ant_rf_shrink
halbtc8723b1ant_dac_swing
halbtc8723b1ant_adc_backoff
halbtc8723b1ant_agc_table
halbtc8723b1ant_coex_alloff
halbtc8723b1ant_ps_tdma_type_by_wifi_rssi
halbtc8723b1ant_set_sw_rf_rx_lpf_corner
halbtc8723b1ant_set_sw_full_time_dac_swing
halbtc8723b1ant_set_adc_backoff
halbtc8723b1ant_set_agc_table
halbtc8723b1ant_set_dac_swing_reg
Conditionally compiled symbols:
halbtc8723b1ant_bt_auto_report
halbtc8723b1ant_set_bt_auto_report
halbtc8723b1ant_action_wifi_only
halbtc8723b1ant_monitor_bt_enable_disable
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Konrad Zapalowicz [Sat, 24 May 2014 15:23:06 +0000 (17:23 +0200)]
staging: rtl8821ae: add static for local symbols in halbtc8723b1ant.c
This commit fixes the sparse warning:
"warning: symbol 'X' was not declared. Should it be static?"
Where the X is one of the following:
glbt_info_src_8723b_2ant
glcoex_ver_date_8723b_2ant
glcoex_ver_8723b_2ant
halbtc8723b2ant_bt_rssi_state
halbtc8723b2ant_wifi_rssi_state
halbtc8723b2ant_monitor_bt_enable_disable
halbtc8723b2ant_monitor_bt_ctr
halbtc8723b2ant_query_bt_info
halbtc8723b2ant_is_wifi_status_changed
halbtc8723b2ant_update_bt_link_info
halbtc8723b2ant_action_algorithm
halbtc8723b2ant_need_to_dec_bt_pwr
halbtc8723b2ant_set_fw_dac_swing_level
halbtc8723b2ant_set_fw_dec_bt_pwr
halbtc8723b2ant_dec_bt_pwr
halbtc8723b2ant_set_bt_auto_report
halbtc8723b2ant_bt_auto_report
halbtc8723b2ant_fw_dac_swing_lvl
halbtc8723b2ant_set_sw_rf_rx_lpf_corner
halbtc8723b2ant_rf_shrink
halbtc8723b2ant_set_sw_penalty_txrate_adaptive
halbtc8723b2ant_low_penalty_ra
halbtc8723b2ant_set_dac_swing_reg
halbtc8723b2ant_set_sw_fulltime_dac_swing
halbtc8723b2ant_dac_swing
halbtc8723b2ant_set_adc_backoff
halbtc8723b2ant_adc_backoff
halbtc8723b2ant_set_agc_table
halbtc8723b2ant_agc_table
halbtc8723b2ant_set_coex_table
halbtc8723b2ant_coex_table
halbtc8723b2ant_coex_table_with_type
halbtc8723b2ant_set_fw_ignore_wlan_act
halbtc8723b2ant_ignore_wlan_act
halbtc8723b2ant_set_fw_ps_tdma
halbtc8723b2ant_sw_mechanism1
halbtc8723b2ant_sw_mechanism2
halbtc8723b2ant_set_ant_path
halbtc8723b2ant_ps_tdma
halbtc8723b2ant_coex_alloff
halbtc8723b2ant_init_coex_dm
halbtc8723b2ant_action_bt_inquiry
halbtc8723b2ant_is_common_action
halbtc8723b2ant_tdma_duration_adjust
halbtc8723b2ant_action_sco
halbtc8723b2ant_action_hid
halbtc8723b2ant_action_a2dp
halbtc8723b2ant_action_a2dp_pan_hs
halbtc8723b2ant_action_pan_edr
halbtc8723b2ant_action_pan_hs
halbtc8723b2ant_action_pan_edr_a2dp
halbtc8723b2ant_action_pan_edr_hid
halbtc8723b2ant_action_hid_a2dp_pan_edr
halbtc8723b2ant_action_hid_a2dp
halbtc8723b2ant_run_coexist_mechanism
halbtc8723b2ant_wifioff_hwcfg
Signed-off-by: Konrad Zapalowicz <bergo.torino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Niv Yehezkel [Sat, 24 May 2014 14:28:07 +0000 (10:28 -0400)]
staging: android: describe use of memory barrier on sync.c
Added comments describing the purpose of using write memory
barrier in the context of sync_timeline_destory.
Signed-off-by: Niv Yehezkel <executerx@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Pascal COMBES [Sat, 24 May 2014 13:58:09 +0000 (15:58 +0200)]
Staging: dgap: Fixed iomem accesses in dgap.c
I changed dereferences from iomem into the adequate ioread function.
Signed-off-by: Pascal COMBES <pascom@orange.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Joe Perches [Sat, 24 May 2014 05:13:20 +0000 (22:13 -0700)]
staging: Convert __FUNCTION__ to __func__
Use the normal mechanism for emitting a function name.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Vitaly Osipov [Sat, 24 May 2014 08:19:27 +0000 (18:19 +1000)]
staging: rtl8712: remove _malloc()
This patch removes all usage of _malloc() and the function itself. Most
uses are straightforward replacements by kmalloc(..., GFP_ATOMIC),
because this was the definition of _malloc(). In a few places it
was possible to use kzalloc() or memdup_user.
A further improvement would be to replace GFP_ATOMIC with GFP_KERNEL
where possible.
Verified by compilation only.
Initial replacement done by running a Coccinelle script along the lines
of:
@@
type T;
expression E;
identifier V;
@@
- V = (T) _malloc(E);
+ V = kmalloc(E, GFP_ATOMIC);
@@
expression E, E1;
@@
- E1 = _malloc(E);
+ E1 = kmalloc(E, GFP_ATOMIC);
Signed-off-by: Vitaly Osipov <vitaly.osipov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
David Matlack [Sat, 24 May 2014 04:04:55 +0000 (21:04 -0700)]
staging: slicoss: clean up use of dev_err
First, don't print pci device information or driver prefixes, this
is already printed by dev_err. Next, don't report error messages
via dev_err when the failing function already reports all errors
via dev_err.
Signed-off-by: David Matlack <matlackdavid@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
David Matlack [Sat, 24 May 2014 04:04:54 +0000 (21:04 -0700)]
staging: slicoss: remove private netdev list
Remove the private linked list of netdev structs. This list isn't
being used anyway.
This patch has no noticable effect.
Signed-off-by: David Matlack <matlackdavid@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg DeAngelis [Sat, 24 May 2014 03:09:11 +0000 (23:09 -0400)]
staging: lustre: Fixed warnings about using 0 as NULL ptr
Fixed a bunch of warnings pointed out by sparse about using 0 to
represent a NULL pointer.
Signed-off-by: Greg DeAngelis <gdeangel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Stephen Boyd [Sat, 24 May 2014 00:16:53 +0000 (17:16 -0700)]
staging/mt29f_spinand: Terminate of match table
Failure to terminate this match table can lead to boot failures
depending on where the compiler places the match table.
Cc: Kamlakant Patel <kamlakant.patel@broadcom.com>
Cc: Mona Anonuevo <manonuevo@micron.com>
Cc: linux-mtd@lists.infradead.org
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Dominique van den Broeck [Fri, 23 May 2014 23:35:24 +0000 (01:35 +0200)]
staging: panel: (coding style) Multiple assignments
Style-only modifications to comply with checkpatch.pl --strict --file.
. Breaks down compound assignments.
Signed-off-by: Dominique van den Broeck <domdevlin@free.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:10 +0000 (14:37 +0100)]
staging: vt6656: CARDvCalculateOFDMRParameter fix indentation rsv_time
In patch
staging: vt6656 camel case clean up CARDvCalculateOFDMRParameter
rsv_time is indented incorrectly.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:09 +0000 (14:37 +0100)]
staging: vt6656: camel case and clean up CARDvSetBSSMode
camel case changes
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:08 +0000 (14:37 +0100)]
staging: vt6656: camel case clean up CARDbRadioPowerOn
camel case changes
pDevice -> priv
bResult -> ret
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:07 +0000 (14:37 +0100)]
staging: vt6656: camel case clean up CARDbRadioPowerOff
camel case changes
pDevice -> priv
bResult -> ret
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:06 +0000 (14:37 +0100)]
staging: vt6656: camel case clean up CARDvUpdateNextTBTT
camel case changes
pDevice -> priv
qwTSF -> tsf
wBeaconInterval -> beacon_interval
pbyData -> data
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:05 +0000 (14:37 +0100)]
staging: vt6656: camel case and clean up CARDvSetFirstNextTBTT
camel case changes
pDevice -> priv
wBeaconInterval -> beacon_interval
qwNextTBTT -> next_tbtt
pbyData -> data
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:04 +0000 (14:37 +0100)]
staging: vt6656: CARDqGetNextTBTT remove camel case
qwTSF -> tsf
wBeaconInterval -> beacon_interval
uBeaconInterval -> beacon_int
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:03 +0000 (14:37 +0100)]
staging: vt6656: CARDbClearCurrentTSF remove camel case
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:02 +0000 (14:37 +0100)]
staging: vt6656: CARDbGetCurrentTSF remove camel case
pDevice -> priv
pqwCurrTSF -> current_tsf
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:01 +0000 (14:37 +0100)]
staging: vt6656: camel case and clean up CARDvAdjustTSF
camel case changes
pDevice -> priv
byRxRate -> rx_rate
qwBSSTimestamp -> time_stamp
qwLocalTSF -> local_tsf
qwTSFOffset -> tsf_offset
pbyData -> data
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:37:00 +0000 (14:37 +0100)]
staging: vt6656: CARDqGetTSFOffset remove camel case
camel case changes
byRxRate -> rx_rate
qwTSF1 -> tsf1
qwTSF2 -> tsf2
qwTSFOffset -> tsf_offset
wRxBcnTSFOffst -> rx_bcn_offset
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:36:59 +0000 (14:36 +0100)]
staging: vt6656: camel case and cleanup CARDbyGetPktType
camel case changes
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:36:58 +0000 (14:36 +0100)]
staging: vt6656: camel case clean up CARDbIsOFDMinBasicRate
camel case change
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 24 May 2014 13:36:57 +0000 (14:36 +0100)]
staging: vt6656: Convert remaining locks to spin_lock_irqsave.
Convert all remaining locks to spin_lock_irqsave and
spin_unlock_irqrestore
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 25 May 2014 17:56:30 +0000 (10:56 -0700)]
Merge tag 'iio-for-3.16c' of git://git./linux/kernel/git/jic23/iio into staging-next
Jonathan writes:
Third round of new stuff for IIO in the 3.16 cycle.
This (rather late) set consists only of fixes to patches earlier in the
cycle or minor fixes for other problems (such as randconfig build issues
picked up by Arnd). I've included the general minor fixes here as it is
very late in the current cycle and they can all wait for the merge window.
* Recent change to hid-sensors introduced a possible infinite loop due to a
typo. In the same series, a report interval unit read was added but in the
case where the hardware doesn't support it a value of 0 would lead to some
nasty issues. The patch sets it to the specified default of msecs.
* Anon inodes were used by IIO without being explicitly selected. This has
been true for a long time so it clearly only effects rather unusual
configurations (rand configs)
* at91 requires the input subsytem but this wasn't explicity in the Kconfig.
* A couple of parts supported by the max1363 driver were using the wrong
iio_chan_spec arrays and hence would missreport their bit depths. This has
been there a long time and was never right so isn't a regression.
Toralf Förster [Sat, 24 May 2014 13:25:00 +0000 (14:25 +0100)]
fix format string missmatch in staging/iio/Documentation/iio_utils.h
spotted by cppcheck
Signed-off-by: Toralf Förster <toralf.foerster@gmx.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Jonathan Cameron [Sat, 24 May 2014 11:52:10 +0000 (12:52 +0100)]
iio:adc:max1363 incorrect resolutions for max11604, max11605, max11610 and max11611.
Cc: Stable@vger.kernel.org>
Reported-by: Erik Habbinga <Erik.Habbinga@schneider-electric.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Acked-by: Hartmut Knaack <knaack.h@gmx.de>
Greg Kroah-Hartman [Sat, 24 May 2014 20:01:06 +0000 (13:01 -0700)]
Revert "staging: r8188eu: Add support for Dlink DWA-121 802.11n Wireless N 150 Pico Adapter"
This reverts commit
599a2c02a784934b9bf0725920c1bc235b9b28ce.
Larry writes:
This patch should be dropped. The 0x2001:0x3308 device is an
RTL8188CU, not RTL8188EU. As a result, it is already in the USB
device tables.
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Kim Nylund <kim@pratsam.com>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:24:05 +0000 (22:24 +0200)]
Staging: bcm: Shortened prototype for GetNextTargetBufferLocation()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:24:04 +0000 (22:24 +0200)]
Staging: bcm: Shortened lines in SearchClsid()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:24:03 +0000 (22:24 +0200)]
Staging: bcm: Shortened lines in deleteSFBySfid()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:24:02 +0000 (22:24 +0200)]
Staging: bcm: Shortened lines in CopyIpAddrToClassifier()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:24:01 +0000 (22:24 +0200)]
Staging: bcm: Replaced member accessing by variable
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:24:00 +0000 (22:24 +0200)]
Staging: bcm: Shortened lines and replaced member accessing in CopyIpAddrToClassifier()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:59 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in CopyClassifierRuleToSF()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:58 +0000 (22:23 +0200)]
Staging: bcm: Replaced member accessing with variable
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:57 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in DeleteClassifierRuleFromSF()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:56 +0000 (22:23 +0200)]
Staging: bcm: Shortened line in DeleteAllClassifiersForSF()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:55 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in CopyToAdapter()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:54 +0000 (22:23 +0200)]
Staging: bcm: Replaced struct member accessing with variable
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:53 +0000 (22:23 +0200)]
Staging: bcm: Outsourced PHS rule applying
This patch outsources the big PHS-rule-applying code chunk into a
function.
It also replaces member accessing by using variables.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:52 +0000 (22:23 +0200)]
Staging: bcm: Shortened some lines in DumpCmControlPacker()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:51 +0000 (22:23 +0200)]
Staging: bcm: Replaced long member-accessing
This patch replaces long object->member.member foo by variable useage.
Increases the reading experience a lot by introducing only one variable
(as it decreases the whoa-wall-of-text experience).
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:50 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in RestoreSFParam()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:49 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in StoreSFParam()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:48 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in StoreCmControlResponseMessage()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:47 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in RestoreCmControlResponseMessage()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:46 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in SetupTargetDsxBuffers()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:45 +0000 (22:23 +0200)]
Staging: bcm: Shortenend lines in GetNextTargetBufferLocation()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:44 +0000 (22:23 +0200)]
Staging: bcm: Shortened line
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:43 +0000 (22:23 +0200)]
Staging: bcm: Simplified by using variables
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:42 +0000 (22:23 +0200)]
Staging: bcm: Outsourced restoring of endianess for ip adresses
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:41 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in get_dsx_sf_data_to_application()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:40 +0000 (22:23 +0200)]
Staging: bcm: Shortened lines in OverrideServiceFlowParams()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Fri, 23 May 2014 20:23:39 +0000 (22:23 +0200)]
Staging: bcm: Shortened code by using variable
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Davide Gianforte [Fri, 23 May 2014 20:06:44 +0000 (22:06 +0200)]
staging: gdm72xx: code cleanup
Checkpatch.pl cleanup
Thanks again to Greg KH and Dan Carpenter for the patience :)
Signed-off-by: Davide Gianforte <davide@gengisdave.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Mitchel Humpherys [Fri, 23 May 2014 20:01:22 +0000 (13:01 -0700)]
staging: ion: WARN when the handle kmap_cnt is going to wrap around
There are certain client bugs (double unmap, for example) that can cause
the handle->kmap_cnt (an unsigned int) to wrap around from zero. This
causes problems when the handle is destroyed because we have:
while (handle->kmap_cnt)
ion_handle_kmap_put(handle);
which takes a long time to complete when kmap_cnt starts at ~0 and can
result in a watchdog timeout.
WARN and bail when kmap_cnt is about to wrap around from zero.
Signed-off-by: Mitchel Humpherys <mitchelh@codeaurora.org>
Acked-by: Colin Cross <ccross@android.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Nicolas Joseph [Fri, 23 May 2014 19:45:08 +0000 (21:45 +0200)]
staging/olpc_dcon: fix checkpatch warnings
WARNING: Missing a blank line after declarations
Signed-off-by: Nicolas Joseph <nicolas.joseph@homecomputing.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Fri, 23 May 2014 17:00:19 +0000 (18:00 +0100)]
staging: vt6656: camel case cleanup CARDbAddBasicRate
camel case changes
pDevice -> priv
wRateIdx -> rate_idx
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Fri, 23 May 2014 17:00:18 +0000 (18:00 +0100)]
staging: vt6656: camel case clean up CARDvUpdateBasicTopRate
Camel case changes
pDevice -> priv
byTopOFDM -> top_ofdm
byTopCCK -> top_cck
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>