From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: Re: [patch] dma: cppi41: off by one in desc_to_chan() Date: Wed, 28 Aug 2013 15:46:27 -0700 Message-ID: <20130828224627.GA4281@kroah.com> References: <20130826083220.GM2748@intel.com> <20130828104844.GG19256@mwanda> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20130828104844.GG19256@mwanda> Sender: kernel-janitors-owner@vger.kernel.org To: Dan Carpenter Cc: Vinod Koul , Dan Williams , Grant Likely , Rob Herring , devicetree@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-usb@vger.kernel.org List-Id: devicetree@vger.kernel.org On Wed, Aug 28, 2013 at 01:48:44PM +0300, Dan Carpenter wrote: > The test here should be ">=" instead of ">". The cdd->chan_busy[] array > has "ALLOC_DECS_NUM" elements. > > Signed-off-by: Dan Carpenter > Acked-by: Vinod Koul > --- > Resending to Greg and the USB people because Vinod says this goes > through the USB tree. Really? Ok, that seems odd, but whatever...