[AMD Public Use] Reviewed-by: Hawking Zhang <[email protected]>
Regards, Hawking From: Lazar, Lijo <[email protected]> Sent: Monday, March 8, 2021 21:16 To: Lazar, Lijo <[email protected]>; [email protected] Cc: Deucher, Alexander <[email protected]>; Xu, Feifei <[email protected]>; Zhang, Hawking <[email protected]> Subject: RE: [PATCH] drm/amdgpu: Check if FB BAR is enabled for ROM read [AMD Public Use] <Ping> From: amd-gfx <[email protected]<mailto:[email protected]>> On Behalf Of Lazar, Lijo Sent: Wednesday, March 3, 2021 10:15 AM To: [email protected]<mailto:[email protected]> Cc: Deucher, Alexander <[email protected]<mailto:[email protected]>>; Xu, Feifei <[email protected]<mailto:[email protected]>>; Zhang, Hawking <[email protected]<mailto:[email protected]>> Subject: [PATCH] drm/amdgpu: Check if FB BAR is enabled for ROM read [AMD Public Use] Some configurations don't have FB BAR enabled. Avoid reading ROM image from FB BAR region in such cases. Signed-off-by: Lijo Lazar <[email protected]<mailto:[email protected]>> Reviewed-by: Hawking Zhang <[email protected]<mailto:[email protected]>> Reviewed-by: Feifei Xu <[email protected]<mailto:[email protected]>> --- drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c index efdf639f6593..f454a6bd0ed6 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c @@ -97,6 +97,10 @@ static bool igp_read_bios_from_vram(struct amdgpu_device *adev) if (amdgpu_device_need_post(adev)) return false; + /* FB BAR not enabled */ + if (pci_resource_len(adev->pdev, 0) == 0) + return false; + adev->bios = NULL; vram_base = pci_resource_start(adev->pdev, 0); bios = ioremap_wc(vram_base, size); -- 2.29.2
_______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
