From: Sui Jingfeng <suijingf...@loongson.cn>

Because the 'else' is not generally useful after the 'return'.

Signed-off-by: Sui Jingfeng <suijingf...@loongson.cn>
---
 drivers/gpu/drm/etnaviv/etnaviv_gem.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gem.c 
b/drivers/gpu/drm/etnaviv/etnaviv_gem.c
index be2f459c66b5..271470723d5e 100644
--- a/drivers/gpu/drm/etnaviv/etnaviv_gem.c
+++ b/drivers/gpu/drm/etnaviv/etnaviv_gem.c
@@ -358,10 +358,11 @@ static inline enum dma_data_direction 
etnaviv_op_to_dma_dir(u32 op)
 {
        if (op & ETNA_PREP_READ)
                return DMA_FROM_DEVICE;
-       else if (op & ETNA_PREP_WRITE)
+
+       if (op & ETNA_PREP_WRITE)
                return DMA_TO_DEVICE;
-       else
-               return DMA_BIDIRECTIONAL;
+
+       return DMA_BIDIRECTIONAL;
 }
 
 int etnaviv_gem_cpu_prep(struct drm_gem_object *obj, u32 op,
-- 
2.25.1

Reply via email to