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


Groups > linux.kernel > #1210677

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

Path csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From Mark Brown <broonie@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH 4/9] spi: sun4i: add DMA support
Date Thu, 20 Aug 2015 21:00:02 +0200
Message-ID <pZDe9-50e-7@gated-at.bofh.it> (permalink)
References <pZz0R-7wx-11@gated-at.bofh.it> <pZz0R-7wx-13@gated-at.bofh.it>
X-Original-To Emilio López <emilio@elopez.com.ar>
MIME-Version 1.0
Content-Type multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ngPZezdD7QsvFaqQ"
Content-Disposition inline
X-Cookie You are fairminded, just and loving.
User-Agent Mutt/1.5.23 (2014-03-12)
X-Sa-Exim-Connect-IP 69.84.245.29
X-Sa-Exim-Mail-From broonie@sirena.org.uk
X-Sa-Exim-Version 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000)
X-Sa-Exim-Scanned Yes (on mezzanine.sirena.org.uk)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 44
Organization linux.* mail to news gateway
X-Original-Cc linux-sunxi@googlegroups.com, Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, Russell King <linux@arm.linux.org.uk>, Maxime Ripard <maxime.ripard@free-electrons.com>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org
X-Original-Date Thu, 20 Aug 2015 11:58:50 -0700
X-Original-Message-ID <20150820185850.GQ12027@sirena.org.uk>
X-Original-References <cover.1440080122.git.hramrach@gmail.com> <ccf776869b0d7fe2c78bcc41d6cd1896bf732296.1440080122.git.hramrach@gmail.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1210677

Show key headers only | View raw


[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?

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

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

csiph-web