linux.git/drivers/staging/rtl8723bs/hal, branch v6.6

staging: rtl8723bs: Use helpers to check broadcast and multicast Ether addresses

2023-08-22T13:47:17+00:00

Use is_multicast_ether_addr() and is_broadcast_ether_addr()
instead of custom macro IS_MCAST() and MacAddr_isBcst(), the buffer
is properly aligned.

Signed-off-by: Ruan Jinjie 
Link: https://lore.kernel.org/r/20230814105623.292541-1-ruanjinjie@huawei.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: replace ternary operator with if-else block

2023-05-28T09:11:29+00:00

replace the ternary operator used for assignment of pHalData->ant_path
with an if-else block for better readability

Signed-off-by: Atin Bainada 
Link: https://lore.kernel.org/r/20230513214338.79833-1-hi@atinb.me
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: remove hal_btcoex_SetChipType()

2023-03-25T08:53:01+00:00

Remove the function hal_btcoex_SetChipType() as it does nothing and
produces the following gcc warning when compiling with W=1.

drivers/staging/rtl8723bs/hal/hal_btcoex.c:1182:30: warning: variable 'pHalData' set but not used [-Wunused-but-set-variable]
 1182 |         struct hal_com_data *pHalData;
      |                              ^~~~~~~~

Signed-off-by: Michael Straube 
Reviewed-by: Hans de Goede 
Link: https://lore.kernel.org/r/20230324084334.12183-1-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: rtw_get_raw_rssi_info() is not used

2023-03-16T08:40:09+00:00

The function rtw_get_raw_rssi_info() is not used anywhere, remove it.

Signed-off-by: Michael Straube 
Reviewed-by: Hans de Goede 
Link: https://lore.kernel.org/r/20230310083449.23775-6-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: linked_info_dump() is not used

2023-03-16T08:40:09+00:00

The function linked_info_dump() is not used anywhere, remove it.

Signed-off-by: Michael Straube 
Reviewed-by: Hans de Goede 
Link: https://lore.kernel.org/r/20230310083449.23775-5-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: isAllSpaceOrTab() is not used

2023-03-16T08:40:09+00:00

The function isAllSpaceOrTab() is not used anywhere, remove it.

Signed-off-by: Michael Straube 
Reviewed-by: Hans de Goede 
Link: https://lore.kernel.org/r/20230310083449.23775-4-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: ParseQualifiedString() is not used

2023-03-16T08:40:09+00:00

The function ParseQualifiedString() is not used anywhere, remove it.

Signed-off-by: Michael Straube 
Reviewed-by: Hans de Goede 
Link: https://lore.kernel.org/r/20230310083449.23775-3-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: MapCharToHexDigit() is not used

2023-03-16T08:40:09+00:00

The function MapCharToHexDigit() is not used anywhere, remove it.

Signed-off-by: Michael Straube 
Reviewed-by: Hans de Goede 
Link: https://lore.kernel.org/r/20230310083449.23775-2-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: hal: Fix codespell-reported spelling mistakes

2023-01-17T18:27:14+00:00

They are appear to be spelling mistakes,
Initially identified in a codespell report and never been addressed so far.

drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:656: regsiters ==> registers
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:1696: beacause ==> because
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:2092: Checl ==> Check
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:2513: checksume ==> checksum
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:2726: sequense ==> sequence
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:2780: vlaue ==> value
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:3409: equall ==> equal, equally

Signed-off-by: Anup Sharma 
Link: https://lore.kernel.org/r/Y61y+flJp9/jEicc@local
Signed-off-by: Greg Kroah-Hartman

staging: rtl8723bs: replace underutilized struct by array variable

2022-11-08T15:26:49+00:00

For structure iqk_matrix_regs_setting, only the "Value" member variable
is utilized whereas the other struct members are only declared but not
utilised. Replace the struct declaration and implementation by an
equivalent variable similar to the only used struct member variable.
While in there, update the macro mixed case names to uppercase style.
The resultant code is simpler and is easy to maintain.

Suggested-by: Dan Carpenter 
Signed-off-by: Deepak R Varma 
Link: https://lore.kernel.org/r/Y2ZF6O1KU3zZ6r3C@qemulion
Signed-off-by: Greg Kroah-Hartman