From 24151963f36a4d1dac41d367bf1178e9de567a1b Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Fri, 24 Aug 2018 22:45:50 +0000 Subject: Revert "drivers/spi/spi_flash: don't allocate unbounded stack memory" This reverts commit c5ee35ff861fe4447fd80119f645fba7bfd3a184. Reason for revert: breaks boards, uncertain if it _really_ works. Change-Id: I9ba2ba877b9a391306f89295c0c1d0e2d011c5ea Signed-off-by: Patrick Georgi Reviewed-on: https://review.coreboot.org/28338 Reviewed-by: Aaron Durbin Reviewed-by: Jean Lucas Tested-by: build bot (Jenkins) --- src/drivers/spi/spi_flash.c | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-) (limited to 'src') diff --git a/src/drivers/spi/spi_flash.c b/src/drivers/spi/spi_flash.c index 82333e9b0f..f2714791db 100644 --- a/src/drivers/spi/spi_flash.c +++ b/src/drivers/spi/spi_flash.c @@ -82,29 +82,24 @@ static int spi_flash_cmd_read(const struct spi_slave *spi, const u8 *cmd, return ret; } +/* TODO: This code is quite possibly broken and overflowing stacks. Fix ASAP! */ +#pragma GCC diagnostic push +#if defined(__GNUC__) && !defined(__clang__) +#pragma GCC diagnostic ignored "-Wstack-usage=" +#endif int spi_flash_cmd_write(const struct spi_slave *spi, const u8 *cmd, size_t cmd_len, const void *data, size_t data_len) { - int ret = 1; - struct spi_op vectors[] = { - [0] = { .dout = cmd, .bytesout = cmd_len, - .din = NULL, .bytesin = 0, }, - [1] = { .dout = data, .bytesout = data_len, - .din = NULL, .bytesin = 0, } - }; - size_t count = ARRAY_SIZE(vectors); - - if (spi_claim_bus(spi)) - return ret; - - if (spi_xfer_vector(spi, vectors, count) == 0) - ret = 0; + int ret; + u8 buff[cmd_len + data_len]; + memcpy(buff, cmd, cmd_len); + memcpy(buff + cmd_len, data, data_len); + ret = do_spi_flash_cmd(spi, buff, cmd_len + data_len, NULL, 0); if (ret) { printk(BIOS_WARNING, "SF: Failed to send write command (%zu bytes): %d\n", data_len, ret); } - spi_release_bus(spi); return ret; } -- cgit v1.2.3