Variable ar assigned a value that is never used. I have also removed all the code that thereby serves no purpose. This was found using a static code analysis program called cppcheck Signed-off-by: Rickard Strandqvist <rickard_strandqvist@xxxxxxxxxxxxxxxxxx> --- drivers/video/fbdev/mb862xx/mb862xxfb_accel.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c b/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c index fe92eed..aa778ba 100644 --- a/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c +++ b/drivers/video/fbdev/mb862xx/mb862xxfb_accel.c @@ -28,7 +28,6 @@ static void mb862xxfb_write_fifo(u32 count, u32 *data, struct fb_info *info) { - struct mb862xxfb_par *par = info->par; static u32 free; u32 total = 0; @@ -188,7 +187,6 @@ static void mb86290fb_imageblit16(u32 *cmd, u16 step, u16 dx, u16 dy, static void mb86290fb_imageblit(struct fb_info *info, const struct fb_image *image) { - int mdr; u32 *cmd = NULL; void (*cmdfn) (u32 *, u16, u16, u16, u16, u16, u32, u32, const struct fb_image *, struct fb_info *) = NULL; @@ -200,7 +198,6 @@ static void mb86290fb_imageblit(struct fb_info *info, u16 dx = image->dx, dy = image->dy; int x2, y2, vxres, vyres; - mdr = (GDC_ROP_COPY << 9); x2 = image->dx + image->width; y2 = image->dy + image->height; vxres = info->var.xres_virtual; @@ -309,8 +306,6 @@ static void mb86290fb_fillrect(struct fb_info *info, void mb862xxfb_init_accel(struct fb_info *info, int xres) { - struct mb862xxfb_par *par = info->par; - if (info->var.bits_per_pixel == 32) { info->fbops->fb_fillrect = cfb_fillrect; info->fbops->fb_copyarea = cfb_copyarea; -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html