phy: xgene: remove unsigned integer comparison with less than zero
authorVinod Koul <vkoul@kernel.org>
Wed, 8 Jul 2020 13:28:06 +0000 (18:58 +0530)
committerVinod Koul <vkoul@kernel.org>
Mon, 13 Jul 2020 06:44:51 +0000 (12:14 +0530)
We get warning with W=1 build:
drivers/phy/phy-xgene.c: In function ‘xgene_phy_xlate’:
drivers/phy/phy-xgene.c:1618:20: warning: comparison of unsigned
expression in ‘< 0’ is always false [-Wtype-limits]
 1618 |  if (args->args[0] < MODE_SATA || args->args[0] >= MODE_MAX) |

args is uint32_t so can never be less than zero, so remove this check

Link: https://lore.kernel.org/r/20200708132809.265967-3-vkoul@kernel.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/phy-xgene.c

index 7a33ec1..b88922e 100644 (file)
@@ -1615,7 +1615,7 @@ static struct phy *xgene_phy_xlate(struct device *dev,
 
        if (args->args_count <= 0)
                return ERR_PTR(-EINVAL);
-       if (args->args[0] < MODE_SATA || args->args[0] >= MODE_MAX)
+       if (args->args[0] >= MODE_MAX)
                return ERR_PTR(-EINVAL);
 
        ctx->mode = args->args[0];