19 Sep, 2016

1 commit

  • We don't want to burry the bridge structures kerneldoc in drm_crtc.h.

    Cc: Archit Taneja
    Reviewed-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160831160913.12991-3-daniel.vetter@ffwll.ch

    Daniel Vetter
     

29 Aug, 2016

3 commits

  • I figured an overview section here is overkill, and better
    to just document the 2 structures themselves well enough.

    v2: Review from Archit:
    - Appease checkpatch in moved code.
    - Spelling fixes in the kerneldoc.

    Reviewed-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-5-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • It's only used in drm_mode_object_get_properties, and we can compute
    it there directly with a bit of code shuffling.

    Reviewed-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-4-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Just for the struct drm_mode_object base class. The header file was
    already partially extracted to help untangle the include loops.

    v2:
    - Also move the generic get/set property ioctls. At first this seemed
    like a bad idea since it requires making drm_mode_crtc_set_obj_prop
    non-static. But eventually that will get split away too (like
    the connector version already is) for both crtc and planes. Hence I
    reconsidered.

    - drm_mode_object.[hc] instead of drm_modeset.[hc], which requires
    renaming the drm_modeset.h header I already started building up.
    This is more consistent (matches the name of the main structure),
    and I want to be able to use drm_modeset.[hc] for the basic modeset
    init/cleanup functionality like drm_mode_config_init.

    Reviewed-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-3-daniel.vetter@ffwll.ch

    Daniel Vetter