i810: fix sparse warnings
authorHannes Eder <hannes@hanneseder.net>
Tue, 6 Jan 2009 22:42:34 +0000 (14:42 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 6 Jan 2009 23:59:27 +0000 (15:59 -0800)
Fix this sparse warnings:

  drivers/video/i810/i810_accel.c:305:3: warning: returning void-valued expression
  drivers/video/i810/i810_accel.c:331:3: warning: returning void-valued expression
  drivers/video/i810/i810_accel.c:370: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>
drivers/video/i810/i810_accel.c

index 76764ea3486a7d31a77e0887548ea17696819672..f5bedee4310a9ac8a3f50abb6852f924ae23c71f 100644 (file)
@@ -301,8 +301,10 @@ void i810fb_fillrect(struct fb_info *info, const struct fb_fillrect *rect)
        u32 dx, dy, width, height, dest, rop = 0, color = 0;
 
        if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
        u32 dx, dy, width, height, dest, rop = 0, color = 0;
 
        if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
-           par->depth == 4) 
-               return cfb_fillrect(info, rect);
+           par->depth == 4) {
+               cfb_fillrect(info, rect);
+               return;
+       }
 
        if (par->depth == 1) 
                color = rect->color;
 
        if (par->depth == 1) 
                color = rect->color;
@@ -327,8 +329,10 @@ void i810fb_copyarea(struct fb_info *info, const struct fb_copyarea *region)
        u32 sx, sy, dx, dy, pitch, width, height, src, dest, xdir;
 
        if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
        u32 sx, sy, dx, dy, pitch, width, height, src, dest, xdir;
 
        if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
-           par->depth == 4)
-               return cfb_copyarea(info, region);
+           par->depth == 4) {
+               cfb_copyarea(info, region);
+               return;
+       }
 
        dx = region->dx * par->depth;
        sx = region->sx * par->depth;
 
        dx = region->dx * par->depth;
        sx = region->sx * par->depth;
@@ -366,8 +370,10 @@ void i810fb_imageblit(struct fb_info *info, const struct fb_image *image)
        u32 fg = 0, bg = 0, size, dst;
        
        if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
        u32 fg = 0, bg = 0, size, dst;
        
        if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
-           par->depth == 4 || image->depth != 1) 
-               return cfb_imageblit(info, image);
+           par->depth == 4 || image->depth != 1) {
+               cfb_imageblit(info, image);
+               return;
+       }
 
        switch (info->var.bits_per_pixel) {
        case 8:
 
        switch (info->var.bits_per_pixel) {
        case 8: