From: Charlene Liu <[email protected]>

[why]
need to add null pointer access check

Reviewed-by: Gabe Teeger <[email protected]>
Signed-off-by: Charlene Liu <[email protected]>
Signed-off-by: George Zhang <[email protected]>
---
 .../drm/amd/display/dc/dio/dcn10/dcn10_link_encoder.c  | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dio/dcn10/dcn10_link_encoder.c 
b/drivers/gpu/drm/amd/display/dc/dio/dcn10/dcn10_link_encoder.c
index 4b36c01a9e7a..00e82d627d91 100644
--- a/drivers/gpu/drm/amd/display/dc/dio/dcn10/dcn10_link_encoder.c
+++ b/drivers/gpu/drm/amd/display/dc/dio/dcn10/dcn10_link_encoder.c
@@ -1189,8 +1189,9 @@ void dcn10_link_encoder_dp_set_phy_pattern(
                set_dp_phy_pattern_prbs7(enc10);
                break;
        case DP_TEST_PATTERN_80BIT_CUSTOM:
-               set_dp_phy_pattern_80bit_custom(
-                       enc10, param->custom_pattern);
+               if (param)
+                       set_dp_phy_pattern_80bit_custom(
+                               enc10, param->custom_pattern);
                break;
        case DP_TEST_PATTERN_CP2520_1:
                set_dp_phy_pattern_hbr2_compliance_cp2520_2(enc10, 1);
@@ -1202,8 +1203,9 @@ void dcn10_link_encoder_dp_set_phy_pattern(
                set_dp_phy_pattern_hbr2_compliance_cp2520_2(enc10, 3);
                break;
        case DP_TEST_PATTERN_VIDEO_MODE: {
-               set_dp_phy_pattern_passthrough_mode(
-                       enc10, param->dp_panel_mode);
+               if (param)
+                       set_dp_phy_pattern_passthrough_mode(
+                               enc10, param->dp_panel_mode);
                break;
        }

--
2.55.0

Reply via email to