瀏覽代碼

s390: Delete unnecessary checks before the function call "debug_unregister"

The debug_unregister() function performs also input parameter validation.
Thus the test around the calls is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Markus Elfring 9 年之前
父節點
當前提交
a6e975c5f8
共有 4 個文件被更改,包括 8 次插入16 次删除
  1. 1 2
      drivers/s390/cio/chsc_sch.c
  2. 3 6
      drivers/s390/cio/cio.c
  3. 2 4
      drivers/s390/cio/qdio_debug.c
  4. 2 4
      drivers/s390/crypto/zcrypt_api.c

+ 1 - 2
drivers/s390/cio/chsc_sch.c

@@ -185,8 +185,7 @@ static int __init chsc_init_dbfs(void)
 	debug_set_level(chsc_debug_log_id, 2);
 	debug_set_level(chsc_debug_log_id, 2);
 	return 0;
 	return 0;
 out:
 out:
-	if (chsc_debug_msg_id)
-		debug_unregister(chsc_debug_msg_id);
+	debug_unregister(chsc_debug_msg_id);
 	return -ENOMEM;
 	return -ENOMEM;
 }
 }
 
 

+ 3 - 6
drivers/s390/cio/cio.c

@@ -76,12 +76,9 @@ static int __init cio_debug_init(void)
 	return 0;
 	return 0;
 
 
 out_unregister:
 out_unregister:
-	if (cio_debug_msg_id)
-		debug_unregister(cio_debug_msg_id);
-	if (cio_debug_trace_id)
-		debug_unregister(cio_debug_trace_id);
-	if (cio_debug_crw_id)
-		debug_unregister(cio_debug_crw_id);
+	debug_unregister(cio_debug_msg_id);
+	debug_unregister(cio_debug_trace_id);
+	debug_unregister(cio_debug_crw_id);
 	return -1;
 	return -1;
 }
 }
 
 

+ 2 - 4
drivers/s390/cio/qdio_debug.c

@@ -366,8 +366,6 @@ void qdio_debug_exit(void)
 {
 {
 	qdio_clear_dbf_list();
 	qdio_clear_dbf_list();
 	debugfs_remove(debugfs_root);
 	debugfs_remove(debugfs_root);
-	if (qdio_dbf_setup)
-		debug_unregister(qdio_dbf_setup);
-	if (qdio_dbf_error)
-		debug_unregister(qdio_dbf_error);
+	debug_unregister(qdio_dbf_setup);
+	debug_unregister(qdio_dbf_error);
 }
 }

+ 2 - 4
drivers/s390/crypto/zcrypt_api.c

@@ -1428,10 +1428,8 @@ int __init zcrypt_debug_init(void)
 void zcrypt_debug_exit(void)
 void zcrypt_debug_exit(void)
 {
 {
 	debugfs_remove(debugfs_root);
 	debugfs_remove(debugfs_root);
-	if (zcrypt_dbf_common)
-		debug_unregister(zcrypt_dbf_common);
-	if (zcrypt_dbf_devices)
-		debug_unregister(zcrypt_dbf_devices);
+	debug_unregister(zcrypt_dbf_common);
+	debug_unregister(zcrypt_dbf_devices);
 }
 }
 
 
 /**
 /**