diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-02-13 15:21:44 +0400 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2014-03-11 09:42:25 +0400 |
commit | da22b89386e8d4dc89525801dfe60f5f8c29668d (patch) | |
tree | 534b9359d62dac3f987933391203dd9c761deae0 /drivers/mtd | |
parent | 74414a945ac992c71766bbf76718c7fddbcbd016 (diff) | |
download | linux-da22b89386e8d4dc89525801dfe60f5f8c29668d.tar.xz |
mtd: remove some duplicative checks
"rc" is an error code here, no need to check it a second time.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd')
-rw-r--r-- | drivers/mtd/rfd_ftl.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/mtd/rfd_ftl.c b/drivers/mtd/rfd_ftl.c index 233b946e5d66..d1cbf26db2c0 100644 --- a/drivers/mtd/rfd_ftl.c +++ b/drivers/mtd/rfd_ftl.c @@ -602,8 +602,7 @@ static int mark_sector_deleted(struct partition *part, u_long old_addr) if (rc) { printk(KERN_ERR PREFIX "error writing '%s' at " "0x%lx\n", part->mbd.mtd->name, addr); - if (rc) - goto err; + goto err; } if (block == part->current_block) part->header_cache[offset + HEADER_MAP_OFFSET] = del; @@ -675,8 +674,7 @@ static int do_writesect(struct mtd_blktrans_dev *dev, u_long sector, char *buf, if (rc) { printk(KERN_ERR PREFIX "error writing '%s' at 0x%lx\n", part->mbd.mtd->name, addr); - if (rc) - goto err; + goto err; } part->sector_map[sector] = addr; @@ -695,8 +693,7 @@ static int do_writesect(struct mtd_blktrans_dev *dev, u_long sector, char *buf, if (rc) { printk(KERN_ERR PREFIX "error writing '%s' at 0x%lx\n", part->mbd.mtd->name, addr); - if (rc) - goto err; + goto err; } block->used_sectors++; block->free_sectors--; |