RE: [PATCH v2] drm/amd/display: Drop unused DCN_BASE variable in dcn314_resource.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[Public]

Reviewed-by: Roman Li <Roman.Li@xxxxxxx>

> -----Original Message-----
> From: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM@xxxxxxx>
> Sent: Wednesday, May 31, 2023 1:46 PM
> To: Pillai, Aurabindo <Aurabindo.Pillai@xxxxxxx>; Mahfooz, Hamza
> <Hamza.Mahfooz@xxxxxxx>; Siqueira, Rodrigo
> <Rodrigo.Siqueira@xxxxxxx>; Wentland, Harry
> <Harry.Wentland@xxxxxxx>; Li, Roman <Roman.Li@xxxxxxx>
> Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; SHANMUGAM, SRINIVASAN
> <SRINIVASAN.SHANMUGAM@xxxxxxx>
> Subject: [PATCH v2] drm/amd/display: Drop unused DCN_BASE variable in
> dcn314_resource.c
>
> Fixes the following W=1 kernel build warning:
>
> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn314/dcn314_resource.c:12
> 8:29: warning: ‘DCN_BASE’ defined but not used [-Wunused-const-variable=]
>   128 | static const struct IP_BASE DCN_BASE = { { { { 0x00000012,
> 0x000000C0, 0x000034C0, 0x00009000, 0x02403C00, 0, 0, 0 } },
>       |                             ^~~~~~~~
>
> Suggested-by: Roman Li <Roman.Li@xxxxxxx>
> Cc: Hamza Mahfooz <hamza.mahfooz@xxxxxxx>
> Cc: Rodrigo Siqueira <Rodrigo.Siqueira@xxxxxxx>
> Cc: Harry Wentland <harry.wentland@xxxxxxx>
> Cc: Aurabindo Pillai <aurabindo.pillai@xxxxxxx>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>
> ---
>
> v2:
>  - Remove even unused IP_BASE_INSTANCE and IP_BASE struct definitions
>    altogether (Roman)
>
>  .../drm/amd/display/dc/dcn314/dcn314_resource.c | 17 -----------------
>  1 file changed, 17 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> b/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> index 3592efcc7fae..837884c4f03a 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> @@ -117,23 +117,6 @@
>  #define regBIF_BX2_BIOS_SCRATCH_6                    0x003e
>  #define regBIF_BX2_BIOS_SCRATCH_6_BASE_IDX           1
>
> -struct IP_BASE_INSTANCE {
> -     unsigned int segment[MAX_SEGMENT];
> -};
> -
> -struct IP_BASE {
> -     struct IP_BASE_INSTANCE instance[MAX_INSTANCE];
> -};
> -
> -static const struct IP_BASE DCN_BASE = { { { { 0x00000012, 0x000000C0,
> 0x000034C0, 0x00009000, 0x02403C00, 0, 0, 0 } },
> -                                     { { 0, 0, 0, 0, 0, 0, 0, 0 } },
> -                                     { { 0, 0, 0, 0, 0, 0, 0, 0 } },
> -                                     { { 0, 0, 0, 0, 0, 0, 0, 0 } },
> -                                     { { 0, 0, 0, 0, 0, 0, 0, 0 } },
> -                                     { { 0, 0, 0, 0, 0, 0, 0, 0 } },
> -                                     { { 0, 0, 0, 0, 0, 0, 0, 0 } } } };
> -
> -
>  #define DC_LOGGER_INIT(logger)
>
>  enum dcn31_clk_src_array_id {
> --
> 2.25.1





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux