Hi Tony,
If there are no further comments on below patches, can you please
push the patches

https://patchwork.kernel.org/patch/83540/
https://patchwork.kernel.org/patch/83541/

-Manjunath 

> -----Original Message-----
> From: G, Manjunath Kondaiah 
> Sent: Thursday, March 04, 2010 7:46 AM
> To: linux-omap@vger.kernel.org
> Cc: G, Manjunath Kondaiah; Tony Lindgren; Shilimkar, Santosh; 
> Raja, Govindraj; Kevin Hilman
> Subject: [PATCH v2] OMAP: DMA: Init CDAC to zero
> 
> The register DMA4_CDAC needs to be initialized to zero
> before starting DMA transfer.
> 
> Cc: Tony Lindgren <t...@atomide.com>
> Cc: Santosh Shilimkar <santosh.shilim...@ti.com>
> Cc: Govindraj R <govindraj.r...@ti.com>
> Cc: Kevin Hilman <khil...@deeprootsystems.com>
> Reported-by:S, Venkatraman <svenk...@ti.com>
> Signed-off-by: Manjunatha GK <manj...@ti.com>
> ---
> It was aligned to reset CDAC to zero in omap_start_dma(int lch)
> instead of creating new API for accessing CDAC register.
> 
> Discussion thread is at:
> http://patchwork.kernel.org/patch/83176/
> http://patchwork.kernel.org/patch/82948/
> 
> v2 changes:
> Fixed multiline comments as per CodingStyle
> 
>  arch/arm/plat-omap/dma.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/plat-omap/dma.c b/arch/arm/plat-omap/dma.c
> index 2ab224c..f6c9bdc 100644
> --- a/arch/arm/plat-omap/dma.c
> +++ b/arch/arm/plat-omap/dma.c
> @@ -936,6 +936,15 @@ void omap_start_dma(int lch)
>  {
>       u32 l;
>  
> +     /*
> +      * The CPC/CDAC register needs to be initialized to zero
> +      * before starting dma transfer.
> +      */
> +     if (cpu_is_omap15xx())
> +             dma_write(0, CPC(lch));
> +     else
> +             dma_write(0, CDAC(lch));
> +
>       if (!omap_dma_in_1510_mode() && dma_chan[lch].next_lch != -1) {
>               int next_lch, cur_lch;
>               char dma_chan_link_map[OMAP_DMA4_LOGICAL_DMA_CH_COUNT];
> -- 
> 1.6.3.3
> 
> --
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

Reply via email to