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:     =?UTF-8?q?Toke=20H=C3=B8iland-J=C3=B8rgensen?= <toke@toke.dk>,
        Kalle Valo <kvalo@kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        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] ath9k: hif_usb: simplify if-if to if-else
Date:   Sun, 24 Apr 2022 17:44:41 +0800
Message-Id: <20220424094441.104937-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?5bZLotuxyeWaYhk+KRvFSMbUbg/Y9AU0rL7ih5Ba76MKq11AA8ATmo1IbCkC?=
 =?us-ascii?Q?zA5d3YbAlLZdjKtcobj8KOdlxSQ7R1hk5PBbPRpF9ACTEBaFKTaADAPhQttU?=
 =?us-ascii?Q?w8Vn5+ehq6Ya/CYLtx4kI6z3CzTJ2IGwd5Mf47cFFn41orZ40O7bWxGS0mB0?=
 =?us-ascii?Q?q1L5cKAQefLotrM4opWiIQ0Q5kJFBwPlrcNqwglX3ARxP4rtLBZ29NpXxqHR?=
 =?us-ascii?Q?h7lpboPksQH/zNeS4Xo4iqwJ3vmq/pEQnUV4/9Jq/+QSCv9Oa4bc1vHUIQZM?=
 =?us-ascii?Q?GleHKpJ80urcT7+C8HZiolBpX5sxbyn0SAtS7BlOkBiWlZXMkIinlGcZMsVN?=
 =?us-ascii?Q?ezast7n6szrwuw7Lb73Zoyk8qj+Bc2f1mvkq9fl6Am/zkE6IxQWj6FmaMFzw?=
 =?us-ascii?Q?JIwR+VwwemDH0Q6nGcgFJYfSYv/XVPinfqUDn9c4S0G6l571YgpAt5hRYlGC?=
 =?us-ascii?Q?XMVP2s1dHSHPuwMCZq0v5i7r0BpPI5FU1ULRtAqn1JOYaHgRt2KDflDT61rZ?=
 =?us-ascii?Q?74gjaqSLzovUVCRQqbV3LnTvdKs5kLJWHRUAtUBO98oPRLxHkc+XwJbCR0Nv?=
 =?us-ascii?Q?AyVWVyjJGw+ddLVMNaI9MgnGWLpnfLDrWAjScK4oUEKsb5hpWVz/p8ysH2Sp?=
 =?us-ascii?Q?w/W+2cc7gQZ0w6zw/ZeigK+zUCGxBeFhkv0wMGUUkg+FxWKEldE4OEkbM7z5?=
 =?us-ascii?Q?hizjP5ozgoz8QVSB0GDrfauQs6bKZ9i58HBkKkEB7K+YD8zRQevz42ZvoMX8?=
 =?us-ascii?Q?Z6K7tAzg24asHB7VxCGnwonRvw2Ln27YDIR7oZ/Nl444LusYI6YGpV+uY/2g?=
 =?us-ascii?Q?5YoNdGEg0IsSG4AX4fTMQKnrfYNpzUZh0AdA1hQ9+Zw8VPSnAiHrmnEkK10Q?=
 =?us-ascii?Q?16AqXLi1fDqSwCuS1GwKTOoueODVFbJXf+dFkYu3zHRsJiPHT25y2OoqAadh?=
 =?us-ascii?Q?zdF4mvNdhJOMwDF1BV4x0G/tUP+QRchUY63eQtwNcHcTXk6rpvJplDoI3e3c?=
 =?us-ascii?Q?NvoUK51NEwhTwlU50X3QoIiuFXaVhoI41TBz+6DNtfgJKUQzlTtublNg2HE4?=
 =?us-ascii?Q?kbfS6UH8s17GIrMlodi6PHx7bD3cQXskxQ2RJWC5XypuYQmlD7/2jK7cB8Cr?=
 =?us-ascii?Q?tQ1dcb2hX6yyAYOPjmpL6gLXs32w4VGUWlL2oniEVeqT96KtzX39ln1asxpw?=
 =?us-ascii?Q?FM/jOWCJMUe7XKcI+vKrqjm5cpY0dYf+Y4krG0fvdNq6LKMUThyDtTwI0m8n?=
 =?us-ascii?Q?NLTj+wCmIUXPZvIUtoGzwLlyvvcS/Xp3RpOeQi0Ui8wxzqdsC1GPm81+MvN7?=
 =?us-ascii?Q?X+0QvLLSAa0G1FWXPins5B4G2hPbVkBuDUkDyvoj2QZIOJKrEC2OBNVpXtn0?=
 =?us-ascii?Q?7SmkWmPx4BYBOtFCsGdYG3vmOcol+PFJugZZ4p48iYCigrt5aP+UAdUPzOD4?=
 =?us-ascii?Q?51/DBluhbab2YIg/6BKBM7ZwbM7nC3kRcc4oVbEo1sTrF1VmDyfWUG9r7LqT?=
 =?us-ascii?Q?yzqRFiO7PX/ieaM1vvbQqV8ck203SY2Z97KHCG7gO3evnjjT/9hj5b+Yb1Zp?=
 =?us-ascii?Q?9nvpGaF2MY9VReWG71+LkqI27HWf69oNwFd1p8VviHdntYbIpdDJ6+Xp4/6B?=
 =?us-ascii?Q?txDY5h1nbZaMt5wplz1sxMhPDgSJ9KBw7a2ULBe7BLAB04vysRjl/KhK+svK?=
 =?us-ascii?Q?NNTDlbKaEfFrpRCUik5CV1BzSuWdoMQLlkifR1xFImMXkBm3hKViMU8DWn3Q?=
 =?us-ascii?Q?tz4T0V4P2w=3D=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b4b7ea6b-a705-4508-14b2-08da25d7173f
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:44:57.9864
 (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: DOwaZ6Cs1q4AOqqt26JiFMA+r28DCdCfMR3ZrpmwPbU+/efoLkRUkmFLF1gUYklt/0qJofyTr6XXKSkcsmXQkQ==
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/ath9k/hif_usb.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/hif_usb.c b/drivers/net/wireless/ath/ath9k/hif_usb.c
index f06eec99de68..518deb5098a2 100644
--- a/drivers/net/wireless/ath/ath9k/hif_usb.c
+++ b/drivers/net/wireless/ath/ath9k/hif_usb.c
@@ -368,10 +368,9 @@ static int __hif_usb_tx(struct hif_device_usb *hif_dev)
 		__skb_queue_head_init(&tx_buf->skb_queue);
 		list_move_tail(&tx_buf->list, &hif_dev->tx.tx_buf);
 		hif_dev->tx.tx_buf_cnt++;
-	}
-
-	if (!ret)
+	} else {
 		TX_STAT_INC(buf_queued);
+	}
 
 	return ret;
 }
-- 
2.35.1