video: fbdev: aty: remove useless variable assignments in aty_var_to_crtc()
authorGustavo A. R. Silva <garsilva@embeddedor.com>
Tue, 4 Jul 2017 15:47:23 +0000 (17:47 +0200)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Tue, 4 Jul 2017 15:47:23 +0000 (17:47 +0200)
Value assigned to variable vdisplay at line 990 is overwritten
at line 1039 before it can be used. Also, variable assignment
at line 987 is the same as at line 1039. This makes such
variable assignments useless.

Remove these variable assignments and the code related.

Addresses-Covertity-ID: 1226900
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
drivers/video/fbdev/aty/atyfb_base.c

index 11026e726b68468d2e8de7e159ad5003001b5610..3142d7f7020f0db37e16ed7abf7c6b21283f5eb9 100644 (file)
@@ -984,12 +984,6 @@ static int aty_var_to_crtc(const struct fb_info *info,
                v_total <<= 1;
        }
 
-       vdisplay = yres;
-#ifdef CONFIG_FB_ATY_GENERIC_LCD
-       if ((par->lcd_table != 0) && (crtc->lcd_gen_cntl & LCD_ON))
-               vdisplay  = par->lcd_height;
-#endif
-
        v_disp--;
        v_sync_strt--;
        v_sync_end--;