diff options
| author | Nishanth Menon <nm@ti.com> | 2026-05-12 12:06:21 -0500 |
|---|---|---|
| committer | Nishanth Menon <nm@ti.com> | 2026-05-22 10:55:55 -0500 |
| commit | fe5272e13e34025a9bb10cc452805e5085492a92 (patch) | |
| tree | a5521e7a498c9d480dfe42987aa2db574a42f209 /drivers/soc | |
| parent | 2b47cc255a952d3bd7ec3c0f38fdd81ccde3cb58 (diff) | |
| download | linux-next-fe5272e13e34025a9bb10cc452805e5085492a92.tar.gz linux-next-fe5272e13e34025a9bb10cc452805e5085492a92.zip | |
soc: ti: knav_dma: Remove dead check on unsigned args.args[0]
smatch warns:
knav_dma.c:390 of_channel_match_helper() warn: unsigned
'args.args[0]' is never less than zero.
of_phandle_args.args[] is uint32_t, so the 'args.args[0] < 0' check
is always false. of_parse_phandle_with_fixed_args() already handles
errors by returning a non-zero code, which is checked immediately
above. Remove the dead check.
Reviewed-by: Sai Sree Kartheek Adivi <s-adivi@ti.com>
Reviewed-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
Link: https://patch.msgid.link/20260512170623.3174416-10-nm@ti.com
Signed-off-by: Nishanth Menon <nm@ti.com>
Diffstat (limited to 'drivers/soc')
| -rw-r--r-- | drivers/soc/ti/knav_dma.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/soc/ti/knav_dma.c b/drivers/soc/ti/knav_dma.c index 462d181ca564..7ba6fd58e9ce 100644 --- a/drivers/soc/ti/knav_dma.c +++ b/drivers/soc/ti/knav_dma.c @@ -387,11 +387,6 @@ static int of_channel_match_helper(struct device_node *np, const char *name, return -ENODEV; } - if (args.args[0] < 0) { - dev_err(kdev->dev, "Missing args for %s\n", name); - return -ENODEV; - } - return args.args[0]; } |