- 26 Nov, 2023 1 commit
-
-
Gary Rookard authored
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.orgSigned-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
- 23 Nov, 2023 39 commits
-
-
Stefan Wahren authored
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.netSigned-off-by:
-
Stefan Wahren authored
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:
Laurent Pinchart <laurent.pinchart@ideasonboard.com> Link: https://lore.kernel.org/r/20231029124837.119832-3-wahrenst@gmx.netSigned-off-by:
-
Stefan Wahren authored
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:
-
Bagas Sanjaya authored
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.comSigned-off-by:
-
Dan Carpenter authored
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.mountainSigned-off-by:
-
Pavan Bobba authored
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:
-
Pavan Bobba authored
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 authored
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 authored
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 authored
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 authored
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 authored
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 authored
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 authored
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 authored
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 authored
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:
-
Philipp Hortmann authored
Remove unused variable bss_start_channel. Signed-off-by:
-
Philipp Hortmann authored
Remove unused struct chnl_txpow_triple. Signed-off-by:
-
Philipp Hortmann authored
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 authored
global_domain is initialized to false and then never used. Signed-off-by:
-
Philipp Hortmann authored
country_watchdog is initialized to 0 and then never changed. Remove related macros and functions. Signed-off-by:
-
Philipp Hortmann authored
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 authored
Remove unused variable dot11d_info->state. Remove unused enum dot11d_state as well. Signed-off-by:
-
Philipp Hortmann authored
Remove unused function copy_mac_addr(). Signed-off-by:
-
Philipp Hortmann authored
Remove unused macros IS_EQUAL_CIE_SRC, UPDATE_CIE_WATCHDOG, UPDATE_CIE_SRC and IS_COUNTRY_IE_VALID. Signed-off-by:
-
Philipp Hortmann authored
Remove unused function dot11d_reset(). Signed-off-by:
-
Philipp Hortmann authored
Remove unused function dot11d_scan_complete(). Signed-off-by:
-
Philipp Hortmann authored
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 authored
Remove unused function dot11d_update_country(). Signed-off-by:
-
Philipp Hortmann authored
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 authored
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 authored
Remove switch for a constant in dot11d_channel_map() as the result will always be the same. Signed-off-by:
-
Philipp Hortmann authored
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 authored
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 authored
Remove unused variable eeprom_chnl_plan. Signed-off-by:
-
Philipp Hortmann authored
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 authored
Remove equation for chnl_plan as the last line sets chnl_plan to a constant. Signed-off-by:
-
Philipp Hortmann authored
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 authored
Change parameter "ch" of set_chan() to u8 to combine functions in the following patch. Signed-off-by:
-
