[PATCH] fbdev: Remove software clipping from drawing functions

Remove software clipping from imageblit, fillrect and copyarea.  Clipping is
not needed because the console layer assures that reads/writes doest not
happen beyond the extents of the framebuffer.  And software clipping tends to
hide bugs, if they do exist.

Signed-off-by: Antonino Daplas <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 1 insertions, 21 deletions

diff --git a/drivers/video/cfbimgblt.c b/drivers/video/cfbimgblt.c
index 4c123abaa84..da664cea7ec 100644
--- a/drivers/video/cfbimgblt.c
+++ b/drivers/video/cfbimgblt.c
@@ -272,33 +272,13 @@ void cfb_imageblit(struct fb_info *p, const struct fb_image *image)
 {
         u32 fgcolor, bgcolor, start_index, bitstart, pitch_index = 0;
         u32 bpl = sizeof(u32), bpp = p->var.bits_per_pixel;
-        u32 width = image->width, height = image->height; 
+        u32 width = image->width;
         u32 dx = image->dx, dy = image->dy;
-        int x2, y2, vxres, vyres;
         u8 __iomem *dst1;
 
         if (p->state != FBINFO_STATE_RUNNING)
                 return;
 
-        vxres = p->var.xres_virtual;
-        vyres = p->var.yres_virtual;
-        /* 
-         * We could use hardware clipping but on many cards you get around
-         * hardware clipping by writing to framebuffer directly like we are
-         * doing here. 
-         */
-        if (image->dx > vxres || image->dy > vyres)
-                return;
-
-        x2 = image->dx + image->width;
-        y2 = image->dy + image->height;
-        dx = image->dx > 0 ? image->dx : 0;
-        dy = image->dy > 0 ? image->dy : 0;
-        x2 = x2 < vxres ? x2 : vxres;
-        y2 = y2 < vyres ? y2 : vyres;
-        width  = x2 - dx;
-        height = y2 - dy;
-
         bitstart = (dy * p->fix.line_length * 8) + (dx * bpp);
         start_index = bitstart & (32 - 1);
         pitch_index = (p->fix.line_length & (bpl - 1)) * 8;