Commit ad71f40a authored by Mark Brown's avatar Mark Brown

Merge remote-tracking branch 'spi/topic/core' into spi-next

parents 62d02e41 c7908a37
...@@ -552,6 +552,9 @@ int spi_register_board_info(struct spi_board_info const *info, unsigned n) ...@@ -552,6 +552,9 @@ int spi_register_board_info(struct spi_board_info const *info, unsigned n)
struct boardinfo *bi; struct boardinfo *bi;
int i; int i;
if (!n)
return -EINVAL;
bi = kzalloc(n * sizeof(*bi), GFP_KERNEL); bi = kzalloc(n * sizeof(*bi), GFP_KERNEL);
if (!bi) if (!bi)
return -ENOMEM; return -ENOMEM;
...@@ -789,27 +792,35 @@ static int spi_transfer_one_message(struct spi_master *master, ...@@ -789,27 +792,35 @@ static int spi_transfer_one_message(struct spi_master *master,
list_for_each_entry(xfer, &msg->transfers, transfer_list) { list_for_each_entry(xfer, &msg->transfers, transfer_list) {
trace_spi_transfer_start(msg, xfer); trace_spi_transfer_start(msg, xfer);
reinit_completion(&master->xfer_completion); if (xfer->tx_buf || xfer->rx_buf) {
reinit_completion(&master->xfer_completion);
ret = master->transfer_one(master, msg->spi, xfer); ret = master->transfer_one(master, msg->spi, xfer);
if (ret < 0) { if (ret < 0) {
dev_err(&msg->spi->dev, dev_err(&msg->spi->dev,
"SPI transfer failed: %d\n", ret); "SPI transfer failed: %d\n", ret);
goto out; goto out;
} }
if (ret > 0) { if (ret > 0) {
ret = 0; ret = 0;
ms = xfer->len * 8 * 1000 / xfer->speed_hz; ms = xfer->len * 8 * 1000 / xfer->speed_hz;
ms += ms + 100; /* some tolerance */ ms += ms + 100; /* some tolerance */
ms = wait_for_completion_timeout(&master->xfer_completion, ms = wait_for_completion_timeout(&master->xfer_completion,
msecs_to_jiffies(ms)); msecs_to_jiffies(ms));
} }
if (ms == 0) { if (ms == 0) {
dev_err(&msg->spi->dev, "SPI transfer timed out\n"); dev_err(&msg->spi->dev,
msg->status = -ETIMEDOUT; "SPI transfer timed out\n");
msg->status = -ETIMEDOUT;
}
} else {
if (xfer->len)
dev_err(&msg->spi->dev,
"Bufferless transfer has length %u\n",
xfer->len);
} }
trace_spi_transfer_stop(msg, xfer); trace_spi_transfer_stop(msg, xfer);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment