Re: [PATCH v5 0/4] Generic zpos property

[Tobias Jakobi <tjakobi@math.uni-bielefeld.de>]

Hello Benjamin,


Benjamin Gaignard wrote:
> version 5:
> rebased on drm-next where Documentation/DocBook/gpu.tmpl doesn't
> exist anymore.
I think the documentation has just moved to Documentation/gpu, so the
zpos property should be documented there then.


With best wishes,
Tobias


> rework sti patch because some plane functions have changed since v4
> 
> version 4:
> make sure that normalized zpos value is stay in the defined property 
> range and warn user if not.
> Fix NULL pointer bug in rcar-du while setting zpos value.
> No changes in the other drivers.
> 
> version 3:
> use kmalloc_array instead of kmalloc.
> Correct normalize_zpos computation (comeback to Mareck original code)
> 
> version 2:
> add a zpos property into drm_plane structure to simplify code.
> This allow to get/set zpos value in core and not in drivers code.
> Fix various remarks.
> 
> version 1:
> refactor Marek's patches to have per plane zpos property instead of only
> one in core.
> 
> Benjamin Gaignard (2):
>   drm: sti: use generic zpos for plane
>   drm: rcar: use generic code for managing zpos plane property
> 
> Marek Szyprowski (2):
>   drm: add generic zpos property
>   drm/exynos: use generic code for managing zpos plane property
> 
>  drivers/gpu/drm/Makefile                  |   2 +-
>  drivers/gpu/drm/drm_atomic.c              |   4 +
>  drivers/gpu/drm/drm_atomic_helper.c       |   6 +
>  drivers/gpu/drm/drm_blend.c               | 227 ++++++++++++++++++++++++++++++
>  drivers/gpu/drm/drm_crtc_internal.h       |   4 +
>  drivers/gpu/drm/exynos/exynos_drm_drv.h   |   2 -
>  drivers/gpu/drm/exynos/exynos_drm_plane.c |  67 ++-------
>  drivers/gpu/drm/exynos/exynos_mixer.c     |   6 +-
>  drivers/gpu/drm/rcar-du/rcar_du_crtc.c    |   2 +-
>  drivers/gpu/drm/rcar-du/rcar_du_drv.h     |   1 -
>  drivers/gpu/drm/rcar-du/rcar_du_kms.c     |   5 -
>  drivers/gpu/drm/rcar-du/rcar_du_plane.c   |   9 +-
>  drivers/gpu/drm/rcar-du/rcar_du_plane.h   |   2 -
>  drivers/gpu/drm/sti/sti_cursor.c          |   4 +-
>  drivers/gpu/drm/sti/sti_gdp.c             |   4 +-
>  drivers/gpu/drm/sti/sti_hqvdp.c           |   4 +-
>  drivers/gpu/drm/sti/sti_mixer.c           |   9 +-
>  drivers/gpu/drm/sti/sti_plane.c           |  76 ++++------
>  drivers/gpu/drm/sti/sti_plane.h           |   7 +-
>  include/drm/drm_crtc.h                    |  30 ++++
>  20 files changed, 324 insertions(+), 147 deletions(-)
>  create mode 100644 drivers/gpu/drm/drm_blend.c
> 
> Cc: Inki Dae <inki.dae@samsung.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Ville Syrjala <ville.syrjala@linux.intel.com>
> Cc: Joonyoung Shim <jy0922.shim@samsung.com>
> Cc: Seung-Woo Kim <sw0312.kim@samsung.com>
> Cc: Andrzej Hajda <a.hajda@samsung.com>
> Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
> Cc: Gustavo Padovan <gustavo@padovan.org>
> Cc: vincent.abriou@st.com
> Cc: fabien.dessenne@st.com
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>