]> www.infradead.org Git - users/jedix/linux-maple.git/commit
drm/msm/a6xx: Remove logically deadcode in a6xx_preempt.c
authorEverest K.C. <everestkc@everestkc.com.np>
Fri, 11 Oct 2024 05:23:14 +0000 (23:23 -0600)
committerRob Clark <robdclark@chromium.org>
Sun, 27 Oct 2024 15:13:19 +0000 (08:13 -0700)
commit45a4f888988a142b34d57a3e02950b3942b8cfc7
tree89dc71178bd8b85381177827333c71d1f09c2df9
parent74c374648ed08efb2ef339656f2764c28c046956
drm/msm/a6xx: Remove logically deadcode in a6xx_preempt.c

The ternary operator never returns -1 as `ring` will never be NULL.
Thus, the ternary operator is not needed.
Fix this by removing the ternary operation and only including the
value it will return when the `ring` is not NULL.

This was reported by Coverity Scan.
https://scan7.scan.coverity.com/#/project-view/51525/11354?selectedIssue=1600286

Fixes: 35d36dc1692f ("drm/msm/a6xx: Add traces for preemption")
Signed-off-by: Everest K.C. <everestkc@everestkc.com.np>
Patchwork: https://patchwork.freedesktop.org/patch/619349/
Signed-off-by: Rob Clark <robdclark@chromium.org>
drivers/gpu/drm/msm/adreno/a6xx_preempt.c