diff options
author | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2010-07-18 14:42:28 +0000 |
---|---|---|
committer | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2010-07-18 14:42:28 +0000 |
commit | 06b30ea42246bf7142515d584fe19263fb83ebc9 (patch) | |
tree | 6602356afa0436123eed615c45a88f3d5071b821 /bitbang_spi.c | |
parent | cd259c81339f4e0c1c6056d0c3c546c73231366c (diff) | |
download | flashrom-06b30ea42246bf7142515d584fe19263fb83ebc9.zip flashrom-06b30ea42246bf7142515d584fe19263fb83ebc9.tar.gz |
Check during SPI bitbang init that SPI bitbang master configuration is correct
Corresponding to flashrom svn r1092.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Michael Karcher <flashrom@mkarcher.dialup.fu-berlin.de>
Diffstat (limited to 'bitbang_spi.c')
-rw-r--r-- | bitbang_spi.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/bitbang_spi.c b/bitbang_spi.c index 9f0e878..b58633a 100644 --- a/bitbang_spi.c +++ b/bitbang_spi.c @@ -54,6 +54,16 @@ static int bitbang_spi_get_miso(void) int bitbang_spi_init(const struct bitbang_spi_master *master, int halfperiod) { + /* BITBANG_SPI_INVALID is 0, so if someone forgot to initialize ->type, + * we catch it here. Same goes for missing initialization of bitbanging + * functions. + */ + if (!master || master->type == BITBANG_SPI_INVALID || !master->set_cs || + !master->set_sck || !master->set_mosi || !master->get_miso) { + msg_perr("Incomplete bitbanging SPI master setting! Please " + "report a bug at flashrom@flashrom.org\n"); + return 1; + } bitbang_spi_master = master; bitbang_spi_half_period = halfperiod; |