From 15d76cb5fc61de83f5ff2e2d86911d18c6735507 Mon Sep 17 00:00:00 2001 From: Mateusz Tomaszkiewicz Date: Wed, 20 Mar 2013 22:15:18 +0100 Subject: [PATCH] SSD1289 SSD2119: fixed warning messages Few type castings in order to suppress warnings (-Wsign-compare): "comparison between signed and unsigned integer expressions" "signed and unsigned type in conditional expression" --- drivers/gdisp/SSD1289/gdisp_lld.c | 8 ++++---- drivers/gdisp/SSD2119/gdisp_lld.c | 8 ++++---- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/gdisp/SSD1289/gdisp_lld.c b/drivers/gdisp/SSD1289/gdisp_lld.c index e2dd79e0..f6531f2c 100644 --- a/drivers/gdisp/SSD1289/gdisp_lld.c +++ b/drivers/gdisp/SSD1289/gdisp_lld.c @@ -429,7 +429,7 @@ void gdisp_lld_draw_pixel(coord_t x, coord_t y, color_t color) { abslines = lines < 0 ? -lines : lines; acquire_bus(); - if (abslines >= cy) { + if ((coord_t)abslines >= cy) { abslines = cy; gap = 0; } else { @@ -447,20 +447,20 @@ void gdisp_lld_draw_pixel(coord_t x, coord_t y, color_t color) { set_viewport(x, row0, cx, 1); stream_start(); j = read_data(); // dummy read - for (j = 0; j < cx; j++) + for (j = 0; (coord_t)j < cx; j++) buf[j] = read_data(); stream_stop(); set_viewport(x, row1, cx, 1); stream_start(); - for (j = 0; j < cx; j++) + for (j = 0; (coord_t)j < cx; j++) write_data(buf[j]); stream_stop(); } } /* fill the remaining gap */ - set_viewport(x, lines > 0 ? (y+gap) : y, cx, abslines); + set_viewport(x, lines > 0 ? (y+(coord_t)gap) : y, cx, abslines); stream_start(); gap = cx*abslines; for(i = 0; i < gap; i++) write_data(bgcolor); diff --git a/drivers/gdisp/SSD2119/gdisp_lld.c b/drivers/gdisp/SSD2119/gdisp_lld.c index b8884e0f..a7ad7b1a 100644 --- a/drivers/gdisp/SSD2119/gdisp_lld.c +++ b/drivers/gdisp/SSD2119/gdisp_lld.c @@ -472,7 +472,7 @@ void gdisp_lld_draw_pixel(coord_t x, coord_t y, color_t color) { abslines = lines < 0 ? -lines : lines; acquire_bus(); - if (abslines >= cy) { + if ((coord_t)abslines >= cy) { abslines = cy; gap = 0; } else { @@ -490,20 +490,20 @@ void gdisp_lld_draw_pixel(coord_t x, coord_t y, color_t color) { set_viewport(x, row0, cx, 1); stream_start(); j = read_data(); // dummy read - for (j = 0; j < cx; j++) + for (j = 0; (coord_t)j < cx; j++) buf[j] = read_data(); stream_stop(); set_viewport(x, row1, cx, 1); stream_start(); - for (j = 0; j < cx; j++) + for (j = 0; (coord_t)j < cx; j++) write_data(buf[j]); stream_stop(); } } /* fill the remaining gap */ - set_viewport(x, lines > 0 ? (y+gap) : y, cx, abslines); + set_viewport(x, lines > 0 ? (y+(coord_t)gap) : y, cx, abslines); stream_start(); gap = cx*abslines; for(i = 0; i < gap; i++) write_data(bgcolor);