2022-07-27 19:04:52

by Maira Canal

Subject: [PATCH] drm/amd/display: Remove unused struct freesync_context

All references to struct freesync_context were removed, so remove the
struct freesync_context itself and its entry on struct dc_stream_state.

Signed-off-by: Maíra Canal <[email protected]>
---
drivers/gpu/drm/amd/display/dc/dc_stream.h | 8 --------
1 file changed, 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dc_stream.h b/drivers/gpu/drm/amd/display/dc/dc_stream.h
index c3d97206ed89..f87f852d4829 100644
--- a/drivers/gpu/drm/amd/display/dc/dc_stream.h
+++ b/drivers/gpu/drm/amd/display/dc/dc_stream.h
@@ -48,11 +48,6 @@ struct dc_stream_status {
bool is_abm_supported;
};

-// TODO: References to this needs to be removed..
-struct freesync_context {
- bool dummy;
-};
-
enum hubp_dmdata_mode {
DMDATA_SW_MODE,
DMDATA_HW_MODE
@@ -184,9 +179,6 @@ struct dc_stream_state {
struct rect src; /* composition area */
struct rect dst; /* stream addressable area */

- // TODO: References to this needs to be removed..
- struct freesync_context freesync_ctx;
-
struct audio_info audio_info;

struct dc_info_packet hdr_static_metadata;
--
2.37.1

2022-07-27 23:15:25

by Rodrigo Siqueira Jordao

Subject: Re: [PATCH] drm/amd/display: Remove unused struct freesync_context

On 2022-07-27 13:50, Maíra Canal wrote:
> All references to struct freesync_context were removed, so remove the
> struct freesync_context itself and its entry on struct dc_stream_state.
>
> Signed-off-by: Maíra Canal <[email protected]>
> ---
> drivers/gpu/drm/amd/display/dc/dc_stream.h | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dc_stream.h b/drivers/gpu/drm/amd/display/dc/dc_stream.h
> index c3d97206ed89..f87f852d4829 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc_stream.h
> +++ b/drivers/gpu/drm/amd/display/dc/dc_stream.h
> @@ -48,11 +48,6 @@ struct dc_stream_status {
> bool is_abm_supported;
> };
>
> -// TODO: References to this needs to be removed..
> -struct freesync_context {
> - bool dummy;
> -};
> -
> enum hubp_dmdata_mode {
> DMDATA_SW_MODE,
> DMDATA_HW_MODE
> @@ -184,9 +179,6 @@ struct dc_stream_state {
> struct rect src; /* composition area */
> struct rect dst; /* stream addressable area */
>
> - // TODO: References to this needs to be removed..
> - struct freesync_context freesync_ctx;
> -
> struct audio_info audio_info;
>
> struct dc_info_packet hdr_static_metadata;

Reviewed-by: Rodrigo Siqueira <[email protected]>

and applied to amd-staging-drm-next.

Thanks
Siqueira