From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cyril Chemparathy Subject: Re: [PATCH 6/8] netdev: separate out davinci cpdma controller code Date: Tue, 03 Aug 2010 20:46:41 -0400 Message-ID: <4C58B871.8070107@ti.com> References: <1280842313-3743-1-git-send-email-cyril@ti.com> <1280842313-3743-7-git-send-email-cyril@ti.com> <27F9C60D11D683428E133F85D2BB4A53043F3C8C2B@dlee03.ent.ti.com> Reply-To: cyril@ti.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: "netdev@vger.kernel.org" , "davinci-linux-open-source@linux.davincidsp.com" , "linux-omap@vger.kernel.org" , "Nori, Sekhar" , "Govindarajan, Sriramakrishnan" To: "Ramirez Luna, Omar" Return-path: In-Reply-To: <27F9C60D11D683428E133F85D2BB4A53043F3C8C2B@dlee03.ent.ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi Omar, [...] >> + if (ctlr->channels[chan_num]) >> + goto err_chan_busy; > > Mem leaking 'chan' > Thanks. Will modify for v2. Regards Cyril.