- 05 Oct, 2023 2 commits
-
-
Philipp Hortmann authored
Remove ibss_maxjoin_chal as it is just set and never evaluated. Signed-off-by:
Philipp Hortmann <philipp.g.hortmann@gmail.com> Link: https://lore.kernel.org/r/7c5d512a69e5832263a0a2e8f4ea3608fe286a09.1696165351.git.philipp.g.hortmann@gmail.comSigned-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Chengfeng Ye authored
As &priv->tx_dev.tx_dev_lock is also acquired by xmit callback which could be call from timer under softirq context, use spin_lock_bh() on it to prevent potential deadlock. hostif_sme_work() --> hostif_sme_set_pmksa() --> hostif_mib_set_request() --> ks_wlan_hw_tx() --> spin_lock(&priv->tx_dev.tx_dev_lock) ks_wlan_start_xmit() --> hostif_data_request() --> ks_wlan_hw_tx() --> spin_lock(&priv->tx_dev.tx_dev_lock) Signed-off-by:
Chengfeng Ye <dg573847474@gmail.com> Link: https://lore.kernel.org/r/20230926161323.41928-1-dg573847474@gmail.comSigned-off-by:
-
- 26 Sep, 2023 27 commits
-
-
Pavan Bobba authored
variable name "byRFType" 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:
-
Arnd Bergmann authored
The rtllib_rxb structure contains a pointer, so this is not a hardware structure but could benefit from loading the pointer in a single instruction rather than assembling it from four or eight individual bytes. Both structures are allocated as part of larger structure that already enforce at least a 4-byte alignment, so there is no reason to ever have to deal with misaligned definitions. Signed-off-by:
Arnd Bergmann <arnd@arndb.de> Link: https://lore.kernel.org/r/20230925155413.471287-2-arnd@kernel.orgSigned-off-by:
-
Philipp Hortmann authored
Resolve circular locking from reorder_spinlock with rx_pkt_pending_timer. Signed-off-by:
-
Philipp Hortmann authored
The following command uses the mutex ieee->wx_mutex: cancel_delayed_work_sync(&ieee->associate_retry_wq) Therefore it cannot be canceled with ieee->wx_mutex locked. rtllib_stop_protocol() is always called with ieee->wx_mutex locked so according line can be unlocked. Signed-off-by:
-
Philipp Hortmann authored
The following commands use the mutex ieee->wx_mutex: cancel_delayed_work_sync(&ieee->associate_retry_wq) cancel_delayed_work_sync(&ieee->associate_procedure_wq) cancel_work_sync(&ieee->wx_sync_scan_wq) Therefore they cannot be canceled with ieee->wx_mutex locked. As the work functions are executed time shifted during normal operation of the driver, proper locking can only be done inside the functions. Signed-off-by:
-
Philipp Hortmann authored
To reverse order of initialization put kfree() to the end of rtllib_softmac_free(). Signed-off-by:
-
Philipp Hortmann authored
rtllib_start_send_beacons() is removed so no need for rtllib_stop_send_beacons() anymore. Signed-off-by:
-
Philipp Hortmann authored
Remove delayed_work start_ibss_wq and the related functions which are not needed anymore. Signed-off-by:
-
Philipp Hortmann authored
Remove unsupported mode IW_MODE_ADHOC from rtllib_softmac.c. Signed-off-by:
-
Philipp Hortmann authored
Remove unsupported mode IW_MODE_ADHOC from rtl_cam.c. Signed-off-by:
-
Philipp Hortmann authored
Remove unsupported mode IW_MODE_ADHOC from rtl_core.c and further files. Signed-off-by:
-
Philipp Hortmann authored
Remove unsupported mode IW_MODE_ADHOC from rtllib_rx.c and all related functions and structs. Signed-off-by:
-
Philipp Hortmann authored
Remove unsupported mode IW_MODE_ADHOC from rtllib_tx.c, rtl819x_TSProc.c, rtl819x_HTProc.c, r8192E_dev.c Signed-off-by:
-
Tree Davies authored
Rename variable StatusCode to status_code to fix checkpatch warning Avoid CamelCase. Signed-off-by:
Tree Davies <tdavies@darkphysics.net> Tested-by:
-
Tree Davies authored
Rename function TSInitialize to rtllib_ts_init to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename function GetTs to rtllib_get_ts to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable Policy to policy to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable pStatusCode to status_code to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable Addr to addr to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable pPendingBA to pending_ba to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable Dst to dst to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable pPendingBa to pending_ba to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable pAdmittedBa to admitted_ba to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Tree Davies authored
Rename variable Time to time to fix checkpatch warning Avoid CamelCase. Signed-off-by:
-
Philipp Hortmann authored
iwe.u.name and proto_name have both 16 bytes of space. But when writing to iwe.u.name the first 10 bytes are used by the fixed string "IEEE802.11". Evaluating the code shows that only the following strings can occur in proto_name: "N-24G", "g", "b", "bg". Therefore it is sufficient to shorten proto_name to 6 bytes. Reported-by:
kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202309171733.Gl96cmYd-lkp@intel.com/Signed-off-by:
-
Philipp Hortmann authored
Fix alignment of structs to even addresses to support all architectures. Reported-by:
-
Bragatheswaran Manickavel authored
They are appear to be spelling mistakes, Initially identified in a codespell report and never been addressed so far. ./rtl8723b_phycfg.c:156: Threre ==> There, three ./rtl8723b_phycfg.c:283: Condig ==> Config ./rtl8723b_phycfg.c:328: Tranceiver ==> Transceiver Signed-off-by:
Bragatheswaran Manickavel <bragathemanick0908@gmail.com> Link: https://lore.kernel.org/r/20230917134940.2746-1-bragathemanick0908@gmail.comSigned-off-by:
-
- 17 Sep, 2023 11 commits
-
-
Jonathan Bergh authored
Fixed various issues relating to comments including: * Lines with '*' in block comments which should be aligned and were not * Corrected comments where closing multiline comment identifier ran over into second newline spuriously * Corrected comment content to correctly reflect hexadecimal for the offsets rather than integers Signed-off-by:Jonathan Bergh <bergh.jonathan@gmail.com> Link: https://lore.kernel.org/r/20230914200732.47659-1-bergh.jonathan@gmail.comSigned-off-by:
-
Jonathan Bergh authored
Removed spurious newlines in define statements so that defines which should be grouped together for readability are grouped as such. Signed-off-by:
-
Pavan Bobba authored
Array names starting with "byVT3253B0_" updated like below: a.type encoding info dropped from names b.camelcase names replaced by snakecase Issue found by checkpatch Signed-off-by:
-
Jonathan Bergh authored
Removed unused/commented code from vme_fake.c which was causing spurious checkpatch warnings. Signed-off-by:
-
Oliver Crumrine authored
Remove typedef in structs cvmx_pip_port_status_t and cvmx_pko_port_status_t, and rename all instances to cvmx_pip_port_status, and cvmx_pko_port_status, respectively Signed-off-by:
Oliver Crumrine <ozlinux@hotmail.com> Link: https://lore.kernel.org/r/PH7PR11MB76434462A6563C5FBDEE51A8BCE0A@PH7PR11MB7643.namprd11.prod.outlook.comSigned-off-by:
-
Oliver Crumrine authored
Remove typedef in enum cmvx_pko_status_t, and rename all instances to cvmx_pko_status Signed-off-by:
-
Oliver Crumrine authored
Remove typedef in struct cvmx_pko_lock_t, and rename all instances to cvmx_pko_lock Signed-off-by:
-
Oliver Crumrine authored
Remove typedef in enum cvmx_pow_wait_t, and rename all instances to cvmx_pow_wait Signed-off-by:
-
Oliver Crumrine authored
Remove typedef in enum cvmx_helper_interface_mode_t, and rename all instances to cvmx_helper_interface_mode Signed-off-by:
-
Oliver Crumrine authored
Remove typedef in enum cvmx_spi_mode_t, and rename all instances to cvmx_spi_mode Signed-off-by:
-
Alexon Oliveira authored
Fixed all CHECK: No space is necessary after a cast as reported by checkpatch to adhere to the Linux kernel coding-style guidelines. Signed-off-by:
Alexon Oliveira <alexondunkan@gmail.com> Link: https://lore.kernel.org/r/603976b29ad4d9a0e4cbd8452ff674ec70a227a8.1693164540.git.alexondunkan@gmail.comSigned-off-by:
-
