drm/amd/display: Refactor on dc_sink structure.

Signed-off-by: Zeyu Fan <Zeyu.Fan@amd.com>
Acked-by: Harry Wentland <Harry.Wentland@amd.com>
Reviewed-by: Tony Cheng <Tony.Cheng@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
index d275fc88f71e20c299de46286ff5ed49c1e25699..5ca72afe8f6f4f2b933a7a1c0e3acb5d5a7a197b 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
@@ -675,10 +675,6 @@ bool dc_link_detect(const struct dc_link *dc_link, bool boot)
 
                sink_init_data.link = &link->public;
                sink_init_data.sink_signal = sink_caps.signal;
-               sink_init_data.dongle_max_pix_clk =
-                       sink_caps.max_hdmi_pixel_clock;
-               sink_init_data.converter_disable_audio =
-                       converter_disable_audio;
 
                dc_sink = dc_sink_create(&sink_init_data);
                if (!dc_sink) {
@@ -686,6 +682,9 @@ bool dc_link_detect(const struct dc_link *dc_link, bool boot)
                        return false;
                }
 
+               dc_sink->dongle_max_pix_clk = sink_caps.max_hdmi_pixel_clock;
+               dc_sink->converter_disable_audio = converter_disable_audio;
+
                sink = DC_SINK_TO_CORE(dc_sink);
                link->public.local_sink = &sink->public;
 
@@ -1361,7 +1360,7 @@ enum dc_status dc_link_validate_mode_timing(
                struct core_link *link,
                const struct dc_crtc_timing *timing)
 {
-       uint32_t max_pix_clk = stream->sink->dongle_max_pix_clk;
+       uint32_t max_pix_clk = stream->sink->public.dongle_max_pix_clk;
 
        /* A hack to avoid failing any modes for EDID override feature on
         * topology change such as lower quality cable for DP or different dongle

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
index c38f71e4538177a6123e9374d5a805c3bb30276c..326019407cb1b50fdef39f7ac98ed2b167306904 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
@@ -1193,7 +1193,7 @@ enum dc_status resource_map_pool_resources(
                        pipe_ctx->stream_enc);
 
                /* TODO: Add check if ASIC support and EDID audio */
-               if (!stream->sink->converter_disable_audio &&
+               if (!stream->sink->public.converter_disable_audio &&
                        dc_is_audio_capable_signal(pipe_ctx->stream->signal) &&
                        stream->public.audio_info.mode_count) {
                        pipe_ctx->audio = find_first_free_audio(

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_sink.c b/drivers/gpu/drm/amd/display/dc/core/dc_sink.c
index 0a805ea1159aba2a110e34714e625a3257e39a28..8a204731301dbee7437a2103ca557313fc6d5041 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc_sink.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_sink.c
@@ -60,8 +60,8 @@ static bool construct(struct sink *sink, const struct dc_sink_init_data *init_pa
        sink->protected.public.sink_signal = init_params->sink_signal;
        sink->protected.link = core_link;
        sink->protected.ctx = core_link->ctx;
-       sink->protected.dongle_max_pix_clk = init_params->dongle_max_pix_clk;
-       sink->protected.converter_disable_audio =
+       sink->protected.public.dongle_max_pix_clk = init_params->dongle_max_pix_clk;
+       sink->protected.public.converter_disable_audio =
                        init_params->converter_disable_audio;
 
        return true;

diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
index e2c2a0bf764dc5a34746a6acd6c489270b45f8bd..c6c0cf500ae8db247d53df87fa570c3e1b90ef48 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -653,6 +653,8 @@ struct dc_sink {
        enum signal_type sink_signal;
        struct dc_edid dc_edid; /* raw edid */
        struct dc_edid_caps edid_caps; /* parse display caps */
+       uint32_t dongle_max_pix_clk;
+       bool converter_disable_audio;
 };
 
 void dc_sink_retain(const struct dc_sink *sink);

diff --git a/drivers/gpu/drm/amd/display/dc/inc/core_types.h b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
index b29fca9dfefffb7bb637a89b008aee9aa7ba9a56..faec2292cc27ff0c346c0b63c127213c0b1844a1 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/inc/core_types.h
+++ b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
@@ -109,8 +109,6 @@ struct core_sink {
        /* not used for now */
        struct core_link *link;
        struct dc_context *ctx;
-       uint32_t dongle_max_pix_clk;
-       bool converter_disable_audio;
 };
 
 /************ link *****************/