aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/mtd/onenand
diff options
context:
space:
mode:
authorAaro Koskinen <aaro.koskinen@iki.fi>2016-02-20 15:27:48 -0500
committerBrian Norris <computersforpeace@gmail.com>2016-02-24 13:06:46 -0500
commit5e64c29e98bfbba1b527b0a164f9493f3db9e8cb (patch)
tree9e0d8b236a3060800cc2bde08c74bf9008bb03c3 /drivers/mtd/onenand
parent3b5394a3ccffbfa1d1d448d48742853a862822c4 (diff)
mtd: onenand: fix deadlock in onenand_block_markbad
Commit 5942ddbc500d ("mtd: introduce mtd_block_markbad interface") incorrectly changed onenand_block_markbad() to call mtd_block_markbad instead of onenand_chip's block_markbad function. As a result the function will now recurse and deadlock. Fix by reverting the change. Fixes: 5942ddbc500d ("mtd: introduce mtd_block_markbad interface") Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi> Acked-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com> Cc: <stable@vger.kernel.org> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd/onenand')
-rw-r--r--drivers/mtd/onenand/onenand_base.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/mtd/onenand/onenand_base.c b/drivers/mtd/onenand/onenand_base.c
index 43b3392ffee7..652d01832873 100644
--- a/drivers/mtd/onenand/onenand_base.c
+++ b/drivers/mtd/onenand/onenand_base.c
@@ -2599,6 +2599,7 @@ static int onenand_default_block_markbad(struct mtd_info *mtd, loff_t ofs)
2599 */ 2599 */
2600static int onenand_block_markbad(struct mtd_info *mtd, loff_t ofs) 2600static int onenand_block_markbad(struct mtd_info *mtd, loff_t ofs)
2601{ 2601{
2602 struct onenand_chip *this = mtd->priv;
2602 int ret; 2603 int ret;
2603 2604
2604 ret = onenand_block_isbad(mtd, ofs); 2605 ret = onenand_block_isbad(mtd, ofs);
@@ -2610,7 +2611,7 @@ static int onenand_block_markbad(struct mtd_info *mtd, loff_t ofs)
2610 } 2611 }
2611 2612
2612 onenand_get_device(mtd, FL_WRITING); 2613 onenand_get_device(mtd, FL_WRITING);
2613 ret = mtd_block_markbad(mtd, ofs); 2614 ret = this->block_markbad(mtd, ofs);
2614 onenand_release_device(mtd); 2615 onenand_release_device(mtd);
2615 return ret; 2616 return ret;
2616} 2617}