On Mon, Feb 26, 2018 at 12:18 AM, Monk Liu <[email protected]> wrote: > From: Emily Deng <[email protected]> > > Need to call function amdgpu_ucode_fini_bo to release ucode bo for > psp firmware load type.
Are you sure this is right? I think is this is handled in amdgpu_psp.c already. Alex > > Change-Id: I1c7be8135993e11076c9d46b3cd87615514a9abb > Signed-off-by: Emily Deng <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index 69fb5e50..61696c7 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -1490,6 +1490,7 @@ static int amdgpu_device_ip_fini(struct amdgpu_device > *adev) > > /* disable all interrupts */ > amdgpu_irq_disable_all(adev); > + amdgpu_ucode_fini_bo(adev); > > for (i = adev->num_ip_blocks - 1; i >= 0; i--) { > if (!adev->ip_blocks[i].status.sw) > -- > 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
