[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAOhMmr7vpmgQPSJ7Ga0kbCU++QsoaFnd7KetWp38mszad7=Xmw@mail.gmail.com>
Date: Thu, 24 Jun 2021 00:50:57 -0500
From: Lijun Pan <lijunp213@...il.com>
To: Sukadev Bhattiprolu <sukadev@...ux.ibm.com>
Cc: Networking <netdev@...r.kernel.org>,
Dany Madden <drt@...ux.ibm.com>,
Rick Lindsley <ricklind@...ux.ibm.com>,
Brian King <brking@...ux.ibm.com>,
Cristobal Forno <cforno12@...ux.ibm.com>
Subject: Re: [PATCH net 7/7] ibmvnic: parenthesize a check
On Wed, Jun 23, 2021 at 11:17 PM Sukadev Bhattiprolu
<sukadev@...ux.ibm.com> wrote:
>
> Parenthesize a check to be more explicit and to fix a sparse warning
> seen on some distros.
>
> Fixes: 91dc5d2553fbf ("ibmvnic: fix miscellaneous checks")
> Signed-off-by: Sukadev Bhattiprolu <sukadev@...ux.ibm.com>
> ---
> drivers/net/ethernet/ibm/ibmvnic.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c
> index 363a5d5503ad..697b9714fc76 100644
> --- a/drivers/net/ethernet/ibm/ibmvnic.c
> +++ b/drivers/net/ethernet/ibm/ibmvnic.c
> @@ -3367,7 +3367,7 @@ static int enable_scrq_irq(struct ibmvnic_adapter *adapter,
> /* H_EOI would fail with rc = H_FUNCTION when running
> * in XIVE mode which is expected, but not an error.
> */
> - if (rc && rc != H_FUNCTION)
> + if (rc && (rc != H_FUNCTION))
It is unnecessary and this is not an issue with the latest sparse.
Just curious what was the version on that distro?
Powered by blists - more mailing lists