net: phy: mscc: fix excluded_middle.cocci warnings
authorkernel test robot <lkp@intel.com>
Mon, 16 Nov 2020 15:34:44 +0000 (16:34 +0100)
committerJakub Kicinski <kuba@kernel.org>
Tue, 17 Nov 2020 17:27:39 +0000 (09:27 -0800)
Condition !A || A && B is equivalent to !A || B.

Generated by: scripts/coccinelle/misc/excluded_middle.cocci

Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: kernel test robot <lkp@intel.com>
Signed-off-by: Julia Lawall <julia.lawall@inria.fr>
Reviewed-by: Antoine Tenart <atenart@kernel.org>
Link: https://lore.kernel.org/r/alpine.DEB.2.22.394.2011161633240.2682@hadrien
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/phy/mscc/mscc_ptp.c

index f053729..d8a6145 100644 (file)
@@ -136,7 +136,7 @@ static void vsc85xx_ts_write_csr(struct phy_device *phydev, enum ts_blk blk,
 
        phy_ts_base_write(phydev, MSCC_EXT_PAGE_ACCESS, MSCC_PHY_PAGE_1588);
 
-       if (!cond || (cond && upper))
+       if (!cond || upper)
                phy_ts_base_write(phydev, MSCC_PHY_TS_CSR_DATA_MSB, upper);
 
        phy_ts_base_write(phydev, MSCC_PHY_TS_CSR_DATA_LSB, lower);