• Ville Syrjälä's avatar
    drm/i915: Reorganize .disable hooks for pre-DDI DP · 1a8ff607
    Ville Syrjälä authored
    Most of our DP encoder hooks are split into per-platform variants.
    .disable() an exception, and thus it's a bit messy. Let's split it
    up as well. We'll leave the common parts in a helper called by
    each platform specific hook. Now each platform has mostly its own
    hooks. Some hooks are still shared between vlv and chv, and between
    g4x and ilk. None of the remaining shared hooks have any platform
    checks in them however so duplicating them doesn't seem particularly
    useful.
    
    There is a subtle change on VLV/CHV where we now disable PSR before
    audio, whereas before we disabled PSR after audio. That should be
    totally fine, and PSR is disabled by default anyway. Jani also pointed
    out to me that PSR + audio doesn't seem like a particularly realistic
    combination.
    
    v2: Drop the PSR HAS_DDI check here (Rodrigo)
        Pimp up the commit message a bit based on a chat with Jani
    
    Cc: Jani Nikula <jani.nikula@intel.com>
    Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
    Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20170920151251.5961-1-ville.syrjala@linux.intel.com
    1a8ff607
intel_dp.c 174 KB