From patchwork Thu Jun 10 02:28:50 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Leizhen \(ThunderTown\)" X-Patchwork-Id: 457805 Delivered-To: patch@linaro.org Received: by 2002:a02:735a:0:0:0:0:0 with SMTP id a26csp5980934jae; Wed, 9 Jun 2021 19:31:45 -0700 (PDT) X-Google-Smtp-Source: ABdhPJznaJzozv6P9zw2AMYEjyunubhwbKccEfK5sxTCxDaF7xDPGtJgXCOGieSpQqkXNpsPbcM7 X-Received: by 2002:a17:906:488f:: with SMTP id v15mr2372429ejq.428.1623292305381; Wed, 09 Jun 2021 19:31:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1623292305; cv=none; d=google.com; s=arc-20160816; b=mQKW6Osp4Auvwfvzy6FJmHiNg1s95+DdLjHQOQJ4SByYPhpPipTG7bsEYRAJ53XJg8 46IQiq/sO1Ay7LcA92P7ExPaKaJHPrA2BgExS2J2x01bNbHcb0s1eqfuo7AOUhQ9BWXk hZ0vu+ES0DQe3uyBIkrcg8BzcY6lCwwAsbMhYHjfj78Lf3SguZkJAGOdS1oqwJdcfV6t 5nR8cUwVtWazDXQP1OxpQoHHNmVdgKwM6p8CKPd0Td3Hn1DEXkuzX5FSphG3wo/KCBIE ouUGlL7y2Ti3YG6yjGCvIFwx3vXcCL/x+CDPEhB79mLRnNXDM7HxbQb/4bck6vjPIHfv Yziw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=errors-to:sender:content-transfer-encoding:list-subscribe:list-help :list-post:list-archive:list-unsubscribe:list-id:precedence :mime-version:message-id:date:subject:cc:to:from:dkim-signature; bh=ArG1Caf/03NhQlt3g8gOewSnHd26jlgkMnJRYxP936g=; b=pd3XNJ9vSGB+HcAE6LwHbzkl0Ub9LChid16Gn1LOpYR82vYSSblXHfzGZHiIO9TP18 e3FfvRFdhIFNfiFAkRTApgfDqFUabF58iakwFN+UnFFaMQGI6D29VqXn//JlmYaOUsyN HwJiNswuWADg7EeAUZKNIw+B7IfC7hrnLMQo4PUmpGGMSjB3cEOUtNKIkEkYf/JNphW7 ttnLRdwiTv7pUiK/tRhfJxz/9ckT0ZkVp/8dkXZnrd+ELFy98MZfM+fteSQfk75WQWkK m72uxptQ5spzml2jMy+hHAY0GO2v7etqrHPqELCQLjn8u+LUjOfpFOSPmZaUiOT4HtTp PbfA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@lists.infradead.org header.s=bombadil.20210309 header.b="46El9u/f"; spf=pass (google.com: best guess record for domain of linux-mtd-bounces+patch=linaro.org@lists.infradead.org designates 198.137.202.133 as permitted sender) smtp.mailfrom="linux-mtd-bounces+patch=linaro.org@lists.infradead.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Return-Path: Received: from bombadil.infradead.org (bombadil.infradead.org. [198.137.202.133]) by mx.google.com with ESMTPS id jz1si1146237ejb.461.2021.06.09.19.31.43 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Jun 2021 19:31:45 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-mtd-bounces+patch=linaro.org@lists.infradead.org designates 198.137.202.133 as permitted sender) client-ip=198.137.202.133; Authentication-Results: mx.google.com; dkim=pass header.i=@lists.infradead.org header.s=bombadil.20210309 header.b="46El9u/f"; spf=pass (google.com: best guess record for domain of linux-mtd-bounces+patch=linaro.org@lists.infradead.org designates 198.137.202.133 as permitted sender) smtp.mailfrom="linux-mtd-bounces+patch=linaro.org@lists.infradead.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:CC :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=ArG1Caf/03NhQlt3g8gOewSnHd26jlgkMnJRYxP936g=; b=46El9u/fZb4dvS +qY7obyqJRpHU2NvEypwFUl6m92fWaUgMzYMxiYv4eqkN7R3IGso7r16snknro+8IZwNiTziOXCcx kWgz0UyWyRJMarff1nYfee6pnT5rTIcWHt5VONJ6E4sdhNYgzLLlkPM6DgRNUO9y99PM+1P4X15Qw PAWvBcM/YjsxZWttGUI4FeePVxu+6KeUyAlvi14GXQD2j0iPBD4FsZEJmg+MzWeVeD00gbJ6V6SqR egHa0Nb7f6OxS7tXx0w/k3ZFKW76G5vAH4bURunknIU2gw2P5TKmqhuIw2C1RThIdTS+J2on0fht6 d6jxwLUhClrAytvBrQng==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lrARa-00GRpy-Es; Thu, 10 Jun 2021 02:29:18 +0000 Received: from szxga01-in.huawei.com ([45.249.212.187]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lrARW-00GRoa-Vh for linux-mtd@lists.infradead.org; Thu, 10 Jun 2021 02:29:16 +0000 Received: from dggemv703-chm.china.huawei.com (unknown [172.30.72.56]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4G0nq72vLszWtkk; Thu, 10 Jun 2021 10:24:19 +0800 (CST) Received: from dggpemm500006.china.huawei.com (7.185.36.236) by dggemv703-chm.china.huawei.com (10.3.19.46) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Thu, 10 Jun 2021 10:28:57 +0800 Received: from thunder-town.china.huawei.com (10.174.177.72) by dggpemm500006.china.huawei.com (7.185.36.236) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Thu, 10 Jun 2021 10:28:57 +0800 From: Zhen Lei To: Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , linux-mtd CC: Zhen Lei Subject: [PATCH 1/1] mtd: inftl: remove unnecessary oom message Date: Thu, 10 Jun 2021 10:28:50 +0800 Message-ID: <20210610022850.15397-1-thunder.leizhen@huawei.com> X-Mailer: git-send-email 2.26.0.windows.1 MIME-Version: 1.0 X-Originating-IP: [10.174.177.72] X-ClientProxiedBy: dggems704-chm.china.huawei.com (10.3.19.181) To dggpemm500006.china.huawei.com (7.185.36.236) X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210609_192915_229977_B044D7F7 X-CRM114-Status: UNSURE ( 8.43 ) X-CRM114-Notice: Please train this message. X-Spam-Score: -2.3 (--) X-Spam-Report: Spam detection software, running on the system "bombadil.infradead.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: Fixes scripts/checkpatch.pl warning: WARNING: Possible unnecessary 'out of memory' message Remove it can help us save a bit of memory. Signed-off-by: Zhen Lei --- drivers/mtd/inftlmount.c | 15 ++ 1 file changed, 2 insertions(+), 13 deletions(-) Content analysis details: (-2.3 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at https://www.dnswl.org/, medium trust [45.249.212.187 listed in list.dnswl.org] 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -0.0 SPF_PASS SPF: sender matches SPF record X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-mtd" Errors-To: linux-mtd-bounces+patch=linaro.org@lists.infradead.org Fixes scripts/checkpatch.pl warning: WARNING: Possible unnecessary 'out of memory' message Remove it can help us save a bit of memory. Signed-off-by: Zhen Lei --- drivers/mtd/inftlmount.c | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-) -- 2.26.0.106.g9fadedd ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ diff --git a/drivers/mtd/inftlmount.c b/drivers/mtd/inftlmount.c index 39d024118610264..6276daa296dab62 100644 --- a/drivers/mtd/inftlmount.c +++ b/drivers/mtd/inftlmount.c @@ -259,20 +259,13 @@ static int find_boot_record(struct INFTLrecord *inftl) /* Memory alloc */ inftl->PUtable = kmalloc_array(inftl->nb_blocks, sizeof(u16), GFP_KERNEL); - if (!inftl->PUtable) { - printk(KERN_WARNING "INFTL: allocation of PUtable " - "failed (%zd bytes)\n", - inftl->nb_blocks * sizeof(u16)); + if (!inftl->PUtable) return -ENOMEM; - } inftl->VUtable = kmalloc_array(inftl->nb_blocks, sizeof(u16), GFP_KERNEL); if (!inftl->VUtable) { kfree(inftl->PUtable); - printk(KERN_WARNING "INFTL: allocation of VUtable " - "failed (%zd bytes)\n", - inftl->nb_blocks * sizeof(u16)); return -ENOMEM; } @@ -558,12 +551,8 @@ int INFTL_mount(struct INFTLrecord *s) /* Temporary buffer to store ANAC numbers. */ ANACtable = kcalloc(s->nb_blocks, sizeof(u8), GFP_KERNEL); - if (!ANACtable) { - printk(KERN_WARNING "INFTL: allocation of ANACtable " - "failed (%zd bytes)\n", - s->nb_blocks * sizeof(u8)); + if (!ANACtable) return -ENOMEM; - } /* * First pass is to explore each physical unit, and construct the