diff mbox

opensm/osm_sminfo_rcv.c: Eliminate unneeded return parameter from internal routine

Message ID 4DFA05CF.1080200@dev.mellanox.co.il (mailing list archive)
State Accepted
Delegated to: Alex Netes
Headers show

Commit Message

Hal Rosenstock June 16, 2011, 1:31 p.m. UTC
smi_rcv_process_get_sm only returns 0 so no need for return parameter.

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
---
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Alex Netes July 5, 2011, 3:58 p.m. UTC | #1
Hi Hal,

On 09:31 Thu 16 Jun     , Hal Rosenstock wrote:
> 
> smi_rcv_process_get_sm only returns 0 so no need for return parameter.
> 
> Signed-off-by: Hal Rosenstock <hal@mellanox.com>
> ---

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/opensm/osm_sminfo_rcv.c b/opensm/osm_sminfo_rcv.c
index b6bc558..66eb886 100644
--- a/opensm/osm_sminfo_rcv.c
+++ b/opensm/osm_sminfo_rcv.c
@@ -293,9 +293,9 @@  Exit:
 	OSM_LOG_EXIT(sm->p_log);
 }
 
-static int smi_rcv_process_get_sm(IN osm_sm_t * sm,
-				  IN const osm_remote_sm_t * p_sm,
-				  boolean_t light_sweep)
+static void smi_rcv_process_get_sm(IN osm_sm_t * sm,
+				   IN const osm_remote_sm_t * p_sm,
+				   boolean_t light_sweep)
 {
 	const ib_sm_info_t *p_smi;
 
@@ -405,7 +405,6 @@  static int smi_rcv_process_get_sm(IN osm_sm_t * sm,
 	}
 
 	OSM_LOG_EXIT(sm->p_log);
-	return 0;
 }
 
 static void smi_rcv_process_get_response(IN osm_sm_t * sm,