Skip to contentSkip to conversation

staging: rtl8712: rtl8712_cmd.c: removes unnecessary parentheses

PATCH In review Juliana Rodrigues 2 participants 2 messages Mar 13, 2016 – Mar 21, 2016 Staging

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 staging: rtl8712: rtl8712_cmd.c: removes unnecessary parentheses

This patch removes unnecessary parentheses that were causing multiple checkpatch.pl issues.

---
 drivers/staging/rtl8712/rtl8712_cmd.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)
drivers/staging/rtl8712/rtl8712_cmd.c +7 -7
index 50f4002..bef2b89 100644
@@ -264,9 +264,9 @@ static struct cmd_obj *cmd_hdl_filter(struct _adapter *padapter,
 		 */
 		if (padapter->pwrctrlpriv.pwr_mode > PS_MODE_ACTIVE) {
 			padapter->pwrctrlpriv.pwr_mode = PS_MODE_ACTIVE;
-			_enter_pwrlock(&(padapter->pwrctrlpriv.lock));
+			_enter_pwrlock(&padapter->pwrctrlpriv.lock);
 			r8712_set_rpwm(padapter, PS_STATE_S4);
-			up(&(padapter->pwrctrlpriv.lock));
+			up(&padapter->pwrctrlpriv.lock);
 		}
 		pcmd_r = pcmd;
 		break;
@@ -318,18 +318,18 @@ int r8712_cmd_thread(void *context)
 	struct tx_desc *pdesc;
 	void (*pcmd_callback)(struct _adapter *dev, struct cmd_obj *pcmd);
 	struct _adapter *padapter = context;
-	struct	cmd_priv	*pcmdpriv = &(padapter->cmdpriv);
+	struct	cmd_priv	*pcmdpriv = &padapter->cmdpriv;
 
 	allow_signal(SIGTERM);
 	while (1) {
-		if ((_down_sema(&(pcmdpriv->cmd_queue_sema))) == _FAIL)
+		if ((_down_sema(&pcmdpriv->cmd_queue_sema)) == _FAIL)
 			break;
 		if (padapter->bDriverStopped || padapter->bSurpriseRemoved)
 			break;
 		if (r8712_register_cmd_alive(padapter) != _SUCCESS)
 			continue;
 _next:
-		pcmd = r8712_dequeue_cmd(&(pcmdpriv->cmd_queue));
+		pcmd = r8712_dequeue_cmd(&pcmdpriv->cmd_queue);
 		if (!(pcmd)) {
 			r8712_unregister_cmd_alive(padapter);
 			continue;
@@ -415,7 +415,7 @@ _next:
 	}
 	/* free all cmd_obj resources */
 	do {
-		pcmd = r8712_dequeue_cmd(&(pcmdpriv->cmd_queue));
+		pcmd = r8712_dequeue_cmd(&pcmdpriv->cmd_queue);
 		if (pcmd == NULL)
 			break;
 		r8712_free_cmd_obj(pcmd);
@@ -429,7 +429,7 @@ void r8712_event_handle(struct _adapter *padapter, uint *peventbuf)
 	u8 evt_code, evt_seq;
 	u16 evt_sz;
 	void (*event_callback)(struct _adapter *dev, u8 *pbuf);
-	struct	evt_priv *pevt_priv = &(padapter->evtpriv);
+	struct	evt_priv *pevt_priv = &padapter->evtpriv;
 
 	if (peventbuf == NULL)
 		goto _abort_event_;
Signature · 1 line
2.7.2
maintainer
#1 lore raw
Quoted 7 lines · Juliana Rodrigues

This patch removes unnecessary parentheses that were causing multiple checkpatch.pl issues.

---
 drivers/staging/rtl8712/rtl8712_cmd.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

Does not apply to my tree :(