blob: 6c1493ee58a657625abd043b0ae2a6d7be1bd53f (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
|
From 324a12ebfd6509fd4466500bd2c5a650133ca1ff Mon Sep 17 00:00:00 2001
From: Colin Ian King <colin.king@canonical.com>
Date: Wed, 2 Sep 2015 07:15:38 -0400
Subject: [PATCH 172/222] bcm2708_fb: remove unnecessary initialization of
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);
|