diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-07-04 15:37:09 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2014-08-05 08:53:24 -0400 |
commit | 9c2448780faf013af4333b9476dcaefc94f98c93 (patch) | |
tree | 88c8786c48651655f20e9d97c539cc4b0a0c9d2f | |
parent | f2c6b0f452c3804496f55655fda28c2809e1a58b (diff) |
drm/radeon: remove null test before kfree
Fix checkpatch warning:
WARNING: kfree(NULL) is safe this check is probably not required
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_pm.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_pm.c b/drivers/gpu/drm/radeon/radeon_pm.c index e447e390d09a..a06afee460be 100644 --- a/drivers/gpu/drm/radeon/radeon_pm.c +++ b/drivers/gpu/drm/radeon/radeon_pm.c | |||
@@ -1400,9 +1400,7 @@ static void radeon_pm_fini_old(struct radeon_device *rdev) | |||
1400 | } | 1400 | } |
1401 | 1401 | ||
1402 | radeon_hwmon_fini(rdev); | 1402 | radeon_hwmon_fini(rdev); |
1403 | 1403 | kfree(rdev->pm.power_state); | |
1404 | if (rdev->pm.power_state) | ||
1405 | kfree(rdev->pm.power_state); | ||
1406 | } | 1404 | } |
1407 | 1405 | ||
1408 | static void radeon_pm_fini_dpm(struct radeon_device *rdev) | 1406 | static void radeon_pm_fini_dpm(struct radeon_device *rdev) |
@@ -1421,9 +1419,7 @@ static void radeon_pm_fini_dpm(struct radeon_device *rdev) | |||
1421 | radeon_dpm_fini(rdev); | 1419 | radeon_dpm_fini(rdev); |
1422 | 1420 | ||
1423 | radeon_hwmon_fini(rdev); | 1421 | radeon_hwmon_fini(rdev); |
1424 | 1422 | kfree(rdev->pm.power_state); | |
1425 | if (rdev->pm.power_state) | ||
1426 | kfree(rdev->pm.power_state); | ||
1427 | } | 1423 | } |
1428 | 1424 | ||
1429 | void radeon_pm_fini(struct radeon_device *rdev) | 1425 | void radeon_pm_fini(struct radeon_device *rdev) |