[PATCH] drm/amdgpu: mark symbols static where possible

2016-09-30 Thread Baoyou Xie
We get 2 warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/si.c:908:5: warning: no previous prototype for 'si_pciep_rreg' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/si.c:921:6: warning: no previous prototype for 'si_pciep_wreg' [-Wmissing-prototypes] In fact, these

[PATCH] drm/amdgpu: mark symbols static where possible

2016-09-30 Thread Baoyou Xie
We get 2 warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/si.c:908:5: warning: no previous prototype for 'si_pciep_rreg' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/si.c:921:6: warning: no previous prototype for 'si_pciep_wreg' [-Wmissing-prototypes] In fact, these

Re: [PATCH] drm/amdgpu: mark symbols static where possible

2016-09-19 Thread Alex Deucher
On Mon, Sep 19, 2016 at 6:01 AM, Christian König wrote: > Am 18.09.2016 um 16:09 schrieb Baoyou Xie: >> >> We get 7 warnings when building kernel with W=1: >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous >> prototype for 'amdgpu_pre_soft_reset'

Re: [PATCH] drm/amdgpu: mark symbols static where possible

2016-09-19 Thread Alex Deucher
On Mon, Sep 19, 2016 at 6:01 AM, Christian König wrote: > Am 18.09.2016 um 16:09 schrieb Baoyou Xie: >> >> We get 7 warnings when building kernel with W=1: >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous >> prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes] >>

Re: [PATCH] drm/amdgpu: mark symbols static where possible

2016-09-19 Thread Christian König
Am 18.09.2016 um 16:09 schrieb Baoyou Xie: We get 7 warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c:1548:5: warning: no

Re: [PATCH] drm/amdgpu: mark symbols static where possible

2016-09-19 Thread Christian König
Am 18.09.2016 um 16:09 schrieb Baoyou Xie: We get 7 warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c:1548:5: warning: no

[PATCH] drm/amdgpu: mark symbols static where possible

2016-09-18 Thread Baoyou Xie
We get 7 warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c:1548:5: warning: no previous prototype for

[PATCH] drm/amdgpu: mark symbols static where possible

2016-09-18 Thread Baoyou Xie
We get 7 warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c:1548:5: warning: no previous prototype for

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Christian König
Am 03.09.2016 um 07:57 schrieb Baoyou Xie: We get a few warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/cz_smc.c:51:5: warning: no previous prototype for 'cz_send_msg_to_smc_async' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/cz_smc.c:143:5: warning: no previous

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Christian König
Am 03.09.2016 um 07:57 schrieb Baoyou Xie: We get a few warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/cz_smc.c:51:5: warning: no previous prototype for 'cz_send_msg_to_smc_async' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/cz_smc.c:143:5: warning: no previous

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Huang Rui
On Sat, Sep 03, 2016 at 01:57:14PM +0800, Baoyou Xie wrote: > We get a few warnings when building kernel with W=1: > drivers/gpu/drm/amd/amdgpu/cz_smc.c:51:5: warning: no previous prototype for > 'cz_send_msg_to_smc_async' [-Wmissing-prototypes] > drivers/gpu/drm/amd/amdgpu/cz_smc.c:143:5:

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Huang Rui
On Sat, Sep 03, 2016 at 01:57:14PM +0800, Baoyou Xie wrote: > We get a few warnings when building kernel with W=1: > drivers/gpu/drm/amd/amdgpu/cz_smc.c:51:5: warning: no previous prototype for > 'cz_send_msg_to_smc_async' [-Wmissing-prototypes] > drivers/gpu/drm/amd/amdgpu/cz_smc.c:143:5:

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Edward O'Callaghan
On 09/03/2016 04:23 PM, Baoyou Xie wrote: > in drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c and > drivers/gpu/drm/amd/amdgpu/cz_smc.c, there are two functions named > cz_write_smc_sram_dword, but these two functions only have the same > name, but in fact they are different functions, even

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Edward O'Callaghan
On 09/03/2016 04:23 PM, Baoyou Xie wrote: > in drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c and > drivers/gpu/drm/amd/amdgpu/cz_smc.c, there are two functions named > cz_write_smc_sram_dword, but these two functions only have the same > name, but in fact they are different functions, even

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Edward O'Callaghan
Reviewed-by: Edward O'Callaghan Also, I just noticed cz_write_smc_sram_dword() and if I am not mistaken they seems identical in both: drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c and, drivers/gpu/drm/amd/amdgpu/cz_smc.c with a personal preference to the

Re: [PATCH] drm: amdgpu: mark symbols static where possible

2016-09-03 Thread Edward O'Callaghan
Reviewed-by: Edward O'Callaghan Also, I just noticed cz_write_smc_sram_dword() and if I am not mistaken they seems identical in both: drivers/gpu/drm/amd/powerplay/smumgr/cz_smumgr.c and, drivers/gpu/drm/amd/amdgpu/cz_smc.c with a personal preference to the powerplay version I suppose.

[PATCH] drm: amdgpu: mark symbols static where possible

2016-09-02 Thread Baoyou Xie
We get a few warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/cz_smc.c:51:5: warning: no previous prototype for 'cz_send_msg_to_smc_async' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/cz_smc.c:143:5: warning: no previous prototype for 'cz_write_smc_sram_dword'

[PATCH] drm: amdgpu: mark symbols static where possible

2016-09-02 Thread Baoyou Xie
We get a few warnings when building kernel with W=1: drivers/gpu/drm/amd/amdgpu/cz_smc.c:51:5: warning: no previous prototype for 'cz_send_msg_to_smc_async' [-Wmissing-prototypes] drivers/gpu/drm/amd/amdgpu/cz_smc.c:143:5: warning: no previous prototype for 'cz_write_smc_sram_dword'