@@ -135,7 +135,7 @@ static int g2d_add_cmd(struct g2d_context *ctx, unsigned long cmd,
break;
}
- return TRUE;
+ return 0;
}
/*
@@ -187,7 +187,7 @@ static int g2d_flush(struct g2d_context *ctx)
struct drm_exynos_g2d_set_cmdlist cmdlist = {0};
if (ctx->cmd_nr == 0 && ctx->cmd_buf_nr == 0)
- return FALSE;
+ return -1;
if (ctx->cmdlist_nr >= G2D_MAX_CMD_LIST_NR) {
fprintf(stderr, "Overflow cmdlist.\n");
@@ -13,13 +13,6 @@
#ifndef _FIMG2D_H_
#define _FIMG2D_H_
-#ifndef TRUE
-#define TRUE 0
-#endif
-#ifndef FALSE
-#define FALSE -1
-#endif
-
#define G2D_MAX_CMD_NR 64
#define G2D_MAX_GEM_CMD_NR 64
#define G2D_MAX_CMD_LIST_NR 64
The fimg2d header was defining TRUE and FALSE, but actually these defines are just used once. Remove them, since they don't make the code better readable/understandable. Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de> --- exynos/exynos_fimg2d.c | 4 ++-- exynos/exynos_fimg2d.h | 7 ------- 2 files changed, 2 insertions(+), 9 deletions(-)