diff options
author | Baoyou Xie <baoyou.xie@linaro.org> | 2016-09-03 01:57:14 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-09-14 15:10:37 -0400 |
commit | 761c2e82054fda665bcdec95b9daf3d468c5fd5b (patch) | |
tree | c6a8bbc52ee979e223ef300ba815623b76157e95 /drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | |
parent | 9ca91fdd13ad877f4a303f6fb971aecbea4b67ae (diff) |
drm/amdgpu: mark symbols static where possible
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' [-Wmissing-prototypes]
drivers/gpu/drm/amd/amdgpu/iceland_smc.c:124:6: warning: no previous prototype for 'iceland_start_smc' [-Wmissing-prototypes]
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:3926:6: warning: no previous prototype for 'gfx_v8_0_rlc_stop' [-Wmissing-prototypes]
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c:94:6: warning: no previous prototype for 'amdgpu_job_free_cb' [-Wmissing-prototypes]
....
In fact, these functions are only used in the file in which they are
declared and don't need a declaration, but can be made static.
So this patch marks these functions with 'static'.
Reviewed-by: Christian König <christian.koenig@amd.com>
Acked-by: Huang Rui <ray.huang@amd.com>
Reviewed-by: Edward O'Callaghan <funfunctor@folklore1984.net>
Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c index 040993c998d4..f1c53a2b09c6 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | |||
@@ -616,7 +616,7 @@ static int amdgpu_cgs_irq_put(struct cgs_device *cgs_device, unsigned src_id, un | |||
616 | return amdgpu_irq_put(adev, adev->irq.sources[src_id], type); | 616 | return amdgpu_irq_put(adev, adev->irq.sources[src_id], type); |
617 | } | 617 | } |
618 | 618 | ||
619 | int amdgpu_cgs_set_clockgating_state(struct cgs_device *cgs_device, | 619 | static int amdgpu_cgs_set_clockgating_state(struct cgs_device *cgs_device, |
620 | enum amd_ip_block_type block_type, | 620 | enum amd_ip_block_type block_type, |
621 | enum amd_clockgating_state state) | 621 | enum amd_clockgating_state state) |
622 | { | 622 | { |
@@ -637,7 +637,7 @@ int amdgpu_cgs_set_clockgating_state(struct cgs_device *cgs_device, | |||
637 | return r; | 637 | return r; |
638 | } | 638 | } |
639 | 639 | ||
640 | int amdgpu_cgs_set_powergating_state(struct cgs_device *cgs_device, | 640 | static int amdgpu_cgs_set_powergating_state(struct cgs_device *cgs_device, |
641 | enum amd_ip_block_type block_type, | 641 | enum amd_ip_block_type block_type, |
642 | enum amd_powergating_state state) | 642 | enum amd_powergating_state state) |
643 | { | 643 | { |