• Gwan-gyeong Mun's avatar
    drm/i915/dp: Program VSC Header and DB for Pixel Encoding/Colorimetry Format · 3c053a96
    Gwan-gyeong Mun authored
    Function intel_pixel_encoding_setup_vsc handles vsc header and data block
    setup for pixel encoding / colorimetry format.
    
    Setup VSC header and data block in function intel_pixel_encoding_setup_vsc
    for pixel encoding / colorimetry format as per dp 1.4a spec,
    section 2.2.5.7.1, table 2-119: VSC SDP Header Bytes, section 2.2.5.7.5,
    table 2-120:VSC SDP Payload for DB16 through DB18.
    
    v2:
      Minor style fix. [Maarten]
      Refer to commit ids instead of patchwork. [Maarten]
    
    v6: Rebase
    
    v7:
      Rebase and addressed review comments from Ville.
      Use a structure initializer instead of memset().
      Fix non-standard comment format.
      Remove a referring to specific commit.
      Add a setting of dynamic range bit to  vsc_sdp.DB17.
      Add a setting of bpc which is based on pipe_bpp.
      Remove duplicated checking of connector's ycbcr_420_allowed from
      intel_pixel_encoding_setup_vsc(). It is already checked from
      intel_dp_ycbcr420_config().
      Remove comments for VSC_SDP_EXTENSION_FOR_COLORIMETRY_SUPPORTED. It is
      already implemented on intel_dp_get_colorimetry_status().
    
    v8:
      A missing of setting bpc to VSC setup is the pretty fatal case, it
      replaces DRM_DEBUG_KMS() to MISSING_CASE(). [Maarten]
    
    v9: Use a changed member name of struct dp_sdp. it renamed to db from DB.
    
    Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
    Signed-off-by: default avatarGwan-gyeong Mun <gwan-gyeong.mun@intel.com>
    Reviewed-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20190521121721.32010-4-gwan-gyeong.mun@intel.com
    3c053a96
intel_ddi.c 129 KB