[PATCH v12 00/14] In order to readout DP SDPs, refactors the handling of DP SDPs
gwan-gyeong.mun at intel.com
Thu May 14 06:07:18 UTC 2020
In order to readout DP SDPs (Secondary Data Packet: DP HDR Metadata
Infoframe SDP, DP VSC SDP), it refactors handling DP SDPs codes.
It adds new compute routines for DP HDR Metadata Infoframe SDP
and DP VSC SDP.
And new writing routines of DP SDPs (Secondary Data Packet) that uses
New reading routines of DP SDPs are added for readout.
It adds a logging function for DP VSC SDP.
When receiving video it is very useful to be able to log DP VSC SDP.
This greatly simplifies debugging.
In order to use a common VSC SDP Colorimetry calculating code on PSR,
it uses a new psr vsc sdp compute routine.
v2: Minor style fix
- Add a new drm data structure for DP VSC SDP
- Replace a structure name to drm_dp_vsc_sdp from intel_dp_vsc_sdp
- Move logging functions to drm core [Jani N]
And use drm core's DP VSC SDP logging function
- Explicitly disable unused DIPs (AVI, GCP, VS, SPD, DRM. They will be
used for HDMI), when intel_dp_set_infoframes() function will be called.
- Use struct drm_device logging macros
- Use intel_de_*() functions for register access
- Add warning where a bpc is 6 and a pixel format is RGB.
- Addressed review comments from Uma
Add kernel docs for added data structures
Rename enum dp_colorspace to dp_pixelformat
Polish commit message and comments
Combine the if checks of sdp.HB2 and sdp.HB3
Add 6bpc to packining and unpacking of VSC SDP
v6: Fix enabled infoframe states of lspcon
v7: Fix the wrong check of combination bpc 6 and RGB pixelformat
v9: Add clear comments to hdmi_drm_infoframe_unpack_only() and
hdmi_drm_infoframe_unpack() (Laurent Pinchart)
- Fix packing of VSC SDP where Pixel Encoding/Colorimetry Format is not
- When a PSR is enabled, it needs to add DP_SDP_VSC to infoframes.enable.
- Change a checking of PSR state.
- Skip checking of VSC SDP when a crtc config has psr.
v11: If PSR is disabled by flag, it don't enable psr on pipe compute
v12: Fix an inconsistent indenting
Gwan-gyeong Mun (14):
video/hdmi: Add Unpack only function for DRM infoframe
drm/i915/dp: Read out DP SDPs
drm: Add logging function for DP VSC SDP
drm/i915: Include HDMI DRM infoframe in the crtc state dump
drm/i915: Include DP HDR Metadata Infoframe SDP in the crtc state dump
drm/i915: Include DP VSC SDP in the crtc state dump
drm/i915: Program DP SDPs with computed configs
drm/i915: Add state readout for DP HDR Metadata Infoframe SDP
drm/i915: Add state readout for DP VSC SDP
drm/i915: Fix enabled infoframe states of lspcon
drm/i915: Program DP SDPs on pipe updates
drm/i915: Stop sending DP SDPs on ddi disable
drm/i915/dp: Add compute routine for DP PSR VSC SDP
drm/i915/psr: Use new DP VSC SDP compute routine on PSR
drivers/gpu/drm/drm_dp_helper.c | 174 ++++++++
drivers/gpu/drm/i915/display/intel_ddi.c | 19 +-
drivers/gpu/drm/i915/display/intel_display.c | 63 +++
drivers/gpu/drm/i915/display/intel_dp.c | 406 ++++++++++---------
drivers/gpu/drm/i915/display/intel_dp.h | 15 +-
drivers/gpu/drm/i915/display/intel_lspcon.c | 2 +-
drivers/gpu/drm/i915/display/intel_psr.c | 58 +--
drivers/gpu/drm/i915/display/intel_psr.h | 6 +-
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/video/hdmi.c | 65 ++-
include/drm/drm_dp_helper.h | 3 +
include/linux/hdmi.h | 2 +
12 files changed, 551 insertions(+), 263 deletions(-)
More information about the dri-devel