Re: [PATCH 0/4] drm/exynos: introduce generic zpos property

[Inki Dae <inki.dae@samsung.com>]

Hi Marek,


2016년 01월 05일 21:52에 Marek Szyprowski 이(가) 쓴 글:
> Hello all,
> 
> This patch series is a continuation of rework of blending support in
> Exynos DRM driver. Some background can be found here:
> http://www.spinics.net/lists/dri-devel/msg96969.html
> 
> Daniel Vetter suggested that zpos property should be made generic, with
> well-defined semantics. This patchset is my proposal for such generic
> zpos property:
> - added zpos properties to drm core and plane state structures,
> - added helpers for normalizing zpos properties of given set of planes,
> - well defined semantics: planes are sorted by zpos values and then plane
>   id value if zpos equals.
> 
> Patches 2/4 and 3/4 are fixes for Exynos DRM driver, which are required

Picked them up. Especially, 2/4 is a good patch.

Thanks,
Inki Dae

> to properly implement generic zpos handling. However they can be also
> merged regardless of the rest of the patches (if there is a need for
> further discussion on the generic zpos property feature).
> 
> Patches are based on top of latest exynos-drm-next branch.
> 
> Best regards
> Marek Szyprowski
> Samsung R&D Institute Poland
> 
> 
> Patch summary:
> 
> Marek Szyprowski (4):
>   drm: add generic zpos property
>   drm/exynos: crtc: rework atomic_{begin,flush}
>   drm/exynos: mixer: properly update all planes on the same vblank event
>   drm/exynos: use generic code for managing zpos plane property
> 
>  Documentation/DocBook/gpu.tmpl                | 14 +++++-
>  drivers/gpu/drm/drm_atomic.c                  |  4 ++
>  drivers/gpu/drm/drm_atomic_helper.c           | 52 ++++++++++++++++++++
>  drivers/gpu/drm/drm_crtc.c                    | 13 +++++
>  drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 14 +++---
>  drivers/gpu/drm/exynos/exynos7_drm_decon.c    | 14 +++---
>  drivers/gpu/drm/exynos/exynos_drm_crtc.c      | 20 ++------
>  drivers/gpu/drm/exynos/exynos_drm_drv.h       | 11 ++---
>  drivers/gpu/drm/exynos/exynos_drm_fimd.c      | 14 +++---
>  drivers/gpu/drm/exynos/exynos_drm_plane.c     | 68 +++++++--------------------
>  drivers/gpu/drm/exynos/exynos_mixer.c         | 53 ++++++++++++++++-----
>  include/drm/drm_atomic_helper.h               |  2 +
>  include/drm/drm_crtc.h                        | 13 +++++
>  13 files changed, 187 insertions(+), 105 deletions(-)
>