aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTobias Jakobi <tjakobi@math.uni-bielefeld.de>2017-08-22 10:19:36 -0400
committerInki Dae <inki.dae@samsung.com>2017-08-24 23:03:52 -0400
commitdc500cfb869d776e78e4a62b20b65f8208e2c695 (patch)
treedb3f043ae8f41350a75090ea350e67c212c74634
parent88a5e22a90efd12dfde642ed55d4c669ab658b0f (diff)
drm/exynos: mixer: fix chroma comment in vp_video_buffer()
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@math.uni-bielefeld.de> Signed-off-by: Inki Dae <inki.dae@samsung.com>
-rw-r--r--drivers/gpu/drm/exynos/exynos_mixer.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c
index a998a8dd783c..cbd949a989f9 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,
532 /* setting size of input image */ 532 /* setting size of input image */
533 vp_reg_write(res, VP_IMG_SIZE_Y, VP_IMG_HSIZE(fb->pitches[0]) | 533 vp_reg_write(res, VP_IMG_SIZE_Y, VP_IMG_HSIZE(fb->pitches[0]) |
534 VP_IMG_VSIZE(fb->height)); 534 VP_IMG_VSIZE(fb->height));
535 /* chroma height has to reduced by 2 to avoid chroma distorions */ 535 /* chroma plane for NV12/NV21 is half the height of the luma plane */
536 vp_reg_write(res, VP_IMG_SIZE_C, VP_IMG_HSIZE(fb->pitches[0]) | 536 vp_reg_write(res, VP_IMG_SIZE_C, VP_IMG_HSIZE(fb->pitches[0]) |
537 VP_IMG_VSIZE(fb->height / 2)); 537 VP_IMG_VSIZE(fb->height / 2));
538 538