Am 02.12.18 um 21:52 schrieb Thomas Meyer: > NULL check before some freeing functions is not needed. > > Signed-off-by: Thomas Meyer <thomas@xxxxxxxx> Reviewed-by: Christian König <christian.koenig@xxxxxxx> > --- > > diff -u -p a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c > @@ -816,6 +816,5 @@ out: > void amdgpu_acpi_fini(struct amdgpu_device *adev) > { > unregister_acpi_notifier(&adev->acpi_nb); > - if (adev->atif) > - kfree(adev->atif); > + kfree(adev->atif); > } _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel