[PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple assignments should be avoided.

From	Srishti Sharma <srishtishar@gmail.com>
Newsgroups	linux.kernel
Subject	[PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple assignments should be avoided.
Date	2017-08-20 00:40 +0200
Message-ID	<ugkzo-74v-17@gated-at.bofh.it> (permalink)
Organization	linux.* mail to news gateway

Show all headers | View raw

Fixed a check reported by checkpatch.pl to avoid multiple assignments in a single statement.

Signed-off-by: Srishti Sharma <srishtishar@gmail.com>
---
 drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
index 26017fe..75e3366 100644
--- a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
+++ b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
@@ -1636,7 +1636,8 @@ static void set_fq_affinity(struct dpaa2_eth_priv *priv)
 	 * This may well change at runtime, either through irqbalance or
 	 * through direct user intervention.
 	 */
-	rx_cpu = txc_cpu = cpumask_first(&priv->dpio_cpumask);
+	rx_cpu = cpumask_first(&priv->dpio_cpumask);
+	txc_cpu = cpumask_first(&priv->dpio_cpumask);

 	for (i = 0; i < priv->num_fqs; i++) {
 		fq = &priv->fq[i];
--
2.7.4

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple assignments should be avoided. Srishti Sharma <srishtishar@gmail.com> - 2017-08-20 00:40 +0200
  Re: [PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple  assignments should be avoided. Greg KH <gregkh@linuxfoundation.org> - 2017-08-20 02:10 +0200
    Re: [PATCH] Staging: fsl-dpaa2: ethernet: dpaa2-eth.c: Multiple  assignments should be avoided. Srishti Sharma <srishtishar@gmail.com> - 2017-08-20 03:10 +0200

csiph-web