Add second buffer for YUV blitting, progressbar and FPS display in mpegplayer work correct now.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@20766 a1c6a512-1295-4272-9138-f99709370657

4 files changed, 96 insertions, 81 deletions

diff --git a/firmware/export/dm320.h b/firmware/export/dm320.h
index d6599a6..a1bbc3c 100644
--- a/firmware/export/dm320.h
+++ b/firmware/export/dm320.h
@@ -30,11 +30,13 @@
 #if !defined(__ASSEMBLER__) && !defined(__LD__)
 /* These variables are created during linking (app/boot.lds) */
 extern unsigned long _lcdbuf;
+extern unsigned long _lcdbuf2;
 extern unsigned long _ttbstart;
 #endif
 
 #define TTB_BASE_ADDR    (_ttbstart) /* End of memory */
-#define FRAME            ((short *) (&_lcdbuf))  /* Right before TTB */
+#define FRAME            ((short *) (&_lcdbuf))  /* Right after TTB */
+#define FRAME2            ((short *) (&_lcdbuf2))  /* Right after FRAME */
 
 #define PHY_IO_BASE      0x00030000
 #define DM320_REG(addr)  (*(volatile unsigned short *)(PHY_IO_BASE + (addr)))
@@ -651,7 +653,7 @@ extern unsigned long _ttbstart;
 #define IO_MEM_STICK_DMA_STATUS   DM320_REG(0x0C96)
 
 /* ATM : WBB Need to find these Register values */
-#define IO_ATM_                   DM320_REG(0x0D00
+#define IO_ATM_                   DM320_REG(0x0D00)
 
 /* I2C */
 #define IO_I2C_TXDATA             DM320_REG(0x0D80)
diff --git a/firmware/target/arm/tms320dm320/app.lds b/firmware/target/arm/tms320dm320/app.lds
index 635d9a8..14b4415 100644
--- a/firmware/target/arm/tms320dm320/app.lds
+++ b/firmware/target/arm/tms320dm320/app.lds
@@ -27,8 +27,11 @@ STARTUP(target/arm/tms320dm320/crt0.o)
 /* must be 16Kb (0x4000) aligned */
 #define TTB_SIZE        0x4000
 
-/* Give this 1 meg to allow it to align to the MMU boundary */
-#define LCD_TTB_AREA    0x100000
+/* Give this some memory to allow it to align to the MMU boundary.
+ * Note that since there are two buffers (YUV/RGB) it calculates the approximate
+ * memory needed in steps of 1 Meg.
+ */
+#define LCD_TTB_AREA    0x100000*((LCD_BUFFER_SIZE>>19)+1)
 
 #define DRAMSIZE (MEMORYSIZE * 0x100000) - STUBOFFSET
 
@@ -174,5 +177,11 @@ SECTIONS
         _lcdbuf = .;
         . += LCD_BUFFER_SIZE;
     } > DRAM
+    
+    .lcdbuffer2 (NOLOAD) :
+    {
+        _lcdbuf2 = .;
+        . += LCD_BUFFER_SIZE;
+    } > DRAM
 }
 
diff --git a/firmware/target/arm/tms320dm320/mrobe-500/button-mr500.c b/firmware/target/arm/tms320dm320/mrobe-500/button-mr500.c
index a2729ad..d64a9cd 100644
--- a/firmware/target/arm/tms320dm320/mrobe-500/button-mr500.c
+++ b/firmware/target/arm/tms320dm320/mrobe-500/button-mr500.c
@@ -28,7 +28,6 @@
 #include "kernel.h"
 #include "backlight.h"
 #include "adc.h"
-#include "system.h"
 #include "backlight-target.h"
 #include "lcd-remote-target.h"
 #include "uart-target.h"
diff --git a/firmware/target/arm/tms320dm320/mrobe-500/lcd-mr500.c b/firmware/target/arm/tms320dm320/mrobe-500/lcd-mr500.c
index ec85821..e26a887 100644
--- a/firmware/target/arm/tms320dm320/mrobe-500/lcd-mr500.c
+++ b/firmware/target/arm/tms320dm320/mrobe-500/lcd-mr500.c
@@ -209,6 +209,8 @@ void lcd_init_device(void)
     
     IO_OSD_VIDWINMD=0x0002;
     
+    addr = ((int)FRAME2-CONFIG_SDRAM_START) / 32;
+    
     /* This is a bit messy, the LCD transfers appear to happen in chunks of 32
      * pixels. (based on OF)
      */
@@ -242,6 +244,40 @@ void lcd_init_device(void)
 	IO_VID_ENC_VMOD|=0x01;
 }
 
+#if defined(HAVE_LCD_MODES)
+void lcd_set_mode(int mode)
+{
+	if(mode==LCD_MODE_YUV)
+	{
+		/* Turn off the RGB buffer and enable the YUV buffer */
+		IO_OSD_OSDWINMD0 |=0x04;
+		IO_OSD_VIDWINMD  |=0x01;
+		memset16(FRAME2, 0x0080, LCD_NATIVE_HEIGHT*(LCD_NATIVE_WIDTH+LCD_FUDGE));
+	}
+	else if(mode==LCD_MODE_RGB565)
+	{
+		/* Turn on the RGB window, set it to 16 bit and turn YUV window off */
+		IO_OSD_VIDWINMD  &=~(0x01);
+		IO_OSD_OSDWIN0OFST=LCD_NATIVE_WIDTH / 16;
+    	IO_OSD_OSDWINMD0 |=(1<<13);
+    	IO_OSD_OSDWINMD0 &=~0x04;
+    	lcd_clear_display();
+	}
+	else if(mode==LCD_MODE_PAL256)
+	{
+#if LCD_NATIVE_WIDTH%32!=0
+        IO_OSD_OSDWIN0OFST=LCD_NATIVE_WIDTH / 32+1;
+#else
+        IO_OSD_OSDWIN0OFST=LCD_NATIVE_WIDTH / 32;
+#endif
+
+        IO_OSD_VIDWINMD  &=~(0x01);
+        IO_OSD_OSDWINMD0 &=~(1<<13);
+		IO_OSD_OSDWINMD0 |=0x01;
+	}
+}
+#endif
+
 /* Update a fraction of the display. */
 void lcd_update_rect(int x, int y, int width, int height)
 {
@@ -317,39 +353,6 @@ void lcd_update(void)
 #endif
 }
 
-#if defined(HAVE_LCD_MODES)
-void lcd_set_mode(int mode)
-{
-	if(mode==LCD_MODE_YUV)
-	{
-		/* Turn off the RGB buffer and enable the YUV buffer */
-		IO_OSD_OSDWINMD0 &=~(0x01);
-		IO_OSD_VIDWINMD  |=0x01;
-		memset16(FRAME, 0x0080, LCD_NATIVE_HEIGHT*(LCD_NATIVE_WIDTH+LCD_FUDGE));
-	}
-	else if(mode==LCD_MODE_RGB565)
-	{
-		/* Turn on the RGB window, set it to 16 bit and turn YUV window off */
-		IO_OSD_VIDWINMD  &=~(0x01);
-		IO_OSD_OSDWIN0OFST=LCD_NATIVE_WIDTH / 16;
-    	IO_OSD_OSDWINMD0 |=(1<<13)|0x01;
-    	lcd_clear_display();
-	}
-	else if(mode==LCD_MODE_PAL256)
-	{
-#if LCD_NATIVE_WIDTH%32!=0
-        IO_OSD_OSDWIN0OFST=LCD_NATIVE_WIDTH / 32+1;
-#else
-        IO_OSD_OSDWIN0OFST=LCD_NATIVE_WIDTH / 32;
-#endif
-
-        IO_OSD_VIDWINMD  &=~(0x01);
-        IO_OSD_OSDWINMD0 &=~(1<<13);
-		IO_OSD_OSDWINMD0 |=0x01;
-	}
-}
-#endif
-
 #if defined(HAVE_LCD_MODES) && (HAVE_LCD_MODES & LCD_MODE_PAL256)
 void lcd_blit_pal256(unsigned char *src, int src_x, int src_y, int x, int y,
 	int width, int height)
@@ -362,7 +365,7 @@ void lcd_blit_pal256(unsigned char *src, int src_x, int src_y, int x, int y,
 	{
 		memcpy ( dst, src, width);
 		
-		dst=dst+width+(LCD_NATIVE_WIDTH-x-width)+LCD_FUDGE; 
+		dst=dst+(LCD_NATIVE_WIDTH-x+LCD_FUDGE); 
 		src+=width;
 	}
 	
@@ -425,10 +428,7 @@ void lcd_blit_yuv(unsigned char * const src[3],
                   int src_x, int src_y, int stride,
                   int x, int y, int width, int height)
 {
-    /* Caches for chroma data so it only need be recaculated every other
-       line */
-    unsigned char const * yuv_src[3];
-    off_t z;
+    register unsigned char const * yuv_src[3];
 
     if (!lcd_on)
         return;
@@ -455,50 +455,55 @@ void lcd_blit_yuv(unsigned char * const src[3],
     width &= ~1;
     height>>=1;
 
-    fb_data * dst = FRAME + ((LCD_NATIVE_WIDTH+LCD_FUDGE)*(LCD_NATIVE_HEIGHT-1)) 
+    fb_data * dst = FRAME2 
+        + ((LCD_NATIVE_WIDTH+LCD_FUDGE)*(LCD_NATIVE_HEIGHT-1)) 
         - (LCD_NATIVE_WIDTH+LCD_FUDGE)*x + y ;
 
-    z = stride*src_y;
-    yuv_src[0] = src[0] + z + src_x;
-    yuv_src[1] = src[1] + (z >> 2) + (src_x >> 1);
-    yuv_src[2] = src[2] + (yuv_src[1] - src[1]);
+    /* Scope z */
+    {
+        off_t z;
+        z = stride*src_y;
+        yuv_src[0] = src[0] + z + src_x;
+        yuv_src[1] = src[1] + (z >> 2) + (src_x >> 1);
+        yuv_src[2] = src[2] + (yuv_src[1] - src[1]);
+    }
 
+    register int cbcr_remain=(stride>>1)-(width>>1);
+    register int y_remain=(stride<<1)-width;
+    do
     {
-        do
-        {
-        	register fb_data *c_dst=dst;
-        	register int c_width=width;
-        	unsigned char const * c_yuv_src[3];
-        	c_yuv_src[0] = yuv_src[0];
-        	c_yuv_src[1] = yuv_src[1];
-        	c_yuv_src[2] = yuv_src[2];
-
-        	do
-        	{
-        	/* This needs to be done in a block of 4 pixels */
-        		*c_dst=*c_yuv_src[0]<<8 | *c_yuv_src[1];
-        		*(c_dst+1)=*(c_yuv_src[0]+stride)<<8 | *c_yuv_src[2];
-        		c_dst-=(LCD_NATIVE_WIDTH+LCD_FUDGE);
-        		c_yuv_src[0]++;
-        		*c_dst=*c_yuv_src[0]<<8 | *c_yuv_src[1];
-        		*(c_dst+1)=*(c_yuv_src[0]+stride)<<8 | *c_yuv_src[2];
-        		c_dst-=(LCD_NATIVE_WIDTH+LCD_FUDGE);
-	            c_yuv_src[0]++;
-	            
-            	c_yuv_src[1]++;
-            	c_yuv_src[2]++;
-            	
-        		c_width -= 2;
-    		}
-    		while (c_width > 0);
+    	register fb_data *c_dst=dst;
+    	register int c_width=width;
+
+    	do
+    	{
+    	    /* This needs to be done in a block of 4 pixels */
+    	    
+    		*c_dst=*yuv_src[0]<<8 | *yuv_src[1];
+    		*(c_dst+1)=*(yuv_src[0]+stride)<<8 | *yuv_src[2];
+    		c_dst-=(LCD_NATIVE_WIDTH+LCD_FUDGE);
     		
-    		yuv_src[0] += stride << 1; /* Skip down two luma lines */
-            yuv_src[1] += stride >> 1; /* Skip down one chroma line */
-            yuv_src[2] += stride >> 1;
-            dst+=2;
-        }
-        while (--height > 0);
+    		yuv_src[0]++;
+    		
+    		*c_dst=*yuv_src[0]<<8 | *yuv_src[1];
+    		*(c_dst+1)=*(yuv_src[0]+stride)<<8 | *yuv_src[2];
+    		c_dst-=(LCD_NATIVE_WIDTH+LCD_FUDGE);
+    		
+            yuv_src[0]++;
+            
+        	yuv_src[1]++;
+        	yuv_src[2]++;
+        	
+    		c_width -= 2;
+		}
+		while (c_width > 0);
+		
+		yuv_src[0] += y_remain; /* Skip down two luma lines-width */
+        yuv_src[1] += cbcr_remain; /* Skip down one chroma line-width/2 */
+        yuv_src[2] += cbcr_remain;
+        dst+=2;
     }
+    while (--height > 0);
 }
 
 void lcd_set_contrast(int val) {