mtd: onenand: Remove unnecessary OOM messages
authorJingoo Han <jg1.han@samsung.com>
Thu, 6 Feb 2014 06:15:38 +0000 (15:15 +0900)
committerBrian Norris <computersforpeace@gmail.com>
Tue, 11 Mar 2014 05:42:24 +0000 (22:42 -0700)
The site-specific OOM messages are unnecessary, because they
duplicate the MM subsystem generic OOM message.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
drivers/mtd/onenand/onenand_base.c
drivers/mtd/onenand/samsung.c

index 531ccbcdc04b6c7a4fba61f5e4378ecb14c769a3..e886d7af868f01a47137f61e5dfafb72245beb7b 100644 (file)
@@ -3994,11 +3994,8 @@ int onenand_scan(struct mtd_info *mtd, int maxchips)
        /* Allocate buffers, if necessary */
        if (!this->page_buf) {
                this->page_buf = kzalloc(mtd->writesize, GFP_KERNEL);
-               if (!this->page_buf) {
-                       printk(KERN_ERR "%s: Can't allocate page_buf\n",
-                               __func__);
+               if (!this->page_buf)
                        return -ENOMEM;
-               }
 #ifdef CONFIG_MTD_ONENAND_VERIFY_WRITE
                this->verify_buf = kzalloc(mtd->writesize, GFP_KERNEL);
                if (!this->verify_buf) {
@@ -4011,8 +4008,6 @@ int onenand_scan(struct mtd_info *mtd, int maxchips)
        if (!this->oob_buf) {
                this->oob_buf = kzalloc(mtd->oobsize, GFP_KERNEL);
                if (!this->oob_buf) {
-                       printk(KERN_ERR "%s: Can't allocate oob_buf\n",
-                               __func__);
                        if (this->options & ONENAND_PAGEBUF_ALLOC) {
                                this->options &= ~ONENAND_PAGEBUF_ALLOC;
                                kfree(this->page_buf);
index df7400dd4df847b321bb1fdedbb5ac522e9c5b83..b1a792fd1c23682a63d313b0982ad1bfe666131c 100644 (file)
@@ -872,10 +872,8 @@ static int s3c_onenand_probe(struct platform_device *pdev)
 
        size = sizeof(struct mtd_info) + sizeof(struct onenand_chip);
        mtd = kzalloc(size, GFP_KERNEL);
-       if (!mtd) {
-               dev_err(&pdev->dev, "failed to allocate memory\n");
+       if (!mtd)
                return -ENOMEM;
-       }
 
        onenand = kzalloc(sizeof(struct s3c_onenand), GFP_KERNEL);
        if (!onenand) {