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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html