• Matt Roper's avatar
    drm/i915/gen9: Allow skl_allocate_pipe_ddb() to operate on in-flight state (v3) · c107acfe
    Matt Roper authored
    We eventually want to calculate watermark values at atomic 'check' time
    instead of atomic 'commit' time so that any requested configurations
    that result in impossible watermark requirements are properly rejected.
    The first step along this path is to allocate the DDB at atomic 'check'
    time.  As we perform this transition, allow the main allocation function
    to operate successfully on either an in-flight state or an
    already-commited state.  Once we complete the transition in a future
    patch, we'll come back and remove the unnecessary logic for the
    already-committed case.
    
    v2: Rebase/refactor; we should no longer need to grab extra plane states
        while allocating the DDB since we can pull cached data rates and
        minimum block counts from the CRTC state for any planes that aren't
        being modified by this transaction.
    
    v3:
     - Simplify memsets to clear DDB plane entries.  (Maarten)
     - Drop a redundant memset of plane[pipe][PLANE_CURSOR] that was added
       by an earlier Coccinelle patch.  (Maarten)
     - Assign *num_active at the top of skl_ddb_get_pipe_allocation_limits()
       so that no code paths return without setting it.  (kbuild robot)
    
    Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Signed-off-by: default avatarMatt Roper <matthew.d.roper@intel.com>
    Reviewed-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Link: http://patchwork.freedesktop.org/patch/msgid/1463061971-19638-8-git-send-email-matthew.d.roper@intel.com
    c107acfe
i915_drv.h 115 KB