Re: [PATCH] drm/tegra: Expose color key and plane blending controls to userspace

From: Thierry Reding
Date: Fri Sep 02 2016 - 11:32:30 EST


On Fri, Sep 02, 2016 at 12:33:42PM +0300, Dmitry Osipenko wrote:
> Chromakey is a simple way of video overlay overlap implementation. This
> patch adds 2 new IOCTL's: first - sets color key and is common across of
> all Tegra SoC's, second - sets plane blending controls and allows to
> utilize the color key, this one is exclusive to Tegra20/30.
>
> Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx>
> ---
> drivers/gpu/drm/tegra/dc.c | 150 +++++++++++++++++++++++++++++++++-------
> drivers/gpu/drm/tegra/dc.h | 6 ++
> drivers/gpu/drm/tegra/drm.c | 159 +++++++++++++++++++++++++++++++++++++++++++
> drivers/gpu/drm/tegra/drm.h | 14 ++++
> include/uapi/drm/tegra_drm.h | 34 +++++++++
> 5 files changed, 337 insertions(+), 26 deletions(-)

I think these are really nice features to have, but these would need to
be exposed as properties, rather than custom driver-specific IOCTLs. It
seems to me like the colorkey feature could be implemented in much the
same way as in the Armada, RCar and Nouveau drivers.

As for the blending options, I think they should be exposed in terms of
the zpos property, to allow generic userspace to make use of them. Also
can you explain why this needs to be exclusive to Tegra20 and Tegra30?

Thierry

Attachment: signature.asc
Description: PGP signature