Commit fcb5c172 authored Oct 09, 2019 by Ben Dooks Committed by Sean Paul Oct 10, 2019

drm/msm: make a5xx_show and a5xx_gpu_state_put static

The a5xx_show and a5xx_gpu_state_put objects are not exported
outside of the file, so make them static to avoid the following
warnings from sparse:

drivers/gpu/drm/msm/adreno/a5xx_gpu.c:1292:5: warning: symbol 'a5xx_gpu_state_put' was not declared. Should it be static?
drivers/gpu/drm/msm/adreno/a5xx_gpu.c:1302:6: warning: symbol 'a5xx_show' was not declared. Should it be static?

Reviewed-by: Jordan Crouse <jcrouse@codeaurora.org>
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20191009114607.701-1-ben.dooks@codethink.co.uk

parent df4954e3

drivers/gpu/drm/msm/adreno/a5xx_gpu.c

+3 −3

Original line number	Diff line number	Diff line
		@@ -1289,7 +1289,7 @@ static void a5xx_gpu_state_destroy(struct kref *kref)
		kfree(a5xx_state);
		}

		int a5xx_gpu_state_put(struct msm_gpu_state *state)
		static int a5xx_gpu_state_put(struct msm_gpu_state *state)
		{
		if (IS_ERR_OR_NULL(state))
		return 1;
		@@ -1299,7 +1299,7 @@ int a5xx_gpu_state_put(struct msm_gpu_state *state)


		#if defined(CONFIG_DEBUG_FS) \|\| defined(CONFIG_DEV_COREDUMP)
		void a5xx_show(struct msm_gpu gpu, struct msm_gpu_state state,
		static void a5xx_show(struct msm_gpu gpu, struct msm_gpu_state state,
		struct drm_printer *p)
		{
		int i, j;