Re: [PATCH] video: fbdev: remove redundant self assignment of 'height'

2017-12-29 Thread Bartlomiej Zolnierkiewicz
On Thursday, November 23, 2017 10:55:53 AM Colin King wrote:
> From: Colin Ian King 
> 
> The assignment of height to itself is redundant and can be removed.
> Detected with Coccinelle.
> 
> Signed-off-by: Colin Ian King 

Patch queued for 4.16, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R Institute Poland
Samsung Electronics

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH] video: fbdev: remove redundant self assignment of 'height'

2017-11-23 Thread Colin King
From: Colin Ian King 

The assignment of height to itself is redundant and can be removed.
Detected with Coccinelle.

Signed-off-by: Colin Ian King 
---
 drivers/video/fbdev/vga16fb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/video/fbdev/vga16fb.c b/drivers/video/fbdev/vga16fb.c
index 5f0690c8fc93..2c6a576ed84c 100644
--- a/drivers/video/fbdev/vga16fb.c
+++ b/drivers/video/fbdev/vga16fb.c
@@ -1055,7 +1055,6 @@ static void vga16fb_copyarea(struct fb_info *info, const 
struct fb_copyarea *are
case FB_TYPE_VGA_PLANES:
if (info->fix.type_aux == FB_AUX_VGA_PLANES_VGA4) {
width = width/8;
-   height = height;
line_ofs = info->fix.line_length - width;
 
setmode(1);
-- 
2.14.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel