[PATCH] drm/amdgpu: Move IH clientid defs to separate file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



SOC15 refers to the SOC design we started with vega10.  It differs from previous SOC design from previous asics.  I think it was designed in ~2015, so SOC15.  The soc15 name is fine with me.


Alex

________________________________
From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of Zeng, Oak <Oak.Zeng at amd.com>
Sent: Friday, March 9, 2018 9:52:06 AM
To: Koenig, Christian; Oak Zeng; amd-gfx at lists.freedesktop.org
Subject: RE: [PATCH] drm/amdgpu: Move IH clientid defs to separate file

Thanks Alex, Christian and David for the feedback.

I will:
1. keep AMDGPU_IH_CLIENTID_LEGACY in amdgpu_ih.h
2. Change other client ID's prefix to SOC15
3. Regarding the file name, I think those client ID are introduced since Vega10 and also apply for navi10. Is SOC15 the proper file name? Actually can anybody explain what is SOC15? Or any suggestion for a better file name?

Thanks,
Oak


-----Original Message-----
From: Christian König [mailto:ckoenig.leichtzumerken@xxxxxxxxx]
Sent: Friday, March 09, 2018 2:49 AM
To: Oak Zeng; amd-gfx at lists.freedesktop.org
Cc: Zeng, Oak
Subject: Re: [PATCH] drm/amdgpu: Move IH clientid defs to separate file

Am 09.03.2018 um 00:24 schrieb Oak Zeng:
> This is preparation for sharing client ID definitions between amdgpu
> and amdkfd
>
> Change-Id: Ie0b7b14c9c2366fd896745a51b74a9ba97ee3224
> Signed-off-by: Oak Zeng <Oak.Zeng at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_ih.h          | 44 +--------------
>   drivers/gpu/drm/amd/include/soc15_ih_clientid.h | 72 +++++++++++++++++++++++++
>   2 files changed, 73 insertions(+), 43 deletions(-)
>   create mode 100644 drivers/gpu/drm/amd/include/soc15_ih_clientid.h
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ih.h
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ih.h
> index b8a7dba..62a9869 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ih.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ih.h
> @@ -25,51 +25,9 @@
>   #define __AMDGPU_IH_H__
>
>   #include <linux/chash.h>
> +#include "soc15_ih_clientid.h"
>
>   struct amdgpu_device;
> - /*
> -  * vega10+ IH clients
> - */
> -enum amdgpu_ih_clientid
> -{
> -    AMDGPU_IH_CLIENTID_IH        = 0x00,
> -    AMDGPU_IH_CLIENTID_ACP       = 0x01,
> -    AMDGPU_IH_CLIENTID_ATHUB     = 0x02,
> -    AMDGPU_IH_CLIENTID_BIF       = 0x03,
> -    AMDGPU_IH_CLIENTID_DCE       = 0x04,
> -    AMDGPU_IH_CLIENTID_ISP       = 0x05,
> -    AMDGPU_IH_CLIENTID_PCIE0     = 0x06,
> -    AMDGPU_IH_CLIENTID_RLC       = 0x07,
> -    AMDGPU_IH_CLIENTID_SDMA0     = 0x08,
> -    AMDGPU_IH_CLIENTID_SDMA1     = 0x09,
> -    AMDGPU_IH_CLIENTID_SE0SH     = 0x0a,
> -    AMDGPU_IH_CLIENTID_SE1SH     = 0x0b,
> -    AMDGPU_IH_CLIENTID_SE2SH     = 0x0c,
> -    AMDGPU_IH_CLIENTID_SE3SH     = 0x0d,
> -    AMDGPU_IH_CLIENTID_SYSHUB            = 0x0e,
> -    AMDGPU_IH_CLIENTID_THM       = 0x0f,
> -    AMDGPU_IH_CLIENTID_UVD       = 0x10,
> -    AMDGPU_IH_CLIENTID_VCE0      = 0x11,
> -    AMDGPU_IH_CLIENTID_VMC       = 0x12,
> -    AMDGPU_IH_CLIENTID_XDMA      = 0x13,
> -    AMDGPU_IH_CLIENTID_GRBM_CP           = 0x14,
> -    AMDGPU_IH_CLIENTID_ATS       = 0x15,
> -    AMDGPU_IH_CLIENTID_ROM_SMUIO    = 0x16,
> -    AMDGPU_IH_CLIENTID_DF        = 0x17,
> -    AMDGPU_IH_CLIENTID_VCE1      = 0x18,
> -    AMDGPU_IH_CLIENTID_PWR       = 0x19,
> -    AMDGPU_IH_CLIENTID_UTCL2     = 0x1b,
> -    AMDGPU_IH_CLIENTID_EA        = 0x1c,
> -    AMDGPU_IH_CLIENTID_UTCL2LOG          = 0x1d,
> -    AMDGPU_IH_CLIENTID_MP0       = 0x1e,
> -    AMDGPU_IH_CLIENTID_MP1       = 0x1f,
> -
> -    AMDGPU_IH_CLIENTID_MAX,
> -
> -    AMDGPU_IH_CLIENTID_VCN       = AMDGPU_IH_CLIENTID_UVD
> -};
> -
> -#define AMDGPU_IH_CLIENTID_LEGACY 0
>
>   #define AMDGPU_PAGEFAULT_HASH_BITS 8
>   struct amdgpu_retryfault_hashtable { diff --git
> a/drivers/gpu/drm/amd/include/soc15_ih_clientid.h
> b/drivers/gpu/drm/amd/include/soc15_ih_clientid.h
> new file mode 100644
> index 0000000..e2e8c63
> --- /dev/null
> +++ b/drivers/gpu/drm/amd/include/soc15_ih_clientid.h
> @@ -0,0 +1,72 @@
> +/*
> + * Copyright 2018 Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person
> +obtaining a
> + * copy of this software and associated documentation files (the
> +"Software"),
> + * to deal in the Software without restriction, including without
> +limitation
> + * the rights to use, copy, modify, merge, publish, distribute,
> +sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom
> +the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be
> +included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> +EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> +MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
> +SHALL
> + * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM,
> +DAMAGES OR
> + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
> +OTHERWISE,
> + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE
> +OR
> + * OTHER DEALINGS IN THE SOFTWARE.
> + *
> + */
> +
> +#ifndef __SOC15_IH_CLIENTID_H__
> +#define __SOC15_IH_CLIENTID_H__
> +
> + /*
> +  * vega10+ IH clients
> + */
> +enum amdgpu_ih_clientid {
> +     AMDGPU_IH_CLIENTID_IH           = 0x00,
> +     AMDGPU_IH_CLIENTID_ACP          = 0x01,
> +     AMDGPU_IH_CLIENTID_ATHUB        = 0x02,
> +     AMDGPU_IH_CLIENTID_BIF          = 0x03,
> +     AMDGPU_IH_CLIENTID_DCE          = 0x04,
> +     AMDGPU_IH_CLIENTID_ISP          = 0x05,
> +     AMDGPU_IH_CLIENTID_PCIE0        = 0x06,
> +     AMDGPU_IH_CLIENTID_RLC          = 0x07,
> +     AMDGPU_IH_CLIENTID_SDMA0        = 0x08,
> +     AMDGPU_IH_CLIENTID_SDMA1        = 0x09,
> +     AMDGPU_IH_CLIENTID_SE0SH        = 0x0a,
> +     AMDGPU_IH_CLIENTID_SE1SH        = 0x0b,
> +     AMDGPU_IH_CLIENTID_SE2SH        = 0x0c,
> +     AMDGPU_IH_CLIENTID_SE3SH        = 0x0d,
> +     AMDGPU_IH_CLIENTID_SYSHUB       = 0x0e,
> +     AMDGPU_IH_CLIENTID_THM          = 0x0f,
> +     AMDGPU_IH_CLIENTID_UVD          = 0x10,
> +     AMDGPU_IH_CLIENTID_VCE0         = 0x11,
> +     AMDGPU_IH_CLIENTID_VMC          = 0x12,
> +     AMDGPU_IH_CLIENTID_XDMA         = 0x13,
> +     AMDGPU_IH_CLIENTID_GRBM_CP      = 0x14,
> +     AMDGPU_IH_CLIENTID_ATS          = 0x15,
> +     AMDGPU_IH_CLIENTID_ROM_SMUIO    = 0x16,
> +     AMDGPU_IH_CLIENTID_DF           = 0x17,
> +     AMDGPU_IH_CLIENTID_VCE1         = 0x18,
> +     AMDGPU_IH_CLIENTID_PWR          = 0x19,
> +     AMDGPU_IH_CLIENTID_UTCL2        = 0x1b,
> +     AMDGPU_IH_CLIENTID_EA           = 0x1c,
> +     AMDGPU_IH_CLIENTID_UTCL2LOG     = 0x1d,
> +     AMDGPU_IH_CLIENTID_MP0          = 0x1e,
> +     AMDGPU_IH_CLIENTID_MP1          = 0x1f,
> +
> +     AMDGPU_IH_CLIENTID_MAX,
> +
> +     AMDGPU_IH_CLIENTID_VCN          = AMDGPU_IH_CLIENTID_UVD
> +};
> +
> +#define AMDGPU_IH_CLIENTID_LEGACY 0

Please keep AMDGPU_IH_CLIENTID_LEGACY in amdgpu_ih.h, since that isn't related to SOC15 in any way.

Additional to that the other client IDs could use a new prefix if you move them around, but that isn't mandatory.

Christian.

> +
> +#endif
> +
> +

_______________________________________________
amd-gfx mailing list
amd-gfx at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180309/8226892a/attachment-0001.html>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux