Am 31.01.2018 um 16:31 schrieb Oded Gabbay:
> On Wed, Jan 31, 2018 at 5:28 PM, Christian König
> <ckoenig.leichtzumerken at gmail.com> wrote:
>> Am 31.01.2018 um 16:25 schrieb Oded Gabbay:
>>> On Fri, Jan 5, 2018 at 12:17 AM, Felix Kuehling <Felix.Kuehling at amd.com>
>>> wrote:
>>>> Signed-off-by: Felix Kuehling <Felix.Kuehling at amd.com>
>>>> ---
>>>> 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 <oded.gabbay at gmail.com>
>
> 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
>>> amd-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>>
