staging: rtl8723bs: remove unused BT structures

remove static BT debugging structures no longer in use.

Signed-off-by: Bryan Brattlof <hello@bryanbrattlof.com>
Link: https://lore.kernel.org/r/20210804144812.715695-1-hello@bryanbrattlof.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

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>

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>

staging: rtl8723bs: hal: Delete tests with no effects

Deleted tests with no effects. Detected by Coccinelle.
It seems that these tests have been inadvertently
left while removing BTC_PRINT logs in commit 8e30fb99d729.

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

staging: rtl8723bs: remove commented out DbgPrint logs

remove commented out DbgPrint logs.

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

staging: rtl8723bs: remove empty if-else blocks after BTC_PRINT removal

remove if-else blocks left empty after BTC_PRINT removal.

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

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>

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

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

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

Staging: rtl8723bs: fix names in HalBtc8723b2Ant.h

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

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

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>

Staging: rtl8723bs: fix spaces in HalBtc8723b2Ant.c

This commit fixes the following checkpatch.pl errors:

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#19: FILE: ./hal/HalBtc8723b2Ant.c:19:
+static struct COEX_DM_8723B_2ANT * pCoexDm = &GLCoexDm8723b2Ant;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#21: FILE: ./hal/HalBtc8723b2Ant.c:21:
+static struct COEX_STA_8723B_2ANT * pCoexSta = &GLCoexSta8723b2Ant;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#111: FILE: ./hal/HalBtc8723b2Ant.c:111:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#192: FILE: ./hal/HalBtc8723b2Ant.c:192:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#215: FILE: ./hal/HalBtc8723b2Ant.c:215:
+static void halbtc8723b2ant_MonitorBtCtr(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#265: FILE: ./hal/HalBtc8723b2Ant.c:265:
+static void halbtc8723b2ant_QueryBtInfo(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#282: FILE: ./hal/HalBtc8723b2Ant.c:282:
+static bool halbtc8723b2ant_IsWifiStatusChanged(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#313: FILE: ./hal/HalBtc8723b2Ant.c:313:
+static void halbtc8723b2ant_UpdateBtLinkInfo(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#315: FILE: ./hal/HalBtc8723b2Ant.c:315:
+ struct BTC_BT_LINK_INFO * pBtLinkInfo = &pBtCoexist->btLinkInfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#377: FILE: ./hal/HalBtc8723b2Ant.c:377:
+static u8 halbtc8723b2ant_ActionAlgorithm(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#379: FILE: ./hal/HalBtc8723b2Ant.c:379:
+ struct BTC_BT_LINK_INFO * pBtLinkInfo = &pBtCoexist->btLinkInfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#552: FILE: ./hal/HalBtc8723b2Ant.c:552:
+ struct BTC_COEXIST * pBtCoexist, u8 dacSwingLvl

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#576: FILE: ./hal/HalBtc8723b2Ant.c:576:
+ struct BTC_COEXIST * pBtCoexist, u8 decBtPwrLvl

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#597: FILE: ./hal/HalBtc8723b2Ant.c:597:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, u8 decBtPwrLvl

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#631: FILE: ./hal/HalBtc8723b2Ant.c:631:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, u8 fwDacSwingLvl

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#666: FILE: ./hal/HalBtc8723b2Ant.c:666:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#689: FILE: ./hal/HalBtc8723b2Ant.c:689:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, bool bRxRfShrinkOn

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#723: FILE: ./hal/HalBtc8723b2Ant.c:723:
+ struct BTC_COEXIST * pBtCoexist, bool bLowPenaltyRa

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#751: FILE: ./hal/HalBtc8723b2Ant.c:751:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, bool bLowPenaltyRa

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#785: FILE: ./hal/HalBtc8723b2Ant.c:785:
+static void halbtc8723b2ant_SetDacSwingReg(struct BTC_COEXIST * pBtCoexist, u32 level)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#798: FILE: ./hal/HalBtc8723b2Ant.c:798:
+ struct BTC_COEXIST * pBtCoexist, bool bSwDacSwingOn, u32 swDacSwingLvl

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

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#853: FILE: ./hal/HalBtc8723b2Ant.c:853:
+ struct BTC_COEXIST * pBtCoexist, bool bAgcTableEn

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#913: FILE: ./hal/HalBtc8723b2Ant.c:913:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, bool bAgcTableEn

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#947: FILE: ./hal/HalBtc8723b2Ant.c:947:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#984: FILE: ./hal/HalBtc8723b2Ant.c:984:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1050: FILE: ./hal/HalBtc8723b2Ant.c:1050:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, u8 type

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1099: FILE: ./hal/HalBtc8723b2Ant.c:1099:
+ struct BTC_COEXIST * pBtCoexist, bool bEnable

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1120: FILE: ./hal/HalBtc8723b2Ant.c:1120:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, bool bEnable

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1148: FILE: ./hal/HalBtc8723b2Ant.c:1148:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1187: FILE: ./hal/HalBtc8723b2Ant.c:1187:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1199: FILE: ./hal/HalBtc8723b2Ant.c:1199:
+ struct BTC_COEXIST * pBtCoexist,

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1211: FILE: ./hal/HalBtc8723b2Ant.c:1211:
+ struct BTC_COEXIST * pBtCoexist, u8 antPosType, bool bInitHwCfg, bool bWifiOff

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1214: FILE: ./hal/HalBtc8723b2Ant.c:1214:
+ struct BTC_BOARD_INFO * pBoardInfo = &pBtCoexist->boardInfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1305: FILE: ./hal/HalBtc8723b2Ant.c:1305:
+ struct BTC_COEXIST * pBtCoexist, bool bForceExec, bool bTurnOn, u8 type

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1437: FILE: ./hal/HalBtc8723b2Ant.c:1437:
+static void halbtc8723b2ant_CoexAllOff(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1453: FILE: ./hal/HalBtc8723b2Ant.c:1453:
+static void halbtc8723b2ant_InitCoexDm(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1465: FILE: ./hal/HalBtc8723b2Ant.c:1465:
+static void halbtc8723b2ant_ActionBtInquiry(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1493: FILE: ./hal/HalBtc8723b2Ant.c:1493:
+static bool halbtc8723b2ant_IsCommonAction(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#1594: FILE: ./hal/HalBtc8723b2Ant.c:1594:
+ struct BTC_COEXIST * pBtCoexist, bool bScoHid, bool bTxPause, u8 maxInterval

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2044: FILE: ./hal/HalBtc8723b2Ant.c:2044:
+static void halbtc8723b2ant_ActionSco(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2099: FILE: ./hal/HalBtc8723b2Ant.c:2099:
+static void halbtc8723b2ant_ActionHid(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2160: FILE: ./hal/HalBtc8723b2Ant.c:2160:
+static void halbtc8723b2ant_ActionA2dp(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2241: FILE: ./hal/HalBtc8723b2Ant.c:2241:
+static void halbtc8723b2ant_ActionA2dpPanHs(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2291: FILE: ./hal/HalBtc8723b2Ant.c:2291:
+static void halbtc8723b2ant_ActionPanEdr(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2349: FILE: ./hal/HalBtc8723b2Ant.c:2349:
+static void halbtc8723b2ant_ActionPanHs(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2399: FILE: ./hal/HalBtc8723b2Ant.c:2399:
+static void halbtc8723b2ant_ActionPanEdrA2dp(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2460: FILE: ./hal/HalBtc8723b2Ant.c:2460:
+static void halbtc8723b2ant_ActionPanEdrHid(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2524: FILE: ./hal/HalBtc8723b2Ant.c:2524:
+static void halbtc8723b2ant_ActionHidA2dpPanEdr(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2584: FILE: ./hal/HalBtc8723b2Ant.c:2584:
+static void halbtc8723b2ant_ActionHidA2dp(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2659: FILE: ./hal/HalBtc8723b2Ant.c:2659:
+static void halbtc8723b2ant_RunCoexistMechanism(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2758: FILE: ./hal/HalBtc8723b2Ant.c:2758:
+static void halbtc8723b2ant_WifiOffHwCfg(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2783: FILE: ./hal/HalBtc8723b2Ant.c:2783:
+static void halbtc8723b2ant_InitHwConfig(struct BTC_COEXIST * pBtCoexist, bool bBackUp)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2817: FILE: ./hal/HalBtc8723b2Ant.c:2817:
+void EXhalbtc8723b2ant_PowerOnSetting(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2819: FILE: ./hal/HalBtc8723b2Ant.c:2819:
+ struct BTC_BOARD_INFO * pBoardInfo = &pBtCoexist->boardInfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2869: FILE: ./hal/HalBtc8723b2Ant.c:2869:
+void EXhalbtc8723b2ant_InitHwConfig(struct BTC_COEXIST * pBtCoexist, bool bWifiOnly)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2874: FILE: ./hal/HalBtc8723b2Ant.c:2874:
+void EXhalbtc8723b2ant_InitCoexDm(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2881: FILE: ./hal/HalBtc8723b2Ant.c:2881:
+void EXhalbtc8723b2ant_DisplayCoexInfo(struct BTC_COEXIST * pBtCoexist)

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2883: FILE: ./hal/HalBtc8723b2Ant.c:2883:
+ struct BTC_BOARD_INFO * pBoardInfo = &pBtCoexist->boardInfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2884: FILE: ./hal/HalBtc8723b2Ant.c:2884:
+ struct BTC_STACK_INFO * pStackInfo = &pBtCoexist->stackInfo;

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#2885: FILE: ./hal/HalBtc8723b2Ant.c:2885:
+ struct BTC_BT_LINK_INFO * pBtLinkInfo = &pBtCoexist->btLinkInfo;

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

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

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

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

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

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

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#3362: FILE: ./hal/HalBtc8723b2Ant.c:3362:
+ struct BTC_COEXIST * pBtCoexist, u8 *tmpBuf, u8 length

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#3506: FILE: ./hal/HalBtc8723b2Ant.c:3506:
+void EXhalbtc8723b2ant_HaltNotify(struct BTC_COEXIST * pBtCoexist)

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

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
#3531: FILE: ./hal/HalBtc8723b2Ant.c:3531:
+void EXhalbtc8723b2ant_Periodical(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-5-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

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>

staging: HalBtc8723b2Ant: remove useless typedef's

This commit fixes the following checkpatch.pl warnings:

WARNING: do not add new typedefs
#19: FILE: hal/HalBtc8723b2Ant.h:19:
+typedef enum _BT_INFO_SRC_8723B_2ANT {

WARNING: do not add new typedefs
#26: FILE: hal/HalBtc8723b2Ant.h:26:
+typedef enum _BT_8723B_2ANT_BT_STATUS {

WARNING: do not add new typedefs
#36: FILE: hal/HalBtc8723b2Ant.h:36:
+typedef enum _BT_8723B_2ANT_COEX_ALGO {

WARNING: do not add new typedefs
#51: FILE: hal/HalBtc8723b2Ant.h:51:
+typedef struct _COEX_DM_8723B_2ANT {

WARNING: do not add new typedefs
#104: FILE: hal/HalBtc8723b2Ant.h:104:
+typedef struct _COEX_STA_8723B_2ANT {

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

staging: rtl8723bs: hal: Correct typos in comments

Correct typos in comments.
Misspellings found using checkpatch.pl.

Signed-off-by: R Veera Kumar <vkor@vkten.in>
Link: https://lore.kernel.org/r/20200326143023.13681-1-vkor@vkten.in
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

staging: rtl8723bs: hal: fix condition with no effect

fix warning reorted by coccicheck
WARNING: possible condition with no effect (if == else)

Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
Link: https://lore.kernel.org/r/20200125134604.GA4247@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

staging: rtl8723bs: hal: move common code to macro

In halbtc8723b2ant_TdmaDurationAdjust function,below piece of code is
repeated many times.

halbtc8723b2ant_PsTdma(pBtCoexist, NORMAL_EXEC, true, val);
pCoexDm->psTdmaDuAdjType = val;

So replace the same with "HAL_BTC8723B2ANT_DMA_DURATION_ADJUST" MACRO.

Signed-off-by: Hariprasad Kelam <hariprasad.kelam@gmail.com>
-----
changes in v2: clean the changelog with proper indent
-----
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

staging: rtl8723bs: Replace license boilerplate with SPDX identifiers

This satisfies a checkpatch.pl warning and is the preferred method for
notating the license due to its lack of ambiguity.

Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

staging: rtl8723bs: fix spelling mistakes: "dismatch" and "Inviation"

Trivial fix to spelling mistakes in message text and comments

"dismatch" -> "mismatch"
"Inviation" -> "Invitation"

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

staging: rtl8723bs: Fix initialization of static variables

Do not initialize static to 0
Do not initialize static to false

Signed-off-by: Justin Vreeland <justin@jvreeland.com>
Reviewed-by: Bastien Nocera <hadess@hadess.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

staging: Add rtl8723bs sdio wifi driver

The rtl8723bs is found on quite a few systems used by Linux users,
such as on Atom systems (Intel Computestick and various other
Atom based devices) and on many (budget) ARM boards such as
the CHIP.

The plan moving forward with this is for the new clean,
written from scratch, rtl8xxxu driver to eventually gain
support for sdio devices. But there is no clear timeline
for that, so lets add this driver included in staging for now.

Cc: Bastien Nocera <hadess@hadess.net>
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Jes Sorensen <jes.sorensen@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>