diff options
| author | Peter Stuge <peter@stuge.se> | 2008-07-07 06:38:51 +0000 |
|---|---|---|
| committer | Peter Stuge <peter@stuge.se> | 2008-07-07 06:38:51 +0000 |
| commit | a4793182e892c71660e24dbe12198a653f7ec1e6 (patch) | |
| tree | c3e5bb7b62a251f94b1da1b0c9fd623c8286a013 /util/flashrom/flash.h | |
| parent | 5dc95de5e64638d5b95d7fd1fb7ef87887bfdbdc (diff) | |
| download | coreboot-a4793182e892c71660e24dbe12198a653f7ec1e6.tar.xz | |
flashrom: Trivial SPI cleanups
While writing a new SPI driver I fixed some things in the SPI code:
All calls to spi_command() had unneccessary #define duplications, and in some
cases the read count define could theoretically become harmful because NULL was
passed for the read buffer. Avoid a crash, should someone change the #defines.
I also noticed that the only caller of spi_page_program() was the it87 driver,
and spi_page_program() could only call back into the it87 driver. Removed the
function for easier-to-follow code and made it8716f_spi_page_program() static.
The ichspi driver's static page functions are already static.
Signed-off-by: Peter Stuge <peter@stuge.se>
Acked-by: Peter Stuge <peter@stuge.se>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3418 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'util/flashrom/flash.h')
| -rw-r--r-- | util/flashrom/flash.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/util/flashrom/flash.h b/util/flashrom/flash.h index cfe3062944..a810a7c2d7 100644 --- a/util/flashrom/flash.h +++ b/util/flashrom/flash.h @@ -423,7 +423,6 @@ int spi_chip_read(struct flashchip *flash, uint8_t *buf); uint8_t spi_read_status_register(); void spi_disable_blockprotect(void); void spi_byte_program(int address, uint8_t byte); -void spi_page_program(int block, uint8_t *buf, uint8_t *bios); void spi_nbyte_read(int address, uint8_t *bytes, int len); /* 82802ab.c */ @@ -447,7 +446,6 @@ int it87xx_probe_spi_flash(const char *name); int it8716f_spi_command(unsigned int writecnt, unsigned int readcnt, const unsigned char *writearr, unsigned char *readarr); int it8716f_spi_chip_read(struct flashchip *flash, uint8_t *buf); int it8716f_spi_chip_write(struct flashchip *flash, uint8_t *buf); -void it8716f_spi_page_program(int block, uint8_t *buf, uint8_t *bios); /* jedec.c */ uint8_t oddparity(uint8_t val); |