RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic

2020-06-05 Thread Gao, Likun
- From: Zhang, Hawking Sent: Friday, June 5, 2020 10:14 PM To: Gao, Likun ; amd-gfx@lists.freedesktop.org Cc: Xu, Feifei Subject: RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic [AMD Public Use] It looks more reasonable to me to move the AISC type check to the following

RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic

2020-06-05 Thread Zhang, Hawking
Hawking -Original Message- From: Zhang, Hawking Sent: Friday, June 5, 2020 22:10 To: Zhang, Hawking ; Gao, Likun ; amd-gfx@lists.freedesktop.org Cc: Xu, Feifei Subject: RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic [AMD Public Use] Wait, I see the risk. the amdgpu_ttm

RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic

2020-06-05 Thread Zhang, Hawking
Likun ; amd-gfx@lists.freedesktop.org Cc: Xu, Feifei Subject: RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic [AMD Public Use] Does it mean there is legacy VBIOS that even doesn't have firmwareinfo table and introduce crash? I think we expect the atomfirmware interfac

RE: [PATCH] drm/amdgpu: skip fb size query for navi10 previous asic

2020-06-05 Thread Zhang, Hawking
[AMD Public Use] Does it mean there is legacy VBIOS that even doesn't have firmwareinfo table and introduce crash? I think we expect the atomfirmware interface return 0 for ASICs prior to Sienna Cichlid Regards, Hawking -Original Message- From: Gao, Likun Sent: Friday, June 5, 2020