> -----Original Message----- > From: Kevin Hilman [mailto:khil...@deeprootsystems.com] > Sent: Wednesday, November 10, 2010 9:54 PM > To: G, Manjunath Kondaiah > Cc: linux-omap@vger.kernel.org; > linux-arm-ker...@lists.infradead.org; Cousson, Benoit; > Shilimkar, Santosh > Subject: Re: [PATCH v3 10/13] OMAP: DMA: Convert DMA library > into DMA platform Driver > [...] > >> > +#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.
No. This macro is used in omap2+ hwmod db through plat/dma.h which includes mach/dma.h > > 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. If so, I can remove this macro and hard code in hwmod db. -Manjunath-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
