diff options
author | Boris Brezillon <boris.brezillon@bootlin.com> | 2018-12-02 10:54:32 +0100 |
---|---|---|
committer | Jagan Teki <jagan@amarulasolutions.com> | 2018-12-06 00:45:36 +0530 |
commit | 08898e8b22d74a4511eadee9b06b11aab43e809c (patch) | |
tree | 9860510a9a98670eb68422d9a6da6981d839d091 /include/configs/mxs.h | |
parent | 7371944a71690abafd0717b5d5f72c67e9f0f414 (diff) |
mtd: sf: Make sf_mtd.c more robust
SPI flash based MTD devs can be registered/unregistered at any time
through the sf probe command or the spi_flash_free() function.
This commit does not try to fix the root cause as it would probably
require rewriting most of the code and have an mtd_info object
instance per spi_flash object (not to mention that the the spi-flash
layer is likely to be replaced by a spi-nor layer ported from Linux).
Instead, we try to be as safe as can be by checking the code returned
by del_mtd_device() and complain loudly when there's nothing we can
do about the deregistration failure. When that happens we also reset
sf_mtd_info.priv to NULL, and check for NULL pointer in the mtd hooks
so that -ENODEV is returned instead of hitting a NULL pointer
dereference exception when the MTD instance is later accessed by a user.
Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
Tested-by: Heiko Schocher <hs@denx.de>
Diffstat (limited to 'include/configs/mxs.h')
0 files changed, 0 insertions, 0 deletions