From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id CCA0521959CB2 for ; Sun, 27 Jan 2019 16:31:13 -0800 (PST) From: Wei Yang Subject: [PATCH 2/5] libnvdimm, namespace: remove special handling when count == 0 Date: Mon, 28 Jan 2019 08:30:15 +0800 Message-Id: <20190128003018.4087-2-richardw.yang@linux.intel.com> In-Reply-To: <20190128003018.4087-1-richardw.yang@linux.intel.com> References: <20190128003018.4087-1-richardw.yang@linux.intel.com> MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: linux-nvdimm@lists.01.org Cc: zwisler@kernel.org List-ID: This is reasonable to continue directly when count == 0, while this case could be merged in following for loop. count == 0, j == 0 => j >= count, loop continues. This patch just removes the special handling to make the code look more neat. Signed-off-by: Wei Yang --- drivers/nvdimm/namespace_devs.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/nvdimm/namespace_devs.c b/drivers/nvdimm/namespace_devs.c index 234c0c79726a..0d64c4b38a56 100644 --- a/drivers/nvdimm/namespace_devs.c +++ b/drivers/nvdimm/namespace_devs.c @@ -2485,8 +2485,6 @@ static int init_active_labels(struct nd_region *nd_region) count = nd_label_active_count(ndd); dev_dbg(ndd->dev, "count: %d\n", count); - if (!count) - continue; for (j = 0; j < count; j++) { struct nd_namespace_label *label; -- 2.19.1 _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm