Re: [PATCH] drm/amd/display: Remove redundant header files

From: Alex Deucher
Date: Tue Aug 19 2025 - 11:04:18 EST


Applied. Thanks!

On Tue, Aug 19, 2025 at 10:33 AM Liao Yuanhong <liaoyuanhong@xxxxxxxx> wrote:
>
> The header file "dc_stream.h" is already included on line 1507. Remove the
> redundant include.
>
> This is because the header file was initially included towards the latter
> part of the code. Subsequent commits had to include the header file again
> earlier in the code. In my opinion, this doesn't count as a fix; it just
> requires removing the redundant header inclusion.
>
> Signed-off-by: Liao Yuanhong <liaoyuanhong@xxxxxxxx>
> ---
> drivers/gpu/drm/amd/display/dc/dc.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
> index 1ab05eabbddb..2ceaf861edb9 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc.h
> +++ b/drivers/gpu/drm/amd/display/dc/dc.h
> @@ -1854,8 +1854,6 @@ void dc_3dlut_func_retain(struct dc_3dlut *lut);
> void dc_post_update_surfaces_to_stream(
> struct dc *dc);
>
> -#include "dc_stream.h"
> -
> /**
> * struct dc_validation_set - Struct to store surface/stream associations for validation
> */
> --
> 2.34.1
>