Re: [PATCH 0/2 v5] dmaengine: rcar-audmapp: independent from SH_DMAE_BASE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent, Vinod

Thank you for your feedback

> > This time I added SoC/platform side setting patches too (= 3) - 6)).
> > SoC/platform side setting needs many entries for this rcar-audmapp,
> > because it has many combinations.
> > But, I believe this is very normal DMAEngine style, not special.
> 
> Vinod commented on 22/12/2014 (Message-ID: <20141222151447.GL16827@xxxxxxxxx>)
> 
> "I think this makes sense. Going thru the driver, it was clear that we were
> not really gaining anything for using dmaengine API here. So agree that lets
> use dmanegine for 1st dmac thru dmaengine library and then configure this in
> your sound driver.."
> 
> My understanding is that a solution specific to the sound driver was 
> preferred, instead of a generic DMA engine driver. Have I missed something ?

Grr... my understanding was that
"1st DMAC will use dmaengine library (= sound framework has sound-dma-xxx functions)
 2nd DMAC will use normal dmaengine API"

But, I need to fixup sound driver ?
 - 1st DMAC = normal DMAEngine API
 - 2nd DMAC = part of sound driver

Sorry, for discuss it again here, but I want flexible switching
for 1st/2nd DMAC (because of 1st/2nd DMAC path combination).
So, using same DMAEngine interface from sound driver is easy to
implement/understanding.
 - 1st DMAC = normal DMAEngine API
 - 2nd DMAC = normal DMAEngine API

Best regards
---
Kuninori Morimoto
--
To unsubscribe from this list: send the line "unsubscribe dmaengine" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux