[PATCH v5 0/4] Generic zpos property

[Daniel Vetter]

On Tue, Jul 12, 2016 at 04:47:15PM +0200, Benjamin Gaignard wrote:
> What is the required tool to edit this .csv file ?

You're favourite text editor, it's a csv (= comman separated values). Just
pls check the end result in the generated docs with

$ make htmldocs

The makefiles should complain if you don't have all the sphinx bits needed
installed.
-Daniel

> 
> 2016-07-12 16:01 GMT+02:00 Daniel Vetter <daniel at ffwll.ch>:
> > On Fri, Jul 08, 2016 at 09:06:18AM +0200, Benjamin Gaignard wrote:
> >> htmldocs get information directly from the comments in .h and .c files
> >
> > You're both correct: There's another table which is supposed to list _all_
> > properties. It's a bit a mess, and should be split up/cleaned up, but
> > that's a longer-term thing. Please add zpos there too.
> >
> > See Documentation/gpu/kms-properties.csv
> >
> > Cheers, Daniel
> >>
> >> 2016-07-07 18:09 GMT+02:00 Tobias Jakobi <tjakobi at math.uni-bielefeld.de>:
> >> > Benjamin Gaignard wrote:
> >> >> zpos new fields are correctly documented in drm-kms.html after running
> >> >> make htmldocs.
> >> > I'm not sure I understand. Where does htmldocs get the information from
> >> > then?
> >> >
> >> > - Tobias
> >> >
> >> >
> >> >> Benjamin
> >> >>
> >> >> 2016-07-07 16:01 GMT+02:00 Tobias Jakobi <tjakobi at 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 at samsung.com>
> >> >>>> Cc: Daniel Vetter <daniel at ffwll.ch>
> >> >>>> Cc: Ville Syrjala <ville.syrjala at linux.intel.com>
> >> >>>> Cc: Joonyoung Shim <jy0922.shim at samsung.com>
> >> >>>> Cc: Seung-Woo Kim <sw0312.kim at samsung.com>
> >> >>>> Cc: Andrzej Hajda <a.hajda at samsung.com>
> >> >>>> Cc: Krzysztof Kozlowski <k.kozlowski at samsung.com>
> >> >>>> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie at samsung.com>
> >> >>>> Cc: Tobias Jakobi <tjakobi at math.uni-bielefeld.de>
> >> >>>> Cc: Gustavo Padovan <gustavo at padovan.org>
> >> >>>> Cc: vincent.abriou at st.com
> >> >>>> Cc: fabien.dessenne at st.com
> >> >>>> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> >> >>>>
> >> >>>
> >> >>
> >> >>
> >> >>
> >> >
> >>
> >>
> >>
> >> --
> >> Benjamin Gaignard
> >>
> >> Graphic Working Group
> >>
> >> Linaro.org │ Open source software for ARM SoCs
> >>
> >> Follow Linaro: Facebook | Twitter | Blog
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
> 
> 
> 
> -- 
> Benjamin Gaignard
> 
> Graphic Working Group
> 
> Linaro.org │ Open source software for ARM SoCs
> 
> Follow Linaro: Facebook | Twitter | Blog

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch