On Wed, 15 Dec 2010, G, Manjunath Kondaiah wrote:
> On Wed, Dec 15, 2010 at 04:39:17PM +0530, G, Manjunath Kondaiah wrote:
> > On Tue, Dec 14, 2010 at 07:25:23PM -0700, Paul Walmsley wrote:
> > > On Sat, 4 Dec 2010, G, Manjunath Kondaiah wrote:
> > >
> > > > Add OMAP2420 DMA hwmod data and also add required
> > > > DMA device attributes.
> > > >
> > > > Signed-off-by: G, Manjunath Kondaiah <manjugk@xxxxxx>
> > > > Cc: Benoit Cousson <b-cousson@xxxxxx>
> > > > Cc: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
> > > > Cc: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> > > >
> > > > +/* dma_system -> L3 */
> > > > +static struct omap_hwmod_ocp_if omap2420_dma_system__l3 = {
> > > > + .master = &omap2420_dma_system_hwmod,
> > > > + .slave = &omap2420_l3_main_hwmod,
> > > > + .clk = "l3_div_ck",
> > >
> > > This clock does not exist on OMAP2420. Did you test this patch on 2420?
> >
> > ok. will be replaced with "sdma_ick". I don't have 2420 for testing. I
> > tested it on 2430. My understanding is that, DMA clock interface is same
> > for 2420 and 2430. Correct me if am wrong.
>
> Looking into omap3/omap4 structures, this entry takes core l3 clock.
>
> This should be "sdma_fck" since it's parent is core_l3_ck.
The OMAP2xxx SDMA data can use either sdma_fck or core_l3_ck for its
functional clock. At some point we will probably remove sdma_fck from the
2xxx data. That's a relic of the pre-clkdev days, so you might as well
just use core_l3_ck. Maybe just stick with sdma_ick for the interface
clock for right now.
- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
