Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Wan Jiabing <wanjiabing@vivo.com>
To:     Kalle Valo <kvalo@kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, ath10k@lists.infradead.org,
        linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     kael_w@yeah.net, Wan Jiabing <wanjiabing@vivo.com>
Subject: [PATCH] ath10k: simplify if-if to if-else
Date:   Sun, 24 Apr 2022 17:45:22 +0800
Message-Id: <20220424094522.105262-1-wanjiabing@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xAY+T5WNEGHew43cwT67OUG6S+uwrdGrgtSWLZXbupbTM8FjUKo7uSjzNxWe?=
 =?us-ascii?Q?jova++dfbQnhPK1kQyInpFSuY7o4Xsu5mSbL0zaN+LOlvor0zUr12u7hDskC?=
 =?us-ascii?Q?fwRoQTol3NPHMn1jmC4Kw4Yfdx81+k580G42hl8W4B4USiYkF8lQ3zS4TXif?=
 =?us-ascii?Q?kNjgIf36vRjdD8Img5lv0hTVTSCarlR5eZQfwzItyy+FQ6izunyincFe6q6W?=
 =?us-ascii?Q?VyFcJIfhtOiF0a247wySmfk34KhJdA9Yf2K0UyWBnIFGpRxvM5HwVuKJyqGX?=
 =?us-ascii?Q?O11nLjb1o/Hs5jGbacUAxKBADUoSrAse9jMIFTzuaYZFl1nWkNvnXpg1s6ud?=
 =?us-ascii?Q?MFwGQRW+hX34uZN7ucZki5t/iB6kmc6b6LkRHMCO9/g8Xg489Xg/ToPcop66?=
 =?us-ascii?Q?w+DLhjb4YYUhrnly4/k17Ll6Y/MsMrg4VxQNcUbmQZJ/HsKUiNjMdKFd7zUH?=
 =?us-ascii?Q?olRGwUJrmMiXx4VWSz8ca9LGTJFKz1OpQn3RCvhFTwzFleSrhz2Oyqtezku9?=
 =?us-ascii?Q?FjzEaLYs3yVFagC96t1y7Ipmj6+5DP9YUU3iIwozGEIZEhSTf7vNVvNDRZZe?=
 =?us-ascii?Q?P1GG6063kXaqFgJkoyYaEGHEuwE7wcRkhz2ex08JAW+f9BHu/5C2RU9IlC0g?=
 =?us-ascii?Q?EiZcHRJFp5sV0EPxdNp0LbkusFxgEkUyRLEGckAphThV4k8vNRNXWksI/r0z?=
 =?us-ascii?Q?oDGndpq98jAF2rb3udivJMDjIybROXQL2aBmFN+gNZNQJ1i1RBArS796k/Fm?=
 =?us-ascii?Q?oWESnUS2r1rkul4VC89rCHIGUdDDi3pbgXsU6nOj+SsNZ19JiQ/MFR3PA8OA?=
 =?us-ascii?Q?uOiNf5DGrTOljYFfi7BgIKG/TXJsto+ONLiu9E+mXCx+OBxOtBXMqL/U4ZiH?=
 =?us-ascii?Q?C8efQSWIaTHQIufGg6sRGL5wU0rlgM65Gqqlmq5wFNvg58pI1can+TkYQavY?=
 =?us-ascii?Q?0BR86ngq1NscsrXPJpEDcP5dMxVHGedFEOJzm7aYzVDHZG0zyIlbHlOfRjgO?=
 =?us-ascii?Q?Iiyq4J1WURqsYBi2DJy9sO/nAwMtuaSUgtWfCJroLEo7t0xY9PSSeRP0O3aL?=
 =?us-ascii?Q?P1ZepuDWBsRevc/3dpU6vhGJZWzkUhipalCxw5d/byX7mBEIxV4grsz7uDnO?=
 =?us-ascii?Q?QAdg7g1gUOf7rKwaZ4WIV6k5hUvIbsI2HP9NPLtHZLH1XBsbOfWTWbp9QNAL?=
 =?us-ascii?Q?9IbUVEW+6+x0BrzGVItC6yay80qz4qIBqw4g5Ca5UWbbxlSAHrgHyeN6Oea2?=
 =?us-ascii?Q?UlbZlfhKS2WIwdsdjIqnegPp/E8fkwvonwKce+FRafT0zGq/wWky+HHjt6/i?=
 =?us-ascii?Q?4uJFNRQHjUujQ/mkD1Rmd5ME/ZpTDTvjohWmS1Y7ohG7LFOdXtPape0UtRIQ?=
 =?us-ascii?Q?+3KUrjt95aC00I1HsD+hzUCKVcTDwyrlMxwdm3h+lX2AV0+ShEyRJWKDUvZ3?=
 =?us-ascii?Q?IKw80x0YQx/vZ+nRluGO/zHsDwWCGY7WMsll/Q8BkPn0RRyGPZwEI2LX87+z?=
 =?us-ascii?Q?O6jfjWUS/eO/U/ZZyLnXhxOcSrC2IiKMk0v+DHAobo/xSCekte9QyY511lFt?=
 =?us-ascii?Q?h0JLA/hlJSoLuuPaLtC4lqQDQ80+0aQFU+Wyb41fkNGpCTM6tiPX1z+wMT8N?=
 =?us-ascii?Q?k3JIRvE3KcBQ/S7AVvUghpZWvDfXPEqInmah10NfYWWX8VepN8EhCJdLfhCI?=
 =?us-ascii?Q?UnbsCGZEqgNAEoNra3sI4wX0oEF3uJH6uk8aw7YD8290KdCC/aB10sO1tWgP?=
 =?us-ascii?Q?zmRA6vcaMw=3D=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a4292273-a2fc-4cd3-b8f5-08da25d72d98
X-MS-Exchange-CrossTenant-AuthSource: SG2PR06MB3367.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Apr 2022 09:45:34.1557
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: qt0FVFiysD68plnCZ2Oe1jOKOGy+r/VQU9Ruy/q+Hfq6MdO3dJg3RMTUr9YQfEDyTuh1HzYuGznLjYtImeKQig==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TY2PR06MB2925
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org

Use if and else instead of if(A) and if (!A).

Signed-off-by: Wan Jiabing <wanjiabing@vivo.com>
---
 drivers/net/wireless/ath/ath10k/mac.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index d804e19a742a..c460f6f8d4bb 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -4119,11 +4119,10 @@ void ath10k_offchan_tx_work(struct work_struct *work)
 		peer = ath10k_peer_find(ar, vdev_id, peer_addr);
 		spin_unlock_bh(&ar->data_lock);
 
-		if (peer)
+		if (peer) {
 			ath10k_warn(ar, "peer %pM on vdev %d already present\n",
 				    peer_addr, vdev_id);
-
-		if (!peer) {
+		} else {
 			ret = ath10k_peer_create(ar, NULL, NULL, vdev_id,
 						 peer_addr,
 						 WMI_PEER_TYPE_DEFAULT);
-- 
2.35.1