linux-kernel - Re: [PATCH] drm/amd/display: mark amdgpu_dm_connector_funcs

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: <5c384fc1-4f4b-2ef1-2349-060448f81be6@amd.com>
Date:   Tue, 2 May 2023 08:32:32 -0400
From:   Hamza Mahfooz <hamza.mahfooz@....com>
To:     Arnd Bergmann <arnd@...nel.org>,
        Harry Wentland <harry.wentland@....com>,
        Leo Li <sunpeng.li@....com>,
        Rodrigo Siqueira <Rodrigo.Siqueira@....com>,
        Alex Deucher <alexander.deucher@....com>,
        Christian König <christian.koenig@....com>
Cc:     Stylon Wang <stylon.wang@....com>,
        kernel test robot <lkp@...el.com>,
        Arnd Bergmann <arnd@...db.de>, Alex Hung <alex.hung@....com>,
        Qingqing Zhuo <qingqing.zhuo@....com>,
        "Pan, Xinhui" <Xinhui.Pan@....com>, linux-kernel@...r.kernel.org,
        amd-gfx@...ts.freedesktop.org, Hans de Goede <hdegoede@...hat.com>,
        Aurabindo Pillai <aurabindo.pillai@....com>,
        Hersen Wu <hersenxs.wu@....com>,
        dri-devel@...ts.freedesktop.org, Daniel Vetter <daniel@...ll.ch>,
        Wenchieh Chien <wenchieh.chien@....com>,
        David Airlie <airlied@...il.com>
Subject: Re: [PATCH] drm/amd/display: mark amdgpu_dm_connector_funcs_force
 static

On 5/1/23 10:31, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@...db.de>
> 
> A global function without a header prototype has made it into
> linux-next during the merge window:
> 
> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:6339:6: error: no previous prototype for 'amdgpu_dm_connector_funcs_force' [-Werror=missing-prototypes]
> 
> Mark the function static instead, as there are no other
> callers outside this file.
> 
> Fixes: 0ba4a784a145 ("drm/amd/display: implement force function in amdgpu_dm_connector_funcs")
> Reported-by: kernel test robot <lkp@...el.com>
> Link: https://lore.kernel.org/oe-kbuild-all/202304251640.JClqTim9-lkp@intel.com/
> Signed-off-by: Arnd Bergmann <arnd@...db.de>

Applied, thanks!

> ---
> This was previously reported by a bot for the drm-next tree but remains
> broken in linux-next-20230428. Sending it out as I needed this fix
> for my local builds.
> ---
>   drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 3647d21d688f..2bbb2988942d 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -6336,7 +6336,7 @@ amdgpu_dm_connector_late_register(struct drm_connector *connector)
>   	return 0;
>   }
>   
> -void amdgpu_dm_connector_funcs_force(struct drm_connector *connector)
> +static void amdgpu_dm_connector_funcs_force(struct drm_connector *connector)
>   {
>   	struct amdgpu_dm_connector *aconnector = to_amdgpu_dm_connector(connector);
>   	struct dc_link *dc_link = aconnector->dc_link;
-- 
Hamza