[bug report] drm/amd/display: access hpo dp link encoder only through link resource

Dan Carpenter dan.carpenter at oracle.com
Mon Jan 31 16:23:11 UTC 2022


Hello Wenjing Liu,

The patch 3d38a5839ea8: "drm/amd/display: access hpo dp link encoder
only through link resource" from Nov 26, 2021, leads to the following
Smatch static checker warning:

	drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dp.c:2961 dc_link_dp_sync_lt_end()
	error: NULL dereference inside function dp_disable_link_phy()

drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dp.c
    2954 bool dc_link_dp_sync_lt_end(struct dc_link *link, bool link_down)
    2955 {
    2956         /* If input parameter is set, shut down phy.
    2957          * Still shouldn't turn off dp_receiver (DPCD:600h)
    2958          */
    2959         if (link_down == true) {
    2960                 struct dc_link_settings link_settings = link->cur_link_settings;
--> 2961                 dp_disable_link_phy(link, NULL, link->connector_signal);
                                                   ^^^^
This NULL will lead to an Oops.

    2962                 if (dp_get_link_encoding_format(&link_settings) == DP_8b_10b_ENCODING)
    2963                         dp_set_fec_ready(link, NULL, false);
    2964         }
    2965 
    2966         link->sync_lt_in_progress = false;
    2967         return true;
    2968 }

regards,
dan carpenter


More information about the amd-gfx mailing list