diff options
| author | Hungyu Lin <dennylin0707@gmail.com> | 2026-04-12 20:24:16 +0300 |
|---|---|---|
| committer | Hans Verkuil <hverkuil+cisco@kernel.org> | 2026-05-06 10:05:56 +0300 |
| commit | c8e13b0dcb7084152cf8274f24e4c5a4ffec6439 (patch) | |
| tree | 44e312a48d4096c116d818551a15fd07dff4e1bd | |
| parent | f86ed548386e3050e5f8f25b450d09dc009d9a88 (diff) | |
| download | linux-c8e13b0dcb7084152cf8274f24e4c5a4ffec6439.tar.xz | |
media: tegra-video: tegra210: remove redundant NULL check in dequeue_buf_done
list_first_entry() does not returns NULL when the list is known to be
non-empty. The NULL check before list_del_init() is therefore
redundant.
Remove the unnecessary check.
Signed-off-by: Hungyu Lin <dennylin0707@gmail.com>
Reviewed-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
| -rw-r--r-- | drivers/staging/media/tegra-video/tegra210.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/media/tegra-video/tegra210.c b/drivers/staging/media/tegra-video/tegra210.c index da99f19a39e7..a6606768fa03 100644 --- a/drivers/staging/media/tegra-video/tegra210.c +++ b/drivers/staging/media/tegra-video/tegra210.c @@ -362,8 +362,7 @@ dequeue_buf_done(struct tegra_vi_channel *chan) buf = list_first_entry(&chan->done, struct tegra_channel_buffer, queue); - if (buf) - list_del_init(&buf->queue); + list_del_init(&buf->queue); spin_unlock(&chan->done_lock); return buf; |