lkml.org 
[lkml]   [2003]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] Fix for image-blitting in tdfxfb driver in 2.6.0-test1
    Date
    The following patch fixes some bugs in the tdfxfb driver in 2.6.0-test1. 
    Image-blitting in software or hardware should now work. Please apply.

    This patch fixes

    1) Problems with the driver's pseudo-palette in truecolor modes. Palette
    entries were being mangled, so the foreground and background colours of text
    written to the console was incorrect (this affected image-blitting of text
    whether either the software routine cfb_imageblit or the accelerated
    tdfxfb_imageblit was used).

    2) Critical bug in tdfxfb_imageblit. Large blits would try and request more
    fifo entries from the hardware than are available, causing an infinite loop.
    I've fixed this by requesting fifo entries in small chunks rather than trying
    to do it all at once.

    3) Various big-endian fixes.

    I've also moved the tdfxfb entry in fbmem.c to the 'with resource management'
    section, because it now has it.

    Cheers,
    Rich

    --- drivers/video/tdfxfb.c.orig 2003-07-20 02:35:33.000000000 -0500
    +++ drivers/video/tdfxfb.c 2003-07-21 00:44:13.000000000 -0500
    @@ -1,3 +1,4 @@
    +
    /*
    *
    * tdfxfb.c
    @@ -177,7 +178,7 @@
    .fb_pan_display = tdfxfb_pan_display,
    .fb_fillrect = tdfxfb_fillrect,
    .fb_copyarea = tdfxfb_copyarea,
    - .fb_imageblit = cfb_imageblit,
    + .fb_imageblit = tdfxfb_imageblit,
    .fb_sync = banshee_wait_idle,
    .fb_cursor = soft_cursor,
    };
    @@ -316,7 +317,9 @@

    static inline void banshee_make_room(struct tdfx_par *par, int size)
    {
    - while((tdfx_inl(par, STATUS) & 0x1f) < size);
    + /* Note: The Voodoo3's onboard FIFO has 32 slots. This loop
    + * won't quit if you ask for more. */
    + while((tdfx_inl(par, STATUS) & 0x1f) < size-1);
    }

    static int banshee_wait_idle(struct fb_info *info)
    @@ -746,6 +749,7 @@
    #if defined(__BIG_ENDIAN)
    switch (info->var.bits_per_pixel) {
    case 8:
    + case 24:
    reg.miscinit0 &= ~(1 << 30);
    reg.miscinit0 &= ~(1 << 31);
    break;
    @@ -753,7 +757,6 @@
    reg.miscinit0 |= (1 << 30);
    reg.miscinit0 |= (1 << 31);
    break;
    - case 24:
    case 32:
    reg.miscinit0 |= (1 << 30);
    reg.miscinit0 &= ~(1 << 31);
    @@ -771,6 +774,9 @@
    return 0;
    }

    +/* A handy macro shamelessly pinched from matroxfb */
    +#define CNVT_TOHW(val,width) ((((val)<<(width))+0x7FFF-(val))>>16)
    +
    static int tdfxfb_setcolreg(unsigned regno, unsigned red, unsigned green,
    unsigned blue,unsigned transp,struct fb_info *info)
    {
    @@ -788,13 +794,10 @@
    break;
    /* Truecolor has no hardware color palettes. */
    case FB_VISUAL_TRUECOLOR:
    - rgbcol = (red << info->var.red.offset) |
    - (green << info->var.green.offset) |
    - (blue << info->var.blue.offset) |
    - (transp << info->var.transp.offset);
    - if (info->var.bits_per_pixel <= 16)
    - ((u16*)(info->pseudo_palette))[regno] = rgbcol;
    - else
    + rgbcol = (CNVT_TOHW( red, info->var.red.length) << info->var.red.offset) |
    + (CNVT_TOHW( green, info->var.green.length) << info->var.green.offset) |
    + (CNVT_TOHW( blue, info->var.blue.length) << info->var.blue.offset) |
    + (CNVT_TOHW( transp, info->var.transp.length) << info->var.transp.offset);
    ((u32*)(info->pseudo_palette))[regno] = rgbcol;
    break;
    default:
    @@ -934,6 +937,7 @@
    {
    struct tdfx_par *par = (struct tdfx_par *) info->par;
    int size = image->height * ((image->width * image->depth + 7)>>3);
    + int fifo_free;
    int i, stride = info->fix.line_length;
    u32 bpp = info->var.bits_per_pixel;
    u32 dstfmt = stride | ((bpp+((bpp==8) ? 0 : 8)) << 13);
    @@ -946,10 +950,22 @@
    cfb_imageblit(info, image);
    return;
    } else {
    - banshee_make_room(par, 8 + ((size + 3) >> 2));
    + banshee_make_room(par, 8);
    + switch (info->fix.visual) {
    + case FB_VISUAL_PSEUDOCOLOR:
    tdfx_outl(par, COLORFORE, image->fg_color);
    tdfx_outl(par, COLORBACK, image->bg_color);
    + break;
    + case FB_VISUAL_TRUECOLOR:
    + default:
    + tdfx_outl(par, COLORFORE, ((u32*)(info->pseudo_palette))[image->fg_color]);
    + tdfx_outl(par, COLORBACK, ((u32*)(info->pseudo_palette))[image->bg_color]);
    + }
    +#ifdef __BIG_ENDIAN
    + srcfmt = 0x400000 | BIT(20);
    +#else
    srcfmt = 0x400000;
    +#endif
    }

    tdfx_outl(par, SRCXY, 0);
    @@ -959,13 +975,22 @@
    tdfx_outl(par, DSTFORMAT, dstfmt);
    tdfx_outl(par, DSTSIZE, image->width | (image->height << 16));

    + /* A count of how many free FIFO entries we've requested.
    + * When this goes negative, we need to request more. */
    + fifo_free = 0;
    +
    /* Send four bytes at a time of data */
    for (i = (size >> 2) ; i > 0; i--) {
    + if(--fifo_free < 0) {
    + fifo_free=31;
    + banshee_make_room(par,fifo_free);
    + }
    tdfx_outl(par, LAUNCH_2D,*(u32*)chardata);
    chardata += 4;
    }

    /* Send the leftovers now */
    + banshee_make_room(par,3);
    i = size%4;
    switch (i) {
    case 0: break;
    --- drivers/video/fbmem.c.orig 2003-05-26 23:06:02.000000000 -0500
    +++ drivers/video/fbmem.c 2003-07-21 01:01:32.000000000 -0500
    @@ -214,6 +214,9 @@
    #ifdef CONFIG_FB_RIVA
    { "rivafb", rivafb_init, rivafb_setup },
    #endif
    +#ifdef CONFIG_FB_3DFX
    + { "tdfxfb", tdfxfb_init, tdfxfb_setup },
    +#endif
    #ifdef CONFIG_FB_RADEON
    { "radeonfb", radeonfb_init, radeonfb_setup },
    #endif
    @@ -294,9 +297,6 @@
    * Chipset specific drivers that don't use resource management (yet)
    */

    -#ifdef CONFIG_FB_3DFX
    - { "tdfxfb", tdfxfb_init, tdfxfb_setup },
    -#endif
    #ifdef CONFIG_FB_SGIVW
    { "sgivwfb", sgivwfb_init, sgivwfb_setup },
    #endif
    \
     
     \ /
      Last update: 2005-03-22 13:47    [W:0.032 / U:0.796 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site