Re: [PATCH] drm/amd/display: fix a missing prototype warning in DCN303

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

 



[AMD Official Use Only]


Hi Siqueira,

Thanks for the heads up, lets apply Isabella's patch instead.

--

Regards,
Jay

From: Siqueira, Rodrigo <Rodrigo.Siqueira@xxxxxxx>
Sent: Wednesday, December 8, 2021 10:17 AM
To: Pillai, Aurabindo <Aurabindo.Pillai@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>; Isabella Basso <isabbasso@xxxxxxxxxx>
Cc: Wentland, Harry <Harry.Wentland@xxxxxxx>; Siqueira, Rodrigo <Rodrigo.Siqueira@xxxxxxx>; kernel test robot <lkp@xxxxxxxxx>
Subject: Re: [PATCH] drm/amd/display: fix a missing prototype warning in DCN303
 


On 2021-12-07 2:18 p.m., Aurabindo Pillai wrote:
> [Why&How]
> A missing include statement triggered a warning when running
> a build with W=1:
>
>>> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn303/dcn303_init.c:30:6: warning: no previous prototype for 'dcn303_hw_sequencer_construct' [-Wmissing-prototypes]
>        30 | void dcn303_hw_sequencer_construct(struct dc *dc)
>           |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Fixes: cd6d421e3d1a (drm/amd/display: Initial DC support for Beige Goby)
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Signed-off-by: Aurabindo Pillai <aurabindo.pillai@xxxxxxx>
> ---
>   drivers/gpu/drm/amd/display/dc/dcn303/dcn303_init.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_init.c b/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_init.c
> index aa5dbbade2bd..d59b24a972bc 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_init.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_init.c
> @@ -7,6 +7,7 @@
>  
>   #include "dcn303_hwseq.h"
>   #include "dcn30/dcn30_init.h"
> +#include "dcn303_init.h"
>   #include "dc.h"
>  
>   void dcn303_hw_sequencer_construct(struct dc *dc)
>

Hi Jay,

Isabella made the below patch that fixes this issue in multiple files:

  https://patchwork.freedesktop.org/patch/465821/?series=97701&rev=1

Maybe we can go ahead with Isabella's patch?

Thanks
Siqueira

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

  Powered by Linux