drm/amd/display: Add comments to improve the code readability
authorRodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Tue, 26 Mar 2024 14:38:55 +0000 (08:38 -0600)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 12 Apr 2024 04:34:32 +0000 (00:34 -0400)
This commit just introduce some basic comments that helps to understand
the overall behavior of some structs.

Acked-by: Hamza Mahfooz <hamza.mahfooz@amd.com>
Signed-off-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/inc/core_types.h
drivers/gpu/drm/amd/display/dc/resource/dce80/dce80_resource.c
drivers/gpu/drm/amd/display/dc/resource/dcn201/dcn201_resource.c

index 3fab7c5bf09325211ef32dbe0b8765e7a5bd1f47..028b2f971e36de6205249229e9782fe49f207899 100644 (file)
@@ -339,7 +339,9 @@ struct stream_resource {
 };
 
 struct plane_resource {
+       /* scl_data is scratch space required to program a plane */
        struct scaler_data scl_data;
+       /* Below pointers to hw objects are required to enable the plane */
        struct hubp *hubp;
        struct mem_input *mi;
        struct input_pixel_processor *ipp;
index 35a2cce0c2b8e609475a2c8e9bd798fec6381994..56ee45e12b461cedaf4d892849cd904d6df0e9c4 100644 (file)
@@ -56,7 +56,6 @@
 #include "dce/dce_aux.h"
 #include "dce/dce_abm.h"
 #include "dce/dce_i2c.h"
-/* TODO remove this include */
 
 #ifndef mmMC_HUB_RDREQ_DMIF_LIMIT
 #include "gmc/gmc_7_1_d.h"
index 914b234d7f6b7f41c5d5af0d9f7ab52b4cac7390..0d2974c9c823b297f6c7613f0fb693c2f86b1c7c 100644 (file)
@@ -182,6 +182,7 @@ static struct _vcs_dpi_soc_bounding_box_st dcn201_soc = {
                                .socclk_mhz = 1254.0,
                                .dram_speed_mts = 14000.0,
                        },
+                       /* state4 is not an actual state, just defines unsupported for dml*/
                        {
                                .state = 4,
                                .dscclk_mhz = 400.0,