diff options
author | 2013-11-05 17:59:07 +0800 | |
---|---|---|
committer | 2013-11-07 10:16:21 -0800 | |
commit | a749d13acd8e079ed4c77a9456d842dc94af8f17 (patch) | |
tree | a985aec3449dfb56af521cf13237aebf617335a7 /tools/perf/scripts/python/export-to-postgresql.py | |
parent | mtd: nand: use a local variable to simplify the nand_scan_tail (diff) | |
download | linux-dev-a749d13acd8e079ed4c77a9456d842dc94af8f17.tar.xz linux-dev-a749d13acd8e079ed4c77a9456d842dc94af8f17.zip |
mtd: atmel_nand: fix bug driver will in a dead lock if no nand detected
In the atmel driver probe function, the code shows like following:
atmel_nand_probe(...) {
...
err_nand_ioremap:
platform_driver_unregister(&atmel_nand_nfc_driver);
return res;
}
If no nand flash detected, the driver probe function will goto
err_nand_ioremap label.
Then platform_driver_unregister() will be called. It will get the
lock of atmel_nand device since it is parent of nfc_device. The
problem is the lock is already hold by atmel_nand_probe itself.
So system will be in a dead lock.
This patch just simply removed to platform_driver_unregister() call.
When atmel_nand driver is quit the platform_driver_unregister() will
be called in atmel_nand_remove().
[Brian: the NAND platform probe really has no business
registering/unregistering another driver; this fixes the deadlock, but
we should follow up the likely racy behavior here with a better
architecture]
Signed-off-by: Josh Wu <josh.wu@atmel.com>
Cc: <stable@vger.kernel.org> # 3.12
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions