linux.git
16 months agostaging: rtl8192e: Remove variable dot11d_info
Philipp Hortmann [Fri, 8 Dec 2023 18:19:51 +0000 (19:19 +0100)]
staging: rtl8192e: Remove variable dot11d_info

Remove variable dot11d_info as it is unused. Remove unused struct
channel_list and empty struct rt_dot11d_info.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/2438971feb0c02c5c72776f57201a4011f249190.1701989555.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove variable channel_map
Philipp Hortmann [Fri, 8 Dec 2023 18:19:45 +0000 (19:19 +0100)]
staging: rtl8192e: Remove variable channel_map

Remove variable channel_map as it is initialized but not used. Remove
channel_array as well as it is without channel_map unused.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/19ca762b4b9d5a3e857b31c22f0688324ca3bd9d.1701989555.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove function rtllib_update_active_chan_map()
Philipp Hortmann [Fri, 8 Dec 2023 18:19:32 +0000 (19:19 +0100)]
staging: rtl8192e: Remove function rtllib_update_active_chan_map()

Remove function rtllib_update_active_chan_map() as "active_channel_map"
and "channel_map" have the same content.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/bae3ddeffe96e4ac7d929127ce3a72cd23fae8dd.1701989555.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bTxEnableFwCalcDur
Gary Rookard [Wed, 6 Dec 2023 23:04:04 +0000 (18:04 -0500)]
staging: rtl8192e: renamed variable bTxEnableFwCalcDur

Coding style issue, checkpatch Avoid CamelCase,
rename it. bTxEnableFwCalcDur -> tx_enable_fw_calc_dur

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231206230404.1721-6-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable MPDU_Density
Gary Rookard [Wed, 6 Dec 2023 23:04:03 +0000 (18:04 -0500)]
staging: rtl8192e: renamed variable MPDU_Density

Coding style issue, checkpatch Avoid CamelCase,
rename it. MPDU_Density -> mpdu_density

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231206230404.1721-5-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable AMPDU_Factor
Gary Rookard [Wed, 6 Dec 2023 23:04:02 +0000 (18:04 -0500)]
staging: rtl8192e: renamed variable AMPDU_Factor

Coding style issue, checkpatch Avoid CamelCase,
rename it. AMPDU_Factor -> ampdu_factor

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231206230404.1721-4-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bAMPDUEnable
Gary Rookard [Wed, 6 Dec 2023 23:04:01 +0000 (18:04 -0500)]
staging: rtl8192e: renamed variable bAMPDUEnable

Coding style issue, checkpatch Avoid CamelCase,
rename it. bAMPDUEnable -> ampdu_enable

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231206230404.1721-3-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bAMSDU_Support
Gary Rookard [Wed, 6 Dec 2023 23:04:00 +0000 (18:04 -0500)]
staging: rtl8192e: renamed variable bAMSDU_Support

Coding style issue, checkpatch Avoid CamelCase,
rename it. bAMSDU_Suppport -> amsdu_support

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231206230404.1721-2-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: vchiq_dev: Use %p to log pointer address
Umang Jain [Thu, 7 Dec 2023 08:38:37 +0000 (14:08 +0530)]
staging: vc04_services: vchiq_dev: Use %p to log pointer address

Solves the following Smatch warnings:
vchiq_release() warn: argument 7 to %lx specifier is cast from pointer

%p will print the hashed pointer to dynamic debug.
In order to print the unmodified pointer address, one can use the
`no_hash_pointers` via kernel parameters.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231207083837.153843-3-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: vchiq_arm: Use %p to log pointer address
Umang Jain [Thu, 7 Dec 2023 08:38:36 +0000 (14:08 +0530)]
staging: vc04_services: vchiq_arm: Use %p to log pointer address

Solves the following Smatch warnings:
service_callback() warn: argument 7 to %lx specifier is cast from pointer
service_callback() warn: argument 11 to %lx specifier is cast from pointer
service_callback() warn: argument 12 to %lx specifier is cast from pointer
service_callback() warn: argument 13 to %lx specifier is cast from pointer

%p will print the hashed pointer to dynamic debug.
In order to print the unmodified pointer address, one can use the
`no_hash_pointers` via kernel parameters.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231207083837.153843-2-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: Drop vchiq_log_debug() in favour of dev_dbg
Umang Jain [Tue, 5 Dec 2023 08:41:57 +0000 (14:11 +0530)]
staging: vc04_services: Drop vchiq_log_debug() in favour of dev_dbg

Drop vchiq_log_debug() macro which wraps dev_dbg(). Introduce the usage
of dev_dbg() directly.

Meanwhile at it, drop the usage of __func__ from the logs.
Dynamic debug supports this via the 'f'  decorator flag.

Remove the entry from TODO regarding custom logging. VC04 is now
aligned according to the standard kernel logging mechanisms.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231205084157.73819-5-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: Drop vchiq_log_trace() in favour of dev_dbg
Umang Jain [Tue, 5 Dec 2023 08:41:56 +0000 (14:11 +0530)]
staging: vc04_services: Drop vchiq_log_trace() in favour of dev_dbg

Drop vchiq_log_trace() macro which wraps dev_dbg(). Introduce the usage
of dev_dbg() directly.

Meanwhile at it, drop the usage of __func__ from the logs.
Dynamic debug supports this via the 'f'  decorator flag.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231205084157.73819-4-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: Drop vchiq_log_warning() in favour of dev_warn
Umang Jain [Tue, 5 Dec 2023 08:41:55 +0000 (14:11 +0530)]
staging: vc04_services: Drop vchiq_log_warning() in favour of dev_warn

Drop vchiq_log_warning() macro which wraps dev_dbg(). Introduce the usage
of dev_warn() directly.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231205084157.73819-3-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: Drop vchiq_log_error() in favour of dev_err
Umang Jain [Tue, 5 Dec 2023 08:41:54 +0000 (14:11 +0530)]
staging: vc04_services: Drop vchiq_log_error() in favour of dev_err

Drop vchiq_log_error() macro which wraps dev_dbg(). Introduce the usage
of dev_err() directly.

Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231205084157.73819-2-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable nAMSDU_MaxSize
Gary Rookard [Tue, 5 Dec 2023 23:16:23 +0000 (18:16 -0500)]
staging: rtl8192e: renamed variable nAMSDU_MaxSize

Coding style issue, checkpatch Avoid CamelCase,
rename it nAMSDU_MaxSize -> amsdu_max_size

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231205231623.23070-6-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bRegSuppCCK
Gary Rookard [Tue, 5 Dec 2023 23:16:22 +0000 (18:16 -0500)]
staging: rtl8192e: renamed variable bRegSuppCCK

Coding style issue, checkpatch Avoid CamelCase,
rename it bRegSuppCCK -> reg_supp_cck

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231205231623.23070-5-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bRegBW40MHz
Gary Rookard [Tue, 5 Dec 2023 23:16:21 +0000 (18:16 -0500)]
staging: rtl8192e: renamed variable bRegBW40MHz

Coding style issue, checkpatch Avoid CamelCase,
rename it bRegBW40MHz -> reg_bw_40mhz

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231205231623.23070-4-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bRegShortGI40MHz
Gary Rookard [Tue, 5 Dec 2023 23:16:20 +0000 (18:16 -0500)]
staging: rtl8192e: renamed variable bRegShortGI40MHz

Coding style issue, checkpatch Avoid CamelCase,
rename it bRegShortGI40MHz -> reg_short_gi_40mhz

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231205231623.23070-3-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bRegShortGI20MHz
Gary Rookard [Tue, 5 Dec 2023 23:16:19 +0000 (18:16 -0500)]
staging: rtl8192e: renamed variable bRegShortGI20MHz

Coding style issue, checkpatch Avoid CamelCase,
rename it bRegShortGI20MHz -> reg_short_gi_20mhz

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231205231623.23070-2-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: Do not pass NULL to vchiq_log_error()
Umang Jain [Tue, 28 Nov 2023 20:19:22 +0000 (01:49 +0530)]
staging: vc04_services: Do not pass NULL to vchiq_log_error()

vchiq_add_connected_callback() logs using vchiq_log_error() macro,
but passes NULL instead of a struct device pointer. Fix it.

vchiq_add_connected_callback() is not used anywhere in the vc04_services
as of now. It will be used when we add new drivers(VC shared memory and
bcm2835-isp), hence it kept as it is for now.

Fixes: 1d8915cf8899 ("staging: vc04: Convert vchiq_log_error() to use dynamic debug")
Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231128201926.489269-2-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: vchiq_core: Log through struct vchiq_instance
Umang Jain [Tue, 28 Nov 2023 20:18:45 +0000 (01:48 +0530)]
staging: vc04_services: vchiq_core: Log through struct vchiq_instance

The handle_to_service() helper can return NULL, so `service` pointer
can indeed be set to NULL. So, do not log through service pointer
(which can cause NULL-deference), instead, use the vchiq_instance
function argument to get access to the struct device.

Fixes: f67af5940d6d ("staging: vc04: Convert(and rename) vchiq_log_info() to use dynamic debug")
Reviewed-by: Ricardo B. Marliere <ricardo@marliere.net>
Signed-off-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/20231128201845.489237-1-umang.jain@ideasonboard.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable nDataRate
Gary Rookard [Tue, 28 Nov 2023 18:17:27 +0000 (13:17 -0500)]
staging: rtl8192e: renamed variable nDataRate

Coding style issue, checkpatch Avoid CamelCase,
rename it nDataRate -> data-rate.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231128181727.19504-4-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable bCurBW40MHz
Gary Rookard [Tue, 28 Nov 2023 18:17:26 +0000 (13:17 -0500)]
staging: rtl8192e: renamed variable bCurBW40MHz

Coding style issue, checkpatch Avoid CamelCase,
rename it bCurBW40MHz -> cur_bw_40mhz.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231128181727.19504-3-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable nMcsRate
Gary Rookard [Tue, 28 Nov 2023 18:17:25 +0000 (13:17 -0500)]
staging: rtl8192e: renamed variable nMcsRate

Coding style issue, checkpatch Avoid CamelCase,
rename it nMcsRate -> mcs_rate.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Link: https://lore.kernel.org/r/20231128181727.19504-2-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable TsCommonInfo
Tree Davies [Mon, 27 Nov 2023 05:43:05 +0000 (21:43 -0800)]
Staging: rtl8192e: Rename variable TsCommonInfo

Rename variable TsCommonInfo to ts_common_info to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-16-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable pAdmittedBA
Tree Davies [Mon, 27 Nov 2023 05:43:04 +0000 (21:43 -0800)]
Staging: rtl8192e: Rename variable pAdmittedBA

Rename variable pAdmittedBA to admitted_ba to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-15-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable pBaTimeoutVal
Tree Davies [Mon, 27 Nov 2023 05:43:03 +0000 (21:43 -0800)]
Staging: rtl8192e: Rename variable pBaTimeoutVal

Rename variable pBaTimeoutVal to ba_timeout_value to fix checkpatch
warning Avoid CamelCase, and make the name consistent with variables
of same name/type.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-14-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable pBaParamSet
Tree Davies [Mon, 27 Nov 2023 05:43:02 +0000 (21:43 -0800)]
Staging: rtl8192e: Rename variable pBaParamSet

Rename variable pBaParamSet to ba_param_set to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-13-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable DelbaParamSet
Tree Davies [Mon, 27 Nov 2023 05:43:01 +0000 (21:43 -0800)]
Staging: rtl8192e: Rename variable DelbaParamSet

Rename variable DelbaParamSet to del_ba_param_set to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-12-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable TsAddBaTimer
Tree Davies [Mon, 27 Nov 2023 05:43:00 +0000 (21:43 -0800)]
Staging: rtl8192e: Rename variable TsAddBaTimer

Rename variable TsAddBaTimer to ts_add_ba_timer to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-11-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable TxCurSeq
Tree Davies [Mon, 27 Nov 2023 05:42:59 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable TxCurSeq

Rename variable TxCurSeq to tx_cur_seq to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-10-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable pTsCommonInfo
Tree Davies [Mon, 27 Nov 2023 05:42:58 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable pTsCommonInfo

Rename variable pTsCommonInfo to ts_common_info to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-9-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable pDialogToken
Tree Davies [Mon, 27 Nov 2023 05:42:57 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable pDialogToken

Rename variable pDialogToken to dialog_token to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-8-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable TxPendingBARecord
Tree Davies [Mon, 27 Nov 2023 05:42:56 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable TxPendingBARecord

Rename variable TxPendingBARecord to tx_pending_ba_record to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-7-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable TxAdmittedBARecord
Tree Davies [Mon, 27 Nov 2023 05:42:55 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable TxAdmittedBARecord

Rename variable TxAdmittedBARecord to tx_admitted_ba_record to fix checkpatch
warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-6-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable TSpec
Tree Davies [Mon, 27 Nov 2023 05:42:54 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable TSpec

Rename variable TSpec to tspec to fix checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-5-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable Delba
Tree Davies [Mon, 27 Nov 2023 05:42:53 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable Delba

Rename variable Delba to del_ba to fix checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-4-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable BAReq
Tree Davies [Mon, 27 Nov 2023 05:42:52 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable BAReq

Rename variable BAReq to ba_req to fix checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-3-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable pTxTs
Tree Davies [Mon, 27 Nov 2023 05:42:51 +0000 (21:42 -0800)]
Staging: rtl8192e: Rename variable pTxTs

Rename variable pTxTs to ts to fix checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Link: https://lore.kernel.org/r/20231127054305.148276-2-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove function rtl92e_update_rx_pkt_timestamp()
Philipp Hortmann [Fri, 24 Nov 2023 21:31:27 +0000 (22:31 +0100)]
staging: rtl8192e: Remove function rtl92e_update_rx_pkt_timestamp()

mac_time and last_rx_desc_tsf are only used in
rtl92e_update_rx_pkt_timestamp(). Depending on a condition one is equal
to the other or vice versa. But since those are not used anywhere else
the function rtl92e_update_rx_pkt_timestamp() is just dead code.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/89a21fa17b32d66e07514bfad5b604d5d4835e25.1700860759.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function HTConstructInfoElement()
Philipp Hortmann [Fri, 24 Nov 2023 21:31:21 +0000 (22:31 +0100)]
staging: rtl8192e: Remove unused function HTConstructInfoElement()

Remove unused function HTConstructInfoElement().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/ed49162367b03a848895638adcdfe3594d4219b3.1700860759.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function rtllib_probe_resp()
Philipp Hortmann [Fri, 24 Nov 2023 21:31:15 +0000 (22:31 +0100)]
staging: rtl8192e: Remove unused function rtllib_probe_resp()

Remove unused function rtllib_probe_resp().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/be709ffea087df6f960075a51c0eab89f86ef8b2.1700860759.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function rtllib_get_beacon_()
Philipp Hortmann [Fri, 24 Nov 2023 21:31:08 +0000 (22:31 +0100)]
staging: rtl8192e: Remove unused function rtllib_get_beacon_()

Remove unused function rtllib_get_beacon_().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/ed825d5fb1946530c310afca1d3f27e46da35f06.1700860759.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function rtllib_send_beacon()
Philipp Hortmann [Fri, 24 Nov 2023 21:31:01 +0000 (22:31 +0100)]
staging: rtl8192e: Remove unused function rtllib_send_beacon()

Remove unused function rtllib_send_beacon().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/fca983a276ae67dc9cd124e236fe9d210fd997d3.1700860759.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused timer beacon_timer
Philipp Hortmann [Fri, 24 Nov 2023 21:30:47 +0000 (22:30 +0100)]
staging: rtl8192e: Remove unused timer beacon_timer

Driver does not support AP Mode therefore no beacons need to be send.
Remove unused timer beacon_timer.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/b16b6b4525ede8e2a218cb39d23f9ef2a3a745a9.1700860759.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function rtllib_get_beacon()
Philipp Hortmann [Fri, 24 Nov 2023 21:30:39 +0000 (22:30 +0100)]
staging: rtl8192e: Remove unused function rtllib_get_beacon()

Remove unused function rtllib_get_beacon().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/d76eeb7e273f4fcfec4ae8879e56d237363ebde2.1700860758.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused interrupt for IMR_BcnInt
Philipp Hortmann [Fri, 24 Nov 2023 21:30:23 +0000 (22:30 +0100)]
staging: rtl8192e: Remove unused interrupt for IMR_BcnInt

Driver does not support AP Mode therefore no beacons need to be send.
Remove unused interrupt for IMR_BcnInt.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/64a1f672dbe2ed2c6a660fa1044bd058fe3ba91a.1700860758.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable HTIOTActIsMgntUseCCK6M
Gary Rookard [Thu, 23 Nov 2023 14:43:37 +0000 (09:43 -0500)]
staging: rtl8192e: renamed variable HTIOTActIsMgntUseCCK6M

Renamed from Pascal/CamelCase to Snake case the variable
HTIOTActIsMgntUseCCK6M.
HTIOTActIsMgntUseCCK6M -> ht_iot_act_is_mgnt_use_cck_6m

Linux kernel coding style (cleanup), checkpatch Avoid CamelCase.
Driver/module rtl8192e compiles.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231123144337.13112-6-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable HTIOTPeerDetermine
Gary Rookard [Thu, 23 Nov 2023 14:43:36 +0000 (09:43 -0500)]
staging: rtl8192e: renamed variable HTIOTPeerDetermine

Renamed from Pascal/CamelCase to Snake case the variable
HTIOTPeerDetermine.
HTIOTPeerDetermine -> ht_iot_peer_determine

Linux kernel coding style (cleanup), checkpatch Avoid CamelCase.
Driver/module rtl8192e compiles.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231123144337.13112-5-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable IsHTHalfNmodeAPs
Gary Rookard [Thu, 23 Nov 2023 14:43:35 +0000 (09:43 -0500)]
staging: rtl8192e: renamed variable IsHTHalfNmodeAPs

Renamed from Pascal/CamelCase to Snake case the variable
IsHTHalfNmodeAPs.
ISHTHalfNmodeAPs -> is_ht_half_nmode_aps

Linux kernel coding style (cleanup), checkpatch Avoid CamelCase.
Driver/module rtl8192e compiles.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231123144337.13112-4-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable TXCountToDataRate
Gary Rookard [Thu, 23 Nov 2023 14:43:34 +0000 (09:43 -0500)]
staging: rtl8192e: renamed variable TXCountToDataRate

Renamed from Pascal/CamelCase to Snake case the variable
TXCountToDataRate.
TXCountToDataRate -> tx_count_to_data_rate

Linux kernel coding style (cleanup), checkpatch Avoid CamelCase.
Driver/module rtl8192e compiles.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231123144337.13112-3-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: renamed variable HTMcsToDataRate
Gary Rookard [Thu, 23 Nov 2023 14:43:33 +0000 (09:43 -0500)]
staging: rtl8192e: renamed variable HTMcsToDataRate

Renamed variable from Pascal/CamelCase to Snake case the variable
HTMcsToDataRate.
HTMcsToDataRate -> ht_mcs_to_data_rate

Linux Kernel coding style (cleanup), checkpatch Avoid CamelCase.
Driver/module rtl8192e compiles.

Signed-off-by: Gary Rookard <garyrookard@fastmail.org>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231123144337.13112-2-garyrookard@fastmail.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vchiq_arm: move state dump to debugfs
Stefan Wahren [Sun, 29 Oct 2023 12:48:37 +0000 (13:48 +0100)]
staging: vchiq_arm: move state dump to debugfs

Besides the IOCTL interface the VCHIQ character device also provides
a state dump of the whole VCHIQ driver via read. Moving the state dump
function to debugfs has a lot advantages:

- following changes on state dump doesn't break userspace ABI
- debug doesn't depend on VCHIQ_CDEV
- dump code simplifies a lot and reduce the chance of buffer overflows

Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
Tested-by: "Ricardo B. Marliere" <ricardo@marliere.net>
Link: https://lore.kernel.org/r/20231029124837.119832-4-wahrenst@gmx.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vchiq_core: Shorten bulk TX/RX pending dump
Stefan Wahren [Sun, 29 Oct 2023 12:48:36 +0000 (13:48 +0100)]
staging: vchiq_core: Shorten bulk TX/RX pending dump

The calculation for the bulk TX/RX pending is complex and
reaches 99 chars per line. So move the size determination
below the pending calculation and get the rid of the
ternary operator.

Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://lore.kernel.org/r/20231029124837.119832-3-wahrenst@gmx.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vchiq_core: Make vchiq_dump_service_state static
Stefan Wahren [Sun, 29 Oct 2023 12:48:35 +0000 (13:48 +0100)]
staging: vchiq_core: Make vchiq_dump_service_state static

The function vchiq_dump_service_state() is only used by vchiq_dump_state()
within vchiq_core.c. So move the definition of vchiq_dump_state() below
vchiq_dump_service_state() in order to make it static.

Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://lore.kernel.org/r/20231029124837.119832-2-wahrenst@gmx.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agodrivers: staging: vme_user: Describe VME_BUS and VME_TSI148
Bagas Sanjaya [Mon, 13 Nov 2023 09:38:38 +0000 (16:38 +0700)]
drivers: staging: vme_user: Describe VME_BUS and VME_TSI148

Help description for both options only tells users to enable them
without description of what VME bridge and TSI148 device are.

Briefly describe them.

Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
Link: https://lore.kernel.org/r/20231113093839.7687-1-bagasdotme@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vc04_services: remove unnecessary NULL check
Dan Carpenter [Tue, 31 Oct 2023 08:59:34 +0000 (11:59 +0300)]
staging: vc04_services: remove unnecessary NULL check

We ensured that "service" as non-NULL in the previous if statement so
there is no need to check again here.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Umang Jain <umang.jain@ideasonboard.com>
Link: https://lore.kernel.org/r/1dff4d9b-d067-4525-95e0-ffdc1185cccd@moroto.mountain
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from variable name "apTailTD"
Pavan Bobba [Tue, 31 Oct 2023 05:34:34 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from variable name "apTailTD"

variable name "apTailTD" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/482553f089fe86dc7ebecd96c9397cfaa9c7bdf9.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from variable name "pCurrTD"
Pavan Bobba [Tue, 31 Oct 2023 05:34:33 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from variable name "pCurrTD"

variable name "pCurrTD" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/06dc61ae807dcaa1dd8fdbf18686f0f4c20be634.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from function name "CARDvSafeResetTx"
Pavan Bobba [Tue, 31 Oct 2023 05:34:32 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from function name "CARDvSafeResetTx"

function name "CARDvSafeResetTx" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/785a163ab01555bd0a7ed87484d727560d22cd95.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from function name "CARDbRadioPowerOff"
Pavan Bobba [Tue, 31 Oct 2023 05:34:31 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from function name "CARDbRadioPowerOff"

function name "CARDbRadioPowerOff" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/5b264db5a6d5ac936e02f0a7f465e648504a049e.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from function name "CARDqGetNextTBTT"
Pavan Bobba [Tue, 31 Oct 2023 05:34:30 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from function name "CARDqGetNextTBTT"

function name "CARDqGetNextTBTT" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/8e142b33245e6933fb178080b0b14371939a84e6.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from variable name "qwNextTBTT"
Pavan Bobba [Tue, 31 Oct 2023 05:34:29 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from variable name "qwNextTBTT"

variable name "qwTSFOffset" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/6b1fe342f7577c46a980cdc1bee0c7f560f05d87.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from variable name "wBeaconInterval"
Pavan Bobba [Tue, 31 Oct 2023 05:34:28 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from variable name "wBeaconInterval"

variable name "wBeaconInterval" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/a8aa6227fb8dadfdebb0a6ab74cff9730358c765.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from function name "CARDbSetBeaconPeriod"
Pavan Bobba [Tue, 31 Oct 2023 05:34:27 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from function name "CARDbSetBeaconPeriod"

function name "CARDbSetBeaconPeriod" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/8d4c10ac86f80bb46d5cb1f18079276c3326e5dd.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from function name "CARDqGetTSFOffset"
Pavan Bobba [Tue, 31 Oct 2023 05:34:26 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from function name "CARDqGetTSFOffset"

function name "CARDqGetTSFOffset" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/b6b86bd65669cae59ca10171ecee8aa4a13c4a26.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from variable name "qwTSFOffset"
Pavan Bobba [Tue, 31 Oct 2023 05:34:25 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from variable name "qwTSFOffset"

variable name "qwTSFOffset" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/035b910c92233a7252a4b312e0cfe5adbcbbae79.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: vt6655: Type encoding info dropped from variable name "qwBSSTimestamp"
Pavan Bobba [Tue, 31 Oct 2023 05:34:24 +0000 (11:04 +0530)]
staging: vt6655: Type encoding info dropped from variable name "qwBSSTimestamp"

variable name "qwBSSTimestamp" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/977e5e538610d28dce6a02a6d475ee336fc0899c.1698730318.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused variable bss_start_channel
Philipp Hortmann [Sun, 19 Nov 2023 22:16:05 +0000 (23:16 +0100)]
staging: rtl8192e: Remove unused variable bss_start_channel

Remove unused variable bss_start_channel.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/a13a8421d43ef1baa2077af1d4955059eba52d79.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused struct chnl_txpow_triple
Philipp Hortmann [Sun, 19 Nov 2023 22:15:53 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused struct chnl_txpow_triple

Remove unused struct chnl_txpow_triple.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/f66c7cd95ae460ccb376e9edce11b2b955c908cc.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function dot11d_init()
Philipp Hortmann [Sun, 19 Nov 2023 22:15:47 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused function dot11d_init()

The variable dot11d_info->channel_map is initialized in dot11d_init() and
in dot11d_channel_map(). dot11d_init() is called only once just before
dot11d_channel_map(). Therefore dot11d_init() can be removed.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/d2e8d81ec9eefc7b7eeb9f6fd6a28b28db6b40f0.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused variable global_domain
Philipp Hortmann [Sun, 19 Nov 2023 22:15:39 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused variable global_domain

global_domain is initialized to false and then never used.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/5e7a0753ed9d6d934203fe29ab8ecef41090f055.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused variable country_watchdog
Philipp Hortmann [Sun, 19 Nov 2023 22:15:32 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused variable country_watchdog

country_watchdog is initialized to 0 and then never changed. Remove
related macros and functions.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/958f2ebbedb51e71e11479f734b4df27484160e1.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused variables from struct rt_dot11d_info
Philipp Hortmann [Sun, 19 Nov 2023 22:15:26 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused variables from struct rt_dot11d_info

Remove unused variables country_len, country_buffer, country_src_addr and
max_tx_power_list. Remove comments about usage as well.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/eeb9915d68c353554973e1056893371d15381f88.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused variable dot11d_info->state
Philipp Hortmann [Sun, 19 Nov 2023 22:15:21 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused variable dot11d_info->state

Remove unused variable dot11d_info->state. Remove unused enum
dot11d_state as well.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/2f1d4851a7a370c5ef367568c2b098bb9fbf0ce7.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function copy_mac_addr()
Philipp Hortmann [Sun, 19 Nov 2023 22:15:15 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused function copy_mac_addr()

Remove unused function copy_mac_addr().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/d157456121388489784ce34e5f3daf91126f80d7.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused macros IS_EQUAL_CIE_SRC and friends
Philipp Hortmann [Sun, 19 Nov 2023 22:15:10 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused macros IS_EQUAL_CIE_SRC and friends

Remove unused macros IS_EQUAL_CIE_SRC, UPDATE_CIE_WATCHDOG,
UPDATE_CIE_SRC and IS_COUNTRY_IE_VALID.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/6d514e946983fb8ef4a3d7d3cb2774261a593d2f.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function dot11d_reset()
Philipp Hortmann [Sun, 19 Nov 2023 22:15:02 +0000 (23:15 +0100)]
staging: rtl8192e: Remove unused function dot11d_reset()

Remove unused function dot11d_reset().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/09306a7f674e0cf1244c47ef8ddfc5ae147124f8.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function dot11d_scan_complete()
Philipp Hortmann [Sun, 19 Nov 2023 22:14:56 +0000 (23:14 +0100)]
staging: rtl8192e: Remove unused function dot11d_scan_complete()

Remove unused function dot11d_scan_complete().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/065a8223b80730360ee3223067ee612737e777a7.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove IS_DOT11D_ENABLE(ieee)
Philipp Hortmann [Sun, 19 Nov 2023 22:14:51 +0000 (23:14 +0100)]
staging: rtl8192e: Remove IS_DOT11D_ENABLE(ieee)

Variable "enabled" is initialized to false and never changed. Therefore
IS_DOT11D_ENABLE(ieee) returns always false. Remove dead code.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/9e737b2246e0b73c3ad84a4ee09f5e94bf778b3c.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused function dot11d_update_country()
Philipp Hortmann [Sun, 19 Nov 2023 22:14:42 +0000 (23:14 +0100)]
staging: rtl8192e: Remove unused function dot11d_update_country()

Remove unused function dot11d_update_country().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/364462eae2271f3925b362a439ec27d50dd899ef.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unexecuted rtllib_extract_country_ie()
Philipp Hortmann [Sun, 19 Nov 2023 22:14:36 +0000 (23:14 +0100)]
staging: rtl8192e: Remove unexecuted rtllib_extract_country_ie()

Variable "enabled" is initialized to false and never changed. Therefore
IS_DOT11D_ENABLE(ieee) returns always false. Because of this all code in
rtllib_extract_country_ie() is never executed.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/7b88c62d3cc5c3fb670b5448c7381c7c099b3518.1700431464.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove constant index from channel_array[]
Philipp Hortmann [Sat, 18 Nov 2023 08:52:08 +0000 (09:52 +0100)]
staging: rtl8192e: Remove constant index from channel_array[]

Used index of channel_array[] is always COUNTRY_CODE_WORLD_WIDE_13. Remove
index and store only used entry in channel_array. This shortens the code
and increases readability.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/379293f2b48d176aaafb023d36aac9bb057f67c7.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove switch for a constant in dot11d_channel_map()
Philipp Hortmann [Sat, 18 Nov 2023 08:52:01 +0000 (09:52 +0100)]
staging: rtl8192e: Remove switch for a constant in dot11d_channel_map()

Remove switch for a constant in dot11d_channel_map() as the result will
always be the same.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/d81caa0894bdaaca471010cb3512f3459a2e7df4.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove check if channel_array[channel_plan].len != 0
Philipp Hortmann [Sat, 18 Nov 2023 08:51:54 +0000 (09:51 +0100)]
staging: rtl8192e: Remove check if channel_array[channel_plan].len != 0

channel_plan is constant COUNTRY_CODE_WORLD_WIDE_13. Remove equation to
check length of array as it is always not 0.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/0cfcc32a06033908c469ef0273de2485dbfd6b82.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove equation to check limits of channel
Philipp Hortmann [Sat, 18 Nov 2023 08:51:48 +0000 (09:51 +0100)]
staging: rtl8192e: Remove equation to check limits of channel

channel_plan is constant COUNTRY_CODE_WORLD_WIDE_13. Remove equation to
check limits of channel as those are always in limit.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/1f91cf8145b304b09b37734a2a504da394833378.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused variable eeprom_chnl_plan
Philipp Hortmann [Sat, 18 Nov 2023 08:51:42 +0000 (09:51 +0100)]
staging: rtl8192e: Remove unused variable eeprom_chnl_plan

Remove unused variable eeprom_chnl_plan.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/ffc592edd61332d2d8f9b0da68fc956d76e68fe0.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove constant variable chnl_plan
Philipp Hortmann [Sat, 18 Nov 2023 08:51:34 +0000 (09:51 +0100)]
staging: rtl8192e: Remove constant variable chnl_plan

Remove constant variable chnl_plan and replace it with its constant.
Remove equation that limits maximum value of chnl_plan as it is always
false.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/e46f16227877fe9853c9d15992e7fd27a0ceeae6.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove equation that results in constant for chnl_plan
Philipp Hortmann [Sat, 18 Nov 2023 08:51:25 +0000 (09:51 +0100)]
staging: rtl8192e: Remove equation that results in constant for chnl_plan

Remove equation for chnl_plan as the last line sets chnl_plan to a
constant.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/d6fbe3d2297d1d5502b183785f14df51a81e3331.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Unwind pointer to pointer to rtl92e_set_channel()
Philipp Hortmann [Sat, 18 Nov 2023 08:51:18 +0000 (09:51 +0100)]
staging: rtl8192e: Unwind pointer to pointer to rtl92e_set_channel()

Replace pointer to function to pointer to rtl92e_set_channel() with
pointer to rtl92e_set_channel(). This increases readability of the code.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/a96bac1f34e669a3b2fc4940a0a2d850564a5975.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Change parameter "ch" of set_chan() to u8
Philipp Hortmann [Sat, 18 Nov 2023 08:51:11 +0000 (09:51 +0100)]
staging: rtl8192e: Change parameter "ch" of set_chan() to u8

Change parameter "ch" of set_chan() to u8 to combine functions in the
following patch.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/4a76c0e2384d67410d383fdf860d0e0859555d1e.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agostaging: rtl8192e: Remove unused return value of rtl92e_set_channel()
Philipp Hortmann [Sat, 18 Nov 2023 08:51:05 +0000 (09:51 +0100)]
staging: rtl8192e: Remove unused return value of rtl92e_set_channel()

Remove unused return value of rtl92e_set_channel().

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/0c0e7c72a10731ae7ed49c8161136b0f0b63d7a0.1700296319.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bDisable_AddBa
Tree Davies [Mon, 13 Nov 2023 19:59:10 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bDisable_AddBa

Rename variable bDisable_AddBa to disable_add_ba to fix
checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-8-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bOverwritePending
Tree Davies [Mon, 13 Nov 2023 19:59:09 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bOverwritePending

Rename variable bOverwritePending to overwrite_pending to fix
checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-7-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bUsingBa
Tree Davies [Mon, 13 Nov 2023 19:59:08 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bUsingBa

Rename variable bUsingBa to using_ba to fix
checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-6-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bAddBaReqDelayed
Tree Davies [Mon, 13 Nov 2023 19:59:07 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bAddBaReqDelayed

Rename variable bAddBaReqDelayed to add_ba_req_delayed to fix
checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-5-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bAddBaReqInProgress
Tree Davies [Mon, 13 Nov 2023 19:59:06 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bAddBaReqInProgress

Rename variable bAddBaReqInProgress to add_ba_req_in_progress to fix
checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-4-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bCurrentAMPDUEnable
Tree Davies [Mon, 13 Nov 2023 19:59:05 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bCurrentAMPDUEnable

Rename variable bCurrentAMPDUEnable to current_ampdu_enable to fix
checkpatch warning Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-3-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoStaging: rtl8192e: Rename variable bSendDELBA
Tree Davies [Mon, 13 Nov 2023 19:59:04 +0000 (11:59 -0800)]
Staging: rtl8192e: Rename variable bSendDELBA

Rename variable bSendDELBA to send_del_ba to fix checkpatch warning
Avoid CamelCase.

Signed-off-by: Tree Davies <tdavies@darkphysics.net>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20231113195910.8423-2-tdavies@darkphysics.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
16 months agoMAINTAINERS: Mark VME subsystem as orphan
Bagas Sanjaya [Thu, 23 Nov 2023 08:34:03 +0000 (15:34 +0700)]
MAINTAINERS: Mark VME subsystem as orphan

Martyn Welch lost his access to VME hardware [1]; and Manohar Vanga has been
MIA since early January 2014 (his last message was [2]). Martyn admitted
that the subsystem is basically orphan, so mark it as such. As a bonus,
add CREDITS entries for the former subsystem maintainers.

Link: https://lore.kernel.org/r/fe8ac0db-d6cc-41bc-b926-484b418e1720@collabora.com/
Link: https://lore.kernel.org/r/CAEktxaFL=3cmU4vZS2akiAR2vG-3d+9HwTZvBvf5JXuThHoOKg@mail.gmail.com/
Cc: Martyn Welch <martyn.welch@collabora.com>
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
Acked-by: Martyn Welch <martyn@welchs.me.uk>
Link: https://lore.kernel.org/r/20231123083406.12129-2-bagasdotme@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>