[PATCH] drm/amd/powerplay: fix compare_const_fl.cocci warnings

Julia Lawall julia.lawall at lip6.fr
Tue Jan 5 06:46:46 PST 2016


X != NULL, or even !X is more normmal in the kernel.

Generated by: scripts/coccinelle/misc/compare_const_fl.cocci

CC: Jammy Zhou <Jammy.Zhou at amd.com>
Signed-off-by: Fengguang Wu <fengguang.wu at intel.com>
Signed-off-by: Julia Lawall <julia.lawall at lip6.fr>
---

 smumgr.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

--- a/drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/smumgr.c
+++ b/drivers/gpu/drm/amd/amdgpu/../powerplay/smumgr/smumgr.c
@@ -70,7 +70,7 @@ int smum_fini(struct pp_smumgr *smumgr)

 int smum_get_argument(struct pp_smumgr *smumgr)
 {
-	if (NULL != smumgr->smumgr_funcs->get_argument)
+	if (smumgr->smumgr_funcs->get_argument != NULL)
 		return smumgr->smumgr_funcs->get_argument(smumgr);

 	return 0;
@@ -79,7 +79,7 @@ int smum_get_argument(struct pp_smumgr *
 int smum_download_powerplay_table(struct pp_smumgr *smumgr,
 								void **table)
 {
-	if (NULL != smumgr->smumgr_funcs->download_pptable_settings)
+	if (smumgr->smumgr_funcs->download_pptable_settings != NULL)
 		return smumgr->smumgr_funcs->download_pptable_settings(smumgr,
 									table);

@@ -88,7 +88,7 @@ int smum_download_powerplay_table(struct

 int smum_upload_powerplay_table(struct pp_smumgr *smumgr)
 {
-	if (NULL != smumgr->smumgr_funcs->upload_pptable_settings)
+	if (smumgr->smumgr_funcs->upload_pptable_settings != NULL)
 		return smumgr->smumgr_funcs->upload_pptable_settings(smumgr);

 	return 0;


More information about the dri-devel mailing list