diff options
| author | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2008-05-16 21:11:53 +0000 |
|---|---|---|
| committer | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2008-05-16 21:11:53 +0000 |
| commit | fc52409252eb3f6f05547cf28b0e44793a39a6a4 (patch) | |
| tree | 323b4985608c2ce9fad53abc52839d40e4865edb | |
| parent | 1b18b3c076713aa73a9c8251dc0b83a80f413897 (diff) | |
| download | flashrom-fc52409252eb3f6f05547cf28b0e44793a39a6a4.tar.gz flashrom-fc52409252eb3f6f05547cf28b0e44793a39a6a4.tar.bz2 flashrom-fc52409252eb3f6f05547cf28b0e44793a39a6a4.zip | |
I looked at the datasheet and erase_sector_39sf020() is totally and completely wrong
It was a straight cut'n'paste from SST 28SF040 code and the person doing
the cut'n'paste didn't even bother to check the data sheet. The SST
39SF020 is completely incompatible with the 28SF040.
No need for replacement. According to the data sheet, standard JEDEC
commands will work and we have those commands in the tree already.
Corresponding to flashrom svn r242 and coreboot v2 svn r3331.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Joseph Smith <joe@settoplinux.org>
| -rw-r--r-- | sst39sf020.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/sst39sf020.c b/sst39sf020.c index c4fd9570..f6fcc373 100644 --- a/sst39sf020.c +++ b/sst39sf020.c @@ -22,21 +22,6 @@ #include <stdint.h> #include "flash.h" -#define AUTO_PG_ERASE1 0x20 -#define AUTO_PG_ERASE2 0xD0 - -static __inline__ int erase_sector_39sf020(volatile uint8_t *bios, - unsigned long address) -{ - *bios = AUTO_PG_ERASE1; - *(bios + address) = AUTO_PG_ERASE2; - - /* wait for Toggle bit ready */ - toggle_ready_jedec(bios); - - return 0; -} - int write_39sf020(struct flashchip *flash, uint8_t *buf) { int i; |