Re: [PATCH 10/14] exynos/fimg2d: remove default case from g2d_get_blend_op()

On 2015년 08월 24일 23:14, Tobias Jakobi wrote:

We now validate the blending mode via g2d_validate_mode() prior to feeding it to g2d_get_blend_op().

Signed-off-by: Tobias Jakobi tjakobi@math.uni-bielefeld.de

exynos/exynos_fimg2d.c | 5 ----- 1 file changed, 5 deletions(-)

diff --git a/exynos/exynos_fimg2d.c b/exynos/exynos_fimg2d.c index 4274a94..d708e91 100644 --- a/exynos/exynos_fimg2d.c +++ b/exynos/exynos_fimg2d.c @@ -91,11 +91,6 @@ static unsigned int g2d_get_blend_op(enum e_g2d_op op) SET_BF(val, G2D_COEFF_MODE_SRC_ALPHA, 0, 0, 0, G2D_COEFF_MODE_SRC_ALPHA, 1, 0, 0); break;

• default:

• fprintf(stderr, "Not support operation(%d).\n", op);
  

• SET_BF(val, G2D_COEFF_MODE_ONE, 0, 0, 0, G2D_COEFF_MODE_ZERO,
  

• 		0, 0, 0);
  

• break;
  

With this, how about changing above switch and case statement to if statement?

Thanks, Inki Dae

}

return val.val;