diff options
author | Adrian Hunter <ext-adrian.hunter@nokia.com> | 2007-10-12 03:34:01 -0400 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2007-10-14 08:37:53 -0400 |
commit | 3cd3a86b6fdcb80ad097850a0990bcf6251422fe (patch) | |
tree | 287cdca45092f67a98af79853134e71545cfdf12 /drivers/mtd/onenand/onenand_base.c | |
parent | 30a7eb298af849bae3050e0a3f4c130995da22b0 (diff) |
[MTD] [OneNAND] Avoid deadlock in erase callback; release chip lock first.
When the erase callback performs some other action on the flash, it's
highly likely to deadlock unless we actually release the chip lock
before calling it.
This patch mirrors that same change already done for NAND.
Signed-off-by: Adrian Hunter <ext-adrian.hunter@nokia.com>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Diffstat (limited to 'drivers/mtd/onenand/onenand_base.c')
-rw-r--r-- | drivers/mtd/onenand/onenand_base.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/mtd/onenand/onenand_base.c b/drivers/mtd/onenand/onenand_base.c index a8c426bf47a4..dd2835569092 100644 --- a/drivers/mtd/onenand/onenand_base.c +++ b/drivers/mtd/onenand/onenand_base.c | |||
@@ -1711,13 +1711,14 @@ static int onenand_erase(struct mtd_info *mtd, struct erase_info *instr) | |||
1711 | erase_exit: | 1711 | erase_exit: |
1712 | 1712 | ||
1713 | ret = instr->state == MTD_ERASE_DONE ? 0 : -EIO; | 1713 | ret = instr->state == MTD_ERASE_DONE ? 0 : -EIO; |
1714 | /* Do call back function */ | ||
1715 | if (!ret) | ||
1716 | mtd_erase_callback(instr); | ||
1717 | 1714 | ||
1718 | /* Deselect and wake up anyone waiting on the device */ | 1715 | /* Deselect and wake up anyone waiting on the device */ |
1719 | onenand_release_device(mtd); | 1716 | onenand_release_device(mtd); |
1720 | 1717 | ||
1718 | /* Do call back function */ | ||
1719 | if (!ret) | ||
1720 | mtd_erase_callback(instr); | ||
1721 | |||
1721 | return ret; | 1722 | return ret; |
1722 | } | 1723 | } |
1723 | 1724 | ||