[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110729111538.GJ25640@n2100.arm.linux.org.uk>
Date: Fri, 29 Jul 2011 12:15:38 +0100
From: Russell King - ARM Linux <linux@....linux.org.uk>
To: Viresh Kumar <viresh.kumar@...com>
Cc: linus.walleij@...aro.org, vinod.koul@...el.com,
dan.j.williams@...el.com, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, armando.visconti@...com,
shiraz.hashim@...com, vipin.kumar@...com, rajeev-dlh.kumar@...com,
deepak.sikri@...com, vipulkumar.samar@...com, amit.virdi@...com,
pratyush.anand@...com, bhupesh.sharma@...com,
viresh.linux@...il.com
Subject: Re: [PATCH 18/18] dmaengine/amba-pl08x: Call pl08x_free_txd()
instead of calling kfree() directly
On Fri, Jul 29, 2011 at 04:19:28PM +0530, Viresh Kumar wrote:
> pl08x_prep_channel_resources() is calling kfree() directly for txd(). To
> maintain consistency in code call pl08x_free_txd() instead.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@...com>
> ---
> drivers/dma/amba-pl08x.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c
> index a72255c..b2a95ce 100644
> --- a/drivers/dma/amba-pl08x.c
> +++ b/drivers/dma/amba-pl08x.c
> @@ -1193,7 +1193,7 @@ static int pl08x_prep_channel_resources(struct pl08x_dma_chan *plchan,
>
> num_llis = pl08x_fill_llis_for_desc(pl08x, txd);
> if (!num_llis) {
> - kfree(txd);
> + pl08x_free_txd(pl08x, txd);
pl08x_free_txd() is supposed to be called under the channel spinlock,
which is why it isn't used here. We don't want to hold the spinlock
throughout the LLI filling because that could cause some problems,
and potentially worsen IRQ latency.
It's something which needs more work...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists