diff options
| author | Thorsten Leemhuis <fedora@leemhuis.info> | 2018-09-17 10:25:17 +0200 |
|---|---|---|
| committer | Thorsten Leemhuis <fedora@leemhuis.info> | 2018-09-17 10:25:17 +0200 |
| commit | 55110e107ebd2f5aa5de2cb0d2308572ed08a039 (patch) | |
| tree | 080d1133e366420cc767baacf10698e1dec2c958 /drm-vc4-Fix-the-no-scaling-case-on-multi-planar-YUV-formats.patch | |
| parent | edd9d30f1568309a66331272bab541a20690e07e (diff) | |
| parent | 5cd25ebdc6fd8ccd7ba001da9b52644e93d9d9ba (diff) | |
| download | kernel-4.18.8-150.vanilla.knurd.1.fc27.tar.gz kernel-4.18.8-150.vanilla.knurd.1.fc27.tar.xz kernel-4.18.8-150.vanilla.knurd.1.fc27.zip | |
Merge remote-tracking branch 'origin/f27' into f27-user-thl-vanilla-fedorakernel-4.18.8-150.vanilla.knurd.1.fc27
Diffstat (limited to 'drm-vc4-Fix-the-no-scaling-case-on-multi-planar-YUV-formats.patch')
| -rw-r--r-- | drm-vc4-Fix-the-no-scaling-case-on-multi-planar-YUV-formats.patch | 88 |
1 files changed, 88 insertions, 0 deletions
diff --git a/drm-vc4-Fix-the-no-scaling-case-on-multi-planar-YUV-formats.patch b/drm-vc4-Fix-the-no-scaling-case-on-multi-planar-YUV-formats.patch new file mode 100644 index 000000000..195ced13b --- /dev/null +++ b/drm-vc4-Fix-the-no-scaling-case-on-multi-planar-YUV-formats.patch @@ -0,0 +1,88 @@ +From patchwork Wed Jul 25 12:29:07 2018 +Content-Type: text/plain; charset="utf-8" +MIME-Version: 1.0 +Content-Transfer-Encoding: 7bit +Subject: drm/vc4: Fix the "no scaling" case on multi-planar YUV formats +From: Boris Brezillon <boris.brezillon@bootlin.com> +X-Patchwork-Id: 240917 +Message-Id: <20180725122907.13702-1-boris.brezillon@bootlin.com> +To: Eric Anholt <eric@anholt.net> +Cc: David Airlie <airlied@linux.ie>, + Boris Brezillon <boris.brezillon@bootlin.com>, stable@vger.kernel.org, + dri-devel@lists.freedesktop.org +Date: Wed, 25 Jul 2018 14:29:07 +0200 + +When there's no scaling requested ->is_unity should be true no matter +the format. + +Also, when no scaling is requested and we have a multi-planar YUV +format, we should leave ->y_scaling[0] to VC4_SCALING_NONE and only +set ->x_scaling[0] to VC4_SCALING_PPF. + +Doing this fixes an hardly visible artifact (seen when using modetest +and a rather big overlay plane in YUV420). + +Fixes: fc04023fafec ("drm/vc4: Add support for YUV planes.") +Cc: <stable@vger.kernel.org> +Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com> +Reviewed-by: Eric Anholt <eric@anholt.net> +--- + drivers/gpu/drm/vc4/vc4_plane.c | 25 ++++++++++++------------- + 1 file changed, 12 insertions(+), 13 deletions(-) + +diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c +index cfb50fedfa2b..a3275fa66b7b 100644 +--- a/drivers/gpu/drm/vc4/vc4_plane.c ++++ b/drivers/gpu/drm/vc4/vc4_plane.c +@@ -297,6 +297,9 @@ static int vc4_plane_setup_clipping_and_scaling(struct drm_plane_state *state) + vc4_state->y_scaling[0] = vc4_get_scaling_mode(vc4_state->src_h[0], + vc4_state->crtc_h); + ++ vc4_state->is_unity = (vc4_state->x_scaling[0] == VC4_SCALING_NONE && ++ vc4_state->y_scaling[0] == VC4_SCALING_NONE); ++ + if (num_planes > 1) { + vc4_state->is_yuv = true; + +@@ -312,24 +315,17 @@ static int vc4_plane_setup_clipping_and_scaling(struct drm_plane_state *state) + vc4_get_scaling_mode(vc4_state->src_h[1], + vc4_state->crtc_h); + +- /* YUV conversion requires that scaling be enabled, +- * even on a plane that's otherwise 1:1. Choose TPZ +- * for simplicity. ++ /* YUV conversion requires that horizontal scaling be enabled, ++ * even on a plane that's otherwise 1:1. Looks like only PPF ++ * works in that case, so let's pick that one. + */ +- if (vc4_state->x_scaling[0] == VC4_SCALING_NONE) +- vc4_state->x_scaling[0] = VC4_SCALING_TPZ; +- if (vc4_state->y_scaling[0] == VC4_SCALING_NONE) +- vc4_state->y_scaling[0] = VC4_SCALING_TPZ; ++ if (vc4_state->is_unity) ++ vc4_state->x_scaling[0] = VC4_SCALING_PPF; + } else { + vc4_state->x_scaling[1] = VC4_SCALING_NONE; + vc4_state->y_scaling[1] = VC4_SCALING_NONE; + } + +- vc4_state->is_unity = (vc4_state->x_scaling[0] == VC4_SCALING_NONE && +- vc4_state->y_scaling[0] == VC4_SCALING_NONE && +- vc4_state->x_scaling[1] == VC4_SCALING_NONE && +- vc4_state->y_scaling[1] == VC4_SCALING_NONE); +- + /* No configuring scaling on the cursor plane, since it gets + non-vblank-synced updates, and scaling requires requires + LBM changes which have to be vblank-synced. +@@ -672,7 +668,10 @@ static int vc4_plane_mode_set(struct drm_plane *plane, + vc4_dlist_write(vc4_state, SCALER_CSC2_ITR_R_601_5); + } + +- if (!vc4_state->is_unity) { ++ if (vc4_state->x_scaling[0] != VC4_SCALING_NONE || ++ vc4_state->x_scaling[1] != VC4_SCALING_NONE || ++ vc4_state->y_scaling[0] != VC4_SCALING_NONE || ++ vc4_state->y_scaling[1] != VC4_SCALING_NONE) { + /* LBM Base Address. */ + if (vc4_state->y_scaling[0] != VC4_SCALING_NONE || + vc4_state->y_scaling[1] != VC4_SCALING_NONE) { |