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