Skip to contentSkip to conversation

staging: rtl8723bs: remove redundant parentheses

PATCH In review Ben O'Bryan 2 participants 2 messages May 31 – Jun 1 Staging

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 staging: rtl8723bs: remove redundant parentheses

Remove redundant parentheses around type checks and align conditional lines to improve readability

---

This is my first patch to the kernel so please let me know if I've done anything wrong.

This patch is mostly for me to verify my environment. Assuming everything goes smoothly, I hope to continue to contribute more formatting patches as I get more comfortable contributing.

 drivers/staging/rtl8723bs/core/rtw_xmit.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/staging/rtl8723bs/core/rtw_xmit.c +3 -3
index 444966c0de7f..cc38c0c950bb 100644
@@ -2476,9 +2476,9 @@ struct xmit_buf *dequeue_pending_xmitbuf_under_survey(struct xmit_priv *pxmitpri
 
 			type = GetFrameSubType(pxmitbuf->pbuf + TXDESC_OFFSET);
 
-			if ((type == WIFI_PROBEREQ) ||
-				(type == WIFI_DATA_NULL) ||
-				(type == WIFI_QOS_DATA_NULL)) {
+			if (type == WIFI_PROBEREQ ||
+			    type == WIFI_DATA_NULL ||
+			    type == WIFI_QOS_DATA_NULL) {
 				list_del_init(&pxmitbuf->list);
 				break;
 			}
Signature · 1 line
2.52.0
#1 lore raw
Quoted 4 lines · Ben O'Bryan

Remove redundant parentheses around type checks and align conditional lines to improve readability

Quoted 9 lines

---

This is my first patch to the kernel so please let me know if I've done anything wrong.

This patch is mostly for me to verify my environment. Assuming everything goes smoothly, I hope to continue to contribute more formatting patches as I get more comfortable contributing.

We don't mind the extra parentheses in staging so we disabled that checkpatch warning.

regards,
dan carpenter