diff options
author | Hannes Eder <hannes@hanneseder.net> | 2009-01-06 17:42:32 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-01-06 18:59:26 -0500 |
commit | 2899bb09418786132079bb6495ba8f1ce86eb0c6 (patch) | |
tree | f85e1ad82b586a48ef0c295b90e88d295b5bc619 | |
parent | 6875145940efdc9851df8aaa251453d33cda62a4 (diff) |
viafb: fix sparse warnings
Fix this sparse warnings:
drivers/video/via/viafbdev.c:871:3: warning: returning void-valued expression
drivers/video/via/viafbdev.c:938:3: warning: returning void-valued expression
drivers/video/via/viafbdev.c:995:3: warning: returning void-valued expression
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Cc: Krzysztof Helt <krzysztof.h1@poczta.fm>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/video/via/viafbdev.c | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/drivers/video/via/viafbdev.c b/drivers/video/via/viafbdev.c index e21fe5b6f9ff..37b433a08ce8 100644 --- a/drivers/video/via/viafbdev.c +++ b/drivers/video/via/viafbdev.c | |||
@@ -870,8 +870,10 @@ static void viafb_fillrect(struct fb_info *info, | |||
870 | u32 col = 0, rop = 0; | 870 | u32 col = 0, rop = 0; |
871 | int pitch; | 871 | int pitch; |
872 | 872 | ||
873 | if (!viafb_accel) | 873 | if (!viafb_accel) { |
874 | return cfb_fillrect(info, rect); | 874 | cfb_fillrect(info, rect); |
875 | return; | ||
876 | } | ||
875 | 877 | ||
876 | if (!rect->width || !rect->height) | 878 | if (!rect->width || !rect->height) |
877 | return; | 879 | return; |
@@ -937,8 +939,10 @@ static void viafb_copyarea(struct fb_info *info, | |||
937 | 939 | ||
938 | DEBUG_MSG(KERN_INFO "viafb_copyarea!!\n"); | 940 | DEBUG_MSG(KERN_INFO "viafb_copyarea!!\n"); |
939 | 941 | ||
940 | if (!viafb_accel) | 942 | if (!viafb_accel) { |
941 | return cfb_copyarea(info, area); | 943 | cfb_copyarea(info, area); |
944 | return; | ||
945 | } | ||
942 | 946 | ||
943 | if (!area->width || !area->height) | 947 | if (!area->width || !area->height) |
944 | return; | 948 | return; |
@@ -994,8 +998,10 @@ static void viafb_imageblit(struct fb_info *info, | |||
994 | int i; | 998 | int i; |
995 | int pitch; | 999 | int pitch; |
996 | 1000 | ||
997 | if (!viafb_accel) | 1001 | if (!viafb_accel) { |
998 | return cfb_imageblit(info, image); | 1002 | cfb_imageblit(info, image); |
1003 | return; | ||
1004 | } | ||
999 | 1005 | ||
1000 | udata = (u32 *) image->data; | 1006 | udata = (u32 *) image->data; |
1001 | 1007 | ||