1
0
Fork 0
forked from len0rd/rockbox

Changed the LCD_COLOR pixel value format to packed RGB (unsigned int). Now all LCDs with depth > 1 use the same datatype. Added macros for easy pixel value definition.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@7912 a1c6a512-1295-4272-9138-f99709370657
This commit is contained in:
Jens Arnold 2005-11-16 21:09:23 +00:00
parent 15046f99b3
commit 5b2cba17aa
15 changed files with 94 additions and 148 deletions

View file

@ -282,17 +282,14 @@ static void addclock(void)
#define DRAW_WIDTH (LCD_WIDTH + LETTER_WIDTH*2)
#if LCD_DEPTH > 1
static const unsigned face_colors[] =
{
#ifdef HAVE_LCD_COLOR
static const struct rgb face_colors[] =
{
LCD_BLACK, {0, 0, LCD_MAX_BLUE}, {LCD_MAX_RED, 0, 0}
};
LCD_BLACK, LCD_RGBPACK(0, 0, 255), LCD_RGBPACK(255, 0, 0)
#else
static const int face_colors[] =
{
0, 2*LCD_MAX_LEVEL/3, LCD_MAX_LEVEL/3
};
LCD_BLACK, LCD_LIGHTGRAY, LCD_DARKGRAY
#endif
};
#endif
static int scrollit(void)

View file

@ -152,18 +152,16 @@ static const struct face faces[6] =
};
#if LCD_DEPTH > 1
static const unsigned face_colors[6] =
{
#ifdef HAVE_LCD_COLOR
static const struct rgb face_colors[6] =
{
{LCD_MAX_RED, 0, 0}, {LCD_MAX_RED, 0, 0}, {0, LCD_MAX_GREEN, 0},
{0, LCD_MAX_GREEN, 0}, {0, 0, LCD_MAX_BLUE}, {0, 0, LCD_MAX_BLUE}
};
LCD_RGBPACK(255, 0, 0), LCD_RGBPACK(255, 0, 0), LCD_RGBPACK(0, 255, 0),
LCD_RGBPACK(0, 255, 0), LCD_RGBPACK(0, 0, 255), LCD_RGBPACK(0, 0, 255)
#else
static const int face_colors[6] =
{
2*LCD_MAX_LEVEL/3, 2*LCD_MAX_LEVEL/3, LCD_MAX_LEVEL/3, LCD_MAX_LEVEL/3, 0, 0
};
LCD_LIGHTGRAY, LCD_LIGHTGRAY, LCD_DARKGRAY,
LCD_DARKGRAY, LCD_BLACK, LCD_BLACK
#endif
};
#endif
enum {

View file

@ -51,11 +51,11 @@ void gray_update_rect(int x, int y, int width, int height);
void gray_set_position(int x, int by);
void gray_set_drawmode(int mode);
int gray_get_drawmode(void);
void gray_set_foreground(int brightness);
int gray_get_foreground(void);
void gray_set_background(int brightness);
int gray_get_background(void);
void gray_set_drawinfo(int mode, int fg_brightness, int bg_brightness);
void gray_set_foreground(unsigned brightness);
unsigned gray_get_foreground(void);
void gray_set_background(unsigned brightness);
unsigned gray_get_background(void);
void gray_set_drawinfo(int mode, unsigned fg_brightness, unsigned bg_brightness);
void gray_setfont(int newfont);
int gray_getstringsize(const unsigned char *str, int *w, int *h);

View file

@ -53,7 +53,7 @@ int gray_get_drawmode(void)
}
/* Set the foreground shade for subsequent drawing operations */
void gray_set_foreground(int brightness)
void gray_set_foreground(unsigned brightness)
{
unsigned data = MULU16(_gray_info.depth, brightness & 0xFF) + 127;
@ -61,14 +61,14 @@ void gray_set_foreground(int brightness)
}
/* Return the current foreground shade */
int gray_get_foreground(void)
unsigned gray_get_foreground(void)
{
return (_gray_info.fg_brightness * 255 + (_gray_info.depth >> 1))
/ _gray_info.depth;
}
/* Set the background shade for subsequent drawing operations */
void gray_set_background(int brightness)
void gray_set_background(unsigned brightness)
{
unsigned data = MULU16(_gray_info.depth, brightness & 0xFF) + 127;
@ -76,14 +76,14 @@ void gray_set_background(int brightness)
}
/* Return the current background shade */
int gray_get_background(void)
unsigned gray_get_background(void)
{
return (_gray_info.bg_brightness * 255 + (_gray_info.depth >> 1))
/ _gray_info.depth;
}
/* Set draw mode, foreground and background shades at once */
void gray_set_drawinfo(int mode, int fg_brightness, int bg_brightness)
void gray_set_drawinfo(int mode, unsigned fg_brightness, unsigned bg_brightness)
{
gray_set_drawmode(mode);
gray_set_foreground(fg_brightness);

View file

@ -67,16 +67,6 @@ use F3 to see how many mines are left (supposing all your flags are correct)
#endif
#if LCD_DEPTH > 1
#if HAVE_LCD_COLOR
#define LIGHT_GRAY ((struct rgb){2*LCD_MAX_RED/3, 2*LCD_MAX_GREEN/3, 2*LCD_MAX_BLUE/3})
#define DARK_GRAY ((struct rgb){LCD_MAX_RED/3, LCD_MAX_GREEN/3, LCD_MAX_BLUE/3})
#else
#define LIGHT_GRAY (2*LCD_MAX_LEVEL/3)
#define DARK_GRAY (LCD_MAX_LEVEL/3)
#endif
#endif
/* here is a global api struct pointer. while not strictly necessary,
it's nice not to have to pass the api pointer in all function calls
in the plugin */
@ -385,7 +375,7 @@ int minesweeper(void)
for(i=0;i<height;i++){
for(j=0;j<width;j++){
#if LCD_DEPTH > 1
rb->lcd_set_foreground(DARK_GRAY);
rb->lcd_set_foreground(LCD_DARKGRAY);
rb->lcd_drawrect(j*8,i*8,8,8);
rb->lcd_set_foreground(LCD_BLACK);
#else
@ -404,7 +394,7 @@ int minesweeper(void)
rb->lcd_drawline(j*8+2,i*8+5,j*8+5,i*8+2);
} else {
#if LCD_DEPTH > 1
rb->lcd_set_foreground(LIGHT_GRAY);
rb->lcd_set_foreground(LCD_LIGHTGRAY);
rb->lcd_fillrect(j*8+1,i*8+1,6,6);
rb->lcd_set_foreground(LCD_BLACK);
#else

View file

@ -59,10 +59,10 @@
#endif
#if LCD_DEPTH > 1
#if HAVE_LCD_COLOR
#define MEDIUM_GRAY ((struct rgb){LCD_MAX_RED/2, LCD_MAX_GREEN/2, LCD_MAX_BLUE/2})
#ifdef HAVE_LCD_COLOR
#define MEDIUM_GRAY LCD_RGBPACK(127, 127, 127)
#else
#define MEDIUM_GRAY (LCD_MAX_LEVEL/2)
#define MEDIUM_GRAY LCD_BRIGHTNESS(127)
#endif
#endif

View file

@ -135,13 +135,13 @@ static struct plugin_api* rb;
#endif
#if LCD_DEPTH>1
static const unsigned colors[4] = {
#ifdef HAVE_LCD_COLOR
static const unsigned struct rgb colors[4] = {
{ 0, 0, 0 }, { LCD_MAX_RED, 0, 0 }, { 0, 0, 0 }, { LCD_MAX_RED, 0, 0 }
};
LCD_BLACK, LCD_RGBPACK(255, 0, 0), LCD_BLACK, LCD_RGBPACK(255, 0, 0)
#else
static const int colors[4] = { LCD_BLACK, LCD_MAX_LEVEL/2, LCD_BLACK, LCD_MAX_LEVEL/2 };
LCD_BLACK, LCD_BRIGHTNESS(127), LCD_BLACK, LCD_BRIGHTNESS(127)
#endif
};
#endif
static const unsigned char suits[4][8] = {

View file

@ -362,16 +362,6 @@ static unsigned char num_inverse[10][8]= {
#error SUDOKU: Unsupported LCD size
#endif
#if LCD_DEPTH > 1
#if HAVE_LCD_COLOR
#define LIGHT_GRAY ((struct rgb){2*LCD_MAX_RED/3, 2*LCD_MAX_GREEN/3, 2*LCD_MAX_BLUE/3})
#define DARK_GRAY ((struct rgb){LCD_MAX_RED/3, LCD_MAX_GREEN/3, LCD_MAX_BLUE/3})
#else
#define LIGHT_GRAY (2*LCD_MAX_LEVEL/3)
#define DARK_GRAY (LCD_MAX_LEVEL/3)
#endif
#endif
/* here is a global api struct pointer. while not strictly necessary,
it's nice not to have to pass the api pointer in all function calls
in the plugin */