All the NULL checks are pointless, clk_*() routines already deal with NULL
just fine.

Signed-off-by: Lubomir Rintel <lkund...@v3.sk>
---
 drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 53 ++++++++++-----------------
 1 file changed, 19 insertions(+), 34 deletions(-)

diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c 
b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
index 798fdbc8ecdb..fb37787449bb 100644
--- a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
+++ b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c
@@ -1487,55 +1487,40 @@ static int etnaviv_gpu_clk_enable(struct etnaviv_gpu 
*gpu)
 {
        int ret;
 
-       if (gpu->clk_reg) {
-               ret = clk_prepare_enable(gpu->clk_reg);
-               if (ret)
-                       return ret;
-       }
+       ret = clk_prepare_enable(gpu->clk_reg);
+       if (ret)
+               return ret;
 
-       if (gpu->clk_bus) {
-               ret = clk_prepare_enable(gpu->clk_bus);
-               if (ret)
-                       goto disable_clk_reg;
-       }
+       ret = clk_prepare_enable(gpu->clk_bus);
+       if (ret)
+               goto disable_clk_reg;
 
-       if (gpu->clk_core) {
-               ret = clk_prepare_enable(gpu->clk_core);
-               if (ret)
-                       goto disable_clk_bus;
-       }
+       ret = clk_prepare_enable(gpu->clk_core);
+       if (ret)
+               goto disable_clk_bus;
 
-       if (gpu->clk_shader) {
-               ret = clk_prepare_enable(gpu->clk_shader);
-               if (ret)
-                       goto disable_clk_core;
-       }
+       ret = clk_prepare_enable(gpu->clk_shader);
+       if (ret)
+               goto disable_clk_core;
 
        return 0;
 
 disable_clk_core:
-       if (gpu->clk_core)
-               clk_disable_unprepare(gpu->clk_core);
+       clk_disable_unprepare(gpu->clk_core);
 disable_clk_bus:
-       if (gpu->clk_bus)
-               clk_disable_unprepare(gpu->clk_bus);
+       clk_disable_unprepare(gpu->clk_bus);
 disable_clk_reg:
-       if (gpu->clk_reg)
-               clk_disable_unprepare(gpu->clk_reg);
+       clk_disable_unprepare(gpu->clk_reg);
 
        return ret;
 }
 
 static int etnaviv_gpu_clk_disable(struct etnaviv_gpu *gpu)
 {
-       if (gpu->clk_shader)
-               clk_disable_unprepare(gpu->clk_shader);
-       if (gpu->clk_core)
-               clk_disable_unprepare(gpu->clk_core);
-       if (gpu->clk_bus)
-               clk_disable_unprepare(gpu->clk_bus);
-       if (gpu->clk_reg)
-               clk_disable_unprepare(gpu->clk_reg);
+       clk_disable_unprepare(gpu->clk_shader);
+       clk_disable_unprepare(gpu->clk_core);
+       clk_disable_unprepare(gpu->clk_bus);
+       clk_disable_unprepare(gpu->clk_reg);
 
        return 0;
 }
-- 
2.26.2

Reply via email to