summaryrefslogtreecommitdiffstats
path: root/apps
diff options
context:
space:
mode:
authorJens Arnold <amiconn@rockbox.org>2008-01-06 23:48:30 +0000
committerJens Arnold <amiconn@rockbox.org>2008-01-06 23:48:30 +0000
commitdbe24fcbacef8003704e736bedb6d64468e89899 (patch)
tree6477eb41c667c966d484191141818e2ed8374d08 /apps
parent43bc13e4b28497443664cd7117e8f96ba8795079 (diff)
downloadrockbox-dbe24fcbacef8003704e736bedb6d64468e89899.tar.gz
rockbox-dbe24fcbacef8003704e736bedb6d64468e89899.zip
Doom LCD update: * Speed up the C versions slightly (colour and greyscale). * Fix the ASM version for H300 (caused line flicker, probably since buffered LCD writes were enabled). Slightly slower, but still ~8% faster than plain C. Removed the C version of this ASM because of the same problem (was only there for reference). * Added ASM optimised version for X5 (~8.5% faster).
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@16009 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'apps')
-rw-r--r--apps/plugins/doom/i_video.c204
1 files changed, 127 insertions, 77 deletions
diff --git a/apps/plugins/doom/i_video.c b/apps/plugins/doom/i_video.c
index fe2f91b699..16024d8aee 100644
--- a/apps/plugins/doom/i_video.c
+++ b/apps/plugins/doom/i_video.c
@@ -495,102 +495,152 @@ static void I_UploadNewPalette(int pal)
void I_FinishUpdate (void)
{
+ int count;
+ byte *src = d_screens[0];
#if (CONFIG_LCD == LCD_H300) && !defined(SIMULATOR)
-
-#if 1
- /* ASM screen update (drops 200 tics (100 asm)) */
- asm (
- "move.w #33,(%[LCD]) \n" /* Setup the LCD controller */
- "clr.w (%[LCD2]) \n"
- "move.w #34,(%[LCD]) \n" /* End LCD controller setup */
- "clr.l %%d1 \n"
- "clr.l %%d0 \n"
- "widthloop: \n"
- "move.l (%[screenptr])+, %%d0 \n"
- "swap.w %%d0 \n"
- "move.w %%d0, %%d1 \n"
- "lsr.l #8,%%d1 \n"
- "move.w (%[palette], %%d1.l:2), (%[LCD2]) \n"
- "move.b %%d0,%%d1 \n"
- "move.w (%[palette], %%d1.l:2), (%[LCD2]) \n"
- "swap.w %%d0 \n"
- "move.w %%d0, %%d1 \n"
- "lsr.l #8,%%d1 \n"
- "move.w (%[palette], %%d1.l:2), (%[LCD2]) \n"
- "move.b %%d0,%%d1 \n"
- "move.w (%[palette], %%d1.l:2), (%[LCD2]) \n"
- "subq.l #4,%[WIDTH] \n"
- "bne widthloop \n"
- "move.w #220,%[WIDTH] \n"
- "subq.l #1,%[HEIGHT] \n"
- "bne widthloop \n"
- : /* outputs */
- : /* inputs */
- [screenptr] "a" (d_screens[0]),
- [palette] "a" (palette),
- [LCD] "a" (0xf0000000),
- [LCD2] "a" (0xf0000002),
- [WIDTH] "d" (220),
- [HEIGHT] "d" (176)
- : /* clobbers */
- "d0", "d1"
+ count = SCREENWIDTH*SCREENHEIGHT;
+
+ /* ASM screen update (drops ~300 tics) */
+ asm volatile (
+ "move.w #33, (%[LCD]) \n" /* Setup the LCD controller */
+ "nop \n"
+ "clr.w (%[LCD2]) \n"
+ "nop \n"
+ "move.w #34, (%[LCD]) \n" /* End LCD controller setup */
+ "clr.l %%d1 \n"
+ ".loop: \n"
+ "move.l (%[scrp])+, %%d0 \n"
+ "swap.w %%d0 \n"
+ "move.w %%d0, %%d1 \n"
+ "lsr.l #8,%%d1 \n"
+ "move.w (%[pal], %%d1.l:2), (%[LCD2]) \n"
+ "move.b %%d0,%%d1 \n"
+ "swap.w %%d0 \n"
+ "nop \n"
+ "move.w (%[pal], %%d1.l:2), (%[LCD2]) \n"
+ "move.w %%d0, %%d1 \n"
+ "lsr.l #8,%%d1 \n"
+ "nop \n"
+ "move.w (%[pal], %%d1.l:2), (%[LCD2]) \n"
+ "move.b %%d0,%%d1 \n"
+ "nop \n"
+ "move.w (%[pal], %%d1.l:2), (%[LCD2]) \n"
+ "subq.l #4,%[cnt] \n"
+ "bne.b .loop \n"
+ : /* outputs */
+ [scrp]"+a"(src),
+ [cnt] "+d"(count)
+ : /* inputs */
+ [pal] "a" (palette),
+ [LCD] "a" (0xf0000000),
+ [LCD2]"a" (0xf0000002)
+ : /* clobbers */
+ "d0", "d1"
+ );
+#elif (CONFIG_LCD == LCD_X5) && !defined(SIMULATOR)
+ count = SCREENWIDTH*SCREENHEIGHT;
+
+ /* ASM screen update (drops ~230 tics) */
+ asm volatile (
+ "clr.w (%[LCD]) \n" /* Setup the LCD controller */
+ "move.w #(33<<1), (%[LCD]) \n"
+ "clr.w (%[LCD2]) \n"
+ "clr.w (%[LCD2]) \n"
+ "clr.w (%[LCD]) \n" /* End LCD controller setup */
+ "move.w #(34<<1), (%[LCD]) \n"
+ "clr.l %%d1 \n"
+ ".loop: \n"
+ "move.l (%[scrp])+, %%d0 \n"
+ "swap.w %%d0 \n"
+ "move.w %%d0, %%d1 \n"
+ "lsr.l #8,%%d1 \n"
+ "move.w (%[pal], %%d1.l:2), %%d2 \n"
+ "move.l %%d2, %%d3 \n"
+ "lsr.l #7, %%d3 \n"
+ "move.w %%d3, (%[LCD2]) \n"
+ "lsl.l #1, %%d2 \n"
+ "move.w %%d2, (%[LCD2]) \n"
+ "move.b %%d0,%%d1 \n"
+ "move.w (%[pal], %%d1.l:2), %%d2 \n"
+ "move.l %%d2, %%d3 \n"
+ "lsr.l #7, %%d3 \n"
+ "move.w %%d3, (%[LCD2]) \n"
+ "lsl.l #1, %%d2 \n"
+ "move.w %%d2, (%[LCD2]) \n"
+ "swap.w %%d0 \n"
+ "move.w %%d0, %%d1 \n"
+ "lsr.l #8,%%d1 \n"
+ "move.w (%[pal], %%d1.l:2), %%d2 \n"
+ "move.l %%d2, %%d3 \n"
+ "lsr.l #7, %%d3 \n"
+ "move.w %%d3, (%[LCD2]) \n"
+ "lsl.l #1, %%d2 \n"
+ "move.w %%d2, (%[LCD2]) \n"
+ "move.b %%d0,%%d1 \n"
+ "move.w (%[pal], %%d1.l:2), %%d2 \n"
+ "move.l %%d2, %%d3 \n"
+ "lsr.l #7, %%d3 \n"
+ "move.w %%d3, (%[LCD2]) \n"
+ "lsl.l #1, %%d2 \n"
+ "move.w %%d2, (%[LCD2]) \n"
+ "subq.l #4,%[cnt] \n"
+ "bne.b .loop \n"
+ : /* outputs */
+ [scrp]"+a"(src),
+ [cnt] "+d"(count)
+ : /* inputs */
+ [pal] "a" (palette),
+ [LCD] "a" (0xf0008000),
+ [LCD2]"a" (0xf0008002)
+ : /* clobbers */
+ "d0", "d1", "d2", "d3"
);
#else
- /* C version of above (drops 100 tics) */
-
- // Start the write
- *(volatile unsigned short *) 0xf0000000 = 0x21; // register
- *(volatile unsigned short *) 0xf0000002 = 0; // value
- *(volatile unsigned short *) 0xf0000000 = 0x22; // GRAM
-
- unsigned char *screenptr=d_screens[0];
- int hcnt=LCD_HEIGHT*LCD_WIDTH;
-
- while(hcnt--)
- {
- *(volatile unsigned short *)0xf0000002 = palette[*screenptr];
- screenptr++;
- }
-#endif
-
-#else
- unsigned char paletteIndex;
- int y;
-
#ifdef HAVE_LCD_COLOR
#if(LCD_HEIGHT>LCD_WIDTH)
if(rotate_screen)
{
- int x;
- for (y=0; y<LCD_HEIGHT; y++)
+ int y;
+
+ for (y = 0; y < SCREENHEIGHT; y++)
{
- for (x=0; x < LCD_WIDTH; x++)
+ fb_data *dst = rb->lcd_framebuffer + LCD_WIDTH - y;
+ count = SCREENWIDTH;
+
+ do
{
- paletteIndex = d_screens[0][SCREENWIDTH*(SCREENHEIGHT-1-x) + y];
- rb->lcd_framebuffer[y*LCD_WIDTH + x] = palette[paletteIndex];
+ *dst = palette[*src++];
+ dst += LCD_WIDTH;
}
+ while (--count);
}
}
else
#endif
- for (y = 0; y < LCD_HEIGHT *LCD_WIDTH; y++)
{
- paletteIndex = d_screens[0][y];
- rb->lcd_framebuffer[y] = palette[paletteIndex];
+ fb_data *dst = rb->lcd_framebuffer;
+ count = SCREENWIDTH*SCREENHEIGHT;
+
+ do
+ *dst++ = palette[*src++];
+ while (--count);
}
rb->lcd_update();
#else /* !HAVE_LCD_COLOR */
- int x;
+ unsigned char *dst;
+ int y;
- for (y = 0; y < SCREENHEIGHT; y++)
- {
- for (x = 0; x < SCREENWIDTH; x++)
- {
- paletteIndex = d_screens[0][y*SCREENWIDTH + x];
- greybuffer[x]=palette[paletteIndex];
- }
- grey_ub_gray_bitmap(greybuffer, 0, y, SCREENWIDTH, 1);
- }
+ for (y = 0; y < SCREENHEIGHT; y++)
+ {
+ dst = greybuffer;
+ count = SCREENWIDTH;
+
+ do
+ *dst++ = palette[*src++];
+ while (--count);
+
+ grey_ub_gray_bitmap(greybuffer, 0, y, SCREENWIDTH, 1);
+ }
#endif /* !HAVE_LCD_COLOR */
#endif
}