Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1475252

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

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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@gmail.com>
> ---
>  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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] drm/tegra: Expose color key and plane blending controls to userspace Dmitry Osipenko <digetx@gmail.com> - 2016-09-02 11:40 +0200
  Re: [PATCH] drm/tegra: Expose color key and plane blending controls  to userspace Thierry Reding <thierry.reding@gmail.com> - 2016-09-02 17:40 +0200
    Re: [PATCH] drm/tegra: Expose color key and plane blending controls  to userspace Thierry Reding <thierry.reding@gmail.com> - 2016-09-02 18:00 +0200
      Re: [PATCH] drm/tegra: Expose color key and plane blending controls  to userspace Dmitry Osipenko <digetx@gmail.com> - 2016-09-04 20:40 +0200

csiph-web