[PATCH] drm/sprd: remove pointless assignment
Dan Carpenter
dan.carpenter at oracle.com
Mon Sep 12 15:09:30 UTC 2022
There is no need for the "video_size_step" variable.
Signed-off-by: Dan Carpenter <dan.carpenter at 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
More information about the dri-devel
mailing list