• Daniel Vetter's avatar
    drm/atomic: document how to handle driver private objects · da6c0596
    Daniel Vetter authored
    DK put some nice docs into the commit introducing driver private
    state, but in the git history alone it'll be lost.
    
    Also, since Ville remove the void* usage it's a good opportunity to
    give the driver private stuff some tlc on the doc front.
    
    Finally try to explain why the "let's just subclass drm_atomic_state"
    approach wasn't the greatest, and annotate all those functions as
    deprecated in favour of more standardized driver private states. Also
    note where we could/should extend driver private states going forward
    (atm neither locking nor synchronization is handled in core/helpers,
    which isn't really all that great).
    
    v2: Spelling and phrasing improvements (Alex, DK).
    
    Cc: Harry Wentland <harry.wentland@amd.com>
    Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
    Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
    Cc: Rob Clark <robdclark@gmail.com>
    Cc: Alex Deucher <alexander.deucher@amd.com>
    Cc: Ben Skeggs <bskeggs@redhat.com>
    Reviewed-by: default avatarDhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
    Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
    Signed-off-by: default avatarDaniel Vetter <daniel.vetter@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20171214203054.20141-5-daniel.vetter@ffwll.ch
    da6c0596
drm_atomic.c 71.3 KB