dpaa2-eth: Simplify the calculation of variables

Fix the following coccicheck warnings:

./drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c:1651:36-38: WARNING
!A || A && B is equivalent to !A || B.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
Acked-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Link: https://lore.kernel.org/r/1612260157-128026-1-git-send-email-jiapeng.chong@linux.alibaba.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
index 41e225baf5712a0d9b4c347e8f713b22a2628ee5..c5ceebca8caeb9f1508b4acda4740b535d818a0f 100644 (file)
--- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
+++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c
@@ -1660,7 +1660,7 @@ set_cgtd:
         * CG taildrop threshold, so it won't interfere with it; we also
         * want frames in non-PFC enabled traffic classes to be kept in check)
         */
-       td.enable = !tx_pause || (tx_pause && pfc);
+       td.enable = !tx_pause || pfc;
        if (priv->rx_cgtd_enabled == td.enable)
                return;