From mboxrd@z Thu Jan 1 00:00:00 1970 From: Artem Bityutskiy Subject: Re: [PATCH] fs/ubifs: remove unnecessary new_valid_dev check Date: Thu, 01 Oct 2015 16:35:10 +0300 Message-ID: <1443706510.16309.52.camel@gmail.com> References: <1443446807-4838-1-git-send-email-bywxiaobai@163.com> <1443535053-4673-1-git-send-email-bywxiaobai@163.com> Reply-To: dedekind1@gmail.com Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Al Viro , Adrian Hunter , "linux-mtd@lists.infradead.org" , linux-fsdevel , LKML To: Richard Weinberger , Yaowei Bai Return-path: Received: from mga03.intel.com ([134.134.136.65]:44743 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752961AbbJANfO (ORCPT ); Thu, 1 Oct 2015 09:35:14 -0400 In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, 2015-09-30 at 10:11 +0200, Richard Weinberger wrote: > > { > > - if (new_valid_dev(rdev)) { > > - dev->new = cpu_to_le32(new_encode_dev(rdev)); > > - return sizeof(dev->new); > > - } else { > > - dev->huge = cpu_to_le64(huge_encode_dev(rdev)); > > - return sizeof(dev->huge); > > - } > > + dev->new = cpu_to_le32(new_encode_dev(rdev)); > > + return sizeof(dev->new); > > } > > Reviewed-by: Richard Weinberger Signed-off-by: Artem Bityutskiy