mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Nicolin Chen <b42378@freescale.com>
Cc: <akpm@linux-foundation.org>, <joe@perches.com>, <nsekhar@ti.com>,
	<khilman@deeprootsystems.com>, <linux@arm.linux.org.uk>,
	<dan.j.williams@intel.com>, <vinod.koul@intel.com>,
	<m.chehab@samsung.com>, <hjk@hansjkoch.de>,
	<gregkh@linuxfoundation.org>, <perex@perex.cz>,
	<lgirdwood@gmail.com>, <broonie@kernel.org>,
	<rmk+kernel@arm.linux.org.uk>, <eric.y.miao@gmail.com>,
	<haojian.zhuang@gmail.com>, <linux-kernel@vger.kernel.org>,
	<davinci-linux-open-source@linux.davincidsp.com>,
	<linux-arm-kernel@lists.infradead.org>,
	<dmaengine@vger.kernel.org>, <linux-media@vger.kernel.org>,
	<alsa-devel@alsa-project.org>
Subject: Re: [PATCH][RESEND 6/8] ALSA: memalloc: use gen_pool_dma_alloc() to allocate iram buffer
Date: Mon, 04 Nov 2013 09:49:58 +0100	[thread overview]
Message-ID: <s5h38ncsg95.wl%tiwai@suse.de> (raw)
In-Reply-To: <db9e59951cdcf55b4903c0358ebe220e58155bc1.1383306365.git.b42378@freescale.com>

At Fri, 1 Nov 2013 19:48:19 +0800,
Nicolin Chen wrote:
> 
> Since gen_pool_dma_alloc() is introduced, we implement it to simplify code.
> 
> Signed-off-by: Nicolin Chen <b42378@freescale.com>

Acked-by: Takashi Iwai <tiwai@suse.de>


> ---
>  sound/core/memalloc.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/sound/core/memalloc.c b/sound/core/memalloc.c
> index 9d93f02..5e1c7bc 100644
> --- a/sound/core/memalloc.c
> +++ b/sound/core/memalloc.c
> @@ -184,11 +184,7 @@ static void snd_malloc_dev_iram(struct snd_dma_buffer *dmab, size_t size)
>  	/* Assign the pool into private_data field */
>  	dmab->private_data = pool;
>  
> -	dmab->area = (void *)gen_pool_alloc(pool, size);
> -	if (!dmab->area)
> -		return;
> -
> -	dmab->addr = gen_pool_virt_to_phys(pool, (unsigned long)dmab->area);
> +	dmab->area = gen_pool_dma_alloc(pool, size, &dmab->addr);
>  }
>  
>  /**
> -- 
> 1.8.4
> 
> 

  reply	other threads:[~2013-11-04  8:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-01 11:48 [PATCH][RESEND 0/8] Add and implement gen_pool_dma_alloc() Nicolin Chen
2013-11-01 11:48 ` [PATCH][RESEND 1/8] lib/genalloc: add a helper function for DMA buffer allocation Nicolin Chen
2013-11-01 11:48 ` [PATCH][RESEND 2/8] ARM: davinci: use gen_pool_dma_alloc() to sram.c Nicolin Chen
2013-11-01 11:48 ` [PATCH][RESEND 3/8] dma: mmp_tdma: use gen_pool_dma_alloc() to allocate descriptor Nicolin Chen
2013-11-12  3:50   ` Vinod Koul
2013-11-01 11:48 ` [PATCH][RESEND 4/8] [media] coda: use gen_pool_dma_alloc() to allocate iram buffer Nicolin Chen
2013-11-01 11:48 ` [PATCH][RESEND 5/8] uio: uio_pruss: use gen_pool_dma_alloc() to allocate sram memory Nicolin Chen
2013-12-09  1:39   ` Greg KH
2013-11-01 11:48 ` [PATCH][RESEND 6/8] ALSA: memalloc: use gen_pool_dma_alloc() to allocate iram buffer Nicolin Chen
2013-11-04  8:49   ` Takashi Iwai [this message]
2013-11-01 11:48 ` [PATCH][RESEND 7/8] ASoC: davinci: use gen_pool_dma_alloc() in davinci-pcm.c Nicolin Chen
2013-11-01 18:07   ` Mark Brown
2013-11-01 11:48 ` [PATCH][RESEND 8/8] ASoC: pxa: use gen_pool_dma_alloc() to allocate dma buffer Nicolin Chen
2013-11-01 18:08   ` Mark Brown

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=s5h38ncsg95.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=b42378@freescale.com \
    --cc=broonie@kernel.org \
    --cc=dan.j.williams@intel.com \
    --cc=davinci-linux-open-source@linux.davincidsp.com \
    --cc=dmaengine@vger.kernel.org \
    --cc=eric.y.miao@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=haojian.zhuang@gmail.com \
    --cc=hjk@hansjkoch.de \
    --cc=joe@perches.com \
    --cc=khilman@deeprootsystems.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=m.chehab@samsung.com \
    --cc=nsekhar@ti.com \
    --cc=perex@perex.cz \
    --cc=rmk+kernel@arm.linux.org.uk \
    --cc=vinod.koul@intel.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

all inboxes | Powered by JetHome®