diff --git a/android/src/org/rockbox/RockboxFramebuffer.java b/android/src/org/rockbox/RockboxFramebuffer.java index 0a60182..db6825f 100644 --- a/android/src/org/rockbox/RockboxFramebuffer.java +++ b/android/src/org/rockbox/RockboxFramebuffer.java @@ -29,26 +29,33 @@ import android.content.Context; import android.graphics.Bitmap; import android.graphics.Canvas; import android.util.Log; +import android.view.Display; import android.view.KeyEvent; import android.view.MotionEvent; import android.view.View; +import android.view.WindowManager; public class RockboxFramebuffer extends View { private Bitmap btm; private ByteBuffer native_buf; private MediaButtonReceiver media_monitor; + private int lcd_width; + private int lcd_height; - public RockboxFramebuffer(Context c, int lcd_width, - int lcd_height, ByteBuffer native_fb) + public RockboxFramebuffer(Context c) { super(c); /* Needed so we can catch KeyEvents */ setFocusable(true); setFocusableInTouchMode(true); setClickable(true); + Display display = ((WindowManager) + c.getSystemService(Context.WINDOW_SERVICE)). + getDefaultDisplay(); + lcd_height = display.getHeight(); + lcd_width = display.getWidth(); btm = Bitmap.createBitmap(lcd_width, lcd_height, Bitmap.Config.RGB_565); - native_buf = native_fb; media_monitor = new MediaButtonReceiver(c); media_monitor.register(); /* the service needs to know the about us */ @@ -61,9 +68,16 @@ public class RockboxFramebuffer extends View } @SuppressWarnings("unused") + private void set_native_buf(ByteBuffer native_fb) + { + native_buf = native_fb; + java_lcd_update(); + } + private void java_lcd_update() { - btm.copyPixelsFromBuffer(native_buf); + if (native_buf != null) + btm.copyPixelsFromBuffer(native_buf); postInvalidate(); } @@ -71,7 +85,8 @@ public class RockboxFramebuffer extends View private void java_lcd_update_rect(int x, int y, int w, int h) { /* can't copy a partial buffer */ - btm.copyPixelsFromBuffer(native_buf); + if (native_buf != null) + btm.copyPixelsFromBuffer(native_buf); postInvalidate(x, y, x+w, y+h); } diff --git a/apps/bitmaps/native/SOURCES b/apps/bitmaps/native/SOURCES index b3bd2c4..99424db 100644 --- a/apps/bitmaps/native/SOURCES +++ b/apps/bitmaps/native/SOURCES @@ -1,5 +1,13 @@ #ifdef HAVE_LCD_BITMAP +#ifdef DYNAMIC_LCD_SIZE +/* This is a hack to prevent buildtime errors popping up */ +#undef LCD_WIDTH +#undef LCD_HEIGHT +#define LCD_WIDTH 0 +#define LCD_HEIGHT 0 +#endif + /* Rockbox logo */ #if (LCD_DEPTH == 1) #if (LCD_WIDTH == 160) @@ -9,6 +17,12 @@ rockboxlogo.128x42x1.bmp #else rockboxlogo.112x30x1.bmp #endif +#elif defined(DYNAMIC_LCD_SIZE) +/* Currently only one screen size is supported. + * Ideally, we should compile in multiple logos and choose the correct one + * at runtime (and optionally discard the others using INIT_ATTR). + */ +rockboxlogo.320x98x16.bmp #elif (LCD_WIDTH == 128) && (LCD_DEPTH == 2) rockboxlogo.128x42x2.bmp #elif (LCD_WIDTH == 128) && (LCD_DEPTH == 16) @@ -41,7 +55,9 @@ rockboxlogo.640x198x16.bmp /* USB logo */ #ifdef HAVE_LCD_COLOR -#if LCD_WIDTH > 176 +#if defined(DYNAMIC_LCD_SIZE) +usblogo.176x48x16.bmp +#elif LCD_WIDTH > 176 usblogo.176x48x16.bmp #elif LCD_WIDTH >= 128 usblogo.128x37x16.bmp diff --git a/apps/debug_menu.c b/apps/debug_menu.c index bedc98a..830c5a6 100644 --- a/apps/debug_menu.c +++ b/apps/debug_menu.c @@ -340,8 +340,13 @@ static bool dbg_buffering_thread(void) screens[i].putsf(0, line++, "alloc: %6ld/%ld", audio_filebufused(), (long) filebuflen); -#if LCD_HEIGHT > 80 || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_HEIGHT > 80) - if (screens[i].lcdheight > 80) + if ((LCD_HEIGHT > 80 || +#ifdef HAVE_REMOTE_LCD + LCD_REMOTE_HEIGHT > 80 +#else + 0 +#endif + ) && screens[i].lcdheight > 80) { gui_scrollbar_draw(&screens[i],0, line*8, screens[i].lcdwidth, 6, filebuflen, 0, audio_filebufused(), HORIZONTAL); @@ -354,19 +359,22 @@ static bool dbg_buffering_thread(void) filebuflen, 0, (long)d.buffered_data, HORIZONTAL); line++; } -#endif screens[i].putsf(0, line++, "usefl: %6ld/%ld", (long)(d.useful_data), (long)filebuflen); -#if LCD_HEIGHT > 80 || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_HEIGHT > 80) - if (screens[i].lcdheight > 80) + if ((LCD_HEIGHT > 80 || +#ifdef HAVE_REMOTE_LCD + LCD_REMOTE_HEIGHT > 80 +#else + 0 +#endif + ) && screens[i].lcdheight > 80) { gui_scrollbar_draw(&screens[i],0, line*8, screens[i].lcdwidth, 6, filebuflen, 0, d.useful_data, HORIZONTAL); line++; } -#endif screens[i].putsf(0, line++, "data_rem: %ld", (long)d.data_rem); diff --git a/apps/gui/skin_engine/skin_backdrops.c b/apps/gui/skin_engine/skin_backdrops.c index e337774..e33163b 100644 --- a/apps/gui/skin_engine/skin_backdrops.c +++ b/apps/gui/skin_engine/skin_backdrops.c @@ -121,6 +121,15 @@ bool skin_backdrops_preload(void) snprintf(temp, size, "%s/%s.bmp", get_user_file_path(BACKDROP_DIR, 0, dir, sizeof(dir)), global_settings.backdrop_file); +#ifdef DYNAMIC_LCD_SIZE + if (!file_exists(temp)) + { + snprintf(temp, size, "%s/%s.%dx%dx%d.bmp", + get_user_file_path(BACKDROP_DIR, 0, dir, sizeof(dir)), + global_settings.backdrop_file, screens[screen].lcdwidth, + screens[screen].lcdheight, screens[screen].depth); + } +#endif filename = temp; } if (*filename && *filename != '-') diff --git a/apps/gui/skin_engine/skin_engine.c b/apps/gui/skin_engine/skin_engine.c index 25baeb2..1db66a1 100644 --- a/apps/gui/skin_engine/skin_engine.c +++ b/apps/gui/skin_engine/skin_engine.c @@ -30,6 +30,7 @@ #include "settings.h" #include "wps.h" #include "file.h" +#include "filefuncs.h" #include "buffer.h" #if CONFIG_TUNER #include "radio.h" @@ -229,6 +230,15 @@ struct gui_wps *skin_get_gwps(enum skinnable_screens skin, enum screen_type scre snprintf(buf, sizeof buf, "%s/%s.%s", get_user_file_path(WPS_DIR, false, path, sizeof(path)), setting, ext); +#ifdef DYNAMIC_LCD_SIZE + if (!file_exists(buf)) + { + snprintf(buf, sizeof buf, "%s/%s.%dx%dx%d.%s", + get_user_file_path(WPS_DIR, false, path, sizeof(path)), + setting, screens[screen].lcdwidth, + screens[screen].lcdheight, screens[screen].depth, ext); + } +#endif } cpu_boost(true); skin_load(skin, screen, buf, true); diff --git a/apps/gui/skin_engine/skin_fonts.h b/apps/gui/skin_engine/skin_fonts.h index 698ed4f..48bef2c 100644 --- a/apps/gui/skin_engine/skin_fonts.h +++ b/apps/gui/skin_engine/skin_fonts.h @@ -31,11 +31,7 @@ #ifndef _SKINFONTS_H_ #define _SKINFONTS_H_ -#if LCD_HEIGHT > 160 -#define SKIN_FONT_SIZE (1024*10) -#else -#define SKIN_FONT_SIZE (1024*3) -#endif +#define SKIN_FONT_SIZE (LCD_HEIGHT > 160 ? 1024*10 : 1024*3) #define GLYPHS_TO_CACHE 256 void skin_font_init(void); diff --git a/apps/gui/skin_engine/skin_parser.c b/apps/gui/skin_engine/skin_parser.c index ad3e7e4..b203e6f 100644 --- a/apps/gui/skin_engine/skin_parser.c +++ b/apps/gui/skin_engine/skin_parser.c @@ -1630,7 +1630,8 @@ bool skin_data_load(enum screen_type screen, struct wps_data *wps_data, if (isfile) { /* get the bitmap dir */ - char *dot = strrchr(buf, '.'); + char *slash = strrchr(buf, '/'); + char *dot = strchr(slash, '.'); strlcpy(bmpdir, buf, dot - buf + 1); } else diff --git a/apps/main.c b/apps/main.c index 950168d..7aef2dd 100644 --- a/apps/main.c +++ b/apps/main.c @@ -345,6 +345,7 @@ static void init(void) lcd_remote_init(); #endif font_init(); + screen_init(); show_logo(); button_init(); backlight_init(); diff --git a/apps/plugin.c b/apps/plugin.c index 316c7c6..b1f43cf 100644 --- a/apps/plugin.c +++ b/apps/plugin.c @@ -99,7 +99,7 @@ static int close_wrapper(int fd); static int creat_wrapper(const char *pathname, mode_t mode); #endif -static const struct plugin_api rockbox_api = { +static struct plugin_api rockbox_api = { /* lcd */ #ifdef HAVE_LCD_CONTRAST @@ -124,7 +124,7 @@ static const struct plugin_api rockbox_api = { lcd_icon, lcd_double_height, #else - &lcd_framebuffer[0][0], + NULL, /* will get filled in later */ lcd_update_rect, lcd_set_drawmode, lcd_get_drawmode, @@ -788,7 +788,8 @@ int plugin_load(const char* plugin, const void* parameter) #else plugin_size = 0; #endif - + rockbox_api.lcd_framebuffer = lcd_framebuffer; + *(p_hdr->api) = &rockbox_api; #if defined HAVE_LCD_BITMAP && LCD_DEPTH > 1 diff --git a/apps/recorder/bmp.c b/apps/recorder/bmp.c index 81ed8bc..17058da 100644 --- a/apps/recorder/bmp.c +++ b/apps/recorder/bmp.c @@ -64,6 +64,7 @@ #undef DEBUGF #define DEBUGF(...) #endif +#include "buffer.h" #ifdef __GNUC__ #define STRUCT_PACKED __attribute__((packed)) @@ -181,7 +182,7 @@ struct bmp_args { short read_width; short width; short depth; - unsigned char buf[BM_MAX_WIDTH * 4]; + unsigned char *buf; struct uint8_rgb *palette; #if (LCD_DEPTH > 1 || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)) && \ defined(HAVE_BMP_SCALING) || defined(PLUGIN) @@ -191,6 +192,8 @@ struct bmp_args { #endif }; +static unsigned char *ba_buf; + static unsigned int read_part_line(struct bmp_args *ba) { const int padded_width = ba->padded_width; @@ -708,9 +711,13 @@ int read_bmp_fd(int fd, memset(bitmap, 0, totalsize); + if (!ba_buf) + ba_buf = buffer_alloc(BM_MAX_WIDTH*4); + struct bmp_args ba = { .fd = fd, .padded_width = padded_width, .read_width = read_width, .width = src_dim.width, .depth = depth, .palette = palette, + .buf = ba_buf, #if (LCD_DEPTH > 1 || (defined(HAVE_REMOTE_LCD) && LCD_REMOTE_DEPTH > 1)) && \ defined(HAVE_BMP_SCALING) || defined(PLUGIN) .cur_row = 0, .cur_col = 0, .part = {0,0} diff --git a/apps/recorder/keyboard.c b/apps/recorder/keyboard.c index 1b2d76e..c748add 100644 --- a/apps/recorder/keyboard.c +++ b/apps/recorder/keyboard.c @@ -371,10 +371,8 @@ int kbd_input(char* text, int buflen) const unsigned char *p; int len = 0; -#if LCD_WIDTH >= 160 && LCD_HEIGHT >= 96 - struct screen *sc = &screens[l]; - - if (sc->getwidth() >= 160 && sc->getheight() >= 96) + if (LCD_WIDTH >= 160 && LCD_HEIGHT >= 96 && + screens[l].getwidth() >= 160 && screens[l].getheight() >= 96) { p = "ABCDEFG abcdefg !?\" @#$%+'\n" "HIJKLMN hijklmn 789 &_()-`\n" @@ -389,7 +387,6 @@ int kbd_input(char* text, int buflen) pm->max_line_len = 26; } else -#endif /* LCD_WIDTH >= 160 && LCD_HEIGHT >= 96 */ { p = "ABCDEFG !?\" @#$%+'\n" "HIJKLMN 789 &_()-`\n" diff --git a/apps/screen_access.c b/apps/screen_access.c index b83e842..00794d1 100644 --- a/apps/screen_access.c +++ b/apps/screen_access.c @@ -105,8 +105,6 @@ struct screen screens[NB_SCREENS] = { { .screen_type=SCREEN_MAIN, - .lcdwidth=LCD_WIDTH, - .lcdheight=LCD_HEIGHT, .depth=LCD_DEPTH, .getnblines=&screen_helper_getnblines, #if defined(HAVE_LCD_COLOR) @@ -316,3 +314,9 @@ void screen_clear_area(struct screen * display, int xstart, int ystart, display->set_drawmode(DRMODE_SOLID); } #endif + +void screen_init(void) +{ + screens[SCREEN_MAIN].lcdwidth = LCD_WIDTH; + screens[SCREEN_MAIN].lcdheight = LCD_HEIGHT; +} diff --git a/apps/screen_access.h b/apps/screen_access.h index efe232b..599ca76 100644 --- a/apps/screen_access.h +++ b/apps/screen_access.h @@ -177,4 +177,6 @@ void screen_clear_area(struct screen * display, int xstart, int ystart, */ extern struct screen screens[NB_SCREENS]; +void screen_init(void) INIT_ATTR; + #endif /*_SCREEN_ACCESS_H_*/ diff --git a/apps/settings_list.c b/apps/settings_list.c index 2922bcd..68cda62 100644 --- a/apps/settings_list.c +++ b/apps/settings_list.c @@ -205,7 +205,7 @@ static const char graphic_numeric[] = "graphic,numeric"; #define DEFAULT_SBSNAME "-" #define DEFAULT_FMS_NAME DEFAULT_WPSNAME -#ifdef HAVE_LCD_BITMAP +#if defined(HAVE_LCD_BITMAP) && !defined(DYNAMIC_LCD_SIZE) #if LCD_HEIGHT <= 64 #define DEFAULT_FONTNAME "08-Rockfont" @@ -224,7 +224,7 @@ static const char graphic_numeric[] = "graphic,numeric"; #endif #else - #define DEFAULT_FONTNAME "" + #define DEFAULT_FONTNAME "27-Adobe-Helvetica" #endif #ifdef HAVE_REMOTE_LCD @@ -732,7 +732,7 @@ const struct settings_list settings[] = { LANG_SCROLL_BAR, SCROLLBAR_LEFT, "scrollbar","off,left,right", NULL, 3, ID2P(LANG_OFF), ID2P(LANG_LEFT), ID2P(LANG_RIGHT)), INT_SETTING(F_THEMESETTING, scrollbar_width, LANG_SCROLLBAR_WIDTH, 6, - "scrollbar width",UNIT_INT, 3, MAX(LCD_WIDTH/10,25), 1, + "scrollbar width",UNIT_INT, 3, 25, 1, NULL, NULL, NULL), #if CONFIG_KEYPAD == RECORDER_PAD OFFON_SETTING(F_THEMESETTING,buttonbar, LANG_BUTTON_BAR ,true,"buttonbar", NULL), @@ -919,9 +919,9 @@ const struct settings_list settings[] = { false, "Screen Scrolls Out Of View", gui_list_screen_scroll_out_of_view), INT_SETTING(F_PADTITLE, scroll_step, LANG_SCROLL_STEP, 6, "scroll step", - UNIT_PIXEL, 1, LCD_WIDTH, 1, NULL, NULL, lcd_scroll_step), + UNIT_PIXEL, 1, 400/*LCD_WIDTH*/, 1, NULL, NULL, lcd_scroll_step), INT_SETTING(F_PADTITLE, screen_scroll_step, LANG_SCREEN_SCROLL_STEP, 16, - "screen scroll step", UNIT_PIXEL, 1, LCD_WIDTH, 1, NULL, NULL, + "screen scroll step", UNIT_PIXEL, 1, 400/*LCD_WIDTH*/, 1, NULL, NULL, gui_list_screen_scroll_step), #endif /* HAVE_LCD_BITMAP */ OFFON_SETTING(0,scroll_paginated,LANG_SCROLL_PAGINATED, diff --git a/firmware/bidi.c b/firmware/bidi.c index 2f4e137..ffd18c6 100644 --- a/firmware/bidi.c +++ b/firmware/bidi.c @@ -137,10 +137,10 @@ static void arabjoin(unsigned short * stringprt, int length) unsigned short *bidi_l2v(const unsigned char *str, int orientation) { - static unsigned short utf16_buf[SCROLL_LINE_SIZE]; + unsigned short utf16_buf[SCROLL_LINE_SIZE]; unsigned short *target, *tmp; #ifndef BOOTLOADER - static unsigned short bidi_buf[SCROLL_LINE_SIZE]; + unsigned short bidi_buf[SCROLL_LINE_SIZE]; unsigned short *heb_str; /* *broken_str */ int block_start, block_end, block_type, block_length, i; int length = utf8length(str); diff --git a/firmware/drivers/lcd-16bit.c b/firmware/drivers/lcd-16bit.c index be4f21f..7979b54 100644 --- a/firmware/drivers/lcd-16bit.c +++ b/firmware/drivers/lcd-16bit.c @@ -43,8 +43,12 @@ enum fill_opt { }; /*** globals ***/ -fb_data lcd_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] +fb_data *lcd_framebuffer; + +#ifndef DYNAMIC_LCD_SIZE +fb_data _lcd_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH] IRAM_LCDFRAMEBUFFER CACHEALIGN_AT_LEAST_ATTR(16); +#endif static fb_data* lcd_backdrop = NULL; @@ -54,8 +58,6 @@ static struct viewport default_vp = { .x = 0, .y = 0, - .width = LCD_WIDTH, - .height = LCD_HEIGHT, .font = FONT_SYSFIXED, .drawmode = DRMODE_SOLID, .fg_pattern = LCD_DEFAULT_FG, @@ -75,6 +77,12 @@ void lcd_init(void) /* Call device specific init */ lcd_init_device(); scroll_init(); + + default_vp.width = LCD_WIDTH; + default_vp.height = LCD_HEIGHT; +#ifndef DYNAMIC_LCD_SIZE + lcd_framebuffer = &_lcd_framebuffer[0][0]; +#endif } /*** Viewports ***/ @@ -200,7 +208,7 @@ int lcd_getstringsize(const unsigned char *str, int *w, int *h) /*** low-level drawing functions ***/ -#define LCDADDR(x, y) (&lcd_framebuffer[(y)][(x)]) +#define LCDADDR(x, y) (lcd_framebuffer + y * LCD_WIDTH + x) static void ICODE_ATTR setpixel(fb_data *address) { @@ -244,7 +252,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 diff --git a/firmware/export/config.h b/firmware/export/config.h index 3a73283..7c8f3fb 100644 --- a/firmware/export/config.h +++ b/firmware/export/config.h @@ -579,6 +579,14 @@ Lyre prototype 1 */ #endif #endif /* HAVE_REMOTE_LCD */ +#ifdef DYNAMIC_LCD_SIZE +#undef LCD_WIDTH +#undef LCD_HEIGHT + +#define LCD_WIDTH ({extern int lcd_width; lcd_width;}) +#define LCD_HEIGHT ({extern int lcd_height; lcd_height;}) +#endif + /* define this in the target config.h to use a different size */ #ifndef CONFIG_DEFAULT_ICON_HEIGHT #define CONFIG_DEFAULT_ICON_HEIGHT 8 diff --git a/firmware/export/config/application.h b/firmware/export/config/application.h index b731f0c..b2b1bf5 100644 --- a/firmware/export/config/application.h +++ b/firmware/export/config/application.h @@ -42,13 +42,7 @@ /* LCD dimensions * * overriden by configure for application builds */ -#ifndef LCD_WIDTH -#define LCD_WIDTH 320 -#endif - -#ifndef LCD_HEIGHT -#define LCD_HEIGHT 480 -#endif +#define DYNAMIC_LCD_SIZE #define LCD_DEPTH 16 #define LCD_PIXELFORMAT 565 diff --git a/firmware/export/lcd.h b/firmware/export/lcd.h index 79231d1..073ffa8 100644 --- a/firmware/export/lcd.h +++ b/firmware/export/lcd.h @@ -407,7 +407,7 @@ static inline unsigned lcd_color_to_native(unsigned color) #define LCD_FBHEIGHT LCD_HEIGHT #endif /* The actual framebuffer */ -extern fb_data lcd_framebuffer[LCD_FBHEIGHT][LCD_FBWIDTH]; +extern fb_data *lcd_framebuffer; /** Port-specific functions. Enable in port config file. **/ #ifdef HAVE_REMOTE_LCD_AS_MAIN diff --git a/firmware/export/scroll_engine.h b/firmware/export/scroll_engine.h index 0fe6fe4..022bbba 100644 --- a/firmware/export/scroll_engine.h +++ b/firmware/export/scroll_engine.h @@ -49,7 +49,7 @@ void lcd_remote_scroll_stop_line(const struct viewport* vp, int y); struct scrollinfo { struct viewport* vp; - char line[SCROLL_LINE_SIZE]; + char *line; #ifdef HAVE_LCD_CHARCELLS int len; /* length of line in chars */ #endif @@ -68,7 +68,7 @@ struct scrollinfo struct scroll_screen_info { - struct scrollinfo * const scroll; + struct scrollinfo * scroll; const int num_scroll; /* number of scrollable lines (also number of scroll structs) */ int lines; /* Number of currently scrolling lines */ long ticks; /* # of ticks between updates*/ diff --git a/firmware/font.c b/firmware/font.c index e973108..68f92b5 100644 --- a/firmware/font.c +++ b/firmware/font.c @@ -38,20 +38,13 @@ #include "rbunicode.h" #include "diacritic.h" #include "rbpaths.h" +#include "buffer.h" #define MAX_FONTSIZE_FOR_16_BIT_OFFSETS 0xFFDB /* max static loadable font buffer size */ #ifndef MAX_FONT_SIZE -#if LCD_HEIGHT > 64 -#if MEM > 2 -#define MAX_FONT_SIZE 60000 -#else -#define MAX_FONT_SIZE 10000 -#endif -#else -#define MAX_FONT_SIZE 4000 -#endif +#define MAX_FONT_SIZE (LCD_HEIGHT > 64 ? (MEM > 2 ? 60000 : 10000) : 4000) #endif #ifndef FONT_HEADER_SIZE @@ -76,11 +69,11 @@ extern struct font sysfont; /* structure filled in by font_load */ static struct font font_ui; /* static buffer allocation structures */ -static unsigned char main_buf[MAX_FONT_SIZE]; +static unsigned char *main_buf; #ifdef HAVE_REMOTE_LCD #define REMOTE_FONT_SIZE 10000 static struct font remote_font_ui; -static unsigned char remote_buf[REMOTE_FONT_SIZE]; +static unsigned char *remote_buf; #endif /* system font table, in order of FONT_xxx definition */ @@ -97,8 +90,10 @@ void font_init(void) int i = SYSTEMFONTCOUNT; while (i= 0; y--) @@ -214,7 +239,7 @@ void screen_dump(void) #endif #elif LCD_DEPTH == 16 dst_end = dst + LCD_WIDTH; - src = lcd_framebuffer[y]; + src = &lcd_framebuffer[y]; do { diff --git a/firmware/scroll_engine.c b/firmware/scroll_engine.c index 70bbcbe..9efdee2 100644 --- a/firmware/scroll_engine.c +++ b/firmware/scroll_engine.c @@ -33,6 +33,7 @@ #ifdef HAVE_REMOTE_LCD #include "lcd-remote.h" #endif +#include "buffer.h" #include "scroll_engine.h" static const char scroll_tick_table[16] = { @@ -45,16 +46,12 @@ static void scroll_thread(void); static char scroll_stack[DEFAULT_STACK_SIZE*3]; static const char scroll_name[] = "scroll"; -static struct scrollinfo lcd_scroll[LCD_SCROLLABLE_LINES]; - #ifdef HAVE_REMOTE_LCD -static struct scrollinfo lcd_remote_scroll[LCD_REMOTE_SCROLLABLE_LINES]; static struct event_queue scroll_queue; #endif struct scroll_screen_info lcd_scroll_info = { - .scroll = lcd_scroll, .lines = 0, .ticks = 12, .delay = HZ/2, @@ -71,7 +68,6 @@ struct scroll_screen_info lcd_scroll_info = #ifdef HAVE_REMOTE_LCD struct scroll_screen_info lcd_remote_scroll_info = { - .scroll = lcd_remote_scroll, .lines = 0, .ticks = 12, .delay = HZ/2, @@ -341,6 +337,7 @@ static void scroll_thread(void) void scroll_init(void) { + int i; #ifdef HAVE_REMOTE_LCD queue_init(&scroll_queue, true); #endif @@ -348,5 +345,21 @@ void scroll_init(void) sizeof(scroll_stack), 0, scroll_name IF_PRIO(, PRIORITY_USER_INTERFACE) IF_COP(, CPU)); + + struct scrollinfo *lcd_scroll = buffer_alloc(sizeof(struct scrollinfo) * + LCD_SCROLLABLE_LINES); + lcd_scroll_info.scroll = lcd_scroll; + + for (i = 0; i < LCD_SCROLLABLE_LINES; i++) + lcd_scroll[i].line = buffer_alloc(SCROLL_LINE_SIZE); +#ifdef HAVE_REMOTE_LCD + struct scrollinfo *lcd_remote_scroll; + lcd_remote_scroll = buffer_alloc(sizeof(struct scrollinfo) * + LCD_REMOTE_SCROLLABLE_LINES); + lcd_remote_scroll_info.scroll = lcd_remote_scroll; + + for (int i = 0; i < LCD_REMOTE_SCROLLABLE_LINES; i++) + lcd_remote_scroll[i].line = buffer_alloc(SCROLL_LINE_SIZE); +#endif } diff --git a/firmware/target/hosted/android/lcd-android.c b/firmware/target/hosted/android/lcd-android.c index 78b1f12..ca2fc5e 100644 --- a/firmware/target/hosted/android/lcd-android.c +++ b/firmware/target/hosted/android/lcd-android.c @@ -21,6 +21,7 @@ #include +#include #include "config.h" #include "system.h" #include "lcd.h" @@ -36,6 +37,8 @@ static jmethodID java_lcd_update_rect; static bool display_on; +int lcd_width = 320, lcd_height = 480; + void lcd_init_device(void) { JNIEnv e = *env_ptr; @@ -47,26 +50,37 @@ void lcd_init_device(void) * can create a Bitmap which directly maps to it **/ - /* map the framebuffer to a ByteBuffer, this way lcd updates will - * be directly feched from the framebuffer */ - jobject buf = e->NewDirectByteBuffer(env_ptr, - lcd_framebuffer, - (jlong)sizeof(lcd_framebuffer)); jmethodID constructor = e->GetMethodID(env_ptr, RockboxFramebuffer_class, "", "(Landroid/content/Context;" /* Service */ - "II" /* lcd width/height */ - "Ljava/nio/ByteBuffer;)V"); /* ByteBuffer */ + ")V"); + jmethodID fb_setter = e->GetMethodID(env_ptr, + RockboxFramebuffer_class, + "set_native_buf", + "(Ljava/nio/ByteBuffer;)V"); /* ByteBuffer */ RockboxFramebuffer_instance = e->NewObject(env_ptr, RockboxFramebuffer_class, constructor, - RockboxService_instance, - (jint)LCD_WIDTH, - (jint)LCD_HEIGHT, - buf); + RockboxService_instance); + + jfieldID jlcd_width = e->GetFieldID(env_ptr, RockboxFramebuffer_class, + "lcd_width", "I"); + lcd_width = e->GetIntField(env_ptr, RockboxFramebuffer_instance, jlcd_width); + jfieldID jlcd_height = e->GetFieldID(env_ptr, RockboxFramebuffer_class, + "lcd_height", "I"); + lcd_height = e->GetIntField(env_ptr, RockboxFramebuffer_instance, jlcd_height); + size_t fb_size = lcd_width * lcd_height * sizeof(fb_data); + + lcd_framebuffer = malloc(fb_size); + /* map the framebuffer to a ByteBuffer, this way lcd updates will + * be directly feched from the framebuffer */ + jobject buf = e->NewDirectByteBuffer(env_ptr, + lcd_framebuffer, + fb_size); + e->CallVoidMethod(env_ptr, RockboxFramebuffer_instance, fb_setter, buf); /* cache update functions */ java_lcd_update = (*env_ptr)->GetMethodID(env_ptr, @@ -152,6 +166,8 @@ void lcd_yuv_set_options(unsigned options) (void)options; } +#define LCD_PIXEL(x, y) (lcd_framebuffer + y * lcd_width + x) + /* Draw a partial YUV colour bitmap - similiar behavior to lcd_blit_yuv in the core */ void lcd_blit_yuv(unsigned char * const src[3], @@ -167,13 +183,16 @@ void lcd_blit_yuv(unsigned char * const src[3], width &= ~1; linecounter = height >> 1; -#if LCD_WIDTH >= LCD_HEIGHT - dst = &lcd_framebuffer[y][x]; - row_end = dst + width; -#else - dst = &lcd_framebuffer[x][LCD_WIDTH - y - 1]; - row_end = dst + LCD_WIDTH * width; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + { + dst = LCD_PIXEL(x, y); + row_end = dst + width; + } + else + { + dst = LCD_PIXEL(x, LCD_WIDTH - y - 1); + row_end = dst + LCD_WIDTH * width; + } z = stride * src_y; ysrc = src[0] + z + src_x; @@ -212,11 +231,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; y = YFAC*(*ysrc++ - 16); r = y + rv; @@ -232,11 +250,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; } while (dst < row_end); @@ -244,13 +261,16 @@ void lcd_blit_yuv(unsigned char * const src[3], usrc -= width >> 1; vsrc -= width >> 1; -#if LCD_WIDTH >= LCD_HEIGHT - row_end += LCD_WIDTH; - dst += LCD_WIDTH - width; -#else - row_end -= 1; - dst -= LCD_WIDTH*width + 1; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + { + row_end += LCD_WIDTH; + dst += LCD_WIDTH - width; + } + else + { + row_end -= 1; + dst -= LCD_WIDTH*width + 1; + } do { @@ -277,11 +297,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; y = YFAC*(*ysrc++ - 16); r = y + rv; @@ -297,11 +316,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; } while (dst < row_end); @@ -309,19 +327,21 @@ void lcd_blit_yuv(unsigned char * const src[3], usrc += stride >> 1; vsrc += stride >> 1; -#if LCD_WIDTH >= LCD_HEIGHT - row_end += LCD_WIDTH; - dst += LCD_WIDTH - width; -#else - row_end -= 1; - dst -= LCD_WIDTH*width + 1; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + { + row_end += LCD_WIDTH; + dst += LCD_WIDTH - width; + } + else + { + row_end -= 1; + dst -= LCD_WIDTH*width + 1; + } } while (--linecounter > 0); -#if LCD_WIDTH >= LCD_HEIGHT - lcd_update_rect(x, y, width, height); -#else - lcd_update_rect(LCD_WIDTH - y - height, x, height, width); -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + lcd_update_rect(x, y, width, height); + else + lcd_update_rect(LCD_WIDTH - y - height, x, height, width); } diff --git a/firmware/target/hosted/sdl/lcd-bitmap.c b/firmware/target/hosted/sdl/lcd-bitmap.c index 7058b26..f460bfa 100644 --- a/firmware/target/hosted/sdl/lcd-bitmap.c +++ b/firmware/target/hosted/sdl/lcd-bitmap.c @@ -26,6 +26,9 @@ #include "screendump.h" SDL_Surface* lcd_surface; +#ifdef DYNAMIC_LCD_SIZE +extern int lcd_width, lcd_height; +#endif #if LCD_DEPTH <= 8 #ifdef HAVE_BACKLIGHT @@ -84,6 +87,8 @@ unsigned long (*lcd_ex_getpixel)(int, int) = NULL; static const unsigned char colorindex[4] = {128, 85, 43, 0}; #endif +#define LCD_PIXEL(x, y) (lcd_framebuffer + y * LCD_WIDTH + x) + static unsigned long get_lcd_pixel(int x, int y) { #if LCD_DEPTH == 1 @@ -109,7 +114,7 @@ static unsigned long get_lcd_pixel(int x, int y) #if defined(LCD_STRIDEFORMAT) && LCD_STRIDEFORMAT == VERTICAL_STRIDE return *(&lcd_framebuffer[0][0]+LCD_HEIGHT*x+y); #else - return lcd_framebuffer[y][x]; + return *LCD_PIXEL(x, y); #endif #endif #endif @@ -126,7 +131,7 @@ void lcd_update_rect(int x_start, int y_start, int width, int height) sdl_update_rect(lcd_surface, x_start, y_start, width, height, LCD_WIDTH, LCD_HEIGHT, get_lcd_pixel); sdl_gui_update(lcd_surface, x_start, y_start, width, - height + LCD_SPLIT_LINES, SIM_LCD_WIDTH, SIM_LCD_HEIGHT, + height + LCD_SPLIT_LINES, LCD_WIDTH, LCD_HEIGHT, background ? UI_LCD_POSX : 0, background? UI_LCD_POSY : 0); } @@ -154,7 +159,7 @@ void sim_backlight(int value) #endif /* LCD_DEPTH */ sdl_gui_update(lcd_surface, 0, 0, SIM_LCD_WIDTH, SIM_LCD_HEIGHT, - SIM_LCD_WIDTH, SIM_LCD_HEIGHT, + LCD_WIDTH, LCD_HEIGHT, background ? UI_LCD_POSX : 0, background? UI_LCD_POSY : 0); } #endif /* HAVE_BACKLIGHT */ @@ -162,10 +167,14 @@ void sim_backlight(int value) /* initialise simulator lcd driver */ void lcd_init_device(void) { +#ifdef DYNAMIC_LCD_SIZE + lcd_framebuffer = malloc(lcd_width * lcd_height * sizeof(fb_data)); +#endif + #if LCD_DEPTH == 16 lcd_surface = SDL_CreateRGBSurface(SDL_SWSURFACE, - SIM_LCD_WIDTH * display_zoom, - SIM_LCD_HEIGHT * display_zoom, + LCD_WIDTH * display_zoom, + LCD_HEIGHT * display_zoom, LCD_DEPTH, 0, 0, 0, 0); #elif LCD_DEPTH <= 8 lcd_surface = SDL_CreateRGBSurface(SDL_SWSURFACE, @@ -203,7 +212,7 @@ void sim_lcd_ex_update_rect(int x_start, int y_start, int width, int height) sdl_update_rect(lcd_surface, x_start, y_start, width, height, LCD_WIDTH, LCD_HEIGHT, lcd_ex_getpixel); sdl_gui_update(lcd_surface, x_start, y_start, width, - height + LCD_SPLIT_LINES, SIM_LCD_WIDTH, SIM_LCD_HEIGHT, + height + LCD_SPLIT_LINES, LCD_WIDTH, LCD_HEIGHT, background ? UI_LCD_POSX : 0, background ? UI_LCD_POSY : 0); } @@ -255,13 +264,16 @@ void lcd_blit_yuv(unsigned char * const src[3], width &= ~1; linecounter = height >> 1; -#if LCD_WIDTH >= LCD_HEIGHT - dst = &lcd_framebuffer[y][x]; - row_end = dst + width; -#else - dst = &lcd_framebuffer[x][LCD_WIDTH - y - 1]; - row_end = dst + LCD_WIDTH * width; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + { + dst = LCD_PIXEL(x, y); + row_end = dst + width; + } + else + { + dst = LCD_PIXEL(x, LCD_WIDTH - y - 1); + row_end = dst + LCD_WIDTH * width; + } z = stride * src_y; ysrc = src[0] + z + src_x; @@ -300,11 +312,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; y = YFAC*(*ysrc++ - 16); r = y + rv; @@ -320,11 +331,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; } while (dst < row_end); @@ -332,13 +342,16 @@ void lcd_blit_yuv(unsigned char * const src[3], usrc -= width >> 1; vsrc -= width >> 1; -#if LCD_WIDTH >= LCD_HEIGHT - row_end += LCD_WIDTH; - dst += LCD_WIDTH - width; -#else - row_end -= 1; - dst -= LCD_WIDTH*width + 1; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + { + row_end += LCD_WIDTH; + dst += LCD_WIDTH - width; + } + else + { + row_end -= 1; + dst -= LCD_WIDTH*width + 1; + } do { @@ -365,11 +378,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; y = YFAC*(*ysrc++ - 16); r = y + rv; @@ -385,11 +397,10 @@ void lcd_blit_yuv(unsigned char * const src[3], *dst = LCD_RGBPACK_LCD(r >> 9, g >> 8, b >> 9); -#if LCD_WIDTH >= LCD_HEIGHT - dst++; -#else - dst += LCD_WIDTH; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + dst++; + else + dst += LCD_WIDTH; } while (dst < row_end); @@ -397,20 +408,22 @@ void lcd_blit_yuv(unsigned char * const src[3], usrc += stride >> 1; vsrc += stride >> 1; -#if LCD_WIDTH >= LCD_HEIGHT - row_end += LCD_WIDTH; - dst += LCD_WIDTH - width; -#else - row_end -= 1; - dst -= LCD_WIDTH*width + 1; -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + { + row_end += LCD_WIDTH; + dst += LCD_WIDTH - width; + } + else + { + row_end -= 1; + dst -= LCD_WIDTH*width + 1; + } } while (--linecounter > 0); -#if LCD_WIDTH >= LCD_HEIGHT - lcd_update_rect(x, y, width, height); -#else - lcd_update_rect(LCD_WIDTH - y - height, x, height, width); -#endif + if (LCD_WIDTH >= LCD_HEIGHT) + lcd_update_rect(x, y, width, height); + else + lcd_update_rect(LCD_WIDTH - y - height, x, height, width); } #endif /* HAVE_LCD_COLOR */ diff --git a/firmware/target/hosted/sdl/system-sdl.c b/firmware/target/hosted/sdl/system-sdl.c index 6937c37..3d7729a 100644 --- a/firmware/target/hosted/sdl/system-sdl.c +++ b/firmware/target/hosted/sdl/system-sdl.c @@ -54,6 +54,10 @@ char having_new_lcd = true; /* Used for player simulator */ bool sim_alarm_wakeup = false; const char *sim_root_dir = NULL; extern int display_zoom; +#ifdef DYNAMIC_LCD_SIZE +int lcd_width = 320; +int lcd_height = 480; +#endif static SDL_Thread *evt_thread = NULL; @@ -208,6 +212,19 @@ void sys_handle_argv(int argc, char *argv[]) background = false; printf("Disabling background image.\n"); } +#ifdef DYNAMIC_LCD_SIZE + else if (!strcmp("--display", argv[x])) + { + x++; + if(x < argc) + { + /* x */ + char *xchar = strchr(argv[x], 'x'); + lcd_width = atoi(argv[x]); + lcd_height = atoi(xchar+1); + } + } +#endif #ifdef HAVE_REMOTE_LCD else if (!strcmp("--noremote", argv[x])) {