From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965117AbbCRVqM (ORCPT ); Wed, 18 Mar 2015 17:46:12 -0400 Received: from mail-wi0-f173.google.com ([209.85.212.173]:33338 "EHLO mail-wi0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965100AbbCRVqJ (ORCPT ); Wed, 18 Mar 2015 17:46:09 -0400 From: Redha Gouicem To: gregkh@linuxfoundation.org Cc: linux-kernel@vger.kernel.org, Redha Gouicem Subject: [PATCH 2/2] Staging: lustre: lnet: lnet: router.c: fix useless statements Date: Wed, 18 Mar 2015 22:45:47 +0100 Message-Id: <1426715147-15371-2-git-send-email-redha.gouicem@gmail.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1426715147-15371-1-git-send-email-redha.gouicem@gmail.com> References: <1426715147-15371-1-git-send-email-redha.gouicem@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch removes useless returns and elses. Signed-off-by: Redha Gouicem --- drivers/staging/lustre/lnet/lnet/router.c | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/router.c b/drivers/staging/lustre/lnet/lnet/router.c index 89a29b7..6fe79e2 100644 --- a/drivers/staging/lustre/lnet/lnet/router.c +++ b/drivers/staging/lustre/lnet/lnet/router.c @@ -270,7 +270,6 @@ static void lnet_shuffle_seed(void) do_gettimeofday(&tv); cfs_srand(tv.tv_sec ^ seed[0], tv.tv_usec ^ seed[1]); seeded = 1; - return; } /* NB expects LNET_LOCK held */ @@ -357,11 +356,10 @@ lnet_add_route(__u32 net, unsigned int hops, lnet_nid_t gateway, if (rc == -EHOSTUNREACH) { /* gateway is not on a local net */ return 0; /* ignore the route entry */ - } else { - CERROR("Error %d creating route %s %d %s\n", rc, - libcfs_net2str(net), hops, - libcfs_nid2str(gateway)); - } + CERROR("Error %d creating route %s %d %s\n", rc, + libcfs_net2str(net), hops, + libcfs_nid2str(gateway)); + return rc; } @@ -607,7 +605,6 @@ lnet_swap_pinginfo(lnet_ping_info_t *info) __swab64s(&stat->ns_nid); __swab32s(&stat->ns_status); } - return; } /** @@ -1011,7 +1008,6 @@ lnet_ping_router_locked(lnet_peer_t *rtr) } lnet_peer_decref_locked(rtr); - return; } int @@ -1084,7 +1080,6 @@ lnet_router_checker_stop(void) rc = LNetEQFree(the_lnet.ln_rc_eqh); LASSERT(rc == 0); - return; } static void @@ -1581,7 +1576,6 @@ EXPORT_SYMBOL(lnet_notify); void lnet_get_tunables(void) { - return; } #else @@ -1670,7 +1664,6 @@ lnet_router_checker(void) lnet_net_unlock(0); running = 0; /* lock only needed for the recursion check */ - return; } /* NB lnet_peers_start_down depends on me, -- 1.9.1