Hi Tobias, On 1 April 2015 at 14:29, Tobias Jakobi <tjakobi@xxxxxxxxxxxxxxxxxxxxx> wrote: > diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c > index 5ab0e32..6822b36 100644 > --- a/drivers/gpu/drm/exynos/exynos_mixer.c > +++ b/drivers/gpu/drm/exynos/exynos_mixer.c > @@ -528,9 +552,8 @@ static void mixer_graph_buffer(struct mixer_context *ctx, int win) > fmt = ARGB8888; > } > > - /* 2x scaling feature */ > - x_ratio = 0; > - y_ratio = 0; > + /* check if mixer supports requested scaling setup */ > + if (mixer_setup_scale(plane, &x_ratio, &y_ratio)) return; > Having return on the same line as the if statement will make checkpatch.pl unhappy :-( Cheers Emil -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html