<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr"> It fixes some headless board VBIOS which will be failed to load. </div> <hr style="display:inline-block;width:98%" tabindex="-1"> <div id="divRplyFwdMsg" dir="ltr">From: Alex Deucher <alexdeucher@gmail.com> Sent: Wednesday, November 8, 2017 10:38:37 PM To: Wang, Ken Cc: amd-gfx list Subject: Re: [PATCH] drm/amdgpu: Remove check which is not valid for certain VBIOS <div> </div> </div> <div class="BodyFragment"> <div class="PlainText">On Wed, Nov 8, 2017 at 1:49 AM, <Ken.Wang@amd.com> wrote: > From: Ken Wang <Ken.Wang@amd.com> What cases does this fix? I'm guessing VFCT or some other platform method for getting the vbios? Acked-by: Alex Deucher <alexander.deucher@amd.com> > > Signed-off-by: Ken Wang <Ken.Wang@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c > index c21adf6..057e1ec 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_bios.c > @@ -59,12 +59,6 @@ static bool check_atom_bios(uint8_t *bios, size_t size) > return false; > } > > - tmp = bios[0x18] | (bios[0x19] << 8); > - if (bios[tmp + 0x14] != 0x0) { > - DRM_INFO("Not an x86 BIOS ROM\n"); > - return false; > - } > - > bios_header_start = bios[0x48] | (bios[0x49] << 8); > if (!bios_header_start) { > DRM_INFO("Can't locate bios header\n"); > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </div> </div> </body> </html>