i810: fix sparse warnings
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>
This commit is contained in:
parent
27ec937c4f
commit
4fa34cd051
@ -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;
|
u32 dx, dy, width, height, dest, rop = 0, color = 0;
|
||||||
|
|
||||||
if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
|
if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
|
||||||
par->depth == 4)
|
par->depth == 4) {
|
||||||
return cfb_fillrect(info, rect);
|
cfb_fillrect(info, rect);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
if (par->depth == 1)
|
if (par->depth == 1)
|
||||||
color = rect->color;
|
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;
|
u32 sx, sy, dx, dy, pitch, width, height, src, dest, xdir;
|
||||||
|
|
||||||
if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
|
if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
|
||||||
par->depth == 4)
|
par->depth == 4) {
|
||||||
return cfb_copyarea(info, region);
|
cfb_copyarea(info, region);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
dx = region->dx * par->depth;
|
dx = region->dx * par->depth;
|
||||||
sx = region->sx * 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;
|
u32 fg = 0, bg = 0, size, dst;
|
||||||
|
|
||||||
if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
|
if (!info->var.accel_flags || par->dev_flags & LOCKUP ||
|
||||||
par->depth == 4 || image->depth != 1)
|
par->depth == 4 || image->depth != 1) {
|
||||||
return cfb_imageblit(info, image);
|
cfb_imageblit(info, image);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
switch (info->var.bits_per_pixel) {
|
switch (info->var.bits_per_pixel) {
|
||||||
case 8:
|
case 8:
|
||||||
|
Loading…
Reference in New Issue
Block a user