From 716ca530e1c4515d8683c9d5be3d56b301758b66 Mon Sep 17 00:00:00 2001 From: James <> Date: Wed, 4 Nov 2015 11:49:21 +0000 Subject: trunk-47381 --- ...remove-unnecessary-initialization-of-resu.patch | 29 ++++++++++++++++++++++ 1 file changed, 29 insertions(+) create mode 100644 target/linux/brcm2708/patches-4.1/0172-bcm2708_fb-remove-unnecessary-initialization-of-resu.patch (limited to 'target/linux/brcm2708/patches-4.1/0172-bcm2708_fb-remove-unnecessary-initialization-of-resu.patch') diff --git a/target/linux/brcm2708/patches-4.1/0172-bcm2708_fb-remove-unnecessary-initialization-of-resu.patch b/target/linux/brcm2708/patches-4.1/0172-bcm2708_fb-remove-unnecessary-initialization-of-resu.patch new file mode 100644 index 0000000..164cbc5 --- /dev/null +++ b/target/linux/brcm2708/patches-4.1/0172-bcm2708_fb-remove-unnecessary-initialization-of-resu.patch @@ -0,0 +1,29 @@ +From 8a969160370faf8c7b09959ca36456d5f96a64d7 Mon Sep 17 00:00:00 2001 +From: Colin Ian King +Date: Wed, 2 Sep 2015 07:15:38 -0400 +Subject: [PATCH 172/203] 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 +--- + 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); -- cgit v1.2.3