diff options
author | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2009-07-14 10:26:56 +0000 |
---|---|---|
committer | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2009-07-14 10:26:56 +0000 |
commit | 142e30fcaa1c3a63a1f0baf0b802ef888a0c250b (patch) | |
tree | c286d89dadacecee2e68b84ce5606467e9ae33e8 /sb600spi.c | |
parent | 78e4e127129398454813d1552b516638837c423e (diff) | |
download | flashrom-142e30fcaa1c3a63a1f0baf0b802ef888a0c250b.tar.gz flashrom-142e30fcaa1c3a63a1f0baf0b802ef888a0c250b.tar.bz2 flashrom-142e30fcaa1c3a63a1f0baf0b802ef888a0c250b.zip |
Use a distinct return code for SPI commands with unsupported/invalid length
Some drivers support only a few combinations of read/write length and
return error otherwise. Having a distinct return code for this error
means we can handle it in upper layers.
Corresponding to flashrom svn r653.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
Diffstat (limited to 'sb600spi.c')
-rw-r--r-- | sb600spi.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sb600spi.c b/sb600spi.c index 937550d0a..865a8c3da 100644 --- a/sb600spi.c +++ b/sb600spi.c @@ -114,14 +114,14 @@ int sb600_spi_send_command(unsigned int writecnt, unsigned int readcnt, if (readcnt > 8) { printf("%s, SB600 SPI controller can not receive %d bytes, " - "which is limited with 8 bytes\n", __func__, readcnt); - return 1; + "it is limited to 8 bytes\n", __func__, readcnt); + return SPI_INVALID_LENGTH; } if (writecnt > 8) { - printf("%s, SB600 SPI controller can not sent %d bytes, " - "which is limited with 8 bytes\n", __func__, writecnt); - return 1; + printf("%s, SB600 SPI controller can not send %d bytes, " + "it is limited to 8 bytes\n", __func__, writecnt); + return SPI_INVALID_LENGTH; } mmio_writeb(cmd, sb600_spibar + 0); |