From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752511AbbCZLB1 (ORCPT ); Thu, 26 Mar 2015 07:01:27 -0400 Received: from mga01.intel.com ([192.55.52.88]:5033 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751402AbbCZLBZ (ORCPT ); Thu, 26 Mar 2015 07:01:25 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,471,1422950400"; d="scan'208";a="698104214" Date: Thu, 26 Mar 2015 16:27:12 +0530 From: Vinod Koul To: Peter Ujfalusi Cc: tony@atomide.com, linux@arm.linux.org.uk, grant.likely@linaro.org, dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, robh+dt@kernel.org, nm@ti.com Subject: Re: [PATCH v2 4/7] dmaengine: omap-dma: Use defines for dma channels and request count Message-ID: <20150326105712.GL32683@intel.com> References: <1426080210-841-1-git-send-email-peter.ujfalusi@ti.com> <1426080210-841-5-git-send-email-peter.ujfalusi@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1426080210-841-5-git-send-email-peter.ujfalusi@ti.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 11, 2015 at 03:23:27PM +0200, Peter Ujfalusi wrote: > Instead of magic numbers in the code, use define for number of logical DMA > channels and DMA requests. > > Signed-off-by: Peter Ujfalusi > --- > drivers/dma/omap-dma.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/dma/omap-dma.c b/drivers/dma/omap-dma.c > index 7dd6dd121681..56c33e93dd24 100644 > --- a/drivers/dma/omap-dma.c > +++ b/drivers/dma/omap-dma.c > @@ -22,6 +22,9 @@ > > #include "virt-dma.h" > > +#define OMAP_SDMA_REQUESTS 127 > +#define OMAP_SDMA_CHANNELS 32 Again why dont we put these things in DT ? -- ~Vinod > + > struct omap_dmadev { > struct dma_device ddev; > spinlock_t lock; > @@ -33,7 +36,7 @@ struct omap_dmadev { > bool legacy; > spinlock_t irq_lock; > uint32_t irq_enable_mask; > - struct omap_chan *lch_map[32]; > + struct omap_chan *lch_map[OMAP_SDMA_CHANNELS]; > }; > > struct omap_chan { > @@ -1115,7 +1118,7 @@ static int omap_dma_probe(struct platform_device *pdev) > > tasklet_init(&od->task, omap_dma_sched, (unsigned long)od); > > - for (i = 0; i < 127; i++) { > + for (i = 0; i < OMAP_SDMA_REQUESTS; i++) { > rc = omap_dma_chan_init(od, i); > if (rc) { > omap_dma_free(od); > -- > 2.3.0 > --