From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753844AbdBRU5w (ORCPT ); Sat, 18 Feb 2017 15:57:52 -0500 Received: from mout.web.de ([212.227.15.4]:62659 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753315AbdBRU5u (ORCPT ); Sat, 18 Feb 2017 15:57:50 -0500 Subject: [PATCH 09/29] IB/mlx4: Move an assignment out of a check in forward_trap() To: linux-rdma@vger.kernel.org, Doug Ledford , Hal Rosenstock , Leon Romanovsky , Matan Barak , Sean Hefty , Yishai Hadas References: <1935365a-bd7c-461e-6a84-0c5d3a501fff@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Message-ID: Date: Sat, 18 Feb 2017 21:57:16 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <1935365a-bd7c-461e-6a84-0c5d3a501fff@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:jIbmM5KMC1y3ZFxucZH10S3lb8i0ULzMCqKyBBxhZoRAsFEGHnr 4x8MNtj0YRFIbL2NXcOXF6UjqFNbkfSf0HouFT88wPAeY5lk7kdTWR9DqHLQed8pEDq/YW7 PKyu6VM/E8aUq0puQVyVNsk3m/YhHx5dhtbDMch7qvU5CUQ0bPnvi5ROUm0/JcKabiInVVR Hzh2n6oBYxnb1kHpAAa/w== X-UI-Out-Filterresults: notjunk:1;V01:K0:9xr4CijQWac=:xfNGOUVl1fxFyXpFpfFNKc g6IM5ZhCfI6EZ/ZnqNtRm4XaMaWY5Imtti2TFTVUoJxth8F91Tnbiiz7napT3WW2pkTcHE8BU 3I6GV/ieBeD64dME1Rv/BLvpxjMUStMGk1Fp/NYN/N4O8l/SZexSF/DXwRzQVLEkma+BzCss1 d0BpeBucpEHwBd5SqJoIgw6MAEfEimctg1mvwaEv0fC1Jer2Y11mvCAWqm5mdrOjJCFdSAD1N Nqz+OtnX+PZrWEheOGWJDkpTe7fDmeNmf5OCBI9Yf1dPWdMBxxdtlRb4NAASlI6lHlE9zz2fB to/oAfR8Po8eDCSEr5lo/HwrtK7vbrUTdi9H4Poazi2OLW8oQ+/M6dvDGjgwzYOFWln1EYSg8 2m+bwK2RB84WytEphRWcYYnTHOOkCY/HzzhtGpXeB8wz/xm4wMfOmzPWzRLUc4pasvAf35V10 FXuru0gIw6j8LBmzOGbnzW0iOMyqmla17DexEoysekIoaNEkOHtEsAa0shLXabMkHPbfdf8DC /IcUBt2c4UEUyvn/6yVEpMbRZWTvfm/NDRGEQ73V36CHHBjgnYUFsNyP+4kJM9JxK8SeiOXHQ O8qf9n8DE9bvLAOhCbUhVnhGe8DFK/vmVhZNAtrsP2Rj00gQ2J00eOnIVD2yNRSHA4QkRjFUn +zIE8xM+w7iehmbAahX27j6tSpmpyYjTzxAQX+7somFaj3bXkSdx3fA9BsguQNIswoTASY5Sl mPNd4Y2W5LPNMGd2Lz/kp37SjwrM3zg/u1KZeb+15gvXAVMhhBbh7MZ1e+etanFaRpSRTsMcJ M0jTI25 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Fri, 17 Feb 2017 22:22:53 +0100 The script "checkpatch.pl" pointed information out like the following. ERROR: do not use assignment in if condition Thus fix the affected source code place. Signed-off-by: Markus Elfring --- drivers/infiniband/hw/mlx4/mad.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/mlx4/mad.c b/drivers/infiniband/hw/mlx4/mad.c index 9518b7244269..860fec8b9601 100644 --- a/drivers/infiniband/hw/mlx4/mad.c +++ b/drivers/infiniband/hw/mlx4/mad.c @@ -401,7 +401,8 @@ static void forward_trap(struct mlx4_ib_dev *dev, u8 port_num, const struct ib_m */ spin_lock_irqsave(&dev->sm_lock, flags); memcpy(send_buf->mad, mad, sizeof *mad); - if ((send_buf->ah = dev->sm_ah[port_num - 1])) + send_buf->ah = dev->sm_ah[port_num - 1]; + if (send_buf->ah) ret = ib_post_send_mad(send_buf, NULL); else ret = -EINVAL; -- 2.11.1