diff options
| author | Vinod Koul <vkoul@kernel.org> | 2020-07-08 18:58:06 +0530 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2020-07-13 12:14:51 +0530 |
| commit | f0c47c74583cc55c7ff9bdafff63bfb33edbb940 (patch) | |
| tree | 6496a24704321284772f814778049cf0da2f7b96 /drivers/phy/phy-xgene.c | |
| parent | c988b7c5fc056d3bec4cb337a57fc2fe6a04081a (diff) | |
phy: xgene: remove unsigned integer comparison with less than zero
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>
Diffstat (limited to 'drivers/phy/phy-xgene.c')
| -rw-r--r-- | drivers/phy/phy-xgene.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/phy/phy-xgene.c b/drivers/phy/phy-xgene.c index 7a33ec12f71b..b88922e7de1d 100644 --- a/drivers/phy/phy-xgene.c +++ b/drivers/phy/phy-xgene.c @@ -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]; |