diff options
| author | Tomer Barletz <barletz@gmail.com> | 2015-08-04 21:00:24 -0700 |
|---|---|---|
| committer | Brian Norris <computersforpeace@gmail.com> | 2015-08-25 13:58:10 -0700 |
| commit | cc7fce80229067890365c1ee196be5d304d36dea (patch) | |
| tree | d821b66a412263b6a1939eb297e5ade5dba20b7b /drivers/mtd/mtd_blkdevs.c | |
| parent | 06bb6f5a69dfc53b79dd5f7afabdcd070a18afbf (diff) | |
| download | linux-cc7fce80229067890365c1ee196be5d304d36dea.tar.gz linux-cc7fce80229067890365c1ee196be5d304d36dea.tar.bz2 linux-cc7fce80229067890365c1ee196be5d304d36dea.zip | |
mtd: blkdevs: fix switch-bool compilation warning
With gcc 5.1 I get:
warning: switch condition has boolean value [-Wswitch-bool]
Signed-off-by: Tomer Barletz <barletz@gmail.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd/mtd_blkdevs.c')
| -rw-r--r-- | drivers/mtd/mtd_blkdevs.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/mtd/mtd_blkdevs.c b/drivers/mtd/mtd_blkdevs.c index 41acc507b22e..88304751eb8a 100644 --- a/drivers/mtd/mtd_blkdevs.c +++ b/drivers/mtd/mtd_blkdevs.c @@ -97,14 +97,13 @@ static int do_blktrans_request(struct mtd_blktrans_ops *tr, if (req->cmd_flags & REQ_DISCARD) return tr->discard(dev, block, nsect); - switch(rq_data_dir(req)) { - case READ: + if (rq_data_dir(req) == READ) { for (; nsect > 0; nsect--, block++, buf += tr->blksize) if (tr->readsect(dev, block, buf)) return -EIO; rq_flush_dcache_pages(req); return 0; - case WRITE: + } else { if (!tr->writesect) return -EIO; @@ -113,9 +112,6 @@ static int do_blktrans_request(struct mtd_blktrans_ops *tr, if (tr->writesect(dev, block, buf)) return -EIO; return 0; - default: - printk(KERN_NOTICE "Unknown request %u\n", rq_data_dir(req)); - return -EIO; } } |