2017년 08월 22일 23:19에 Tobias Jakobi 이(가) 쓴 글: > The current comment sounds like the division op is done to > compensate for some hardware erratum. But the chroma plane > having half the height of the luma plane is just the way > NV12/NV21 is defined, so clarify this behaviour. > > Signed-off-by: Tobias Jakobi <tjakobi@xxxxxxxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/exynos/exynos_mixer.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c > index a998a8dd783c..c6d6f6d42900 100644 > --- a/drivers/gpu/drm/exynos/exynos_mixer.c > +++ b/drivers/gpu/drm/exynos/exynos_mixer.c > @@ -532,7 +532,7 @@ static void vp_video_buffer(struct mixer_context *ctx, > /* setting size of input image */ > vp_reg_write(res, VP_IMG_SIZE_Y, VP_IMG_HSIZE(fb->pitches[0]) | > VP_IMG_VSIZE(fb->height)); > - /* chroma height has to reduced by 2 to avoid chroma distorions */ > + /* the chroma plane for NV12/NV21 is half the height of the luma plane */ WARNING: line over 80 characters #86: FILE: drivers/gpu/drm/exynos/exynos_mixer.c:535: + /* the chroma plane for NV12/NV21 is half the height of the luma plane */ I just removed a word, 'the', in front of 'chroma' word. Thanks, Inki Dae > vp_reg_write(res, VP_IMG_SIZE_C, VP_IMG_HSIZE(fb->pitches[0]) | > VP_IMG_VSIZE(fb->height / 2)); > > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
