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

Inki Dae inki.dae at samsung.com
Mon Aug 31 06:25:43 PDT 2015


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 at 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;
> 



More information about the dri-devel mailing list