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@xxxxxxxx>: > > 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@xxxxxxxxxxxxxxxxxxxxx>: > >> > 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@xxxxxxxxxxxxxxxxxxxxx>: > >> >>> 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@xxxxxxxxxxx> > >> >>>> Cc: Daniel Vetter <daniel@xxxxxxxx> > >> >>>> Cc: Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> > >> >>>> Cc: Joonyoung Shim <jy0922.shim@xxxxxxxxxxx> > >> >>>> Cc: Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx> > >> >>>> Cc: Andrzej Hajda <a.hajda@xxxxxxxxxxx> > >> >>>> Cc: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> > >> >>>> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> > >> >>>> Cc: Tobias Jakobi <tjakobi@xxxxxxxxxxxxxxxxxxxxx> > >> >>>> Cc: Gustavo Padovan <gustavo@xxxxxxxxxxx> > >> >>>> Cc: vincent.abriou@xxxxxx > >> >>>> Cc: fabien.dessenne@xxxxxx > >> >>>> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > >> >>>> > >> >>> > >> >> > >> >> > >> >> > >> > > >> > >> > >> > >> -- > >> 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 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
