There is no need for the "video_size_step" variable.

Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
---
 drivers/gpu/drm/sprd/sprd_dsi.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/sprd/sprd_dsi.c b/drivers/gpu/drm/sprd/sprd_dsi.c
index 12b67a5d5923..c8af4b5516a2 100644
--- a/drivers/gpu/drm/sprd/sprd_dsi.c
+++ b/drivers/gpu/drm/sprd/sprd_dsi.c
@@ -325,8 +325,6 @@ static u16 calc_bytes_per_pixel_x100(int coding)
 
 static u8 calc_video_size_step(int coding)
 {
-       u8 video_size_step;
-
        switch (coding) {
        case COLOR_CODE_16BIT_CONFIG1:
        case COLOR_CODE_16BIT_CONFIG2:
@@ -335,14 +333,14 @@ static u8 calc_video_size_step(int coding)
        case COLOR_CODE_18BIT_CONFIG2:
        case COLOR_CODE_24BIT:
        case COLOR_CODE_COMPRESSTION:
-               return video_size_step = 1;
+               return 1;
        case COLOR_CODE_20BIT_YCC422_LOOSELY:
        case COLOR_CODE_24BIT_YCC422:
        case COLOR_CODE_16BIT_YCC422:
        case COLOR_CODE_30BIT:
        case COLOR_CODE_36BIT:
        case COLOR_CODE_12BIT_YCC420:
-               return video_size_step = 2;
+               return 2;
        default:
                DRM_ERROR("invalid color coding");
                return 0;
-- 
2.35.1

Reply via email to