linux-kernel - Re: [PATCH] drm/amd/display: Remove unused struct freesync

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <13140020-8139-64f9-51b1-2b71c9b673af@amd.com>
Date:   Wed, 27 Jul 2022 18:37:49 -0400
From:   Rodrigo Siqueira Jordao <Rodrigo.Siqueira@....com>
To:     Maíra Canal <mairacanal@...eup.net>,
        Harry Wentland <harry.wentland@....com>,
        Leo Li <sunpeng.li@....com>,
        Alex Deucher <alexander.deucher@....com>,
        christian.koenig@....com, Xinhui.Pan@....com,
        David Airlie <airlied@...ux.ie>,
        Daniel Vetter <daniel@...ll.ch>
Cc:     amd-gfx@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
        mwen@...lia.com, andrealmeid@...eup.net,
        Isabella Basso <isabbasso@...eup.net>, magalilemes00@...il.com,
        tales.aparecida@...il.com
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 <mairacanal@...eup.net>
> ---
>   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 <Rodrigo.Siqueira@....com>

and applied to amd-staging-drm-next.

Thanks
Siqueira