]> 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 3e2ed3d15379ae1de0c0a24eb29b7b08ddb7511c..d1a6398043a3cd8fb93df81e3a47e5882f1c28b0 100644 (file)
@@ -31,6 +31,8 @@
 #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_SURFACES 3
 #define MAX_STREAMS 6
@@ -45,7 +47,7 @@ struct dc_caps {
        uint32_t max_links;
        uint32_t max_audios;
        uint32_t max_slave_planes;
-       uint32_t max_surfaces;
+       uint32_t max_planes;
        uint32_t max_downscale_ratio;
        uint32_t i2c_speed_in_khz;
 
@@ -90,43 +92,39 @@ struct dc_static_screen_events {
 
 /* Forward declaration*/
 struct dc;
-struct dc_surface;
+struct dc_plane_state;
 struct validate_context;
 
 struct dc_cap_funcs {
-#if defined(CONFIG_DRM_AMD_DC_DCN1_0)
        bool (*get_dcc_compression_cap)(const struct dc *dc,
                        const struct dc_dcc_surface_param *input,
                        struct dc_surface_dcc_cap *output);
-#else
-       int i;
-#endif
 };
 
-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,
-                       const struct dc_stream **stream,
+                       struct dc_stream_state **stream,
                        int num_streams,
                        unsigned int *v_pos,
                        unsigned int *nom_v_pos);
 
        bool (*set_gamut_remap)(struct dc *dc,
-                       const struct dc_stream *stream);
+                       const struct dc_stream_state *stream);
 
        bool (*program_csc_matrix)(struct dc *dc,
-                       const struct dc_stream *stream);
+                       struct dc_stream_state *stream);
 
        void (*set_static_screen_events)(struct dc *dc,
-                       const struct dc_stream **stream,
+                       struct dc_stream_state **stream,
                        int num_streams,
                        const struct dc_static_screen_events *events);
 
-       void (*set_dither_option)(const struct dc_stream *stream,
+       void (*set_dither_option)(struct dc_stream_state *stream,
                        enum dc_dither_option option);
 };
 
@@ -141,11 +139,11 @@ struct dc_link_funcs {
                        bool skip_video_pattern);
        void (*set_preferred_link_settings)(struct dc *dc,
                        struct dc_link_settings *link_setting,
-                       const struct dc_link *link);
+                       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,
@@ -160,6 +158,7 @@ struct dc_config {
 
 struct dc_debug {
        bool surface_visual_confirm;
+       bool sanity_checks;
        bool max_disp_clk;
        bool surface_trace;
        bool timing_trace;
@@ -168,7 +167,6 @@ struct dc_debug {
        bool disable_stutter;
        bool disable_dcc;
        bool disable_dfs_bypass;
-#if defined(CONFIG_DRM_AMD_DC_DCN1_0)
        bool disable_dpp_power_gate;
        bool disable_hubp_power_gate;
        bool disable_pplib_wm_range;
@@ -182,7 +180,6 @@ struct dc_debug {
        int percent_of_ideal_drambw;
        int dram_clock_change_latency_ns;
        int always_scale;
-#endif
        bool disable_pplib_clock_request;
        bool disable_clock_gate;
        bool disable_dmcu;
@@ -193,7 +190,7 @@ struct dc_debug {
 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;
@@ -228,6 +225,9 @@ 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);
@@ -236,6 +236,8 @@ 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
  ******************************************************************************/
@@ -292,9 +294,23 @@ 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_context *ctx;
+       int ref_count;
+};
+
+/*
+ * 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_surface {
+struct dc_plane_state {
        struct dc_plane_address address;
 
        struct scaling_taps scaling_quality;
@@ -308,8 +324,8 @@ struct dc_surface {
        struct dc_plane_dcc_param dcc;
        struct dc_hdr_static_metadata hdr_static_ctx;
 
-       const struct dc_gamma *gamma_correction;
-       const struct dc_transfer_func *in_transfer_func;
+       struct dc_gamma *gamma_correction;
+       struct dc_transfer_func *in_transfer_func;
 
        enum dc_color_space color_space;
        enum surface_pixel_format format;
@@ -320,6 +336,14 @@ struct dc_surface {
        bool visible;
        bool flip_immediate;
        bool horizontal_mirror;
+
+       /* private to DC core */
+       struct dc_plane_status status;
+       struct dc_context *ctx;
+
+       /* private to dc_surface.c */
+       enum dc_irq_source irq_source;
+       int ref_count;
 };
 
 struct dc_plane_info {
@@ -343,7 +367,7 @@ struct dc_scaling_info {
 };
 
 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;
@@ -357,34 +381,23 @@ struct dc_surface_update {
        struct dc_transfer_func *in_transfer_func;
        struct dc_hdr_static_metadata *hdr_static_metadata;
 };
-/*
- * 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;
-       bool is_right_eye;
-};
 
 /*
  * 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);
 
 /*
@@ -409,11 +422,11 @@ struct dc_flip_addrs {
  *   This does not trigger a flip.  No surface address is programmed.
  */
 
-bool dc_commit_surfaces_to_stream(
+bool dc_commit_planes_to_stream(
                struct dc *dc,
-               const struct dc_surface **dc_surfaces,
-               uint8_t surface_count,
-               const struct dc_stream *stream);
+               struct dc_plane_state **plane_states,
+               uint8_t new_plane_count,
+               struct dc_stream_state *stream);
 
 bool dc_post_update_surfaces_to_stream(
                struct dc *dc);
@@ -453,8 +466,20 @@ enum surface_update_type {
 /*******************************************************************************
  * Stream Interfaces
  ******************************************************************************/
-struct dc_stream {
-       const struct dc_sink *sink;
+
+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;
+};
+
+struct dc_stream_state {
+       struct dc_sink *sink;
        struct dc_crtc_timing timing;
 
        struct rect src; /* composition area */
@@ -464,7 +489,7 @@ struct dc_stream {
 
        struct freesync_context freesync_ctx;
 
-       const struct dc_transfer_func *out_transfer_func;
+       struct dc_transfer_func *out_transfer_func;
        struct colorspace_transform gamut_remap_matrix;
        struct csc_transform csc_color_matrix;
 
@@ -480,6 +505,21 @@ struct dc_stream {
        /* 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_stream_update {
@@ -488,6 +528,8 @@ struct dc_stream_update {
        struct dc_transfer_func *out_transfer_func;
 };
 
+bool dc_is_stream_unchanged(
+       struct dc_stream_state *old_stream, struct dc_stream_state *stream);
 
 /*
  * Setup stream attributes if no stream updates are provided
@@ -504,32 +546,32 @@ struct dc_stream_update {
  *
  */
 
-void dc_update_surfaces_and_stream(struct dc *dc,
+void dc_update_planes_and_stream(struct dc *dc,
                struct dc_surface_update *surface_updates, int surface_count,
-               const struct dc_stream *dc_stream,
+               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 *stream,
+       const struct dc_stream_state *stream,
        struct dal_logger *dc_logger,
        enum dc_log_type log_type);
 
 uint8_t dc_get_current_stream_count(const struct dc *dc);
-struct dc_stream *dc_get_stream_at_index(const struct dc *dc, uint8_t i);
+struct dc_stream_state *dc_get_stream_at_index(const struct dc *dc, uint8_t i);
 
 /*
  * Return the current frame counter.
  */
-uint32_t dc_stream_get_vblank_counter(const struct dc_stream *stream);
+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
  */
-bool dc_stream_get_scanoutpos(const struct dc_stream *stream,
+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,
@@ -539,11 +581,14 @@ bool dc_stream_get_scanoutpos(const struct dc_stream *stream,
  * Structure to store surface/stream associations for validation
  */
 struct dc_validation_set {
-       const struct dc_stream *stream;
-       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.
  *
@@ -570,7 +615,7 @@ bool dc_validate_resources(
 
 bool dc_validate_guaranteed(
                const struct dc *dc,
-               const struct dc_stream *stream);
+               struct dc_stream_state *stream);
 
 void dc_resource_validate_ctx_copy_construct(
                const struct validate_context *src_ctx,
@@ -578,6 +623,17 @@ void dc_resource_validate_ctx_copy_construct(
 
 void dc_resource_validate_ctx_destruct(struct validate_context *context);
 
+/*
+ * 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 streams are enabled with blank stream; no memory read.
+ */
+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.
@@ -588,7 +644,7 @@ void dc_resource_validate_ctx_destruct(struct validate_context *context);
  */
 bool dc_commit_streams(
                struct dc *dc,
-               const struct dc_stream *streams[],
+               struct dc_stream_state *streams[],
                uint8_t stream_count);
 /*
  * Enable stereo when commit_streams is not required,
@@ -597,30 +653,19 @@ bool dc_commit_streams(
 bool dc_enable_stereo(
        struct dc *dc,
        struct validate_context *context,
-       const struct dc_stream *streams[],
+       struct dc_stream_state *streams[],
        uint8_t stream_count);
 
 /**
  * Create a new default stream for the requested sink
  */
-struct dc_stream *dc_create_stream_for_sink(const struct dc_sink *dc_sink);
-
-void dc_stream_retain(const struct dc_stream *dc_stream);
-void dc_stream_release(const struct dc_stream *dc_stream);
+struct dc_stream_state *dc_create_stream_for_sink(struct dc_sink *dc_sink);
 
-struct dc_stream_status {
-       int primary_otg_inst;
-       int surface_count;
-       const struct dc_surface *surfaces[MAX_SURFACE_NUM];
-
-       /*
-        * link this stream passes through
-        */
-       const struct dc_link *link;
-};
+void dc_stream_retain(struct dc_stream_state *dc_stream);
+void dc_stream_release(struct dc_stream_state *dc_stream);
 
-const struct dc_stream_status *dc_stream_get_status(
-       const struct dc_stream *dc_stream);
+struct dc_stream_status *dc_stream_get_status(
+       struct dc_stream_state *dc_stream);
 
 enum surface_update_type dc_check_update_surfaces_for_stream(
                struct dc *dc,
@@ -629,18 +674,66 @@ enum surface_update_type dc_check_update_surfaces_for_stream(
                struct dc_stream_update *stream_update,
                const struct dc_stream_status *stream_status);
 
+
+void dc_retain_validate_context(struct validate_context *context);
+void dc_release_validate_context(struct validate_context *context);
+
 /*******************************************************************************
  * Link Interfaces
  ******************************************************************************/
 
+struct dpcd_caps {
+       union dpcd_rev dpcd_rev;
+       union max_lane_count max_ln_count;
+       union max_down_spread max_down_spread;
+
+       /* 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;
+
+       uint32_t sink_dev_id;
+       uint32_t branch_dev_id;
+       int8_t branch_dev_name[6];
+       int8_t branch_hw_revision;
+
+       bool allow_invalid_MSA_timing_param;
+       bool panel_mode_edp;
+};
+
+struct dc_link_status {
+       struct dpcd_caps *dpcd_caps;
+};
+
+/* 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;
@@ -651,9 +744,9 @@ 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;
 
@@ -669,32 +762,31 @@ struct dc_link {
        struct ddc_service *ddc;
 
        bool aux_mode;
-};
 
-struct dpcd_caps {
-       union dpcd_rev dpcd_rev;
-       union max_lane_count max_ln_count;
-       union max_down_spread max_down_spread;
+       /* Private to DC core */
 
-       /* 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;
+       const struct core_dc *dc;
 
-       uint32_t sink_dev_id;
-       uint32_t branch_dev_id;
-       int8_t branch_dev_name[6];
-       int8_t branch_hw_revision;
+       struct dc_context *ctx;
 
-       bool allow_invalid_MSA_timing_param;
-       bool panel_mode_edp;
-};
+       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);
@@ -704,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(
@@ -712,7 +804,7 @@ 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_set_abm_disable(const struct dc_link *dc_link);
 
@@ -720,8 +812,8 @@ bool dc_link_set_psr_enable(const struct dc_link *dc_link, bool enable);
 
 bool dc_link_get_psr_state(const struct dc_link *dc_link, uint32_t *psr_state);
 
-bool dc_link_setup_psr(const struct dc_link *dc_link,
-               const struct dc_stream *stream, struct psr_config *psr_config,
+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.
@@ -730,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:
@@ -738,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(
-       const struct dc_link *link,
+       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);
@@ -769,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,
@@ -804,16 +897,23 @@ struct dc_sink {
        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;
 };
@@ -827,11 +927,11 @@ bool dc_sink_set_container_id(struct dc_sink *dc_sink, const struct dc_container
  ******************************************************************************/
 /* TODO: Deprecated once we switch to dc_set_cursor_position */
 bool dc_stream_set_cursor_attributes(
-       const struct dc_stream *stream,
+       const struct dc_stream_state *stream,
        const struct dc_cursor_attributes *attributes);
 
 bool dc_stream_set_cursor_position(
-       const struct dc_stream *stream,
+       struct dc_stream_state *stream,
        const struct dc_cursor_position *position);
 
 /* Newer interfaces  */