Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1210677 > unrolled thread

Re: [PATCH 4/9] spi: sun4i: add DMA support

Started byMark Brown <broonie@kernel.org>
First post2015-08-20 21:00 +0200
Last post2015-08-20 21:00 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 4/9] spi: sun4i: add DMA support Mark Brown <broonie@kernel.org> - 2015-08-20 21:00 +0200

#1210677 — Re: [PATCH 4/9] spi: sun4i: add DMA support

FromMark Brown <broonie@kernel.org>
Date2015-08-20 21:00 +0200
SubjectRe: [PATCH 4/9] spi: sun4i: add DMA support
Message-ID<pZDe9-50e-7@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

On Thu, Aug 20, 2015 at 02:19:46PM -0000, Emilio López wrote:

> -	sun4i_spi_drain_fifo(sspi, SUN4I_FIFO_DEPTH);
> +	if (sun4i_spi_can_dma(master, spi, tfr) && desc_rx) {
> +		/* The receive transfer should be the last one to finish */
> +		dma_wait_for_async_tx(desc_rx);

What if it's a transmit only transfer?  We'll fall over to this...

> +	} else {
> +		sun4i_spi_drain_fifo(sspi, SUN4I_FIFO_DEPTH);
> +	}

...which manually reads data from the FIFO which doesn't seem like what
we want, won't it conflict with the DMA?

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web