summaryrefslogtreecommitdiffstats
path: root/drivers/spi/spi-dw.c
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2015-03-09 15:48:46 +0100
committerMark Brown <broonie@kernel.org>2015-03-09 19:11:13 +0100
commit9f14538ecd1a210eff244a0a2281f6744fe4a59d (patch)
tree3fbe4c565d59def0182e970ab19e039d55c66ebf /drivers/spi/spi-dw.c
parentspi: dw-mid: convert value of dma_width to enum dma_slave_buswidth (diff)
downloadlinux-9f14538ecd1a210eff244a0a2281f6744fe4a59d.tar.xz
linux-9f14538ecd1a210eff244a0a2281f6744fe4a59d.zip
spi: dw-mid: split dma_setup() from dma_transfer()
The patch splits DMA preparatory code to dma_setup() callback. The change also converts transfer_one() to program DMA whenever the transfer is DMA mapped. The change is a follow up of the converion to use SPI core transfer_one_message(). Since the DMA mapped transfers can be interleaved with PIO ones the DMA related configuration should respect that. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-dw.c')
-rw-r--r--drivers/spi/spi-dw.c23
1 files changed, 13 insertions, 10 deletions
diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c
index f3e4092cd8dc..c7c2fcc2b58e 100644
--- a/drivers/spi/spi-dw.c
+++ b/drivers/spi/spi-dw.c
@@ -316,11 +316,11 @@ static int dw_spi_transfer_one(struct spi_master *master,
struct dw_spi *dws = spi_master_get_devdata(master);
struct chip_data *chip = spi_get_ctldata(spi);
u8 imask = 0;
- u8 cs_change = 0;
u16 txlevel = 0;
u16 clk_div = 0;
u32 speed = 0;
u32 cr0 = 0;
+ int ret;
dws->n_bytes = chip->n_bytes;
dws->dma_width = chip->dma_width;
@@ -332,8 +332,6 @@ static int dw_spi_transfer_one(struct spi_master *master,
dws->rx = transfer->rx_buf;
dws->rx_end = dws->rx + transfer->len;
dws->len = transfer->len;
- if (chip != dws->prev_chip)
- cs_change = 1;
spi_enable_chip(dws, 0);
@@ -397,7 +395,13 @@ static int dw_spi_transfer_one(struct spi_master *master,
* Interrupt mode
* we only need set the TXEI IRQ, as TX/RX always happen syncronizely
*/
- if (!dws->dma_mapped && !chip->poll_mode) {
+ if (dws->dma_mapped) {
+ ret = dws->dma_ops->dma_setup(dws);
+ if (ret < 0) {
+ spi_enable_chip(dws, 1);
+ return ret;
+ }
+ } else if (!chip->poll_mode) {
txlevel = min_t(u16, dws->fifo_len / 2, dws->len / dws->n_bytes);
dw_writew(dws, DW_SPI_TXFLTR, txlevel);
@@ -411,11 +415,11 @@ static int dw_spi_transfer_one(struct spi_master *master,
spi_enable_chip(dws, 1);
- if (cs_change)
- dws->prev_chip = chip;
-
- if (dws->dma_mapped)
- dws->dma_ops->dma_transfer(dws, cs_change);
+ if (dws->dma_mapped) {
+ ret = dws->dma_ops->dma_transfer(dws);
+ if (ret < 0)
+ return ret;
+ }
if (chip->poll_mode)
return poll_transfer(dws);
@@ -546,7 +550,6 @@ int dw_spi_add_host(struct device *dev, struct dw_spi *dws)
dws->master = master;
dws->type = SSI_MOTO_SPI;
- dws->prev_chip = NULL;
dws->dma_inited = 0;
dws->dma_addr = (dma_addr_t)(dws->paddr + 0x60);
snprintf(dws->name, sizeof(dws->name), "dw_spi%d", dws->bus_num);