From patchwork Wed Mar 11 17:44:51 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Fabian Frederick X-Patchwork-Id: 449110 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2001:1868:205::9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id E7037140119 for ; Thu, 12 Mar 2015 04:46:46 +1100 (AEDT) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1YVkhE-0002Ot-TU; Wed, 11 Mar 2015 17:45:28 +0000 Received: from mailrelay109.isp.belgacom.be ([195.238.20.136]) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1YVkhB-0001Az-RV for linux-mtd@lists.infradead.org; Wed, 11 Mar 2015 17:45:27 +0000 X-Belgacom-Dynamic: yes X-Cloudmark-SP-Filtered: true X-Cloudmark-SP-Result: v=1.1 cv=hkFBDakPHWYIOok4y3pcivnmCJQbEK2MeX2RAJ8hs1E= c=1 sm=2 a=MKtGQD3n3ToA:10 a=1oJP67jkp3AA:10 a=OHAaWIYZilBwteWFFaAA:9 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A2D3DQDDfQBV/89wsFtcgwaBLLBTBQEBAQEBAQUBdowJi2+BOk4BAQEBAQF8hD0vI4EaN4gzAchvLIYKiXsdhBcFhgCTeIEahXyMYSOBRQE8HIFRPTGCQwEBAQ Received: from 207.112-176-91.adsl-dyn.isp.belgacom.be (HELO localhost.home.) ([91.176.112.207]) by relay.skynet.be with ESMTP; 11 Mar 2015 18:44:57 +0100 From: Fabian Frederick To: linux-kernel@vger.kernel.org Subject: [PATCH 1/1 linux-next] UBIFS: simplify returns Date: Wed, 11 Mar 2015 18:44:51 +0100 Message-Id: <1426095891-17827-1-git-send-email-fabf@skynet.be> X-Mailer: git-send-email 1.9.1 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20150311_104526_046768_FFDB176E X-CRM114-Status: UNSURE ( 7.79 ) X-CRM114-Notice: Please train this message. X-Spam-Score: -0.0 (/) X-Spam-Report: SpamAssassin version 3.4.0 on bombadil.infradead.org summary: Content analysis details: (-0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/, no trust [195.238.20.136 listed in list.dnswl.org] -0.0 RCVD_IN_MSPIKE_H3 RBL: Good reputation (+3) [195.238.20.136 listed in wl.mailspike.net] -0.0 SPF_PASS SPF: sender matches SPF record -0.0 RCVD_IN_MSPIKE_WL Mailspike good senders Cc: Fabian Frederick , linux-mtd@lists.infradead.org, Adrian Hunter , Artem Bityutskiy X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: "linux-mtd" Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org directly return recover_head() and ubifs_leb_unmap() instead of storing value in err and testing it. Signed-off-by: Fabian Frederick --- fs/ubifs/recovery.c | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/fs/ubifs/recovery.c b/fs/ubifs/recovery.c index c640938..a2e1f7f 100644 --- a/fs/ubifs/recovery.c +++ b/fs/ubifs/recovery.c @@ -975,11 +975,8 @@ int ubifs_recover_inl_heads(struct ubifs_info *c, void *sbuf) return err; dbg_rcvry("checking LPT head at %d:%d", c->nhead_lnum, c->nhead_offs); - err = recover_head(c, c->nhead_lnum, c->nhead_offs, sbuf); - if (err) - return err; - return 0; + return recover_head(c, c->nhead_lnum, c->nhead_offs, sbuf); } /** @@ -1002,13 +999,9 @@ static int clean_an_unclean_leb(struct ubifs_info *c, dbg_rcvry("LEB %d len %d", lnum, len); - if (len == 0) { + if (len == 0) /* Nothing to read, just unmap it */ - err = ubifs_leb_unmap(c, lnum); - if (err) - return err; - return 0; - } + return ubifs_leb_unmap(c, lnum); err = ubifs_leb_read(c, lnum, buf, offs, len, 0); if (err && err != -EBADMSG)