drm/amd/display: Fix comments
authorSamson Tam <Samson.Tam@amd.com>
Tue, 9 Mar 2021 22:23:10 +0000 (17:23 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 15 Jun 2022 01:38:40 +0000 (21:38 -0400)
[Why & how]
Fix format and typo of comments.

Acked-by: Alan Liu <HaoPing.Liu@amd.com>
Signed-off-by: Samson Tam <Samson.Tam@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/clk_mgr/dcn20/dcn20_clk_mgr.c
drivers/gpu/drm/amd/display/dc/core/dc_link.c
drivers/gpu/drm/amd/display/dc/gpio/dcn20/hw_translate_dcn20.c
drivers/gpu/drm/amd/display/dc/gpio/dcn21/hw_translate_dcn21.c
drivers/gpu/drm/amd/display/dc/gpio/dcn30/hw_translate_dcn30.c

index cac80ba..fb82e9f 100644 (file)
@@ -436,7 +436,6 @@ void dcn2_read_clocks_from_hw_dentist(struct clk_mgr *clk_mgr_base)
                clk_mgr_base->clks.dppclk_khz = (DENTIST_DIVIDER_RANGE_SCALE_FACTOR
                                * clk_mgr->base.dentist_vco_freq_khz) / dpp_divider;
        }
-
 }
 
 void dcn2_get_clock(struct clk_mgr *clk_mgr,
index 7884530..1998689 100644 (file)
@@ -1379,7 +1379,9 @@ bool dc_link_get_hpd_state(struct dc_link *dc_link)
 static enum hpd_source_id get_hpd_line(struct dc_link *link)
 {
        struct gpio *hpd;
-       enum hpd_source_id hpd_id = HPD_SOURCEID_UNKNOWN;
+       enum hpd_source_id hpd_id;
+
+       hpd_id = HPD_SOURCEID_UNKNOWN;
 
        hpd = get_hpd_gpio(link->ctx->dc_bios, link->link_id,
                           link->ctx->gpio_service);
@@ -1418,7 +1420,9 @@ static enum hpd_source_id get_hpd_line(struct dc_link *link)
 static enum channel_id get_ddc_line(struct dc_link *link)
 {
        struct ddc *ddc;
-       enum channel_id channel = CHANNEL_ID_UNKNOWN;
+       enum channel_id channel;
+
+       channel = CHANNEL_ID_UNKNOWN;
 
        ddc = dal_ddc_service_get_ddc_pin(link->ddc);
 
index 52ba62b..3005ee7 100644 (file)
@@ -150,7 +150,8 @@ static bool offset_to_id(
        /* DDC */
        /* we don't care about the GPIO_ID for DDC
         * in DdcHandle it will use GPIO_ID_DDC_DATA/GPIO_ID_DDC_CLOCK
-        * directly in the create method */
+        * directly in the create method
+        */
        case REG(DC_GPIO_DDC1_A):
                *en = GPIO_DDC_LINE_DDC1;
                return true;
@@ -173,14 +174,16 @@ static bool offset_to_id(
                *en = GPIO_DDC_LINE_DDC_VGA;
                return true;
 
-//     case REG(DC_GPIO_I2CPAD_A): not exit
-//     case REG(DC_GPIO_PWRSEQ_A):
-//     case REG(DC_GPIO_PAD_STRENGTH_1):
-//     case REG(DC_GPIO_PAD_STRENGTH_2):
-//     case REG(DC_GPIO_DEBUG):
+/*
+ *     case REG(DC_GPIO_I2CPAD_A): not exit
+ *     case REG(DC_GPIO_PWRSEQ_A):
+ *     case REG(DC_GPIO_PAD_STRENGTH_1):
+ *     case REG(DC_GPIO_PAD_STRENGTH_2):
+ *     case REG(DC_GPIO_DEBUG):
+ */
        /* UNEXPECTED */
        default:
-//     case REG(DC_GPIO_SYNCA_A): not exist
+/*     case REG(DC_GPIO_SYNCA_A): not exist */
                ASSERT_CRITICAL(false);
                return false;
        }
index 291966e..d734e3a 100644 (file)
@@ -153,7 +153,8 @@ static bool offset_to_id(
        /* DDC */
        /* we don't care about the GPIO_ID for DDC
         * in DdcHandle it will use GPIO_ID_DDC_DATA/GPIO_ID_DDC_CLOCK
-        * directly in the create method */
+        * directly in the create method
+        */
        case REG(DC_GPIO_DDC1_A):
                *en = GPIO_DDC_LINE_DDC1;
                return true;
@@ -173,14 +174,16 @@ static bool offset_to_id(
                *en = GPIO_DDC_LINE_DDC_VGA;
                return true;
 
-//     case REG(DC_GPIO_I2CPAD_A): not exit
-//     case REG(DC_GPIO_PWRSEQ_A):
-//     case REG(DC_GPIO_PAD_STRENGTH_1):
-//     case REG(DC_GPIO_PAD_STRENGTH_2):
-//     case REG(DC_GPIO_DEBUG):
+/*
+ *     case REG(DC_GPIO_I2CPAD_A): not exit
+ *     case REG(DC_GPIO_PWRSEQ_A):
+ *     case REG(DC_GPIO_PAD_STRENGTH_1):
+ *     case REG(DC_GPIO_PAD_STRENGTH_2):
+ *     case REG(DC_GPIO_DEBUG):
+ */
        /* UNEXPECTED */
        default:
-//     case REG(DC_GPIO_SYNCA_A): not exist
+/*     case REG(DC_GPIO_SYNCA_A): not exista */
 #ifdef PALLADIUM_SUPPORTED
                *id = GPIO_ID_HPD;
                *en = GPIO_DDC_LINE_DDC1;
index 3169c56..49d6250 100644 (file)
@@ -155,7 +155,8 @@ static bool offset_to_id(
        /* DDC */
        /* we don't care about the GPIO_ID for DDC
         * in DdcHandle it will use GPIO_ID_DDC_DATA/GPIO_ID_DDC_CLOCK
-        * directly in the create method */
+        * directly in the create method
+        */
        case REG(DC_GPIO_DDC1_A):
                *en = GPIO_DDC_LINE_DDC1;
                return true;
@@ -178,14 +179,16 @@ static bool offset_to_id(
                *en = GPIO_DDC_LINE_DDC_VGA;
                return true;
 
-//     case REG(DC_GPIO_I2CPAD_A): not exit
-//     case REG(DC_GPIO_PWRSEQ_A):
-//     case REG(DC_GPIO_PAD_STRENGTH_1):
-//     case REG(DC_GPIO_PAD_STRENGTH_2):
-//     case REG(DC_GPIO_DEBUG):
+/*
+ *     case REG(DC_GPIO_I2CPAD_A): not exit
+ *     case REG(DC_GPIO_PWRSEQ_A):
+ *     case REG(DC_GPIO_PAD_STRENGTH_1):
+ *     case REG(DC_GPIO_PAD_STRENGTH_2):
+ *     case REG(DC_GPIO_DEBUG):
+ */
        /* UNEXPECTED */
        default:
-//     case REG(DC_GPIO_SYNCA_A): not exist
+/*     case REG(DC_GPIO_SYNCA_A): not exist */
                ASSERT_CRITICAL(false);
                return false;
        }
@@ -369,7 +372,7 @@ static const struct hw_translate_funcs funcs = {
 };
 
 /*
- * dal_hw_translate_dcn10_init
+ * dal_hw_translate_dcn30_init
  *
  * @brief
  * Initialize Hw translate function pointers.