From b37e6bc8c119289aca8740dd5e3b60d72f9d6b40 Mon Sep 17 00:00:00 2001 From: Jonathan Gordon Date: Wed, 22 Feb 2012 21:18:05 +1100 Subject: lcd drivers: Convert lcd_[remote_]framebuffer to a pointer Change all lcd drivers to using a pointer to the static framebuffer instead of directly accessing the static array. This will let us later do fun things like dynamic framebuffer sizes (RaaA) or ability to use different buffers for different layers (dynamic skin backdrops!) Change-Id: I0a4d58a9d7b55e6c932131b929e5d4c9f9414b06 --- firmware/drivers/lcd-16bit-common.c | 18 +++++++++--------- firmware/drivers/lcd-16bit-vert.c | 15 ++++++--------- firmware/drivers/lcd-16bit.c | 15 ++++++--------- firmware/drivers/lcd-1bit-vert.c | 22 ++++++++++++---------- firmware/drivers/lcd-2bit-horz.c | 27 ++++++++++++++------------- firmware/drivers/lcd-2bit-vert.c | 27 ++++++++++++++------------- firmware/drivers/lcd-2bit-vi.c | 30 +++++++++++++++++------------- firmware/drivers/lcd-memframe.c | 4 ++-- firmware/drivers/lcd-remote-1bit-v.c | 2 ++ firmware/drivers/lcd-remote-2bit-vi.c | 2 ++ 10 files changed, 84 insertions(+), 78 deletions(-) (limited to 'firmware/drivers') diff --git a/firmware/drivers/lcd-16bit-common.c b/firmware/drivers/lcd-16bit-common.c index 1a845675ad..c9e88b2137 100644 --- a/firmware/drivers/lcd-16bit-common.c +++ b/firmware/drivers/lcd-16bit-common.c @@ -35,9 +35,9 @@ enum fill_opt { }; /*** globals ***/ -fb_data lcd_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] +fb_data lcd_static_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] IRAM_LCDFRAMEBUFFER CACHEALIGN_AT_LEAST_ATTR(16); - +fb_data *lcd_framebuffer = &lcd_static_framebuffer[0][0]; static fb_data* lcd_backdrop = NULL; static long lcd_backdrop_offset IDATA_ATTR = 0; @@ -234,7 +234,7 @@ void lcd_set_backdrop(fb_data* backdrop) lcd_backdrop = backdrop; if (backdrop) { - lcd_backdrop_offset = (long)backdrop - (long)&lcd_framebuffer[0][0]; + lcd_backdrop_offset = (long)backdrop - (long)lcd_framebuffer; lcd_fastpixelfuncs = lcd_fastpixelfuncs_backdrop; } else @@ -271,7 +271,7 @@ void lcd_drawpixel(int x, int y) && ((unsigned)y < (unsigned)LCD_HEIGHT) #endif ) - lcd_fastpixelfuncs[current_vp->drawmode](LCDADDR(current_vp->x+x, current_vp->y+y)); + lcd_fastpixelfuncs[current_vp->drawmode](FBADDR(current_vp->x+x, current_vp->y+y)); } /* Draw a line */ @@ -346,7 +346,7 @@ void lcd_drawline(int x1, int y1, int x2, int y2) && ((unsigned)y < (unsigned)LCD_HEIGHT) #endif ) - pfunc(LCDADDR(x + current_vp->x, y + current_vp->y)); + pfunc(FBADDR(x + current_vp->x, y + current_vp->y)); if (d < 0) { @@ -459,7 +459,7 @@ void ICODE_ATTR lcd_mono_bitmap_part(const unsigned char *src, int src_x, src += stride * (src_y >> 3) + src_x; /* move starting point */ src_y &= 7; src_end = src + width; - dst_col = LCDADDR(x, y); + dst_col = FBADDR(x, y); if (drmode & DRMODE_INVERSEVID) @@ -747,7 +747,7 @@ static void ICODE_ATTR lcd_alpha_bitmap_part_mix(const fb_data* image, dmask = ~dmask; } - dst_row = LCDADDR(x, y); + dst_row = FBADDR(x, y); int col, row = height; unsigned data, pixels; @@ -1020,10 +1020,10 @@ void lcd_blit_yuv(unsigned char * const src[3], linecounter = height >> 1; #if LCD_WIDTH >= LCD_HEIGHT - dst = &lcd_framebuffer[y][x]; + dst = FBADDR(x, y); row_end = dst + width; #else - dst = &lcd_framebuffer[x][LCD_WIDTH - y - 1]; + dst = FBADDR(LCD_WIDTH - y - 1, x); row_end = dst + LCD_WIDTH * width; #endif diff --git a/firmware/drivers/lcd-16bit-vert.c b/firmware/drivers/lcd-16bit-vert.c index f020d74757..6f59af9f70 100644 --- a/firmware/drivers/lcd-16bit-vert.c +++ b/firmware/drivers/lcd-16bit-vert.c @@ -41,10 +41,7 @@ #define ROW_INC 1 #define COL_INC LCD_HEIGHT -#define LCDADDR(x, y) (&lcd_framebuffer[0][0] + LCD_HEIGHT*(x) + (y)) - #include "lcd-16bit-common.c" - #include "lcd-bitmap-common.c" /*** drawing functions ***/ @@ -54,7 +51,7 @@ void lcd_clear_viewport(void) { fb_data *dst, *dst_end; - dst = LCDADDR(current_vp->x, current_vp->y); + dst = FBADDR(current_vp->x, current_vp->y); dst_end = dst + current_vp->width * LCD_HEIGHT; if (current_vp->drawmode & DRMODE_INVERSEVID) @@ -145,7 +142,7 @@ void lcd_hline(int x1, int x2, int y) x2 = LCD_WIDTH-1; #endif - dst = LCDADDR(x1 , y ); + dst = FBADDR(x1 , y ); dst_end = dst + (x2 - x1) * LCD_HEIGHT; do @@ -230,7 +227,7 @@ void lcd_vline(int x, int y1, int y2) if (fillopt == OPT_NONE && current_vp->drawmode != DRMODE_COMPLEMENT) return; - dst = LCDADDR(x, y1); + dst = FBADDR(x, y1); switch (fillopt) { @@ -333,7 +330,7 @@ void lcd_fillrect(int x, int y, int width, int height) if (fillopt == OPT_NONE && current_vp->drawmode != DRMODE_COMPLEMENT) return; - dst = LCDADDR(x, y); + dst = FBADDR(x, y); dst_end = dst + width * LCD_HEIGHT; do @@ -426,7 +423,7 @@ void ICODE_ATTR lcd_bitmap_part(const fb_data *src, int src_x, int src_y, #endif src += stride * src_x + src_y; /* move starting point */ - dst = LCDADDR(x, y); + dst = FBADDR(x, y); fb_data *dst_end = dst + width * LCD_HEIGHT; do @@ -500,7 +497,7 @@ void ICODE_ATTR lcd_bitmap_transparent_part(const fb_data *src, int src_x, #endif src += stride * src_x + src_y; /* move starting point */ - dst = LCDADDR(x, y); + dst = FBADDR(x, y); dst_end = dst + width * LCD_HEIGHT; do diff --git a/firmware/drivers/lcd-16bit.c b/firmware/drivers/lcd-16bit.c index f7d3cf22d4..71768b6744 100644 --- a/firmware/drivers/lcd-16bit.c +++ b/firmware/drivers/lcd-16bit.c @@ -41,10 +41,7 @@ #define ROW_INC LCD_WIDTH #define COL_INC 1 -#define LCDADDR(x, y) (&lcd_framebuffer[(y)][(x)]) - #include "lcd-16bit-common.c" - #include "lcd-bitmap-common.c" /*** drawing functions ***/ @@ -54,7 +51,7 @@ void lcd_clear_viewport(void) { fb_data *dst, *dst_end; - dst = LCDADDR(current_vp->x, current_vp->y); + dst = FBADDR(current_vp->x, current_vp->y); dst_end = dst + current_vp->height * LCD_WIDTH; if (current_vp->drawmode & DRMODE_INVERSEVID) @@ -173,7 +170,7 @@ void lcd_hline(int x1, int x2, int y) if (fillopt == OPT_NONE && current_vp->drawmode != DRMODE_COMPLEMENT) return; - dst = LCDADDR(x1, y); + dst = FBADDR(x1, y); switch (fillopt) { @@ -241,7 +238,7 @@ void lcd_vline(int x, int y1, int y2) y2 = LCD_HEIGHT-1; #endif - dst = LCDADDR(x , y1); + dst = FBADDR(x , y1); dst_end = dst + (y2 - y1) * LCD_WIDTH; do @@ -333,7 +330,7 @@ void lcd_fillrect(int x, int y, int width, int height) if (fillopt == OPT_NONE && current_vp->drawmode != DRMODE_COMPLEMENT) return; - dst = LCDADDR(x, y); + dst = FBADDR(x, y); dst_end = dst + height * LCD_WIDTH; do @@ -426,7 +423,7 @@ void ICODE_ATTR lcd_bitmap_part(const fb_data *src, int src_x, int src_y, #endif src += stride * src_y + src_x; /* move starting point */ - dst = LCDADDR(x, y); + dst = FBADDR(x, y); do { @@ -500,7 +497,7 @@ void ICODE_ATTR lcd_bitmap_transparent_part(const fb_data *src, int src_x, #endif src += stride * src_y + src_x; /* move starting point */ - dst = LCDADDR(x, y); + dst = FBADDR(x, y); #ifdef CPU_ARM { diff --git a/firmware/drivers/lcd-1bit-vert.c b/firmware/drivers/lcd-1bit-vert.c index fd77ba7578..3a90856819 100644 --- a/firmware/drivers/lcd-1bit-vert.c +++ b/firmware/drivers/lcd-1bit-vert.c @@ -37,14 +37,16 @@ #ifndef LCDFN /* Not compiling for remote - define macros for main LCD. */ #define LCDFN(fn) lcd_ ## fn #define FBFN(fn) fb_ ## fn +#define FBSIZE FRAMEBUFFER_SIZE #define LCDM(ma) LCD_ ## ma #define LCDNAME "lcd_" +#define LCDFB(x,y) FBADDR(x, y) #define MAIN_LCD #endif /*** globals ***/ - -FBFN(data) LCDFN(framebuffer)[LCDM(FBHEIGHT)][LCDM(FBWIDTH)] IRAM_LCDFRAMEBUFFER; +FBFN(data) LCDFN(static_framebuffer)[LCDM(FBHEIGHT)][LCDM(FBWIDTH)] IRAM_LCDFRAMEBUFFER; +FBFN(data) *LCDFN(framebuffer) = &LCDFN(static_framebuffer)[0][0]; static struct viewport default_vp = { @@ -152,17 +154,17 @@ int LCDFN(getstringsize)(const unsigned char *str, int *w, int *h) static void setpixel(int x, int y) { - LCDFN(framebuffer)[y>>3][x] |= BIT_N(y & 7); + *LCDFB(x,y>>3) |= BIT_N(y & 7); } static void clearpixel(int x, int y) { - LCDFN(framebuffer)[y>>3][x] &= ~BIT_N(y & 7); + *LCDFB(x,y>>3) &= ~BIT_N(y & 7); } static void flippixel(int x, int y) { - LCDFN(framebuffer)[y>>3][x] ^= BIT_N(y & 7); + *LCDFB(x,y>>3) ^= BIT_N(y & 7); } static void nopixel(int x, int y) @@ -242,7 +244,7 @@ void LCDFN(clear_display)(void) { unsigned bits = (current_vp->drawmode & DRMODE_INVERSEVID) ? 0xFFu : 0; - memset(LCDFN(framebuffer), bits, sizeof LCDFN(framebuffer)); + memset(LCDFN(framebuffer), bits, FBSIZE); LCDFN(scroll_info).lines = 0; } @@ -422,7 +424,7 @@ void LCDFN(hline)(int x1, int x2, int y) width = x2 - x1 + 1; bfunc = LCDFN(blockfuncs)[current_vp->drawmode]; - dst = &LCDFN(framebuffer)[y>>3][x1]; + dst = LCDFB(x1,y>>3); mask = BIT_N(y & 7); dst_end = dst + width; @@ -478,7 +480,7 @@ void LCDFN(vline)(int x, int y1, int y2) #endif bfunc = LCDFN(blockfuncs)[current_vp->drawmode]; - dst = &LCDFN(framebuffer)[y1>>3][x]; + dst = LCDFB(x,y1>>3); ny = y2 - (y1 & ~7); mask = 0xFFu << (y1 & 7); mask_bottom = 0xFFu >> (~ny & 7); @@ -583,7 +585,7 @@ void LCDFN(fillrect)(int x, int y, int width, int height) } } bfunc = LCDFN(blockfuncs)[current_vp->drawmode]; - dst = &LCDFN(framebuffer)[y>>3][x]; + dst = LCDFB(x,y>>3); ny = height - 1 + (y & 7); mask = 0xFFu << (y & 7); mask_bottom = 0xFFu >> (~ny & 7); @@ -696,7 +698,7 @@ void ICODE_ATTR LCDFN(bitmap_part)(const unsigned char *src, int src_x, src += stride * (src_y >> 3) + src_x; /* move starting point */ src_y &= 7; y -= src_y; - dst = &LCDFN(framebuffer)[y>>3][x]; + dst = LCDFB(x,y>>3); shift = y & 7; ny = height - 1 + shift + src_y; diff --git a/firmware/drivers/lcd-2bit-horz.c b/firmware/drivers/lcd-2bit-horz.c index b4805094ff..3870bac644 100644 --- a/firmware/drivers/lcd-2bit-horz.c +++ b/firmware/drivers/lcd-2bit-horz.c @@ -39,7 +39,8 @@ /*** globals ***/ -unsigned char lcd_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] IRAM_LCDFRAMEBUFFER; +unsigned char lcd_static_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] IRAM_LCDFRAMEBUFFER; +unsigned char *lcd_framebuffer = &lcd_static_framebuffer[0][0]; static const unsigned char pixmask[4] ICONST_ATTR = { 0xC0, 0x30, 0x0C, 0x03 @@ -193,7 +194,7 @@ int lcd_getstringsize(const unsigned char *str, int *w, int *h) static void setpixel(int x, int y) { unsigned mask = pixmask[x & 3]; - fb_data *address = &lcd_framebuffer[y][x>>2]; + fb_data *address = FBADDR(x>>2,y); unsigned data = *address; *address = data ^ ((data ^ fg_pattern) & mask); @@ -202,7 +203,7 @@ static void setpixel(int x, int y) static void clearpixel(int x, int y) { unsigned mask = pixmask[x & 3]; - fb_data *address = &lcd_framebuffer[y][x>>2]; + fb_data *address = FBADDR(x>>2,y); unsigned data = *address; *address = data ^ ((data ^ bg_pattern) & mask); @@ -211,7 +212,7 @@ static void clearpixel(int x, int y) static void clearimgpixel(int x, int y) { unsigned mask = pixmask[x & 3]; - fb_data *address = &lcd_framebuffer[y][x>>2]; + fb_data *address = FBADDR(x>>2,y); unsigned data = *address; *address = data ^ ((data ^ *(address + lcd_backdrop_offset)) & mask); @@ -220,7 +221,7 @@ static void clearimgpixel(int x, int y) static void flippixel(int x, int y) { unsigned mask = pixmask[x & 3]; - fb_data *address = &lcd_framebuffer[y][x>>2]; + fb_data *address = FBADDR(x>>2,y); *address ^= mask; } @@ -396,14 +397,14 @@ void lcd_clear_display(void) { if (current_vp->drawmode & DRMODE_INVERSEVID) { - memset(lcd_framebuffer, fg_pattern, sizeof lcd_framebuffer); + memset(lcd_framebuffer, fg_pattern, FRAMEBUFFER_SIZE); } else { if (lcd_backdrop) - memcpy(lcd_framebuffer, lcd_backdrop, sizeof lcd_framebuffer); + memcpy(lcd_framebuffer, lcd_backdrop, FRAMEBUFFER_SIZE); else - memset(lcd_framebuffer, bg_pattern, sizeof lcd_framebuffer); + memset(lcd_framebuffer, bg_pattern, FRAMEBUFFER_SIZE); } lcd_scroll_info.lines = 0; @@ -583,7 +584,7 @@ void lcd_hline(int x1, int x2, int y) #endif bfunc = lcd_blockfuncs[current_vp->drawmode]; - dst = &lcd_framebuffer[y][x1>>2]; + dst = FBADDR(x1>>2,y); nx = x2 - (x1 & ~3); mask = 0xFFu >> (2 * (x1 & 3)); mask_right = 0xFFu << (2 * (~nx & 3)); @@ -644,7 +645,7 @@ void lcd_vline(int x, int y1, int y2) #endif bfunc = lcd_blockfuncs[current_vp->drawmode]; - dst = &lcd_framebuffer[y1][x>>2]; + dst = FBADDR(x>>2,y1); mask = pixmask[x & 3]; dst_end = dst + (y2 - y1) * LCD_FBWIDTH; @@ -729,7 +730,7 @@ void lcd_fillrect(int x, int y, int width, int height) #endif bfunc = lcd_blockfuncs[current_vp->drawmode]; - dst = &lcd_framebuffer[y][x>>2]; + dst = FBADDR(x>>2,y); nx = width - 1 + (x & 3); mask = 0xFFu >> (2 * (x & 3)); mask_right = 0xFFu << (2 * (~nx & 3)); @@ -836,7 +837,7 @@ void ICODE_ATTR lcd_mono_bitmap_part(const unsigned char *src, int src_x, src_y &= 7; src_end = src + width; - dst = &lcd_framebuffer[y][x >> 2]; + dst = FBADDR(x >> 2,y); dst_end = dst + height * LCD_FBWIDTH; dst_mask = pixmask[x & 3]; @@ -1050,7 +1051,7 @@ void ICODE_ATTR lcd_bitmap_part(const unsigned char *src, int src_x, src += stride * src_y + (src_x >> 2); /* move starting point */ src_x &= 3; x -= src_x; - dst = &lcd_framebuffer[y][x>>2]; + dst = FBADDR(x>>2,y); shift = x & 3; nx = width - 1 + shift + src_x; diff --git a/firmware/drivers/lcd-2bit-vert.c b/firmware/drivers/lcd-2bit-vert.c index 02699099d7..89c2dc365b 100644 --- a/firmware/drivers/lcd-2bit-vert.c +++ b/firmware/drivers/lcd-2bit-vert.c @@ -36,7 +36,8 @@ /*** globals ***/ -fb_data lcd_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] IRAM_LCDFRAMEBUFFER; +fb_data lcd_static_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] IRAM_LCDFRAMEBUFFER; +fb_data *lcd_framebuffer = &lcd_static_framebuffer[0][0]; const unsigned char lcd_dibits[16] ICONST_ATTR = { 0x00, 0x03, 0x0C, 0x0F, 0x30, 0x33, 0x3C, 0x3F, @@ -196,7 +197,7 @@ int lcd_getstringsize(const unsigned char *str, int *w, int *h) static void setpixel(int x, int y) { unsigned mask = pixmask[y & 3]; - fb_data *address = &lcd_framebuffer[y>>2][x]; + fb_data *address = FBADDR(x,y>>2); unsigned data = *address; *address = data ^ ((data ^ fg_pattern) & mask); @@ -205,7 +206,7 @@ static void setpixel(int x, int y) static void clearpixel(int x, int y) { unsigned mask = pixmask[y & 3]; - fb_data *address = &lcd_framebuffer[y>>2][x]; + fb_data *address = FBADDR(x,y>>2); unsigned data = *address; *address = data ^ ((data ^ bg_pattern) & mask); @@ -214,7 +215,7 @@ static void clearpixel(int x, int y) static void clearimgpixel(int x, int y) { unsigned mask = pixmask[y & 3]; - fb_data *address = &lcd_framebuffer[y>>2][x]; + fb_data *address = FBADDR(x,y>>2); unsigned data = *address; *address = data ^ ((data ^ *(address + lcd_backdrop_offset)) & mask); @@ -223,7 +224,7 @@ static void clearimgpixel(int x, int y) static void flippixel(int x, int y) { unsigned mask = pixmask[y & 3]; - fb_data *address = &lcd_framebuffer[y>>2][x]; + fb_data *address = FBADDR(x,y>>2); *address ^= mask; } @@ -399,14 +400,14 @@ void lcd_clear_display(void) { if (current_vp->drawmode & DRMODE_INVERSEVID) { - memset(lcd_framebuffer, fg_pattern, sizeof lcd_framebuffer); + memset(lcd_framebuffer, fg_pattern, FRAMEBUFFER_SIZE); } else { if (lcd_backdrop) - memcpy(lcd_framebuffer, lcd_backdrop, sizeof lcd_framebuffer); + memcpy(lcd_framebuffer, lcd_backdrop, FRAMEBUFFER_SIZE); else - memset(lcd_framebuffer, bg_pattern, sizeof lcd_framebuffer); + memset(lcd_framebuffer, bg_pattern, FRAMEBUFFER_SIZE); } lcd_scroll_info.lines = 0; @@ -589,7 +590,7 @@ void lcd_hline(int x1, int x2, int y) width = x2 - x1 + 1; bfunc = lcd_blockfuncs[current_vp->drawmode]; - dst = &lcd_framebuffer[y>>2][x1]; + dst = FBADDR(x1,y>>2); mask = pixmask[y & 3]; dst_end = dst + width; @@ -645,7 +646,7 @@ void lcd_vline(int x, int y1, int y2) #endif bfunc = lcd_blockfuncs[current_vp->drawmode]; - dst = &lcd_framebuffer[y1>>2][x]; + dst = FBADDR(x,y1>>2); ny = y2 - (y1 & ~3); mask = 0xFFu << (2 * (y1 & 3)); mask_bottom = 0xFFu >> (2 * (~ny & 3)); @@ -751,7 +752,7 @@ void lcd_fillrect(int x, int y, int width, int height) } } bfunc = lcd_blockfuncs[current_vp->drawmode]; - dst = &lcd_framebuffer[y>>2][x]; + dst = FBADDR(x,y>>2); ny = height - 1 + (y & 3); mask = 0xFFu << (2 * (y & 3)); mask_bottom = 0xFFu >> (2 * (~ny & 3)); @@ -863,7 +864,7 @@ void ICODE_ATTR lcd_mono_bitmap_part(const unsigned char *src, int src_x, src += stride * (src_y >> 3) + src_x; /* move starting point */ src_y &= 7; y -= src_y; - dst = &lcd_framebuffer[y>>2][x]; + dst = FBADDR(x,y>>2); shift = y & 3; ny = height - 1 + shift + src_y; mask = 0xFFFFu << (2 * (shift + src_y)); @@ -1060,7 +1061,7 @@ void ICODE_ATTR lcd_bitmap_part(const fb_data *src, int src_x, int src_y, src += stride * (src_y >> 2) + src_x; /* move starting point */ src_y &= 3; y -= src_y; - dst = &lcd_framebuffer[y>>2][x]; + dst = FBADDR(x,y>>2); shift = y & 3; ny = height - 1 + shift + src_y; diff --git a/firmware/drivers/lcd-2bit-vi.c b/firmware/drivers/lcd-2bit-vi.c index 697d1a7783..8dc746f521 100644 --- a/firmware/drivers/lcd-2bit-vi.c +++ b/firmware/drivers/lcd-2bit-vi.c @@ -40,13 +40,17 @@ #define LCDFN(fn) lcd_ ## fn #define FBFN(fn) fb_ ## fn #define LCDM(ma) LCD_ ## ma +#define FBSIZE FRAMEBUFFER_SIZE #define LCDNAME "lcd_" +#define LCDFB(x,y) FBADDR(x, y) #define MAIN_LCD #endif /*** globals ***/ -FBFN(data) LCDFN(framebuffer)[LCDM(FBHEIGHT)][LCDM(FBWIDTH)] IRAM_LCDFRAMEBUFFER; +FBFN(data) LCDFN(static_framebuffer)[LCDM(FBHEIGHT)][LCDM(FBWIDTH)] IRAM_LCDFRAMEBUFFER; +FBFN(data) *LCDFN(framebuffer) = &LCDFN(static_framebuffer)[0][0]; + static const FBFN(data) patterns[4] = {0xFFFF, 0xFF00, 0x00FF, 0x0000}; @@ -214,7 +218,7 @@ int LCDFN(getstringsize)(const unsigned char *str, int *w, int *h) static void setpixel(int x, int y) { unsigned mask = 0x0101 << (y & 7); - FBFN(data) *address = &LCDFN(framebuffer)[y>>3][x]; + FBFN(data) *address = LCDFB(x,y>>3); unsigned data = *address; *address = data ^ ((data ^ fg_pattern) & mask); @@ -223,7 +227,7 @@ static void setpixel(int x, int y) static void clearpixel(int x, int y) { unsigned mask = 0x0101 << (y & 7); - FBFN(data) *address = &LCDFN(framebuffer)[y>>3][x]; + FBFN(data) *address = LCDFB(x,y>>3); unsigned data = *address; *address = data ^ ((data ^ bg_pattern) & mask); @@ -232,7 +236,7 @@ static void clearpixel(int x, int y) static void clearimgpixel(int x, int y) { unsigned mask = 0x0101 << (y & 7); - FBFN(data) *address = &LCDFN(framebuffer)[y>>3][x]; + FBFN(data) *address = LCDFB(x,y>>3); unsigned data = *address; *address = data ^ ((data ^ *(FBFN(data) *)((long)address @@ -242,7 +246,7 @@ static void clearimgpixel(int x, int y) static void flippixel(int x, int y) { unsigned mask = 0x0101 << (y & 7); - FBFN(data) *address = &LCDFN(framebuffer)[y>>3][x]; + FBFN(data) *address = LCDFB(x,y>>3); *address ^= mask; } @@ -420,15 +424,15 @@ void LCDFN(clear_display)(void) if (default_vp.drawmode & DRMODE_INVERSEVID) { memset(LCDFN(framebuffer), patterns[default_vp.fg_pattern & 3], - sizeof LCDFN(framebuffer)); + FBSIZE); } else { if (backdrop) - memcpy(LCDFN(framebuffer), backdrop, sizeof LCDFN(framebuffer)); + memcpy(LCDFN(framebuffer), backdrop, FBSIZE); else memset(LCDFN(framebuffer), patterns[default_vp.bg_pattern & 3], - sizeof LCDFN(framebuffer)); + FBSIZE); } LCDFN(scroll_info).lines = 0; @@ -611,7 +615,7 @@ void LCDFN(hline)(int x1, int x2, int y) width = x2 - x1 + 1; bfunc = LCDFN(blockfuncs)[current_vp->drawmode]; - dst = &LCDFN(framebuffer)[y>>3][x1]; + dst = LCDFB(x1,y>>3); mask = 0x0101 << (y & 7); dst_end = dst + width; @@ -667,7 +671,7 @@ void LCDFN(vline)(int x, int y1, int y2) #endif bfunc = LCDFN(blockfuncs)[current_vp->drawmode]; - dst = &LCDFN(framebuffer)[y1>>3][x]; + dst = LCDFB(x,y1>>3); ny = y2 - (y1 & ~7); mask = (0xFFu << (y1 & 7)) & 0xFFu; mask |= mask << 8; @@ -776,7 +780,7 @@ void LCDFN(fillrect)(int x, int y, int width, int height) } } bfunc = LCDFN(blockfuncs)[current_vp->drawmode]; - dst = &LCDFN(framebuffer)[y>>3][x]; + dst = LCDFB(x,y>>3); ny = height - 1 + (y & 7); mask = (0xFFu << (y & 7)) & 0xFFu; mask |= mask << 8; @@ -890,7 +894,7 @@ void ICODE_ATTR LCDFN(mono_bitmap_part)(const unsigned char *src, int src_x, src += stride * (src_y >> 3) + src_x; /* move starting point */ src_y &= 7; y -= src_y; - dst = &LCDFN(framebuffer)[y>>3][x]; + dst = LCDFB(x,y>>3); shift = y & 7; ny = height - 1 + shift + src_y; @@ -1058,7 +1062,7 @@ void ICODE_ATTR LCDFN(bitmap_part)(const FBFN(data) *src, int src_x, src += stride * (src_y >> 3) + src_x; /* move starting point */ src_y &= 7; y -= src_y; - dst = &LCDFN(framebuffer)[y>>3][x]; + dst = LCDFB(x,y>>3); shift = y & 7; ny = height - 1 + shift + src_y; diff --git a/firmware/drivers/lcd-memframe.c b/firmware/drivers/lcd-memframe.c index dd878876bf..bb1682b074 100644 --- a/firmware/drivers/lcd-memframe.c +++ b/firmware/drivers/lcd-memframe.c @@ -66,7 +66,7 @@ void lcd_update(void) return; /* Copy the Rockbox framebuffer to the second framebuffer */ - lcd_copy_buffer_rect(LCD_FRAMEBUF_ADDR(0, 0), &lcd_framebuffer[0][0], + lcd_copy_buffer_rect(LCD_FRAMEBUF_ADDR(0, 0), FBADDR(0,0), LCD_WIDTH*LCD_HEIGHT, 1); } #endif /* LCD_OPTIMIZED_UPDATE */ @@ -95,7 +95,7 @@ void lcd_update_rect(int x, int y, int width, int height) return; /* nothing left to do */ dst = LCD_FRAMEBUF_ADDR(x, y); - src = &lcd_framebuffer[y][x]; + src = FBADDR(x,y); /* Copy part of the Rockbox framebuffer to the second framebuffer */ if (width < LCD_WIDTH) diff --git a/firmware/drivers/lcd-remote-1bit-v.c b/firmware/drivers/lcd-remote-1bit-v.c index f2588323e7..24b91509c7 100644 --- a/firmware/drivers/lcd-remote-1bit-v.c +++ b/firmware/drivers/lcd-remote-1bit-v.c @@ -24,7 +24,9 @@ /* Compile 1 bit vertical packing LCD driver for remote LCD */ #define LCDFN(fn) lcd_remote_ ## fn #define FBFN(fn) fb_remote_ ## fn +#define FBSIZE FRAMEBUFFER_REMOTE_SIZE #define LCDM(ma) LCD_REMOTE_ ## ma +#define LCDFB(x,y) FBREMOTEADDR(x, y) #define LCDNAME "lcd_remote_" #include "lcd-1bit-vert.c" diff --git a/firmware/drivers/lcd-remote-2bit-vi.c b/firmware/drivers/lcd-remote-2bit-vi.c index 34af8dde6f..fd4de6f6af 100644 --- a/firmware/drivers/lcd-remote-2bit-vi.c +++ b/firmware/drivers/lcd-remote-2bit-vi.c @@ -26,7 +26,9 @@ /* Compile 2 bit vertical interleaved LCD driver for remote LCD */ #define LCDFN(fn) lcd_remote_ ## fn #define FBFN(fn) fb_remote_ ## fn +#define FBSIZE FRAMEBUFFER_REMOTE_SIZE #define LCDM(ma) LCD_REMOTE_ ## ma +#define LCDFB(x,y) FBREMOTEADDR(x, y) #define LCDNAME "lcd_remote_" #include "lcd-2bit-vi.c" -- cgit v1.2.3