Am 31.01.2018 um 16:31 schrieb Oded Gabbay:
On Wed, Jan 31, 2018 at 5:28 PM, Christian König
<[email protected]> wrote:
Am 31.01.2018 um 16:25 schrieb Oded Gabbay:
On Fri, Jan 5, 2018 at 12:17 AM, Felix Kuehling <[email protected]>
wrote:
Signed-off-by: Felix Kuehling <[email protected]>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
index 335e454..7ebe430 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
@@ -78,10 +78,15 @@ void amdgpu_amdkfd_device_probe(struct amdgpu_device
*adev)
switch (adev->asic_type) {
#ifdef CONFIG_DRM_AMDGPU_CIK
case CHIP_KAVERI:
+ case CHIP_HAWAII:
kfd2kgd = amdgpu_amdkfd_gfx_7_get_functions();
break;
#endif
case CHIP_CARRIZO:
+ case CHIP_TONGA:
+ case CHIP_FIJI:
+ case CHIP_POLARIS10:
+ case CHIP_POLARIS11:
Polaris isn't gfx 9 ?
or is it called differently ?
No Polaris are just updated gfx8 variants.
gfx9 is Vega10.
Christian.
OK, thanks. So this patch is fine and is
Reviewed-by: Oded Gabbay <[email protected]>
Having said that, as I wrote to Alex, if that is the case, I think we
should rename all soc-dependent functions from using cik and vi as
identifiers to gfx7/gfx8.
Well that depends on what the name refers to.
gfx7/gfx8 just describe the version of the CP, not the full ASIC.
For example Tonga and Polaris are both gfx8, but have a different SDMA IIRC.
Christian.
Oded
kfd2kgd = amdgpu_amdkfd_gfx_8_0_get_functions();
break;
default:
--
2.7.4
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
