]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/blobdiff - drivers/gpu/drm/amd/display/dc/dc.h
drm/amd/display: Rename more dc_surface stuff to plane_state
[mirror_ubuntu-bionic-kernel.git] / drivers / gpu / drm / amd / display / dc / dc.h
index 823a8722449809ed57e84db07b8cda5f1d2af7c1..d1a6398043a3cd8fb93df81e3a47e5882f1c28b0 100644 (file)
 #include "logger_types.h"
 #include "gpio_types.h"
 #include "link_service_types.h"
+#include "grph_object_ctrl_defs.h"
+#include <inc/hw/opp.h>
 
-#define MAX_TARGETS 6
 #define MAX_SURFACES 3
+#define MAX_STREAMS 6
 #define MAX_SINKS_PER_LINK 4
 
 /*******************************************************************************
  ******************************************************************************/
 
 struct dc_caps {
-       uint32_t max_targets;
+       uint32_t max_streams;
        uint32_t max_links;
        uint32_t max_audios;
        uint32_t max_slave_planes;
+       uint32_t max_planes;
        uint32_t max_downscale_ratio;
        uint32_t i2c_speed_in_khz;
+
+       unsigned int max_cursor_size;
 };
 
 
 struct dc_dcc_surface_param {
-       enum surface_pixel_format format;
        struct dc_size surface_size;
+       enum surface_pixel_format format;
+       enum swizzle_mode_values swizzle_mode;
        enum dc_scan_direction scan;
 };
 
@@ -63,9 +69,6 @@ struct dc_dcc_setting {
 };
 
 struct dc_surface_dcc_cap {
-       bool capable;
-       bool const_color_support;
-
        union {
                struct {
                        struct dc_dcc_setting rgb;
@@ -76,52 +79,71 @@ struct dc_surface_dcc_cap {
                        struct dc_dcc_setting chroma;
                } video;
        };
+
+       bool capable;
+       bool const_color_support;
+};
+
+struct dc_static_screen_events {
+       bool cursor_update;
+       bool surface_update;
+       bool overlay_update;
 };
 
 /* Forward declaration*/
 struct dc;
-struct dc_surface;
+struct dc_plane_state;
 struct validate_context;
 
 struct dc_cap_funcs {
-       int i;
+       bool (*get_dcc_compression_cap)(const struct dc *dc,
+                       const struct dc_dcc_surface_param *input,
+                       struct dc_surface_dcc_cap *output);
 };
 
-struct dc_stream_funcs {
+struct dc_stream_state_funcs {
        bool (*adjust_vmin_vmax)(struct dc *dc,
-                       const struct dc_stream **stream,
+                       struct dc_stream_state **stream,
                        int num_streams,
                        int vmin,
                        int vmax);
+       bool (*get_crtc_position)(struct dc *dc,
+                       struct dc_stream_state **stream,
+                       int num_streams,
+                       unsigned int *v_pos,
+                       unsigned int *nom_v_pos);
 
-       void (*stream_update_scaling)(const struct dc *dc,
-                       const struct dc_stream *dc_stream,
-                       const struct rect *src,
-                       const struct rect *dst);
        bool (*set_gamut_remap)(struct dc *dc,
-                       const struct dc_stream **stream, int num_streams);
-       bool (*set_backlight)(struct dc *dc, unsigned int backlight_level,
-               unsigned int frame_ramp, const struct dc_stream *stream);
-       bool (*init_dmcu_backlight_settings)(struct dc *dc);
-       bool (*set_abm_level)(struct dc *dc, unsigned int abm_level);
-       bool (*set_psr_enable)(struct dc *dc, bool enable);
-       bool (*setup_psr)(struct dc *dc, const struct dc_stream *stream);
+                       const struct dc_stream_state *stream);
+
+       bool (*program_csc_matrix)(struct dc *dc,
+                       struct dc_stream_state *stream);
+
+       void (*set_static_screen_events)(struct dc *dc,
+                       struct dc_stream_state **stream,
+                       int num_streams,
+                       const struct dc_static_screen_events *events);
+
+       void (*set_dither_option)(struct dc_stream_state *stream,
+                       enum dc_dither_option option);
 };
 
 struct link_training_settings;
 
 struct dc_link_funcs {
        void (*set_drive_settings)(struct dc *dc,
-                       struct link_training_settings *lt_settings);
+                       struct link_training_settings *lt_settings,
+                       const struct dc_link *link);
        void (*perform_link_training)(struct dc *dc,
                        struct dc_link_settings *link_setting,
                        bool skip_video_pattern);
        void (*set_preferred_link_settings)(struct dc *dc,
-                       struct dc_link_settings *link_setting);
+                       struct dc_link_settings *link_setting,
+                       struct dc_link *link);
        void (*enable_hpd)(const struct dc_link *link);
        void (*disable_hpd)(const struct dc_link *link);
        void (*set_test_pattern)(
-                       const struct dc_link *link,
+                       struct dc_link *link,
                        enum dp_test_pattern test_pattern,
                        const struct link_training_settings *p_link_settings,
                        const unsigned char *p_custom_pattern,
@@ -136,29 +158,59 @@ struct dc_config {
 
 struct dc_debug {
        bool surface_visual_confirm;
+       bool sanity_checks;
        bool max_disp_clk;
-       bool target_trace;
        bool surface_trace;
        bool timing_trace;
+       bool clock_trace;
        bool validation_trace;
        bool disable_stutter;
        bool disable_dcc;
        bool disable_dfs_bypass;
-       bool disable_power_gate;
+       bool disable_dpp_power_gate;
+       bool disable_hubp_power_gate;
+       bool disable_pplib_wm_range;
+       bool use_dml_wm;
+       bool disable_pipe_split;
+       int sr_exit_time_dpm0_ns;
+       int sr_enter_plus_exit_time_dpm0_ns;
+       int sr_exit_time_ns;
+       int sr_enter_plus_exit_time_ns;
+       int urgent_latency_ns;
+       int percent_of_ideal_drambw;
+       int dram_clock_change_latency_ns;
+       int always_scale;
+       bool disable_pplib_clock_request;
        bool disable_clock_gate;
        bool disable_dmcu;
-       bool disable_hdmi_deep_color;
+       bool disable_psr;
+       bool force_abm_enable;
 };
 
 struct dc {
        struct dc_caps caps;
        struct dc_cap_funcs cap_funcs;
-       struct dc_stream_funcs stream_funcs;
+       struct dc_stream_state_funcs stream_funcs;
        struct dc_link_funcs link_funcs;
        struct dc_config config;
        struct dc_debug debug;
 };
 
+enum frame_buffer_mode {
+       FRAME_BUFFER_MODE_LOCAL_ONLY = 0,
+       FRAME_BUFFER_MODE_ZFB_ONLY,
+       FRAME_BUFFER_MODE_MIXED_ZFB_AND_LOCAL,
+} ;
+
+struct dchub_init_data {
+       int64_t zfb_phys_addr_base;
+       int64_t zfb_mc_base_addr;
+       uint64_t zfb_size_in_byte;
+       enum frame_buffer_mode fb_mode;
+       bool dchub_initialzied;
+       bool dchub_info_valid;
+};
+
 struct dc_init_data {
        struct hw_asic_id asic_id;
        void *driver; /* ctx */
@@ -173,12 +225,19 @@ struct dc_init_data {
        enum dce_environment dce_environment;
 
        struct dc_config flags;
+#ifdef ENABLE_FBC
+       uint64_t fbc_gpu_addr;
+#endif
 };
 
 struct dc *dc_create(const struct dc_init_data *init_params);
 
 void dc_destroy(struct dc **dc);
 
+bool dc_init_dchub(struct dc *dc, struct dchub_init_data *dh_data);
+
+void dc_log_hw_state(struct dc *dc);
+
 /*******************************************************************************
  * Surface Interfaces
  ******************************************************************************/
@@ -188,8 +247,6 @@ enum {
 };
 
 struct dc_hdr_static_metadata {
-       bool is_hdr;
-
        /* display chromaticities and white point in units of 0.00001 */
        unsigned int chromaticity_green_x;
        unsigned int chromaticity_green_y;
@@ -204,19 +261,26 @@ struct dc_hdr_static_metadata {
        uint32_t max_luminance;
        uint32_t maximum_content_light_level;
        uint32_t maximum_frame_average_light_level;
+
+       bool hdr_supported;
+       bool is_hdr;
 };
 
 enum dc_transfer_func_type {
        TF_TYPE_PREDEFINED,
        TF_TYPE_DISTRIBUTED_POINTS,
+       TF_TYPE_BYPASS
 };
 
 struct dc_transfer_func_distributed_points {
-       uint16_t red[TRANSFER_FUNC_POINTS];
-       uint16_t green[TRANSFER_FUNC_POINTS];
-       uint16_t blue[TRANSFER_FUNC_POINTS];
+       struct fixed31_32 red[TRANSFER_FUNC_POINTS];
+       struct fixed31_32 green[TRANSFER_FUNC_POINTS];
+       struct fixed31_32 blue[TRANSFER_FUNC_POINTS];
+
        uint16_t end_exponent;
-       uint16_t x_point_at_y1;
+       uint16_t x_point_at_y1_red;
+       uint16_t x_point_at_y1_green;
+       uint16_t x_point_at_y1_blue;
 };
 
 enum dc_transfer_func_predefined {
@@ -227,14 +291,26 @@ enum dc_transfer_func_predefined {
 };
 
 struct dc_transfer_func {
+       struct dc_transfer_func_distributed_points tf_pts;
        enum dc_transfer_func_type type;
        enum dc_transfer_func_predefined tf;
-       struct dc_transfer_func_distributed_points tf_pts;
+       struct dc_context *ctx;
+       int ref_count;
 };
 
-struct dc_surface {
-       bool visible;
-       bool flip_immediate;
+/*
+ * This structure is filled in by dc_surface_get_status and contains
+ * the last requested address and the currently active address so the called
+ * can determine if there are any outstanding flips
+ */
+struct dc_plane_status {
+       struct dc_plane_address requested_address;
+       struct dc_plane_address current_address;
+       bool is_flip_pending;
+       bool is_right_eye;
+};
+
+struct dc_plane_state {
        struct dc_plane_address address;
 
        struct scaling_taps scaling_quality;
@@ -244,18 +320,30 @@ struct dc_surface {
 
        union plane_size plane_size;
        union dc_tiling_info tiling_info;
+
        struct dc_plane_dcc_param dcc;
-       enum dc_color_space color_space;
+       struct dc_hdr_static_metadata hdr_static_ctx;
+
+       struct dc_gamma *gamma_correction;
+       struct dc_transfer_func *in_transfer_func;
 
+       enum dc_color_space color_space;
        enum surface_pixel_format format;
        enum dc_rotation_angle rotation;
-       bool horizontal_mirror;
        enum plane_stereo_format stereo_format;
 
-       struct dc_hdr_static_metadata hdr_static_ctx;
+       bool per_pixel_alpha;
+       bool visible;
+       bool flip_immediate;
+       bool horizontal_mirror;
+
+       /* private to DC core */
+       struct dc_plane_status status;
+       struct dc_context *ctx;
 
-       const struct dc_gamma *gamma_correction;
-       const struct dc_transfer_func *in_transfer_func;
+       /* private to dc_surface.c */
+       enum dc_irq_source irq_source;
+       int ref_count;
 };
 
 struct dc_plane_info {
@@ -264,21 +352,22 @@ struct dc_plane_info {
        struct dc_plane_dcc_param dcc;
        enum surface_pixel_format format;
        enum dc_rotation_angle rotation;
-       bool horizontal_mirror;
        enum plane_stereo_format stereo_format;
        enum dc_color_space color_space; /*todo: wrong place, fits in scaling info*/
+       bool horizontal_mirror;
        bool visible;
+       bool per_pixel_alpha;
 };
 
 struct dc_scaling_info {
-               struct rect src_rect;
-               struct rect dst_rect;
-               struct rect clip_rect;
-               struct scaling_taps scaling_quality;
+       struct rect src_rect;
+       struct rect dst_rect;
+       struct rect clip_rect;
+       struct scaling_taps scaling_quality;
 };
 
 struct dc_surface_update {
-       const struct dc_surface *surface;
+       struct dc_plane_state *surface;
 
        /* isr safe update parameters.  null means no updates */
        struct dc_flip_addrs *flip_addr;
@@ -289,39 +378,26 @@ struct dc_surface_update {
         */
        /* gamma TO BE REMOVED */
        struct dc_gamma *gamma;
-       struct dc_hdr_static_metadata *hdr_static_metadata;
        struct dc_transfer_func *in_transfer_func;
-       struct dc_transfer_func *out_transfer_func;
-
-
-};
-/*
- * This structure is filled in by dc_surface_get_status and contains
- * the last requested address and the currently active address so the called
- * can determine if there are any outstanding flips
- */
-struct dc_surface_status {
-       struct dc_plane_address requested_address;
-       struct dc_plane_address current_address;
-       bool is_flip_pending;
+       struct dc_hdr_static_metadata *hdr_static_metadata;
 };
 
 /*
  * Create a new surface with default parameters;
  */
-struct dc_surface *dc_create_surface(const struct dc *dc);
-const struct dc_surface_status *dc_surface_get_status(
-               const struct dc_surface *dc_surface);
+struct dc_plane_state *dc_create_plane_state(const struct dc *dc);
+const struct dc_plane_status *dc_plane_get_status(
+               const struct dc_plane_state *plane_state);
 
-void dc_surface_retain(const struct dc_surface *dc_surface);
-void dc_surface_release(const struct dc_surface *dc_surface);
+void dc_plane_state_retain(struct dc_plane_state *plane_state);
+void dc_plane_state_release(struct dc_plane_state *plane_state);
 
-void dc_gamma_retain(const struct dc_gamma *dc_gamma);
-void dc_gamma_release(const struct dc_gamma *dc_gamma);
+void dc_gamma_retain(struct dc_gamma *dc_gamma);
+void dc_gamma_release(struct dc_gamma **dc_gamma);
 struct dc_gamma *dc_create_gamma(void);
 
-void dc_transfer_func_retain(const struct dc_transfer_func *dc_tf);
-void dc_transfer_func_release(const struct dc_transfer_func *dc_tf);
+void dc_transfer_func_retain(struct dc_transfer_func *dc_tf);
+void dc_transfer_func_release(struct dc_transfer_func *dc_tf);
 struct dc_transfer_func *dc_create_transfer_func(void);
 
 /*
@@ -332,130 +408,206 @@ struct dc_transfer_func *dc_create_transfer_func(void);
 struct dc_flip_addrs {
        struct dc_plane_address address;
        bool flip_immediate;
-       /* TODO: DCC format info */
        /* TODO: add flip duration for FreeSync */
 };
 
 /*
- * Optimized flip address update function.
- *
- * After this call:
- *   Surface addresses and flip attributes are programmed.
- *   Surface flip occur at next configured time (h_sync or v_sync flip)
- */
-void dc_flip_surface_addrs(struct dc *dc,
-               const struct dc_surface *const surfaces[],
-               struct dc_flip_addrs flip_addrs[],
-               uint32_t count);
-
-/*
- * Set up surface attributes and associate to a target
- * The surfaces parameter is an absolute set of all surface active for the target.
- * If no surfaces are provided, the target will be blanked; no memory read.
+ * Set up surface attributes and associate to a stream
+ * The surfaces parameter is an absolute set of all surface active for the stream.
+ * If no surfaces are provided, the stream will be blanked; no memory read.
  * Any flip related attribute changes must be done through this interface.
  *
  * After this call:
- *   Surfaces attributes are programmed and configured to be composed into target.
+ *   Surfaces attributes are programmed and configured to be composed into stream.
  *   This does not trigger a flip.  No surface address is programmed.
  */
 
-bool dc_commit_surfaces_to_target(
+bool dc_commit_planes_to_stream(
                struct dc *dc,
-               const struct dc_surface **dc_surfaces,
-               uint8_t surface_count,
-               struct dc_target *dc_target);
+               struct dc_plane_state **plane_states,
+               uint8_t new_plane_count,
+               struct dc_stream_state *stream);
 
-bool dc_pre_update_surfaces_to_target(
-               struct dc *dc,
-               const struct dc_surface *const *new_surfaces,
-               uint8_t new_surface_count,
-               struct dc_target *dc_target);
-
-bool dc_post_update_surfaces_to_target(
+bool dc_post_update_surfaces_to_stream(
                struct dc *dc);
 
-void dc_update_surfaces_for_target(struct dc *dc, struct dc_surface_update *updates,
-               int surface_count, struct dc_target *dc_target);
+/* Surface update type is used by dc_update_surfaces_and_stream
+ * The update type is determined at the very beginning of the function based
+ * on parameters passed in and decides how much programming (or updating) is
+ * going to be done during the call.
+ *
+ * UPDATE_TYPE_FAST is used for really fast updates that do not require much
+ * logical calculations or hardware register programming. This update MUST be
+ * ISR safe on windows. Currently fast update will only be used to flip surface
+ * address.
+ *
+ * UPDATE_TYPE_MED is used for slower updates which require significant hw
+ * re-programming however do not affect bandwidth consumption or clock
+ * requirements. At present, this is the level at which front end updates
+ * that do not require us to run bw_calcs happen. These are in/out transfer func
+ * updates, viewport offset changes, recout size changes and pixel depth changes.
+ * This update can be done at ISR, but we want to minimize how often this happens.
+ *
+ * UPDATE_TYPE_FULL is slow. Really slow. This requires us to recalculate our
+ * bandwidth and clocks, possibly rearrange some pipes and reprogram anything front
+ * end related. Any time viewport dimensions, recout dimensions, scaling ratios or
+ * gamma need to be adjusted or pipe needs to be turned on (or disconnected) we do
+ * a full update. This cannot be done at ISR level and should be a rare event.
+ * Unless someone is stress testing mpo enter/exit, playing with colour or adjusting
+ * underscan we don't expect to see this call at all.
+ */
+
+enum surface_update_type {
+       UPDATE_TYPE_FAST, /* super fast, safe to execute in isr */
+       UPDATE_TYPE_MED,  /* ISR safe, most of programming needed, no bw/clk change*/
+       UPDATE_TYPE_FULL, /* may need to shuffle resources */
+};
 
 /*******************************************************************************
- * Target Interfaces
+ * Stream Interfaces
  ******************************************************************************/
-#define MAX_STREAM_NUM 1
 
-struct dc_target {
-       uint8_t stream_count;
-       const struct dc_stream *streams[MAX_STREAM_NUM];
+struct dc_stream_status {
+       int primary_otg_inst;
+       int plane_count;
+       struct dc_plane_state *plane_states[MAX_SURFACE_NUM];
+
+       /*
+        * link this stream passes through
+        */
+       struct dc_link *link;
 };
 
-/*
- * Target status is returned from dc_target_get_status in order to get the
- * the IRQ source, current frame counter and currently attached surfaces.
- */
-struct dc_target_status {
-       int primary_otg_inst;
-       int cur_frame_count;
-       int surface_count;
-       const struct dc_surface *surfaces[MAX_SURFACE_NUM];
+struct dc_stream_state {
+       struct dc_sink *sink;
+       struct dc_crtc_timing timing;
+
+       struct rect src; /* composition area */
+       struct rect dst; /* stream addressable area */
+
+       struct audio_info audio_info;
+
+       struct freesync_context freesync_ctx;
+
+       struct dc_transfer_func *out_transfer_func;
+       struct colorspace_transform gamut_remap_matrix;
+       struct csc_transform csc_color_matrix;
+
+       enum signal_type output_signal;
+
+       enum dc_color_space output_color_space;
+       enum dc_dither_option dither_option;
+
+       enum view_3d_format view_format;
+
+       bool ignore_msa_timing_param;
+       /* TODO: custom INFO packets */
+       /* TODO: ABM info (DMCU) */
+       /* TODO: PSR info */
+       /* TODO: CEA VIC */
+
+       /* from core_stream struct */
+       struct dc_context *ctx;
+
+       /* used by DCP and FMT */
+       struct bit_depth_reduction_params bit_depth_params;
+       struct clamping_and_pixel_encoding_params clamping;
+
+       int phy_pix_clk;
+       enum signal_type signal;
+
+       struct dc_stream_status status;
+
+       /* from stream struct */
+       int ref_count;
 };
 
-struct dc_target *dc_create_target_for_streams(
-               struct dc_stream *dc_streams[],
-               uint8_t stream_count);
+struct dc_stream_update {
+       struct rect src;
+       struct rect dst;
+       struct dc_transfer_func *out_transfer_func;
+};
+
+bool dc_is_stream_unchanged(
+       struct dc_stream_state *old_stream, struct dc_stream_state *stream);
 
 /*
- * Get the current target status.
+ * Setup stream attributes if no stream updates are provided
+ * there will be no impact on the stream parameters
+ *
+ * Set up surface attributes and associate to a stream
+ * The surfaces parameter is an absolute set of all surface active for the stream.
+ * If no surfaces are provided, the stream will be blanked; no memory read.
+ * Any flip related attribute changes must be done through this interface.
+ *
+ * After this call:
+ *   Surfaces attributes are programmed and configured to be composed into stream.
+ *   This does not trigger a flip.  No surface address is programmed.
+ *
  */
-const struct dc_target_status *dc_target_get_status(
-                                       const struct dc_target* dc_target);
 
-void dc_target_retain(const struct dc_target *dc_target);
-void dc_target_release(const struct dc_target *dc_target);
-void dc_target_log(
-       const struct dc_target *dc_target,
+void dc_update_planes_and_stream(struct dc *dc,
+               struct dc_surface_update *surface_updates, int surface_count,
+               struct dc_stream_state *dc_stream,
+               struct dc_stream_update *stream_update);
+
+/*
+ * Log the current stream state.
+ */
+void dc_stream_log(
+       const struct dc_stream_state *stream,
        struct dal_logger *dc_logger,
        enum dc_log_type log_type);
 
-uint8_t dc_get_current_target_count(const struct dc *dc);
-struct dc_target *dc_get_target_at_index(const struct dc *dc, uint8_t i);
-
-bool dc_target_is_connected_to_sink(
-               const struct dc_target *dc_target,
-               const struct dc_sink *dc_sink);
+uint8_t dc_get_current_stream_count(const struct dc *dc);
+struct dc_stream_state *dc_get_stream_at_index(const struct dc *dc, uint8_t i);
 
-uint32_t dc_target_get_vblank_counter(const struct dc_target *dc_target);
+/*
+ * Return the current frame counter.
+ */
+uint32_t dc_stream_get_vblank_counter(const struct dc_stream_state *stream);
 
 /* TODO: Return parsed values rather than direct register read
  * This has a dependency on the caller (amdgpu_get_crtc_scanoutpos)
  * being refactored properly to be dce-specific
  */
-uint32_t dc_target_get_scanoutpos(
-               const struct dc_target *dc_target,
-               uint32_t *vbl,
-               uint32_t *position);
+bool dc_stream_get_scanoutpos(const struct dc_stream_state *stream,
+                                 uint32_t *v_blank_start,
+                                 uint32_t *v_blank_end,
+                                 uint32_t *h_position,
+                                 uint32_t *v_position);
 
 /*
- * Structure to store surface/target associations for validation
+ * Structure to store surface/stream associations for validation
  */
 struct dc_validation_set {
-       const struct dc_target *target;
-       const struct dc_surface *surfaces[MAX_SURFACES];
-       uint8_t surface_count;
+       struct dc_stream_state *stream;
+       struct dc_plane_state *plane_states[MAX_SURFACES];
+       uint8_t plane_count;
 };
 
+bool dc_validate_stream(const struct dc *dc, struct dc_stream_state *stream);
+
+bool dc_validate_plane(const struct dc *dc, const struct dc_plane_state *plane_state);
 /*
  * This function takes a set of resources and checks that they are cofunctional.
  *
  * After this call:
  *   No hardware is programmed for call.  Only validation is done.
  */
+struct validate_context *dc_get_validate_context(
+               const struct dc *dc,
+               const struct dc_validation_set set[],
+               uint8_t set_count);
+
 bool dc_validate_resources(
                const struct dc *dc,
                const struct dc_validation_set set[],
                uint8_t set_count);
 
 /*
- * This function takes a target and checks if it is guaranteed to be supported.
- * Guaranteed means that MAX_COFUNC*target is supported.
+ * This function takes a stream and checks if it is guaranteed to be supported.
+ * Guaranteed means that MAX_COFUNC similar streams are supported.
  *
  * After this call:
  *   No hardware is programmed for call.  Only validation is done.
@@ -463,80 +615,125 @@ bool dc_validate_resources(
 
 bool dc_validate_guaranteed(
                const struct dc *dc,
-               const struct dc_target *dc_target);
+               struct dc_stream_state *stream);
+
+void dc_resource_validate_ctx_copy_construct(
+               const struct validate_context *src_ctx,
+               struct validate_context *dst_ctx);
+
+void dc_resource_validate_ctx_destruct(struct validate_context *context);
 
 /*
- * Set up streams and links associated to targets to drive sinks
- * The targets parameter is an absolute set of all active targets.
+ * TODO update to make it about validation sets
+ * Set up streams and links associated to drive sinks
+ * The streams parameter is an absolute set of all active streams.
  *
  * After this call:
  *   Phy, Encoder, Timing Generator are programmed and enabled.
- *   New targets are enabled with blank stream; no memory read.
+ *   New streams are enabled with blank stream; no memory read.
  */
-bool dc_commit_targets(
+bool dc_commit_context(struct dc *dc, struct validate_context *context);
+
+/*
+ * Set up streams and links associated to drive sinks
+ * The streams parameter is an absolute set of all active streams.
+ *
+ * After this call:
+ *   Phy, Encoder, Timing Generator are programmed and enabled.
+ *   New streams are enabled with blank stream; no memory read.
+ */
+bool dc_commit_streams(
                struct dc *dc,
-               struct dc_target *targets[],
-               uint8_t target_count);
+               struct dc_stream_state *streams[],
+               uint8_t stream_count);
+/*
+ * Enable stereo when commit_streams is not required,
+ * for example, frame alternate.
+ */
+bool dc_enable_stereo(
+       struct dc *dc,
+       struct validate_context *context,
+       struct dc_stream_state *streams[],
+       uint8_t stream_count);
 
-/*******************************************************************************
- * Stream Interfaces
- ******************************************************************************/
-struct dc_stream {
-       const struct dc_sink *sink;
-       struct dc_crtc_timing timing;
+/**
+ * Create a new default stream for the requested sink
+ */
+struct dc_stream_state *dc_create_stream_for_sink(struct dc_sink *dc_sink);
 
-       enum dc_color_space output_color_space;
+void dc_stream_retain(struct dc_stream_state *dc_stream);
+void dc_stream_release(struct dc_stream_state *dc_stream);
 
-       struct rect src; /* viewport in target space*/
-       struct rect dst; /* stream addressable area */
+struct dc_stream_status *dc_stream_get_status(
+       struct dc_stream_state *dc_stream);
 
-       struct audio_info audio_info;
+enum surface_update_type dc_check_update_surfaces_for_stream(
+               struct dc *dc,
+               struct dc_surface_update *updates,
+               int surface_count,
+               struct dc_stream_update *stream_update,
+               const struct dc_stream_status *stream_status);
 
-       bool ignore_msa_timing_param;
 
-       struct freesync_context freesync_ctx;
+void dc_retain_validate_context(struct validate_context *context);
+void dc_release_validate_context(struct validate_context *context);
 
-       const struct dc_transfer_func *out_transfer_func;
-       struct colorspace_transform gamut_remap_matrix;
-       struct csc_transform csc_color_matrix;
+/*******************************************************************************
+ * Link Interfaces
+ ******************************************************************************/
 
-       /* TODO: dithering */
-       /* TODO: custom INFO packets */
-       /* TODO: ABM info (DMCU) */
-       /* TODO: PSR info */
-       /* TODO: CEA VIC */
-};
+struct dpcd_caps {
+       union dpcd_rev dpcd_rev;
+       union max_lane_count max_ln_count;
+       union max_down_spread max_down_spread;
 
-/**
- * Create a new default stream for the requested sink
- */
-struct dc_stream *dc_create_stream_for_sink(const struct dc_sink *dc_sink);
+       /* dongle type (DP converter, CV smart dongle) */
+       enum display_dongle_type dongle_type;
+       /* Dongle's downstream count. */
+       union sink_count sink_count;
+       /* If dongle_type == DISPLAY_DONGLE_DP_HDMI_CONVERTER,
+       indicates 'Frame Sequential-to-lllFrame Pack' conversion capability.*/
+       struct dc_dongle_caps dongle_caps;
 
-void dc_stream_retain(const struct dc_stream *dc_stream);
-void dc_stream_release(const struct dc_stream *dc_stream);
+       uint32_t sink_dev_id;
+       uint32_t branch_dev_id;
+       int8_t branch_dev_name[6];
+       int8_t branch_hw_revision;
 
-struct dc_stream_status {
-       /*
-        * link this stream passes through
-        */
-       const struct dc_link *link;
+       bool allow_invalid_MSA_timing_param;
+       bool panel_mode_edp;
 };
 
-const struct dc_stream_status *dc_stream_get_status(
-       const struct dc_stream *dc_stream);
+struct dc_link_status {
+       struct dpcd_caps *dpcd_caps;
+};
 
-/*******************************************************************************
- * Link Interfaces
- ******************************************************************************/
+/* DP MST stream allocation (payload bandwidth number) */
+struct link_mst_stream_allocation {
+       /* DIG front */
+       const struct stream_encoder *stream_enc;
+       /* associate DRM payload table with DC stream encoder */
+       uint8_t vcp_id;
+       /* number of slots required for the DP stream in transport packet */
+       uint8_t slot_count;
+};
+
+/* DP MST stream allocation table */
+struct link_mst_stream_allocation_table {
+       /* number of DP video streams */
+       int stream_count;
+       /* array of stream allocations */
+       struct link_mst_stream_allocation stream_allocations[MAX_CONTROLLER_NUM];
+};
 
 /*
  * A link contains one or more sinks and their connected status.
  * The currently active signal type (HDMI, DP-SST, DP-MST) is also reported.
  */
 struct dc_link {
-       const struct dc_sink *remote_sinks[MAX_SINKS_PER_LINK];
+       struct dc_sink *remote_sinks[MAX_SINKS_PER_LINK];
        unsigned int sink_count;
-       const struct dc_sink *local_sink;
+       struct dc_sink *local_sink;
        unsigned int link_index;
        enum dc_connection_type type;
        enum signal_type connector_signal;
@@ -547,41 +744,49 @@ struct dc_link {
         */
        struct dc_link_settings reported_link_cap;
        struct dc_link_settings verified_link_cap;
-       struct dc_link_settings max_link_setting;
        struct dc_link_settings cur_link_settings;
        struct dc_lane_settings cur_lane_setting;
+       struct dc_link_settings preferred_link_setting;
 
        uint8_t ddc_hw_inst;
+
+       uint8_t hpd_src;
+
        uint8_t link_enc_hw_inst;
 
-       struct psr_caps psr_caps;
        bool test_pattern_enabled;
        union compliance_test_state compliance_test_state;
-};
 
-struct dpcd_caps {
-       union dpcd_rev dpcd_rev;
-       union max_lane_count max_ln_count;
-       union max_down_spread max_down_spread;
+       void *priv;
 
-       /* dongle type (DP converter, CV smart dongle) */
-       enum display_dongle_type dongle_type;
-       /* Dongle's downstream count. */
-       union sink_count sink_count;
-       /* If dongle_type == DISPLAY_DONGLE_DP_HDMI_CONVERTER,
-       indicates 'Frame Sequential-to-lllFrame Pack' conversion capability.*/
-       bool is_dp_hdmi_s3d_converter;
+       struct ddc_service *ddc;
 
-       bool allow_invalid_MSA_timing_param;
-       bool panel_mode_edp;
-       uint32_t sink_dev_id;
-       uint32_t branch_dev_id;
-       int8_t branch_dev_name[6];
-       int8_t branch_hw_revision;
-};
+       bool aux_mode;
+
+       /* Private to DC core */
+
+       const struct core_dc *dc;
+
+       struct dc_context *ctx;
+
+       struct link_encoder *link_enc;
+       struct graphics_object_id link_id;
+       union ddi_channel_mapping ddi_channel_mapping;
+       struct connector_device_tag_info device_tag;
+       struct dpcd_caps dpcd_caps;
+       unsigned int dpcd_sink_count;
+
+       enum edp_revision edp_revision;
+       bool psr_enabled;
+
+       /* MST record stream using this link */
+       struct link_flags {
+               bool dp_keep_receiver_powered;
+       } wa_flags;
+       struct link_mst_stream_allocation_table mst_stream_alloc_table;
+
+       struct dc_link_status link_status;
 
-struct dc_link_status {
-       struct dpcd_caps *dpcd_caps;
 };
 
 const struct dc_link_status *dc_link_get_status(const struct dc_link *dc_link);
@@ -591,7 +796,7 @@ const struct dc_link_status *dc_link_get_status(const struct dc_link *dc_link);
  * boot time.  They cannot be created or destroyed.
  * Use dc_get_caps() to get number of links.
  */
-const struct dc_link *dc_get_link_at_index(const struct dc *dc, uint32_t link_index);
+struct dc_link *dc_get_link_at_index(const struct dc *dc, uint32_t link_index);
 
 /* Return id of physical connector represented by a dc_link at link_index.*/
 const struct graphics_object_id dc_get_link_id_at_index(
@@ -599,16 +804,17 @@ const struct graphics_object_id dc_get_link_id_at_index(
 
 /* Set backlight level of an embedded panel (eDP, LVDS). */
 bool dc_link_set_backlight_level(const struct dc_link *dc_link, uint32_t level,
-               uint32_t frame_ramp, const struct dc_stream *stream);
+               uint32_t frame_ramp, const struct dc_stream_state *stream);
 
-bool dc_link_init_dmcu_backlight_settings(const struct dc_link *dc_link);
-
-bool dc_link_set_abm_level(const struct dc_link *dc_link, uint32_t level);
+bool dc_link_set_abm_disable(const struct dc_link *dc_link);
 
 bool dc_link_set_psr_enable(const struct dc_link *dc_link, bool enable);
 
-bool dc_link_setup_psr(const struct dc_link *dc_link,
-               const struct dc_stream *stream);
+bool dc_link_get_psr_state(const struct dc_link *dc_link, uint32_t *psr_state);
+
+bool dc_link_setup_psr(struct dc_link *dc_link,
+               const struct dc_stream_state *stream, struct psr_config *psr_config,
+               struct psr_context *psr_context);
 
 /* Request DC to detect if there is a Panel connected.
  * boot - If this call is during initial boot.
@@ -616,7 +822,7 @@ bool dc_link_setup_psr(const struct dc_link *dc_link,
  * true otherwise. True meaning further action is required (status update
  * and OS notification).
  */
-bool dc_link_detect(const struct dc_link *dc_link, bool boot);
+bool dc_link_detect(struct dc_link *dc_link, bool boot);
 
 /* Notify DC about DP RX Interrupt (aka Short Pulse Interrupt).
  * Return:
@@ -624,28 +830,29 @@ bool dc_link_detect(const struct dc_link *dc_link, bool boot);
  * detection.
  * false - no change in Downstream port status. No further action required
  * from DM. */
-bool dc_link_handle_hpd_rx_irq(const struct dc_link *dc_link);
+bool dc_link_handle_hpd_rx_irq(struct dc_link *dc_link,
+               union hpd_irq_data *hpd_irq_dpcd_data);
 
 struct dc_sink_init_data;
 
 struct dc_sink *dc_link_add_remote_sink(
-               const struct dc_link *dc_link,
+               struct dc_link *dc_link,
                const uint8_t *edid,
                int len,
                struct dc_sink_init_data *init_data);
 
 void dc_link_remove_remote_sink(
-       const struct dc_link *link,
-       const struct dc_sink *sink);
+       struct dc_link *link,
+       struct dc_sink *sink);
 
 /* Used by diagnostics for virtual link at the moment */
-void dc_link_set_sink(const struct dc_link *link, struct dc_sink *sink);
+void dc_link_set_sink(struct dc_link *link, struct dc_sink *sink);
 
 void dc_link_dp_set_drive_settings(
        struct dc_link *link,
        struct link_training_settings *lt_settings);
 
-bool dc_link_dp_perform_link_training(
+enum link_training_result dc_link_dp_perform_link_training(
        struct dc_link *link,
        const struct dc_link_settings *link_setting,
        bool skip_video_pattern);
@@ -655,7 +862,7 @@ void dc_link_dp_enable_hpd(const struct dc_link *link);
 void dc_link_dp_disable_hpd(const struct dc_link *link);
 
 bool dc_link_dp_set_test_pattern(
-       const struct dc_link *link,
+       struct dc_link *link,
        enum dp_test_pattern test_pattern,
        const struct link_training_settings *p_link_settings,
        const unsigned char *p_custom_pattern,
@@ -665,6 +872,19 @@ bool dc_link_dp_set_test_pattern(
  * Sink Interfaces - A sink corresponds to a display output device
  ******************************************************************************/
 
+struct dc_container_id {
+       // 128bit GUID in binary form
+       unsigned char  guid[16];
+       // 8 byte port ID -> ELD.PortID
+       unsigned int   portId[2];
+       // 128bit GUID in binary formufacturer name -> ELD.ManufacturerName
+       unsigned short manufacturerName;
+       // 2 byte product code -> ELD.ProductCode
+       unsigned short productCode;
+};
+
+
+
 /*
  * The sink structure contains EDID and other display device properties
  */
@@ -672,32 +892,46 @@ struct dc_sink {
        enum signal_type sink_signal;
        struct dc_edid dc_edid; /* raw edid */
        struct dc_edid_caps edid_caps; /* parse display caps */
+       struct dc_container_id *dc_container_id;
+       uint32_t dongle_max_pix_clk;
+       void *priv;
+       struct stereo_3d_features features_3d[TIMING_3D_FORMAT_MAX];
+       bool converter_disable_audio;
+
+       /* private to DC core */
+       struct dc_link *link;
+       struct dc_context *ctx;
+
+       /* private to dc_sink.c */
+       int ref_count;
 };
 
-void dc_sink_retain(const struct dc_sink *sink);
-void dc_sink_release(const struct dc_sink *sink);
+void dc_sink_retain(struct dc_sink *sink);
+void dc_sink_release(struct dc_sink *sink);
 
 const struct audio **dc_get_audios(struct dc *dc);
 
 struct dc_sink_init_data {
        enum signal_type sink_signal;
-       const struct dc_link *link;
+       struct dc_link *link;
        uint32_t dongle_max_pix_clk;
        bool converter_disable_audio;
 };
 
 struct dc_sink *dc_sink_create(const struct dc_sink_init_data *init_params);
+bool dc_sink_get_container_id(struct dc_sink *dc_sink, struct dc_container_id *container_id);
+bool dc_sink_set_container_id(struct dc_sink *dc_sink, const struct dc_container_id *container_id);
 
 /*******************************************************************************
- * Cursor interfaces - To manages the cursor within a target
+ * Cursor interfaces - To manages the cursor within a stream
  ******************************************************************************/
 /* TODO: Deprecated once we switch to dc_set_cursor_position */
-bool dc_target_set_cursor_attributes(
-       struct dc_target *dc_target,
+bool dc_stream_set_cursor_attributes(
+       const struct dc_stream_state *stream,
        const struct dc_cursor_attributes *attributes);
 
-bool dc_target_set_cursor_position(
-       struct dc_target *dc_target,
+bool dc_stream_set_cursor_position(
+       struct dc_stream_state *stream,
        const struct dc_cursor_position *position);
 
 /* Newer interfaces  */
@@ -706,36 +940,6 @@ struct dc_cursor {
        struct dc_cursor_attributes attributes;
 };
 
-/*
- * Create a new cursor with default values for a given target.
- */
-struct dc_cursor *dc_create_cursor_for_target(
-               const struct dc *dc,
-               struct dc_target *dc_target);
-
-/**
- * Commit cursor attribute changes such as pixel format and dimensions and
- * surface address.
- *
- * After this call:
- *   Cursor address and format is programmed to the new values.
- *   Cursor position is unmodified.
- */
-bool dc_commit_cursor(
-               const struct dc *dc,
-               struct dc_cursor *cursor);
-
-/*
- * Optimized cursor position update
- *
- * After this call:
- *   Cursor position will be programmed as well as enable/disable bit.
- */
-bool dc_set_cursor_position(
-               const struct dc *dc,
-               struct dc_cursor *cursor,
-               struct dc_cursor_position *pos);
-
 /*******************************************************************************
  * Interrupt interfaces
  ******************************************************************************/
@@ -754,38 +958,56 @@ enum dc_irq_source dc_get_hpd_irq_source_at_index(
 
 void dc_set_power_state(
                struct dc *dc,
-               enum dc_acpi_cm_power_state power_state,
-               enum dc_video_power_state video_power_state);
+               enum dc_acpi_cm_power_state power_state);
 void dc_resume(const struct dc *dc);
 
-/*******************************************************************************
- * DDC Interfaces
- ******************************************************************************/
-
-const struct ddc_service *dc_get_ddc_at_index(
-               struct dc *dc, uint32_t link_index);
-
 /*
  * DPCD access interfaces
  */
 
-bool dc_read_dpcd(
+bool dc_read_aux_dpcd(
                struct dc *dc,
                uint32_t link_index,
                uint32_t address,
                uint8_t *data,
                uint32_t size);
 
-bool dc_write_dpcd(
+bool dc_write_aux_dpcd(
                struct dc *dc,
                uint32_t link_index,
                uint32_t address,
                const uint8_t *data,
-       uint32_t size);
+               uint32_t size);
+
+bool dc_read_aux_i2c(
+               struct dc *dc,
+               uint32_t link_index,
+               enum i2c_mot_mode mot,
+               uint32_t address,
+               uint8_t *data,
+               uint32_t size);
+
+bool dc_write_aux_i2c(
+               struct dc *dc,
+               uint32_t link_index,
+               enum i2c_mot_mode mot,
+               uint32_t address,
+               const uint8_t *data,
+               uint32_t size);
+
+bool dc_query_ddc_data(
+               struct dc *dc,
+               uint32_t link_index,
+               uint32_t address,
+               uint8_t *write_buf,
+               uint32_t write_size,
+               uint8_t *read_buf,
+               uint32_t read_size);
 
 bool dc_submit_i2c(
                struct dc *dc,
                uint32_t link_index,
                struct i2c_command *cmd);
 
+
 #endif /* DC_INTERFACE_H_ */