Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1548836

[PATCH 6/6] staging: vchiq_arm: Delete an unnecessary return statement in two functions

From SF Markus Elfring <elfring@users.sourceforge.net>
Newsgroups linux.kernel
Subject [PATCH 6/6] staging: vchiq_arm: Delete an unnecessary return statement in two functions
Date 2016-12-31 23:10 +0100
Message-ID <sUA0F-eE-3@gated-at.bofh.it> (permalink)
References <sUzQZ-8nq-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


From: Markus Elfring <elfring@users.sourceforge.net>
Date: Sat, 31 Dec 2016 22:05:19 +0100

The script "checkpatch.pl" pointed information out like the following.

WARNING: void function return statements are not generally useful

Thus remove such a statement in affected functions.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
 drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index 05a00914dba0..6fe0c72246a2 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -2121,7 +2121,6 @@ vchiq_platform_check_suspend(VCHIQ_STATE_T *state)
 
 out:
 	vchiq_log_trace(vchiq_susp_log_level, "%s exit", __func__);
-	return;
 }
 
 
@@ -2306,7 +2305,6 @@ vchiq_check_suspend(VCHIQ_STATE_T *state)
 
 out:
 	vchiq_log_trace(vchiq_susp_log_level, "%s exit", __func__);
-	return;
 }
 
 
-- 
2.11.0

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/6] staging: vchiq_arm: Fine-tuning for some function  implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2016-12-31 23:00 +0100
  [PATCH 6/6] staging: vchiq_arm: Delete an unnecessary return  statement in two functions SF Markus Elfring <elfring@users.sourceforge.net> - 2016-12-31 23:10 +0100
  [PATCH 3/6] staging: vchiq_arm: One check less in dump_phys_mem()  after error detection SF Markus Elfring <elfring@users.sourceforge.net> - 2016-12-31 23:10 +0100
  [PATCH 4/6] staging: vchiq_arm: Delete an error message for a failed  memory allocation in dump_phys_mem() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-12-31 23:10 +0100
  [PATCH 2/6] staging: vchiq_arm: Adjust 13 checks for null pointers SF Markus Elfring <elfring@users.sourceforge.net> - 2016-12-31 23:10 +0100
  [PATCH 5/6] staging: vchiq_arm: Combine substrings for 24 messages SF Markus Elfring <elfring@users.sourceforge.net> - 2016-12-31 23:10 +0100

csiph-web