* [PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word.
@ 2015-10-19 20:54 David Mosberger-Tang
2015-10-20 10:22 ` Nicolas Ferre
2015-10-20 12:26 ` [RESEND PATCH] " Nicolas Ferre
0 siblings, 2 replies; 6+ messages in thread
From: David Mosberger-Tang @ 2015-10-19 20:54 UTC (permalink / raw)
To: linux-arm-kernel
The DMA-slave configuration depends on the whether <= 8 or > 8 bits
are transferred per word, so we need to call
atmel_spi_dma_slave_config() with the correct value.
Signed-off-by: David Mosberger <davidm@egauge.net>
---
drivers/spi/spi-atmel.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c
index 04e48e5..855cc56 100644
--- a/drivers/spi/spi-atmel.c
+++ b/drivers/spi/spi-atmel.c
@@ -758,7 +758,8 @@ static int atmel_spi_next_xfer_dma_submit(struct spi_master *master,
*plen = len;
- if (atmel_spi_dma_slave_config(as, &slave_config, 8))
+ if (atmel_spi_dma_slave_config(as, &slave_config,
+ xfer->bits_per_word))
goto err_exit;
/* Send both scatterlists */
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word.
2015-10-19 20:54 [PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word David Mosberger-Tang
@ 2015-10-20 10:22 ` Nicolas Ferre
2015-10-20 10:27 ` Mark Brown
2015-10-20 12:26 ` [RESEND PATCH] " Nicolas Ferre
1 sibling, 1 reply; 6+ messages in thread
From: Nicolas Ferre @ 2015-10-20 10:22 UTC (permalink / raw)
To: linux-arm-kernel
Le 19/10/2015 22:54, David Mosberger-Tang a ?crit :
> The DMA-slave configuration depends on the whether <= 8 or > 8 bits
> are transferred per word, so we need to call
> atmel_spi_dma_slave_config() with the correct value.
>
> Signed-off-by: David Mosberger <davidm@egauge.net>
Thanks for having rewritten this patch! It's much more simple like this ;-)
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
For Mark, here is the patchwork entry which can be handy:
https://patchwork.kernel.org/patch/7439931/
Best regards,
> ---
> drivers/spi/spi-atmel.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c
> index 04e48e5..855cc56 100644
> --- a/drivers/spi/spi-atmel.c
> +++ b/drivers/spi/spi-atmel.c
> @@ -758,7 +758,8 @@ static int atmel_spi_next_xfer_dma_submit(struct spi_master *master,
>
> *plen = len;
>
> - if (atmel_spi_dma_slave_config(as, &slave_config, 8))
> + if (atmel_spi_dma_slave_config(as, &slave_config,
> + xfer->bits_per_word))
> goto err_exit;
>
> /* Send both scatterlists */
>
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 6+ messages in thread
* [RESEND PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word
2015-10-19 20:54 [PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word David Mosberger-Tang
2015-10-20 10:22 ` Nicolas Ferre
@ 2015-10-20 12:26 ` Nicolas Ferre
1 sibling, 0 replies; 6+ messages in thread
From: Nicolas Ferre @ 2015-10-20 12:26 UTC (permalink / raw)
To: linux-arm-kernel
From: David Mosberger-Tang <davidm@egauge.net>
The DMA-slave configuration depends on the whether <= 8 or > 8 bits
are transferred per word, so we need to call
atmel_spi_dma_slave_config() with the correct value.
Signed-off-by: David Mosberger <davidm@egauge.net>
Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
drivers/spi/spi-atmel.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c
index 1cc1f1e0d17f..0b6e5ecbc752 100644
--- a/drivers/spi/spi-atmel.c
+++ b/drivers/spi/spi-atmel.c
@@ -773,7 +773,8 @@ static int atmel_spi_next_xfer_dma_submit(struct spi_master *master,
*plen = len;
- if (atmel_spi_dma_slave_config(as, &slave_config, 8))
+ if (atmel_spi_dma_slave_config(as, &slave_config,
+ xfer->bits_per_word))
goto err_exit;
/* Send both scatterlists */
--
2.1.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word.
@ 2015-10-19 20:48 David Mosberger-Tang
0 siblings, 0 replies; 6+ messages in thread
From: David Mosberger-Tang @ 2015-10-19 20:48 UTC (permalink / raw)
To: linux-arm-kernel
The DMA-slave configuration depends on the whether <= 8 or > 8 bits
are transferred per word, so we need to call
atmel_spi_dma_slave_config() with the correct value.
Signed-off-by: David Mosberger <davidm@egauge.net>
---
drivers/spi/spi-atmel.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c
index 04e48e5..855cc56 100644
--- a/drivers/spi/spi-atmel.c
+++ b/drivers/spi/spi-atmel.c
@@ -758,7 +758,8 @@ static int atmel_spi_next_xfer_dma_submit(struct spi_master *master,
*plen = len;
- if (atmel_spi_dma_slave_config(as, &slave_config, 8))
+ if (atmel_spi_dma_slave_config(as, &slave_config,
+ xfer->bits_per_word))
goto err_exit;
/* Send both scatterlists */
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-10-20 12:26 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-19 20:54 [PATCH] spi: atmel: Fix DMA-setup for transfers with more than 8 bits per word David Mosberger-Tang
2015-10-20 10:22 ` Nicolas Ferre
2015-10-20 10:27 ` Mark Brown
2015-10-20 12:26 ` Nicolas Ferre
2015-10-20 12:26 ` [RESEND PATCH] " Nicolas Ferre
-- strict thread matches above, loose matches on Subject: below --
2015-10-19 20:48 [PATCH] " David Mosberger-Tang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).