geemili
/
drm
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

tests/exynos: handle G2D_IMGBUF_COLOR in switch statements 

This fixes a compiler warning about missing handling of enum
values in the switch statements.

Also remove the silent mapping to G2D_IMGBUF_GEM when an
unknown buffer type is encountered. We have full control
about the type here, and if it's unknown then we obviously
have a bug in the code.

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
Acked-by: Emil Velikov <emil.l.velikov@gmail.com>
main
Tobias Jakobi 2015-06-12 20:15:14 +02:00 committed by Emil Velikov
parent b26c8efb65
commit 2bf0d5bf63
1 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
tests/exynos/exynos_fimg2d_test.c
View File

@ -270,9 +270,10 @@ static int g2d_copy_test(struct exynos_device *dev, struct exynos_bo *src,
src_img.user_ptr[0].userptr = userptr; src_img.user_ptr[0].userptr = userptr;
src_img.user_ptr[0].size = size; src_img.user_ptr[0].size = size;
break; break;
case G2D_IMGBUF_COLOR:
default: default:
type = G2D_IMGBUF_GEM; ret = -EFAULT;
break; goto fail;
} }
printf("copy test with %s.\n", printf("copy test with %s.\n",
@ -306,6 +307,7 @@ err_free_userptr:
if (userptr) if (userptr)
free((void *)userptr); free((void *)userptr);
fail:
g2d_fini(ctx); g2d_fini(ctx);
return ret; return ret;
@ -349,9 +351,10 @@ static int g2d_copy_with_scale_test(struct exynos_device *dev,
src_img.user_ptr[0].userptr = userptr; src_img.user_ptr[0].userptr = userptr;
src_img.user_ptr[0].size = size; src_img.user_ptr[0].size = size;
break; break;
case G2D_IMGBUF_COLOR:
default: default:
type = G2D_IMGBUF_GEM; ret = -EFAULT;
break; goto fail;
} }
printf("copy and scale test with %s.\n", printf("copy and scale test with %s.\n",
@ -390,6 +393,7 @@ err_free_userptr:
if (userptr) if (userptr)
free((void *)userptr); free((void *)userptr);
fail:
g2d_fini(ctx); g2d_fini(ctx);
return 0; return 0;
@ -435,9 +439,10 @@ static int g2d_blend_test(struct exynos_device *dev,
src_img.user_ptr[0].userptr = userptr; src_img.user_ptr[0].userptr = userptr;
src_img.user_ptr[0].size = size; src_img.user_ptr[0].size = size;
break; break;
case G2D_IMGBUF_COLOR:
default: default:
type = G2D_IMGBUF_GEM; ret = -EFAULT;
break; goto fail;
} }
printf("blend test with %s.\n", printf("blend test with %s.\n",
@ -487,6 +492,7 @@ err_free_userptr:
if (userptr) if (userptr)
free((void *)userptr); free((void *)userptr);
fail:
g2d_fini(ctx); g2d_fini(ctx);
return 0; return 0;
@ -532,6 +538,7 @@ static int g2d_checkerboard_test(struct exynos_device *dev,
src_img.user_ptr[0].userptr = (unsigned long)checkerboard; src_img.user_ptr[0].userptr = (unsigned long)checkerboard;
src_img.user_ptr[0].size = img_w * img_h * 4; src_img.user_ptr[0].size = img_w * img_h * 4;
break; break;
case G2D_IMGBUF_COLOR:
default: default:
ret = -EFAULT; ret = -EFAULT;
goto fail; goto fail;