diff options
author | Wen Yang <wen.yang99@zte.com.cn> | 2018-12-05 02:43:19 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2018-12-05 17:50:26 -0500 |
commit | 8288b2e5ae01cabd078836618fd651574343cbfc (patch) | |
tree | 9143975520c7dca8d3ebd57c9b299b0786b1c097 /drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | |
parent | d4295e12796e747f9a624a56cd54de51fb2b3bdd (diff) |
drm/amdgpu/acpi: NULL check before some freeing functions is not needed
kfree(NULL) is safe, so removes NULL check before freeing the mem.
This patch also fix the ifnullfree.cocci warnings.
Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Wen Yang <wen.yang99@zte.com.cn>
CC: Alex Deucher <alexander.deucher@amd.com>
CC: christian.koenig@amd.com
CC: "David (ChunMing) Zhou" <David1.Zhou@amd.com>
CC: David Airlie <airlied@linux.ie> (maintainer:DRM DRIVERS)
CC: Lyude Paul <lyude@redhat.com>
CC: Rex Zhu <Rex.Zhu@amd.com>
CC: Jim Qu <Jim.Qu@amd.com>
CC: amd-gfx@lists.freedesktop.org
CC: dri-devel@lists.freedesktop.org
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c index 47db65926d71..4376b17ca594 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | |||
@@ -886,6 +886,5 @@ void amdgpu_acpi_get_backlight_caps(struct amdgpu_device *adev, | |||
886 | void amdgpu_acpi_fini(struct amdgpu_device *adev) | 886 | void amdgpu_acpi_fini(struct amdgpu_device *adev) |
887 | { | 887 | { |
888 | unregister_acpi_notifier(&adev->acpi_nb); | 888 | unregister_acpi_notifier(&adev->acpi_nb); |
889 | if (adev->atif) | 889 | kfree(adev->atif); |
890 | kfree(adev->atif); | ||
891 | } | 890 | } |