<div dir="ltr">Hi,<div><br></div><div>Any news on those patches ?</div><div>Could we rely on them to upstream z-order fix on sti driver ?</div><div><br></div><div>Regards,</div><div>Benjamin</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-01-27 15:44 GMT+01:00 Marek Szyprowski <span dir="ltr"><<a href="mailto:m.szyprowski@samsung.com" target="_blank">m.szyprowski@samsung.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello all,<br>
<br>
This patch series is a continuation of rework of blending support in<br>
Exynos DRM driver. Some background can be found here:<br>
<a href="http://www.spinics.net/lists/dri-devel/msg96969.html" rel="noreferrer" target="_blank">http://www.spinics.net/lists/dri-devel/msg96969.html</a><br>
<br>
Daniel Vetter suggested that zpos property should be made generic, with<br>
well-defined semantics. This patchset is my proposal for such generic<br>
zpos property:<br>
- added zpos properties to drm core and plane state structures,<br>
- added helpers for normalizing zpos properties of given set of planes,<br>
- well defined semantics: planes are sorted by zpos values and then plane<br>
  id value if zpos equals.<br>
<br>
The next part is a result of my survey on introducing generic plane<br>
blending related properties:<br>
<a href="http://www.spinics.net/lists/dri-devel/msg98106.html" rel="noreferrer" target="_blank">http://www.spinics.net/lists/dri-devel/msg98106.html</a><br>
<br>
Ville Syrjälä suggested that blending mode should be defined by a pair<br>
of 2 factors, which define generic blending equation. I've implemented<br>
this approach. I hope that this code can be a base for further discussion<br>
on generic blending support. For more information, see patch 4/5.<br>
<br>
Patches are based on top of v4.5-rc1 release.<br>
<br>
Best regards<br>
Marek Szyprowski<br>
Samsung R&D Institute Poland<br>
<br>
Changelog:<br>
<br>
v5:<br>
- added patch with new generic blending and alpha properties<br>
- changed patchseries topic to "drm/exynos: introduce generic zpos and<br>
  blending properties"<br>
- moved all generic code to separate drm_blend.c file<br>
<br>
v4: <a href="http://www.spinics.net/lists/dri-devel/msg98654.html" rel="noreferrer" target="_blank">http://www.spinics.net/lists/dri-devel/msg98654.html</a><br>
- made min/max values of zpos property driver dependent (request of<br>
  Ville Syrjälä)<br>
- adjusted some comments<br>
<br>
v3: <a href="http://www.spinics.net/lists/dri-devel/msg98207.html" rel="noreferrer" target="_blank">http://www.spinics.net/lists/dri-devel/msg98207.html</a><br>
- on request of Daniel Vetter, moved all normalization process to DRM<br>
  core, drivers can simply use plane_state->normalized_zpos in their<br>
  atomic_check/update callbacks with no additional changes needed<br>
- updated documentation<br>
<br>
v2: <a href="http://www.spinics.net/lists/dri-devel/msg98093.html" rel="noreferrer" target="_blank">http://www.spinics.net/lists/dri-devel/msg98093.html</a><br>
- dropped 2 fixes for Exynos DRM, which got merged in meantime<br>
- added more comments and kernel docs for core functions as suggested<br>
  by Daniel Vetter<br>
- reworked initialization of zpos properties (moved assiging property<br>
  class to common code), now the code in the driver is even simpler<br>
- while reworking of intialization of zpos property code, did the same<br>
  change to generic rotation property<br>
<br>
v1: <a href="http://www.spinics.net/lists/dri-devel/msg97709.html" rel="noreferrer" target="_blank">http://www.spinics.net/lists/dri-devel/msg97709.html</a><br>
- initial version<br>
<br>
Patch summary:<br>
<br>
Marek Szyprowski (5):<br>
  drm: add generic zpos property<br>
  drm/exynos: use generic code for managing zpos plane property<br>
  drm: simplify initialization of rotation property<br>
  drm: add generic blending related properties<br>
  drm/exynos: add support for blending properties<br>
<br>
 Documentation/DocBook/gpu.tmpl                  |  57 +++-<br>
 drivers/gpu/drm/Makefile                        |   2 +-<br>
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c |  10 +-<br>
 drivers/gpu/drm/drm_atomic.c                    |  16 ++<br>
 drivers/gpu/drm/drm_atomic_helper.c             |   6 +<br>
 drivers/gpu/drm/drm_blend.c                     | 337 ++++++++++++++++++++++++<br>
 drivers/gpu/drm/drm_crtc.c                      |  29 +-<br>
 drivers/gpu/drm/drm_crtc_internal.h             |   3 +<br>
 drivers/gpu/drm/exynos/exynos_drm_drv.h         |   7 +-<br>
 drivers/gpu/drm/exynos/exynos_drm_plane.c       | 128 +++++----<br>
 drivers/gpu/drm/exynos/exynos_mixer.c           |   6 +-<br>
 drivers/gpu/drm/i915/intel_display.c            |   6 +-<br>
 drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c       |   3 +-<br>
 drivers/gpu/drm/omapdrm/omap_drv.c              |   3 +-<br>
 include/drm/drm_crtc.h                          |  36 ++-<br>
 include/uapi/drm/drm_mode.h                     | 101 +++++++<br>
 16 files changed, 669 insertions(+), 81 deletions(-)<br>
 create mode 100644 drivers/gpu/drm/drm_blend.c<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.9.2<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="border-collapse:collapse;font-family:arial,sans-serif;font-size:13px"><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px">Benjamin Gaignard </p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px">Graphic Working Group</p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span lang="EN-US" style="font-size:10pt;color:rgb(0,176,80)"><span style="color:rgb(0,68,252)"><a href="http://www.linaro.org/" style="color:rgb(0,0,204)" target="_blank">Linaro.org</a></span><b> </b></span><b><span lang="EN-US" style="font-size:10pt">│ </span></b><span lang="EN-US" style="font-size:10pt">Open source software for ARM SoCs</span></p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><u></u></p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span lang="EN-US" style="font-size:10pt">Follow <b>Linaro: </b></span><span style="font-size:10pt;color:rgb(0,68,252)"><a href="http://www.facebook.com/pages/Linaro" style="color:rgb(0,0,204)" target="_blank"><span style="color:blue">Facebook</span></a></span><span style="font-size:10pt"> | <span style="color:rgb(0,68,252)"><a href="http://twitter.com/#!/linaroorg" style="color:rgb(0,0,204)" target="_blank"><span style="color:blue">Twitter</span></a></span> | <span style="color:rgb(0,68,252)"><a href="http://www.linaro.org/linaro-blog/" style="color:rgb(0,0,204)" target="_blank"><span style="color:blue">Blog</span></a></span></span></p></span></div>
</div>