Commit 92576c84 authored by Manuel Palenzuela's avatar Manuel Palenzuela Committed by Greg Kroah-Hartman
Browse files

Staging: rtl8723bs: core: rtw_cmd: Fixed four if-statement coding style issues 



Fixed four cases where the if-statement coding style wasn't following
the guidelines.

Signed-off-by: default avatarManuel Palenzuela <manuelpalenzuelamerino@gmail.com>
Link: https://lore.kernel.org/r/20201029120228.7029-1-manuelpalenzuelamerino@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 071dc178

drivers/staging/rtl8723bs/core/rtw_cmd.c

+4 −4
Original line number Diff line number Diff line
@@ -344,7 +344,7 @@ int rtw_enqueue_cmd(struct cmd_priv *pcmdpriv, struct cmd_obj *cmd_obj) 
	cmd_obj->padapter = padapter;



	res = rtw_cmd_filter(pcmdpriv, cmd_obj);

	if (_FAIL == res) {

	if (res == _FAIL) {

		rtw_free_cmd_obj(cmd_obj);

		goto exit;

	}
@@ -460,7 +460,7 @@ int rtw_cmd_thread(void *context) 


		cmd_start_time = jiffies;



		if (_FAIL == rtw_cmd_filter(pcmdpriv, pcmd)) {

		if (rtw_cmd_filter(pcmdpriv, pcmd) == _FAIL) {

			pcmd->res = H2C_DROPPED;

			goto post_process;

		}
@@ -908,7 +908,7 @@ u8 rtw_disassoc_cmd(struct adapter *padapter, u32 deauth_timeout_ms, bool enqueu 
		res = rtw_enqueue_cmd(cmdpriv, cmdobj);

	} else {

		/* no need to enqueue, do the cmd hdl directly and free cmd parameter */

		if (H2C_SUCCESS != disconnect_hdl(padapter, (u8 *)param))

		if (disconnect_hdl(padapter, (u8 *)param) != H2C_SUCCESS)

			res = _FAIL;

		kfree(param);

	}
@@ -1249,7 +1249,7 @@ u8 rtw_set_chplan_cmd(struct adapter *padapter, u8 chplan, u8 enqueue, u8 swconf 
		res = rtw_enqueue_cmd(pcmdpriv, pcmdobj);

	} else {

		/* no need to enqueue, do the cmd hdl directly and free cmd parameter */

		if (H2C_SUCCESS != set_chplan_hdl(padapter, (unsigned char *)setChannelPlan_param))

		if (set_chplan_hdl(padapter, (unsigned char *)setChannelPlan_param) != H2C_SUCCESS)

			res = _FAIL;



		kfree(setChannelPlan_param);