diff mbox series

drm/sprd: remove pointless assignment

Message ID Yx9LqimHvKwLIr/G@kili (mailing list archive)
State New, archived
Headers show
Series drm/sprd: remove pointless assignment | expand

Commit Message

Dan Carpenter Sept. 12, 2022, 3:09 p.m. UTC 
There is no need for the "video_size_step" variable.

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

Patch

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;