diff options
author | Mike Frysinger <vapier@gentoo.org> | 2008-10-02 01:55:38 -0400 |
---|---|---|
committer | Stefan Roese <sr@denx.de> | 2008-10-02 10:30:32 +0200 |
commit | 2215987e100d2a841ae6d48a7cc9bb83fcf22737 (patch) | |
tree | 37ea6fa0da6ddb6d1d8e11a2ea2abea5b30de3eb /lib_sh | |
parent | 8fd4166c467a46773f80208bda1ec3b4757747bc (diff) | |
download | u-boot-imx-2215987e100d2a841ae6d48a7cc9bb83fcf22737.zip u-boot-imx-2215987e100d2a841ae6d48a7cc9bb83fcf22737.tar.gz u-boot-imx-2215987e100d2a841ae6d48a7cc9bb83fcf22737.tar.bz2 |
cfi_flash: do not reset flash when probe fails
The CFI flash driver starts at flash_init() which calls down into
flash_get_size(). This starts by calling flash_detect_cfi(). If said
function fails, flash_get_size() finishes by attempting to reset the
flash. Unfortunately, it does this with an info->portwidth set to 0x10
which filters down into flash_make_cmd() and that happily smashes the
stack by sticking info->portwidth bytes into a cfiword_t variable that
lives on the stack. On a 64bit system you probably won't notice, but
killing the last 8 bytes on a 32bit system usually leads to a corrupt
return address. Which is what happens on a Blackfin system.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Stefan Roese <sr@denx.de>
Diffstat (limited to 'lib_sh')
0 files changed, 0 insertions, 0 deletions