drm/amd/display: Clean up dp_blank functions
authorDavid Francis <David.Francis@amd.com>
Wed, 24 Oct 2018 14:08:02 +0000 (10:08 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 19 Nov 2018 20:27:37 +0000 (15:27 -0500)
[Why]
Unused variable "refresh" and incorrect comment formatting

[How]
Remove variable, reindent comments

Signed-off-by: David Francis <David.Francis@amd.com>
Reviewed-by: Tony Cheng <Tony.Cheng@amd.com>
Acked-by: Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dce/dce_stream_encoder.c
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_stream_encoder.c

index c47c81883d3cc64bbe950e614705ca0b7833c18a..cce0d18f91da66ea5a3c641eba20ee524d5798a7 100644 (file)
@@ -908,7 +908,6 @@ static void dce110_stream_encoder_dp_blank(
        struct stream_encoder *enc)
 {
        struct dce110_stream_encoder *enc110 = DCE110STRENC_FROM_STRENC(enc);
-       uint32_t retries = 0;
        uint32_t  reg1 = 0;
        uint32_t max_retries = DP_BLANK_MAX_RETRY * 10;
 
@@ -926,30 +925,28 @@ static void dce110_stream_encoder_dp_blank(
         * (2 = start of the next vertical blank) */
        REG_UPDATE(DP_VID_STREAM_CNTL, DP_VID_STREAM_DIS_DEFER, 2);
        /* Larger delay to wait until VBLANK - use max retry of
-       * 10us*3000=30ms. This covers 16.6ms of typical 60 Hz mode +
-       * a little more because we may not trust delay accuracy.
-       */
+        * 10us*3000=30ms. This covers 16.6ms of typical 60 Hz mode +
+        * a little more because we may not trust delay accuracy.
+        */
        max_retries = DP_BLANK_MAX_RETRY * 150;
 
        /* disable DP stream */
        REG_UPDATE(DP_VID_STREAM_CNTL, DP_VID_STREAM_ENABLE, 0);
 
        /* the encoder stops sending the video stream
-       * at the start of the vertical blanking.
-       * Poll for DP_VID_STREAM_STATUS == 0
-       */
+        * at the start of the vertical blanking.
+        * Poll for DP_VID_STREAM_STATUS == 0
+        */
 
        REG_WAIT(DP_VID_STREAM_CNTL, DP_VID_STREAM_STATUS,
                        0,
                        10, max_retries);
 
-       ASSERT(retries <= max_retries);
-
        /* Tell the DP encoder to ignore timing from CRTC, must be done after
-       * the polling. If we set DP_STEER_FIFO_RESET before DP stream blank is
-       * complete, stream status will be stuck in video stream enabled state,
-       * i.e. DP_VID_STREAM_STATUS stuck at 1.
-       */
+        * the polling. If we set DP_STEER_FIFO_RESET before DP stream blank is
+        * complete, stream status will be stuck in video stream enabled state,
+        * i.e. DP_VID_STREAM_STATUS stuck at 1.
+        */
 
        REG_UPDATE(DP_STEER_FIFO, DP_STEER_FIFO_RESET, true);
 }
index 6f9078f3c4d39a353686ec9b5380cc27c63ba922..b8b5525a389abf0926d94e9c20b5522b63ba06be 100644 (file)
@@ -766,7 +766,6 @@ void enc1_stream_encoder_dp_blank(
        struct stream_encoder *enc)
 {
        struct dcn10_stream_encoder *enc1 = DCN10STRENC_FROM_STRENC(enc);
-       uint32_t retries = 0;
        uint32_t  reg1 = 0;
        uint32_t max_retries = DP_BLANK_MAX_RETRY * 10;
 
@@ -803,8 +802,6 @@ void enc1_stream_encoder_dp_blank(
                        0,
                        10, max_retries);
 
-       ASSERT(retries <= max_retries);
-
        /* Tell the DP encoder to ignore timing from CRTC, must be done after
         * the polling. If we set DP_STEER_FIFO_RESET before DP stream blank is
         * complete, stream status will be stuck in video stream enabled state,