[PATCH] drm/vc4: Fix the "no scaling" case on multi-planar YUV formats

Eric Anholt eric at anholt.net
Fri Jul 27 20:55:03 UTC 2018


Boris Brezillon <boris.brezillon at bootlin.com> writes:

> 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 at vger.kernel.org>
> Signed-off-by: Boris Brezillon <boris.brezillon at bootlin.com>

Reviewed-by: Eric Anholt <eric at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180727/5fec858d/attachment.sig>


More information about the dri-devel mailing list