From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v3 10/13] OMAP: DMA: Convert DMA library into DMA platform Driver Date: Wed, 10 Nov 2010 08:24:00 -0800 Message-ID: <87sjz9cfkv.fsf@deeprootsystems.com> References: <1288099513-1854-1-git-send-email-manjugk@ti.com> <1288099513-1854-11-git-send-email-manjugk@ti.com> <8739rakubj.fsf@deeprootsystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pv0-f174.google.com ([74.125.83.174]:39245 "EHLO mail-pv0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756497Ab0KJQYF (ORCPT ); Wed, 10 Nov 2010 11:24:05 -0500 Received: by pva4 with SMTP id 4so144488pva.19 for ; Wed, 10 Nov 2010 08:24:02 -0800 (PST) In-Reply-To: (Manjunath Kondaiah G.'s message of "Wed, 10 Nov 2010 19:32:14 +0530") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "G, Manjunath Kondaiah" Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "Cousson, Benoit" , "Shilimkar, Santosh" "G, Manjunath Kondaiah" writes: >> -----Original Message----- >> From: Kevin Hilman [mailto:khilman@deeprootsystems.com] >> Sent: Wednesday, November 10, 2010 3:56 AM >> To: G, Manjunath Kondaiah >> Cc: linux-omap@vger.kernel.org; >> linux-arm-kernel@lists.infradead.org; Cousson, Benoit; >> Shilimkar, Santosh >> Subject: Re: [PATCH v3 10/13] OMAP: DMA: Convert DMA library >> into DMA platform Driver >> >> "G, Manjunath Kondaiah" writes: >> >> > Convert DMA library into DMA platform driver and make use >> > of platform data provided by HWMOD data base for OMAP2PLUS onwards. >> > For OMAP1 processors, the DMA driver in mach-omap uses resource >> > structures for getting platform data. >> > >> > Signed-off-by: G, Manjunath Kondaiah >> > Cc: Benoit Cousson >> > Cc: Kevin Hilman >> > Cc: Santosh Shilimkar >> >> [...] >> >> > diff --git a/arch/arm/mach-omap2/include/mach/dma.h >> b/arch/arm/mach-omap2/include/mach/dma.h >> > index d0a7d5b..d13c5c0 100644 >> > --- a/arch/arm/mach-omap2/include/mach/dma.h >> > +++ b/arch/arm/mach-omap2/include/mach/dma.h >> > @@ -29,4 +29,7 @@ >> > #ifndef __ASM_ARCH_OMAP2_DMA_H >> > #define __ASM_ARCH_OMAP2_DMA_H >> > >> > +/* Should be part of hwmod data base ? */ >> > +#define OMAP_DMA4_LOGICAL_DMA_CH_COUNT 32 /* >> REVISIT: Is this 32 + 2? */ >> > + >> >> There are no users of this in this patch, all users are removed. > > This macro is used in hwmod data base. Do you mean hard coding this > value in hwmod db and remove header? What I mean is that from a reviewers perspective of this patch, you added a new #define that is not used anywhere in this patch. a #define should be added along with any users of it. That being said, the number of channels of the DMA IP sounds like something that should indeed be associated with the hwmod as it is a fixed HW feature. Kevin