target/linux/brcm2708/patches-4.1/0172-bcm2708_fb-remove-unnecessary-initialization-of-resu.patch

   1 From 8a969160370faf8c7b09959ca36456d5f96a64d7 Mon Sep 17 00:00:00 2001
   2 From: Colin Ian King <colin.king@canonical.com>
   3 Date: Wed, 2 Sep 2015 07:15:38 -0400
   4 Subject: [PATCH 172/203] bcm2708_fb: remove unnecessary initialization of
   5  result
   6
   7 static analysis by cppcheck detected an unnecessary initialization
   8 of variable 'result' which is re-assigned almost immediately after
   9 the initialization.  Remove the redundant initialization.
  10
  11 [drivers/video/fbdev/bcm2708_fb.c:406]: (performance) Variable
  12   'result' is reassigned a value before the old one has been used.
  13
  14 Signed-off-by: Colin Ian King <colin.king@canonical.com>
  15 ---
  16  drivers/video/fbdev/bcm2708_fb.c | 2 +-
  17  1 file changed, 1 insertion(+), 1 deletion(-)
  18
  19 --- a/drivers/video/fbdev/bcm2708_fb.c
  20 +++ b/drivers/video/fbdev/bcm2708_fb.c
  21 @@ -420,7 +420,7 @@ static int bcm2708_fb_blank(int blank_mo
  22
  23  static int bcm2708_fb_pan_display(struct fb_var_screeninfo *var, struct fb_info *info)
  24  {
  25 -       s32 result = -1;
  26 +       s32 result;
  27         info->var.xoffset = var->xoffset;
  28         info->var.yoffset = var->yoffset;
  29         result = bcm2708_fb_set_par(info);