alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Lars-Peter Clausen <lars@metafoo.de>
To: Vasily Khoruzhick <anarsoul@gmail.com>
Cc: alsa-devel@alsa-project.org, Kukjin Kim <kgene.kim@samsung.com>,
	Arnd Bergmann <arnd@arndb.de>, Sangbeom Kim <sbkim73@samsung.com>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>
Subject: Re: [PATCH 1/2] ASoC: samsung: s3c24{xx, 12}-i2s: port to use generic dmaengine API
Date: Tue, 20 May 2014 11:31:34 +0200	[thread overview]
Message-ID: <537B20F6.1010108@metafoo.de> (raw)
In-Reply-To: <CA+E=qVddxDAZ6Fkx5aBy1AL7xbpe64Yet4utTqfsSScQUxpXLA@mail.gmail.com>

On 05/20/2014 11:25 AM, Vasily Khoruzhick wrote:
> On Mon, May 19, 2014 at 11:34 PM, Lars-Peter Clausen <lars@metafoo.de> wrote:
>> On 05/19/2014 10:18 PM, Vasily Khoruzhick wrote:
>>>
>>> On Mon, May 19, 2014 at 8:11 PM, Vasily Khoruzhick <anarsoul@gmail.com>
>>> wrote:
>>>>
>>>> Use dmaengine instead of legacy s3c24xx DMA API for s3c24xx and s3c2412
>>>
>>>
>>> Please discard this series, I'd like to use
>>> dma_request_slave_channel_compat() to request a dma channel here,
>>> to simpify migration to DT later.
>>
>>
>> When you use the generic DMAengine PCM driver, migration to DT essentially
>> comes for free, you don't need to do anything special other than not
>> specifying the NO_DT flag.
>>
>> - Lars
>
> Hi Lars, all,
>
> But how do I have to pass DMA channel number for non-DT boot? I've
> seen that s3c64xx uses DMA resources at least for I2S and SPI,
> but Arnd suggested to use platform data. What's preffered way?

I don't think it matters, both isn't really that nice, but we currently do 
not have a nice way to specify the DMA channel on non-DT platforms. What's 
missing the a equivalent of the GPIO descriptor lookup tables, but for DMA 
channels.

- Lars

      reply	other threads:[~2014-05-20  9:31 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-19 17:11 [PATCH 1/2] ASoC: samsung: s3c24{xx, 12}-i2s: port to use generic dmaengine API Vasily Khoruzhick
2014-05-19 17:11 ` [PATCH 2/2] ASoC: samsung: drop support for legacy S3C24XX DMA API Vasily Khoruzhick
2014-05-19 20:18 ` [PATCH 1/2] ASoC: samsung: s3c24{xx, 12}-i2s: port to use generic dmaengine API Vasily Khoruzhick
2014-05-19 20:34   ` Lars-Peter Clausen
2014-05-20  9:25     ` Vasily Khoruzhick
2014-05-20  9:31       ` Lars-Peter Clausen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=537B20F6.1010108@metafoo.de \
    --to=lars@metafoo.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=anarsoul@gmail.com \
    --cc=arnd@arndb.de \
    --cc=broonie@kernel.org \
    --cc=kgene.kim@samsung.com \
    --cc=lgirdwood@gmail.com \
    --cc=sbkim73@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).