[AMD Official Use Only - AMD Internal Distribution Only] Hit send too quickly. please drop the vendor since it is not needed.
With that fixed. The patch is Reviewed-by: Hawking Zhang <hawking.zh...@amd.com> Regards, Hawking -----Original Message----- From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Zhang, Hawking Sent: Wednesday, April 30, 2025 17:17 To: Zhou1, Tao <tao.zh...@amd.com>; amd-gfx@lists.freedesktop.org Cc: Zhou1, Tao <tao.zh...@amd.com> Subject: RE: [PATCH 7/7] drm/amdgpu: set vram type for GC 9.5.0 [AMD Official Use Only - AMD Internal Distribution Only] [AMD Official Use Only - AMD Internal Distribution Only] "vendor" is not needed at this stage. Regards, Hawking -----Original Message----- From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Tao Zhou Sent: Wednesday, April 30, 2025 16:26 To: amd-gfx@lists.freedesktop.org Cc: Zhou1, Tao <tao.zh...@amd.com> Subject: [PATCH 7/7] drm/amdgpu: set vram type for GC 9.5.0 Set vram type and vendor so we can take different actions per the type. Signed-off-by: Tao Zhou <tao.zh...@amd.com> --- drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c index 464015fc2012..ac21197bf0de 100644 --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c @@ -64,6 +64,7 @@ #include "amdgpu_ras.h" #include "amdgpu_xgmi.h" +#include "atom.h" /* add these here since we already include dce12 headers and these are for DCN */ #define mmHUBP0_DCSURF_PRI_VIEWPORT_DIMENSION 0x055d @@ -2075,6 +2076,11 @@ static void gmc_v9_4_3_init_vram_info(struct amdgpu_device *adev) { adev->gmc.vram_type = AMDGPU_VRAM_TYPE_HBM; adev->gmc.vram_width = 128 * 64; + + if (amdgpu_ip_version(adev, GC_HWIP, 0) == IP_VERSION(9, 5, 0)) { + adev->gmc.vram_type = AMDGPU_VRAM_TYPE_HBM3E; + adev->gmc.vram_vendor = SAMSUNG; + } } static int gmc_v9_0_sw_init(struct amdgpu_ip_block *ip_block) -- 2.34.1