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. 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
