Hi Ville, Thank you for the patch. On Wednesday 17 April 2013 22:18:01 ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > tests/modetest/buffers.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c > index 5086381..6b117b4 100644 > --- a/tests/modetest/buffers.c > +++ b/tests/modetest/buffers.c > @@ -601,7 +601,7 @@ fill_smpte(const struct format_info *info, void > *planes[3], unsigned int width, #define BLUE 0 > > static void > -make_pwetty(void *data, int width, int height, int stride) > +make_pwetty(void *data, int width, int height, int stride, int rgb16) What about passing the format 4cc instead ? > { > #ifdef HAVE_CAIRO > cairo_surface_t *surface; > @@ -609,7 +609,7 @@ make_pwetty(void *data, int width, int height, int > stride) int x, y; > > surface = cairo_image_surface_create_for_data(data, > - CAIRO_FORMAT_ARGB32, > + rgb16 ? CAIRO_FORMAT_RGB16_565 : CAIRO_FORMAT_ARGB32, > width, height, > stride); > cr = cairo_create(surface); > @@ -716,6 +716,7 @@ static void > fill_tiles_rgb16(const struct rgb_info *rgb, unsigned char *mem, > unsigned int width, unsigned int height, unsigned int stride) > { > + unsigned char *mem_base = mem; > unsigned int x, y; > > for (y = 0; y < height; ++y) { > @@ -732,6 +733,8 @@ fill_tiles_rgb16(const struct rgb_info *rgb, unsigned > char *mem, } > mem += stride; > } > + > + make_pwetty(mem_base, width, height, stride, 1); > } > > static void > @@ -777,7 +780,7 @@ fill_tiles_rgb32(const struct rgb_info *rgb, unsigned > char *mem, mem += stride; > } > > - make_pwetty(mem_base, width, height, stride); > + make_pwetty(mem_base, width, height, stride, 0); > } > > static void -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel