[Freedreno] [PATCH 2/2] drm/msm/a4xx: add a405_registers for a405 device

Jordan Crouse jcrouse at codeaurora.org
Mon May 18 14:39:09 UTC 2020


On Sat, May 09, 2020 at 08:38:46PM +0800, Shawn Guo wrote:
> A405 device has a different set of registers than a4xx_registers.  It
> has no VMIDMT or XPU registers, and VBIF registers are different.  Let's
> add a405_registers for a405 device.
> 
> As adreno_is_a405() works only after adreno_gpu_init() gets called, the
> assignments get moved down after adreno_gpu_init().

Reviewed-by: Jordan Crouse <jcrouse at codeauorora.org>

> Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> ---
>  drivers/gpu/drm/msm/adreno/a4xx_gpu.c | 53 +++++++++++++++++++++++++--
>  1 file changed, 50 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/adreno/a4xx_gpu.c b/drivers/gpu/drm/msm/adreno/a4xx_gpu.c
> index 70de59751188..9e244982974e 100644
> --- a/drivers/gpu/drm/msm/adreno/a4xx_gpu.c
> +++ b/drivers/gpu/drm/msm/adreno/a4xx_gpu.c
> @@ -445,6 +445,52 @@ static const unsigned int a4xx_registers[] = {
>  	~0 /* sentinel */
>  };
>  
> +static const unsigned int a405_registers[] = {
> +	/* RBBM */
> +	0x0000, 0x0002, 0x0004, 0x0021, 0x0023, 0x0024, 0x0026, 0x0026,
> +	0x0028, 0x002B, 0x002E, 0x0034, 0x0037, 0x0044, 0x0047, 0x0066,
> +	0x0068, 0x0095, 0x009C, 0x0170, 0x0174, 0x01AF,
> +	/* CP */
> +	0x0200, 0x0233, 0x0240, 0x0250, 0x04C0, 0x04DD, 0x0500, 0x050B,
> +	0x0578, 0x058F,
> +	/* VSC */
> +	0x0C00, 0x0C03, 0x0C08, 0x0C41, 0x0C50, 0x0C51,
> +	/* GRAS */
> +	0x0C80, 0x0C81, 0x0C88, 0x0C8F,
> +	/* RB */
> +	0x0CC0, 0x0CC0, 0x0CC4, 0x0CD2,
> +	/* PC */
> +	0x0D00, 0x0D0C, 0x0D10, 0x0D17, 0x0D20, 0x0D23,
> +	/* VFD */
> +	0x0E40, 0x0E4A,
> +	/* VPC */
> +	0x0E60, 0x0E61, 0x0E63, 0x0E68,
> +	/* UCHE */
> +	0x0E80, 0x0E84, 0x0E88, 0x0E95,
> +	/* GRAS CTX 0 */
> +	0x2000, 0x2004, 0x2008, 0x2067, 0x2070, 0x2078, 0x207B, 0x216E,
> +	/* PC CTX 0 */
> +	0x21C0, 0x21C6, 0x21D0, 0x21D0, 0x21D9, 0x21D9, 0x21E5, 0x21E7,
> +	/* VFD CTX 0 */
> +	0x2200, 0x2204, 0x2208, 0x22A9,
> +	/* GRAS CTX 1 */
> +	0x2400, 0x2404, 0x2408, 0x2467, 0x2470, 0x2478, 0x247B, 0x256E,
> +	/* PC CTX 1 */
> +	0x25C0, 0x25C6, 0x25D0, 0x25D0, 0x25D9, 0x25D9, 0x25E5, 0x25E7,
> +	/* VFD CTX 1 */
> +	0x2600, 0x2604, 0x2608, 0x26A9,
> +	/* VBIF version 0x20050000*/
> +	0x3000, 0x3007, 0x302C, 0x302C, 0x3030, 0x3030, 0x3034, 0x3036,
> +	0x3038, 0x3038, 0x303C, 0x303D, 0x3040, 0x3040, 0x3049, 0x3049,
> +	0x3058, 0x3058, 0x305B, 0x3061, 0x3064, 0x3068, 0x306C, 0x306D,
> +	0x3080, 0x3088, 0x308B, 0x308C, 0x3090, 0x3094, 0x3098, 0x3098,
> +	0x309C, 0x309C, 0x30C0, 0x30C0, 0x30C8, 0x30C8, 0x30D0, 0x30D0,
> +	0x30D8, 0x30D8, 0x30E0, 0x30E0, 0x3100, 0x3100, 0x3108, 0x3108,
> +	0x3110, 0x3110, 0x3118, 0x3118, 0x3120, 0x3120, 0x3124, 0x3125,
> +	0x3129, 0x3129, 0x340C, 0x340C, 0x3410, 0x3410,
> +	~0 /* sentinel */
> +};
> +
>  static struct msm_gpu_state *a4xx_gpu_state_get(struct msm_gpu *gpu)
>  {
>  	struct msm_gpu_state *state = kzalloc(sizeof(*state), GFP_KERNEL);
> @@ -568,13 +614,14 @@ struct msm_gpu *a4xx_gpu_init(struct drm_device *dev)
>  	gpu->perfcntrs = NULL;
>  	gpu->num_perfcntrs = 0;
>  
> -	adreno_gpu->registers = a4xx_registers;
> -	adreno_gpu->reg_offsets = a4xx_register_offsets;
> -
>  	ret = adreno_gpu_init(dev, pdev, adreno_gpu, &funcs, 1);
>  	if (ret)
>  		goto fail;
>  
> +	adreno_gpu->registers = adreno_is_a405(adreno_gpu) ? a405_registers :
> +							     a4xx_registers;
> +	adreno_gpu->reg_offsets = a4xx_register_offsets;
> +
>  	/* if needed, allocate gmem: */
>  	if (adreno_is_a4xx(adreno_gpu)) {
>  		ret = adreno_gpu_ocmem_init(dev->dev, adreno_gpu,
> -- 
> 2.17.1
> 

-- 
The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project


More information about the Freedreno mailing list