staging: rtl8723bs: remove hal_btcoex_SetChipType()

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

drivers/staging/r

staging: rtl8723bs: remove hal_btcoex_SetChipType()

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 <straube.linux@gmail.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20230324084334.12183-1-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove hal_btcoex_SetManualControl

Remove function hal_btcoex_SetManualControl because it is not used.

Signed-off-by: Nam Cao <namcaov@gmail.com>
Link: https://lore.kernel.org/r

staging: rtl8723bs: remove hal_btcoex_SetManualControl

Remove function hal_btcoex_SetManualControl because it is not used.

Signed-off-by: Nam Cao <namcaov@gmail.com>
Link: https://lore.kernel.org/r/ae83ad941013d7a6c47d06f70c0f43087fe2c84f.1662111799.git.namcaov@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove leftover code for other chips

Clean up unreferenced definitions in hal headers and code.

Signed-off-by: Artur Bujdoso <artur.bujdoso@gmail.com>
Link: https://lore.kernel.

staging: rtl8723bs: remove leftover code for other chips

Clean up unreferenced definitions in hal headers and code.

Signed-off-by: Artur Bujdoso <artur.bujdoso@gmail.com>
Link: https://lore.kernel.org/r/YmD/I19v8sJLrOQo@crux
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove unused macros

remove unused macros detected by -Wunused-macros
gcc option.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/20210801134

staging: rtl8723bs: remove unused macros

remove unused macros detected by -Wunused-macros
gcc option.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/20210801134629.23433-1-fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove BT debug code

remove BT dead debug code, probably it was intended
for a specific command line interface.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https:/

staging: rtl8723bs: remove BT debug code

remove BT dead debug code, probably it was intended
for a specific command line interface.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/20210728141858.13024-1-fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: fix camel case in struct wlan_bssid_ex

fix camel case in struct wlan_bssid_ex.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/561065e95ff38f

staging: rtl8723bs: fix camel case in struct wlan_bssid_ex

fix camel case in struct wlan_bssid_ex.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/561065e95ff38f0dbedf030c3acf0498396a1759.1626874164.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: fix camel case name in macro IsLegacyOnly()

fix camel case name in macro IsLegacyOnly()

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/52656

staging: rtl8723bs: fix camel case name in macro IsLegacyOnly()

fix camel case name in macro IsLegacyOnly()

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/52656190cefed1b0f9a2dd6dcb080adf2ca0a89b.1626533647.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove all branchings between 2.4Ghz and 5Ghz band types

remove all code branchings tied to dual band support. The device
works only on 2.4Ghz band so there's no need to check wh

staging: rtl8723bs: remove all branchings between 2.4Ghz and 5Ghz band types

remove all code branchings tied to dual band support. The device
works only on 2.4Ghz band so there's no need to check which
band we are on. Removed all code branches that would be valid
only for 5Ghz for it's dead code. Removed enums, table fields
indexing by band type, function arguments passing current
band type and all other thing related to telling us
which band we are on.

Acked-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/72bb27f6a7b3be607f93f5b406d863dd08376986.1624367071.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove obsolete GLBtcDbgType array declarations

remove obsolete GLBtcDbgType declarations for it was intended
for private debug facilities.

Reviewed-by: Dan Carpenter <dan.carpe

staging: rtl8723bs: remove obsolete GLBtcDbgType array declarations

remove obsolete GLBtcDbgType declarations for it was intended
for private debug facilities.

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/000825941048c41cc53a0cc675fae2b45b781aad.1619794331.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove unnecessary halbtcoutsrc_DbgInit() function

halbtcoutsrc_DbgInit() static function was intended for
manual activation of private tracing facilities.

So remove it.

Review

staging: rtl8723bs: remove unnecessary halbtcoutsrc_DbgInit() function

halbtcoutsrc_DbgInit() static function was intended for
manual activation of private tracing facilities.

So remove it.

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/f6c4f680b2d0ee8850f97532ca76dd405c1d50c3.1619794331.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove hal_btcoex_GetDBG() function

remove unused hal_btcoex_GetDBG() function definition and
prototype.

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fab

staging: rtl8723bs: remove hal_btcoex_GetDBG() function

remove unused hal_btcoex_GetDBG() function definition and
prototype.

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/8b06dc12a65ef15f3e67bd32d097dc4db3f78586.1619794331.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove unsed hal_btcoex_SetDBG() function

remove unused hal_btcoex_SetDBG() function definition and
prototype.

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-b

staging: rtl8723bs: remove unsed hal_btcoex_SetDBG() function

remove unused hal_btcoex_SetDBG() function definition and
prototype.

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/4f8aa4e4312777aff1aea621d26c00e81754025a.1619794331.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove BTC_PRINT logs

Remove BTC_PRINT logs.

BTC_PRINT macro's default behavoiur is _do nothing_.
It's enabled by uncommenting some code in hal/hal_btcoex.c
file.

So just remov

staging: rtl8723bs: remove BTC_PRINT logs

Remove BTC_PRINT logs.

BTC_PRINT macro's default behavoiur is _do nothing_.
It's enabled by uncommenting some code in hal/hal_btcoex.c
file.

So just remove it.

Applied following semantic patch:

@@
@@

- BTC_PRINT(...);

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/9a9a8ce817929f1705b623c470cbafcc75fb3e72.1619794331.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove commented out DBG_871X logs

remove all commented out DBG_871X logs unmatched by
semantic patch.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.ker

staging: rtl8723bs: remove commented out DBG_871X logs

remove all commented out DBG_871X logs unmatched by
semantic patch.

Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/125e216e3bb5bc938e06b15dadfbbf51d9517dde.1617802415.git.fabioaiuto83@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: hal: Add spaces around operators in hal_btcoex.c

Added spaces around operators. Issue detected by checkpatch.pl.
Spaces around operators are preferred to improve readibility.

Si

staging: rtl8723bs: hal: Add spaces around operators in hal_btcoex.c

Added spaces around operators. Issue detected by checkpatch.pl.
Spaces around operators are preferred to improve readibility.

Signed-off-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
Link: https://lore.kernel.org/r/20210406181428.15097-1-fmdefrancesco@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

Staging: rtl8723bs: fix names in hal_btcoex.c

This commit converts names of structs / enums
in hal/hal_btcoex.c from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>

Staging: rtl8723bs: fix names in hal_btcoex.c

This commit converts names of structs / enums
in hal/hal_btcoex.c from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>
Link: https://lore.kernel.org/r/20210317222130.29528-38-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

Staging: rtl8723bs: fix names in HalBtcOutSrc.h

This commit converts names of structs / enums
in hal/HalBtcOutSrc.h from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.

Staging: rtl8723bs: fix names in HalBtcOutSrc.h

This commit converts names of structs / enums
in hal/HalBtcOutSrc.h from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>
Link: https://lore.kernel.org/r/20210317222130.29528-32-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

Staging: rtl8723bs: fix names in hal_com.h

This commit converts names of structs / enums
in include/hal_com.h from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>
L

Staging: rtl8723bs: fix names in hal_com.h

This commit converts names of structs / enums
in include/hal_com.h from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>
Link: https://lore.kernel.org/r/20210317222130.29528-5-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

Staging: rtl8723bs: fix names in rtw_mlme.h

This commit converts names of structs / enums
in include/rtw_mlme.h from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>

Staging: rtl8723bs: fix names in rtw_mlme.h

This commit converts names of structs / enums
in include/rtw_mlme.h from ALL_CAPS format to lowercase

Signed-off-by: Marco Cesati <marcocesati@gmail.com>
Link: https://lore.kernel.org/r/20210317222130.29528-2-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

Staging: rtl8723bs: fix spaces in hal_btcoex.c

This commit fixes the following checkpatch.pl errors:

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#70: FILE: ./hal/hal_btcoex.c:7

Staging: rtl8723bs: fix spaces in hal_btcoex.c

This commit fixes the following checkpatch.pl errors:

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#70: FILE: ./hal/hal_btcoex.c:70:
+static u8 halbtcoutsrc_IsBtCoexistAvailable(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#104: FILE: ./hal/hal_btcoex.c:104:
+static void halbtcoutsrc_LeaveLps(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#117: FILE: ./hal/hal_btcoex.c:117:
+static void halbtcoutsrc_EnterLps(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#130: FILE: ./hal/hal_btcoex.c:130:
+static void halbtcoutsrc_NormalLps(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#152: FILE: ./hal/hal_btcoex.c:152:
+static void halbtcoutsrc_LeaveLowPower(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#187: FILE: ./hal/hal_btcoex.c:187:
+static void halbtcoutsrc_NormalLowPower(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#196: FILE: ./hal/hal_btcoex.c:196:
+static void halbtcoutsrc_DisableLowPower(struct BTC_COEXIST * pBtCoexist, u8 bLowPwrDisable)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#205: FILE: ./hal/hal_btcoex.c:205:
+static void halbtcoutsrc_AggregationCheck(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#283: FILE: ./hal/hal_btcoex.c:283:
+static u32 halbtcoutsrc_GetWifiLinkStatus(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#311: FILE: ./hal/hal_btcoex.c:311:
+static u32 halbtcoutsrc_GetBtPatchVer(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#342: FILE: ./hal/hal_btcoex.c:342:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#448: FILE: ./hal/hal_btcoex.c:448:
+ struct RT_LINK_DETECT_T * plinkinfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#510: FILE: ./hal/hal_btcoex.c:510:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#645: FILE: ./hal/hal_btcoex.c:645:
+static void halbtcoutsrc_DisplayFwPwrModeCmd(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#661: FILE: ./hal/hal_btcoex.c:661:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#673: FILE: ./hal/hal_btcoex.c:673:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#685: FILE: ./hal/hal_btcoex.c:685:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#697: FILE: ./hal/hal_btcoex.c:697:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#709: FILE: ./hal/hal_btcoex.c:709:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#749: FILE: ./hal/hal_btcoex.c:749:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#761: FILE: ./hal/hal_btcoex.c:761:
+ struct BTC_COEXIST * pBtCoexist = (struct BTC_COEXIST *)pBtcContext;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#772: FILE: ./hal/hal_btcoex.c:772:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#785: FILE: ./hal/hal_btcoex.c:785:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#797: FILE: ./hal/hal_btcoex.c:797:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#809: FILE: ./hal/hal_btcoex.c:809:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#821: FILE: ./hal/hal_btcoex.c:821:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#857: FILE: ./hal/hal_btcoex.c:857:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#869: FILE: ./hal/hal_btcoex.c:869:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#891: FILE: ./hal/hal_btcoex.c:891:
+ struct BTC_COEXIST * pBtCoexist = &GLBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#917: FILE: ./hal/hal_btcoex.c:917:
+ struct BTC_COEXIST * pBtCoexist;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#963: FILE: ./hal/hal_btcoex.c:963:
+void EXhalbtcoutsrc_PowerOnSetting(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#975: FILE: ./hal/hal_btcoex.c:975:
+void EXhalbtcoutsrc_InitHwConfig(struct BTC_COEXIST * pBtCoexist, u8 bWifiOnly)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#988: FILE: ./hal/hal_btcoex.c:988:
+void EXhalbtcoutsrc_InitCoexDm(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1003: FILE: ./hal/hal_btcoex.c:1003:
+void EXhalbtcoutsrc_IpsNotify(struct BTC_COEXIST * pBtCoexist, u8 type)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1030: FILE: ./hal/hal_btcoex.c:1030:
+void EXhalbtcoutsrc_LpsNotify(struct BTC_COEXIST * pBtCoexist, u8 type)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1053: FILE: ./hal/hal_btcoex.c:1053:
+void EXhalbtcoutsrc_ScanNotify(struct BTC_COEXIST * pBtCoexist, u8 type)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1082: FILE: ./hal/hal_btcoex.c:1082:
+void EXhalbtcoutsrc_ConnectNotify(struct BTC_COEXIST * pBtCoexist, u8 action)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1108: FILE: ./hal/hal_btcoex.c:1108:
+void EXhalbtcoutsrc_MediaStatusNotify(struct BTC_COEXIST * pBtCoexist, enum RT_MEDIA_STATUS mediaStatus)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1135: FILE: ./hal/hal_btcoex.c:1135:
+void EXhalbtcoutsrc_SpecialPacketNotify(struct BTC_COEXIST * pBtCoexist, u8 pktType)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1166: FILE: ./hal/hal_btcoex.c:1166:
+void EXhalbtcoutsrc_BtInfoNotify(struct BTC_COEXIST * pBtCoexist, u8 *tmpBuf, u8 length)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1184: FILE: ./hal/hal_btcoex.c:1184:
+void EXhalbtcoutsrc_HaltNotify(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1197: FILE: ./hal/hal_btcoex.c:1197:
+void EXhalbtcoutsrc_PnpNotify(struct BTC_COEXIST * pBtCoexist, u8 pnpState)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1213: FILE: ./hal/hal_btcoex.c:1213:
+void EXhalbtcoutsrc_Periodical(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1258: FILE: ./hal/hal_btcoex.c:1258:
+void EXhalbtcoutsrc_DisplayBtCoexInfo(struct BTC_COEXIST * pBtCoexist)

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Marco Cesati <marcocesati@gmail.com>
Link: https://lore.kernel.org/r/20210315170618.2566-18-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove typedefs in hal_btcoex.c

This commit fixes the following checkpatch.pl warning:

WARNING: do not add new typedefs
#23: FILE: hal/hal_btcoex.c:23:
+typedef stru

staging: rtl8723bs: remove typedefs in hal_btcoex.c

This commit fixes the following checkpatch.pl warning:

WARNING: do not add new typedefs
#23: FILE: hal/hal_btcoex.c:23:
+typedef struct _btcoexdbginfo {

Signed-off-by: Marco Cesati <marco.cesati@gmail.com>
Link: https://lore.kernel.org/r/20210312082638.25512-33-marco.cesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove typedefs in hal_com.h

This commit fixes the following checkpatch.pl warning:

WARNING: do not add new typedefs
#168: FILE: include/hal_com.h:168:
+typedef enum

staging: rtl8723bs: remove typedefs in hal_com.h

This commit fixes the following checkpatch.pl warning:

WARNING: do not add new typedefs
#168: FILE: include/hal_com.h:168:
+typedef enum _RT_MEDIA_STATUS {

Signed-off-by: Marco Cesati <marco.cesati@gmail.com>
Link: https://lore.kernel.org/r/20210312082638.25512-12-marco.cesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove typedefs in rtw_mlme.h

This commit fixes the following checkpatch.pl warnings:

WARNING: do not add new typedefs
#84: FILE: include/rtw_mlme.h:84:
+typedef enu

staging: rtl8723bs: remove typedefs in rtw_mlme.h

This commit fixes the following checkpatch.pl warnings:

WARNING: do not add new typedefs
#84: FILE: include/rtw_mlme.h:84:
+typedef enum _RT_SCAN_TYPE {

WARNING: do not add new typedefs
#137: FILE: include/rtw_mlme.h:137:
+typedef struct _RT_LINK_DETECT_T {

Signed-off-by: Marco Cesati <marco.cesati@gmail.com>
Link: https://lore.kernel.org/r/20210312082638.25512-3-marco.cesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove typedefs in HalBtcOutSrc.h

This commit fixes the following checkpatch.pl warnings:

WARNING: do not add new typedefs
#47: FILE: hal/HalBtcOutSrc.h:47:
+typedef

staging: rtl8723bs: remove typedefs in HalBtcOutSrc.h

This commit fixes the following checkpatch.pl warnings:

WARNING: do not add new typedefs
#47: FILE: hal/HalBtcOutSrc.h:47:
+typedef enum _BTC_POWERSAVE_TYPE {

WARNING: do not add new typedefs
#54: FILE: hal/HalBtcOutSrc.h:54:
+typedef enum _BTC_BT_REG_TYPE {

WARNING: do not add new typedefs
#63: FILE: hal/HalBtcOutSrc.h:63:
+typedef enum _BTC_CHIP_INTERFACE {

WARNING: do not add new typedefs
#71: FILE: hal/HalBtcOutSrc.h:71:
+typedef enum _BTC_CHIP_TYPE {

WARNING: do not add new typedefs
#81: FILE: hal/HalBtcOutSrc.h:81:
+typedef enum _BTC_MSG_TYPE {

WARNING: do not add new typedefs
#167: FILE: hal/HalBtcOutSrc.h:167:
+typedef struct _BTC_BOARD_INFO {

WARNING: do not add new typedefs
#177: FILE: hal/HalBtcOutSrc.h:177:
+typedef enum _BTC_DBG_OPCODE {

WARNING: do not add new typedefs
#187: FILE: hal/HalBtcOutSrc.h:187:
+typedef enum _BTC_RSSI_STATE {

WARNING: do not add new typedefs
#200: FILE: hal/HalBtcOutSrc.h:200:
+typedef enum _BTC_WIFI_ROLE {

WARNING: do not add new typedefs
#208: FILE: hal/HalBtcOutSrc.h:208:
+typedef enum _BTC_WIFI_BW_MODE {

WARNING: do not add new typedefs
#215: FILE: hal/HalBtcOutSrc.h:215:
+typedef enum _BTC_WIFI_TRAFFIC_DIR {

WARNING: do not add new typedefs
#221: FILE: hal/HalBtcOutSrc.h:221:
+typedef enum _BTC_WIFI_PNP {

WARNING: do not add new typedefs
#228: FILE: hal/HalBtcOutSrc.h:228:
+typedef enum _BT_WIFI_COEX_STATE {

WARNING: do not add new typedefs
#239: FILE: hal/HalBtcOutSrc.h:239:
+typedef enum _BTC_GET_TYPE {

WARNING: do not add new typedefs
#281: FILE: hal/HalBtcOutSrc.h:281:
+typedef enum _BTC_SET_TYPE {

WARNING: do not add new typedefs
#321: FILE: hal/HalBtcOutSrc.h:321:
+typedef enum _BTC_DBG_DISP_TYPE {

WARNING: do not add new typedefs
#328: FILE: hal/HalBtcOutSrc.h:328:
+typedef enum _BTC_NOTIFY_TYPE_IPS {

WARNING: do not add new typedefs
#334: FILE: hal/HalBtcOutSrc.h:334:
+typedef enum _BTC_NOTIFY_TYPE_LPS {

WARNING: do not add new typedefs
#340: FILE: hal/HalBtcOutSrc.h:340:
+typedef enum _BTC_NOTIFY_TYPE_SCAN {

WARNING: do not add new typedefs
#346: FILE: hal/HalBtcOutSrc.h:346:
+typedef enum _BTC_NOTIFY_TYPE_ASSOCIATE {

WARNING: do not add new typedefs
#352: FILE: hal/HalBtcOutSrc.h:352:
+typedef enum _BTC_NOTIFY_TYPE_MEDIA_STATUS {

WARNING: do not add new typedefs
#358: FILE: hal/HalBtcOutSrc.h:358:
+typedef enum _BTC_NOTIFY_TYPE_SPECIAL_PACKET {

WARNING: do not add new typedefs
#366: FILE: hal/HalBtcOutSrc.h:366:
+typedef enum _BTC_NOTIFY_TYPE_STACK_OPERATION {

WARNING: do not add new typedefs
#374: FILE: hal/HalBtcOutSrc.h:374:
+typedef enum _BTC_ANTENNA_POS {

WARNING: do not add new typedefs
#412: FILE: hal/HalBtcOutSrc.h:412:
+typedef struct _BTC_BT_INFO {

WARNING: do not add new typedefs
#440: FILE: hal/HalBtcOutSrc.h:440:
+typedef struct _BTC_STACK_INFO {

WARNING: do not add new typedefs
#455: FILE: hal/HalBtcOutSrc.h:455:
+typedef struct _BTC_BT_LINK_INFO {

WARNING: do not add new typedefs
#468: FILE: hal/HalBtcOutSrc.h:468:
+typedef struct _BTC_STATISTICS {

WARNING: do not add new typedefs
#487: FILE: hal/HalBtcOutSrc.h:487:
+typedef struct _BTC_COEXIST {

Signed-off-by: Marco Cesati <marco.cesati@gmail.com>
Link: https://lore.kernel.org/r/20210312082638.25512-2-marco.cesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...

staging: rtl8723bs: remove conversion to bool in halbtcoutsrc_Get()

Fix the following coccicheck warning:

drivers/staging/rtl8723bs/hal/hal_btcoex.c:410:59-64: WARNING:
conversion to bool not neede

staging: rtl8723bs: remove conversion to bool in halbtcoutsrc_Get()

Fix the following coccicheck warning:

drivers/staging/rtl8723bs/hal/hal_btcoex.c:410:59-64: WARNING:
conversion to bool not needed here

Signed-off-by: Jason Yan <yanaijie@huawei.com>
Link: https://lore.kernel.org/r/20200504113251.40791-1-yanaijie@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

show more ...