[PATCH v2 7/7] drm/msm/dpu: simplify dpu_encoder_phys_wb_init()

[Dmitry Baryshkov]

There is no need to assign a result to temp varable just to return it
after a goto. Drop the temporary variable and goto and return the result
directly.

Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
---
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c 
b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c
index 6608c00e3c33..e9325cafb1a8 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c
@@ -684,21 +684,18 @@ struct dpu_encoder_phys *dpu_encoder_phys_wb_init(
 {
        struct dpu_encoder_phys *phys_enc = NULL;
        struct dpu_encoder_phys_wb *wb_enc = NULL;
-       int ret = 0;
 
        DPU_DEBUG("\n");
 
        if (!p || !p->parent) {
                DPU_ERROR("invalid params\n");
-               ret = -EINVAL;
-               goto fail_alloc;
+               return ERR_PTR(-EINVAL);
        }
 
        wb_enc = kzalloc(sizeof(*wb_enc), GFP_KERNEL);
        if (!wb_enc) {
                DPU_ERROR("failed to allocate wb phys_enc enc\n");
-               ret = -ENOMEM;
-               goto fail_alloc;
+               return ERR_PTR(-ENOMEM);
        }
 
        phys_enc = &wb_enc->base;
@@ -715,7 +712,4 @@ struct dpu_encoder_phys *dpu_encoder_phys_wb_init(
        DPU_DEBUG("Created dpu_encoder_phys for wb %d\n", phys_enc->hw_wb->idx);
 
        return phys_enc;
-
-fail_alloc:
-       return ERR_PTR(ret);
 }
-- 
2.39.2