diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2015-02-27 10:34:15 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-03-06 14:41:56 -0500 |
commit | b716c4ffc6a2b0bfbcf9619880f335be11b65708 (patch) | |
tree | e9fce46a5b9a33fcfc9593763b78f6bc46c787ea | |
parent | c517d838eb7d07bbe9507871fab3931deccff539 (diff) |
spi: introduce master->handle_err() callback
This callback would be useful to handle an error that occurs in the generic
implementation of transfer_one_message(). The good candidate for this is to
drain FIFO and / or to terminate DMA transfers when timeout happened.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | drivers/spi/spi.c | 3 | ||||
-rw-r--r-- | include/linux/spi/spi.h | 4 |
2 files changed, 7 insertions, 0 deletions
diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index c64a3e59fce3..31d4d9d997e2 100644 --- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c | |||
@@ -851,6 +851,9 @@ out: | |||
851 | if (msg->status == -EINPROGRESS) | 851 | if (msg->status == -EINPROGRESS) |
852 | msg->status = ret; | 852 | msg->status = ret; |
853 | 853 | ||
854 | if (msg->status) | ||
855 | master->handle_err(master, msg); | ||
856 | |||
854 | spi_finalize_current_message(master); | 857 | spi_finalize_current_message(master); |
855 | 858 | ||
856 | return ret; | 859 | return ret; |
diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h index ed9489d893a4..4eaac3a5227b 100644 --- a/include/linux/spi/spi.h +++ b/include/linux/spi/spi.h | |||
@@ -294,6 +294,8 @@ static inline void spi_unregister_driver(struct spi_driver *sdrv) | |||
294 | * transfer_one_message are mutually exclusive; when both | 294 | * transfer_one_message are mutually exclusive; when both |
295 | * are set, the generic subsystem does not call your | 295 | * are set, the generic subsystem does not call your |
296 | * transfer_one callback. | 296 | * transfer_one callback. |
297 | * @handle_err: the subsystem calls the driver to handle and error that occurs | ||
298 | * in the generic implementation of transfer_one_message(). | ||
297 | * @unprepare_message: undo any work done by prepare_message(). | 299 | * @unprepare_message: undo any work done by prepare_message(). |
298 | * @cs_gpios: Array of GPIOs to use as chip select lines; one per CS | 300 | * @cs_gpios: Array of GPIOs to use as chip select lines; one per CS |
299 | * number. Any individual value may be -ENOENT for CS lines that | 301 | * number. Any individual value may be -ENOENT for CS lines that |
@@ -448,6 +450,8 @@ struct spi_master { | |||
448 | void (*set_cs)(struct spi_device *spi, bool enable); | 450 | void (*set_cs)(struct spi_device *spi, bool enable); |
449 | int (*transfer_one)(struct spi_master *master, struct spi_device *spi, | 451 | int (*transfer_one)(struct spi_master *master, struct spi_device *spi, |
450 | struct spi_transfer *transfer); | 452 | struct spi_transfer *transfer); |
453 | void (*handle_err)(struct spi_master *master, | ||
454 | struct spi_message *message); | ||
451 | 455 | ||
452 | /* gpio chip select */ | 456 | /* gpio chip select */ |
453 | int *cs_gpios; | 457 | int *cs_gpios; |