Commit 5b061953 authored by Neil Armstrong's avatar Neil Armstrong Committed by Rob Clark
Browse files

drm/msm: adreno: add defines for gpu & gmu frequency table sizes 



Even if the code uses ARRAY_SIZE() to fill those tables,
it's still a best practice to not use magic values for
tables in structs.

Suggested-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: default avatarAkhil P Oommen <quic_akhilpo@quicinc.com>
Signed-off-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/629393/


Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent 280807dd

drivers/gpu/drm/msm/adreno/a6xx_gmu.h

+7 −4
Original line number Diff line number Diff line
@@ -19,6 +19,9 @@ struct a6xx_gmu_bo { 
	u64 iova;

};



#define GMU_MAX_GX_FREQS	16

#define GMU_MAX_CX_FREQS	4



/*

 * These define the different GMU wake up options - these define how both the

 * CPU and the GMU bring up the hardware
@@ -79,12 +82,12 @@ struct a6xx_gmu { 
	int current_perf_index;



	int nr_gpu_freqs;

	unsigned long gpu_freqs[16];

	u32 gx_arc_votes[16];

	unsigned long gpu_freqs[GMU_MAX_GX_FREQS];

	u32 gx_arc_votes[GMU_MAX_GX_FREQS];



	int nr_gmu_freqs;

	unsigned long gmu_freqs[4];

	u32 cx_arc_votes[4];

	unsigned long gmu_freqs[GMU_MAX_CX_FREQS];

	u32 cx_arc_votes[GMU_MAX_CX_FREQS];



	unsigned long freq;