aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/mtd
diff options
context:
space:
mode:
authorFrank Li <Frank.Li@freescale.com>2010-10-25 23:02:19 -0400
committerDavid Woodhouse <David.Woodhouse@intel.com>2010-10-30 07:30:50 -0400
commit6fe4c590313133ebd5dadb769031489ff178ece1 (patch)
tree7fad4ea814e69270869a08cfeb727ed616224765 /drivers/mtd
parent6411bf6cdfe9b444bc3aa0c23f8cf35460d69432 (diff)
MTD: Fix wrong check register_blkdev return value
register_blkdev return 1..255 when major = 0. if (ret ) { printk(KERN_WARNING "Unable to register %s block device on major %d: %d\n", tr->name, tr->major, ret); mutex_unlock(&mtd_table_mutex); return ret; } Above code will return fail when register_blkdev return allocated major number. Signed-off-by: Frank Li <Frank.Li@freescale.com> Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers/mtd')
-rw-r--r--drivers/mtd/mtd_blkdevs.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/mtd/mtd_blkdevs.c b/drivers/mtd/mtd_blkdevs.c
index a919587ed481..63790e975d20 100644
--- a/drivers/mtd/mtd_blkdevs.c
+++ b/drivers/mtd/mtd_blkdevs.c
@@ -502,13 +502,16 @@ int register_mtd_blktrans(struct mtd_blktrans_ops *tr)
502 mutex_lock(&mtd_table_mutex); 502 mutex_lock(&mtd_table_mutex);
503 503
504 ret = register_blkdev(tr->major, tr->name); 504 ret = register_blkdev(tr->major, tr->name);
505 if (ret) { 505 if (ret < 0) {
506 printk(KERN_WARNING "Unable to register %s block device on major %d: %d\n", 506 printk(KERN_WARNING "Unable to register %s block device on major %d: %d\n",
507 tr->name, tr->major, ret); 507 tr->name, tr->major, ret);
508 mutex_unlock(&mtd_table_mutex); 508 mutex_unlock(&mtd_table_mutex);
509 return ret; 509 return ret;
510 } 510 }
511 511
512 if (ret)
513 tr->major = ret;
514
512 tr->blkshift = ffs(tr->blksize) - 1; 515 tr->blkshift = ffs(tr->blksize) - 1;
513 516
514 INIT_LIST_HEAD(&tr->devs); 517 INIT_LIST_HEAD(&tr->devs);