From: Ping-Ke Shih <pkshih@realtek•com>
To: Ruan Jinjie <ruanjinjie@huawei•com>,
"sgoutham@marvell•com" <sgoutham@marvell•com>,
"davem@davemloft•net" <davem@davemloft•net>,
"edumazet@google•com" <edumazet@google•com>,
"kuba@kernel•org" <kuba@kernel•org>,
"pabeni@redhat•com" <pabeni@redhat•com>,
"jesse.brandeburg@intel•com" <jesse.brandeburg@intel•com>,
"anthony.l.nguyen@intel•com" <anthony.l.nguyen@intel•com>,
"tariqt@nvidia•com" <tariqt@nvidia•com>,
"s.shtylyov@omp•ru" <s.shtylyov@omp•ru>,
"aspriel@gmail•com" <aspriel@gmail•com>,
"franky.lin@broadcom•com" <franky.lin@broadcom•com>,
"hante.meuleman@broadcom•com" <hante.meuleman@broadcom•com>,
"kvalo@kernel•org" <kvalo@kernel•org>,
"richardcochran@gmail•com" <richardcochran@gmail•com>,
"yoshihiro.shimoda.uh@renesas•com"
<yoshihiro.shimoda.uh@renesas•com>,
"u.kleine-koenig@pengutronix•de" <u.kleine-koenig@pengutronix•de>,
"mkl@pengutronix•de" <mkl@pengutronix•de>,
"lee@kernel•org" <lee@kernel•org>,
"set_pte_at@outlook•com" <set_pte_at@outlook•com>,
"linux-arm-kernel@lists•infradead.org"
<linux-arm-kernel@lists•infradead.org>,
"netdev@vger•kernel.org" <netdev@vger•kernel.org>,
"intel-wired-lan@lists•osuosl.org"
<intel-wired-lan@lists•osuosl.org>,
"linux-rdma@vger•kernel.org" <linux-rdma@vger•kernel.org>,
"linux-renesas-soc@vger•kernel.org"
<linux-renesas-soc@vger•kernel.org>,
"linux-wireless@vger•kernel.org" <linux-wireless@vger•kernel.org>,
"brcm80211-dev-list.pdl@broadcom•com"
<brcm80211-dev-list.pdl@broadcom•com>,
"SHA-cyfmac-dev-list@infineon•com"
<SHA-cyfmac-dev-list@infineon•com>
Subject: RE: [PATCH -next 3/6] net/mlx4: Remove an unnecessary ternary operator
Date: Mon, 7 Aug 2023 01:41:33 +0000 [thread overview]
Message-ID: <fd3e9bb40d12421caaddda7279aece7a@realtek.com> (raw)
In-Reply-To: <20230804035346.2879318-4-ruanjinjie@huawei.com>
> -----Original Message-----
> From: Ruan Jinjie <ruanjinjie@huawei•com>
> Sent: Friday, August 4, 2023 11:54 AM
> To: sgoutham@marvell•com; davem@davemloft•net; edumazet@google•com; kuba@kernel•org; pabeni@redhat•com;
> jesse.brandeburg@intel•com; anthony.l.nguyen@intel•com; tariqt@nvidia•com; s.shtylyov@omp•ru;
> aspriel@gmail•com; franky.lin@broadcom•com; hante.meuleman@broadcom•com; kvalo@kernel•org;
> richardcochran@gmail•com; yoshihiro.shimoda.uh@renesas•com; ruanjinjie@huawei•com;
> u.kleine-koenig@pengutronix•de; mkl@pengutronix•de; lee@kernel•org; set_pte_at@outlook•com;
> linux-arm-kernel@lists•infradead.org; netdev@vger•kernel.org; intel-wired-lan@lists•osuosl.org;
> linux-rdma@vger•kernel.org; linux-renesas-soc@vger•kernel.org; linux-wireless@vger•kernel.org;
> brcm80211-dev-list.pdl@broadcom•com; SHA-cyfmac-dev-list@infineon•com
> Subject: [PATCH -next 3/6] net/mlx4: Remove an unnecessary ternary operator
>
> There is a ternary operator, the true or false judgement
> of which is unnecessary in C language semantics.
>
> Signed-off-by: Ruan Jinjie <ruanjinjie@huawei•com>
> ---
> drivers/net/ethernet/mellanox/mlx4/port.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/mellanox/mlx4/port.c b/drivers/net/ethernet/mellanox/mlx4/port.c
> index 256a06b3c096..1c289488d050 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/port.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/port.c
> @@ -176,7 +176,7 @@ static bool mlx4_need_mf_bond(struct mlx4_dev *dev)
> mlx4_foreach_port(i, dev, MLX4_PORT_TYPE_ETH)
> ++num_eth_ports;
>
> - return (num_eth_ports == 2) ? true : false;
> + return num_eth_ports == 2;
It should be one space between '=' and '2'.
> }
>
> int __mlx4_register_mac(struct mlx4_dev *dev, u8 port, u64 mac)
> --
> 2.34.1
>
>
> ------Please consider the environment before printing this e-mail.
next prev parent reply other threads:[~2023-08-07 1:44 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-04 3:53 [PATCH -next 0/6] net: Remove unnecessary ternary operators Ruan Jinjie
2023-08-04 3:53 ` [PATCH -next 1/6] net: thunderx: " Ruan Jinjie
2023-08-07 1:39 ` Ping-Ke Shih
2023-08-07 4:59 ` Ruan Jinjie
2023-08-07 17:34 ` Jacob Keller
2023-08-04 3:53 ` [PATCH -next 2/6] ethernet/intel: " Ruan Jinjie
2023-08-07 17:35 ` Jacob Keller
2023-08-04 3:53 ` [PATCH -next 3/6] net/mlx4: Remove an unnecessary ternary operator Ruan Jinjie
2023-08-07 1:41 ` Ping-Ke Shih [this message]
2023-08-07 5:00 ` Ruan Jinjie
2023-08-04 3:53 ` [PATCH -next 4/6] net: ethernet: renesas: rswitch: " Ruan Jinjie
2023-08-04 3:53 ` [PATCH -next 5/6] net: fjes: " Ruan Jinjie
2023-08-04 3:53 ` [PATCH -next 6/6] brcm80211: " Ruan Jinjie
2023-08-07 1:42 ` Ping-Ke Shih
2023-08-07 5:00 ` Ruan Jinjie
2023-08-04 20:17 ` [PATCH -next 0/6] net: Remove unnecessary ternary operators Jakub Kicinski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fd3e9bb40d12421caaddda7279aece7a@realtek.com \
--to=pkshih@realtek$(echo .)com \
--cc=SHA-cyfmac-dev-list@infineon$(echo .)com \
--cc=anthony.l.nguyen@intel$(echo .)com \
--cc=aspriel@gmail$(echo .)com \
--cc=brcm80211-dev-list.pdl@broadcom$(echo .)com \
--cc=davem@davemloft$(echo .)net \
--cc=edumazet@google$(echo .)com \
--cc=franky.lin@broadcom$(echo .)com \
--cc=hante.meuleman@broadcom$(echo .)com \
--cc=intel-wired-lan@lists$(echo .)osuosl.org \
--cc=jesse.brandeburg@intel$(echo .)com \
--cc=kuba@kernel$(echo .)org \
--cc=kvalo@kernel$(echo .)org \
--cc=lee@kernel$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
--cc=linux-rdma@vger$(echo .)kernel.org \
--cc=linux-renesas-soc@vger$(echo .)kernel.org \
--cc=linux-wireless@vger$(echo .)kernel.org \
--cc=mkl@pengutronix$(echo .)de \
--cc=netdev@vger$(echo .)kernel.org \
--cc=pabeni@redhat$(echo .)com \
--cc=richardcochran@gmail$(echo .)com \
--cc=ruanjinjie@huawei$(echo .)com \
--cc=s.shtylyov@omp$(echo .)ru \
--cc=set_pte_at@outlook$(echo .)com \
--cc=sgoutham@marvell$(echo .)com \
--cc=tariqt@nvidia$(echo .)com \
--cc=u.kleine-koenig@pengutronix$(echo .)de \
--cc=yoshihiro.shimoda.uh@renesas$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox