| Message ID | 1371193161-25224-1-git-send-email-nsekhar@ti.com |
|---|---|
| State | New |
| Headers | show |
On Fri, Jun 14, 2013 at 12:29:20PM +0530, Sekhar Nori wrote: > On the DMA library movement, I was been waiting for acks > from Russell and Mark, but did not get them. OTOH, there > have been no objections as well. I assume the movement > is not a problem since its a movement within arch/arm > and does not end-up bringing new code in. I'm not sure I've actually seen this code? Is this the result of these endless threads about MMC that I keep on getting spammed with - if it is I did ask you guys to let me know when you'd actually figured out what you were doing. Given that there's subsystem work on DMA within ASoC recently I'd really rather this got reviewed.
On 6/14/2013 3:03 PM, Mark Brown wrote: > On Fri, Jun 14, 2013 at 12:29:20PM +0530, Sekhar Nori wrote: > >> On the DMA library movement, I was been waiting for acks >> from Russell and Mark, but did not get them. OTOH, there >> have been no objections as well. I assume the movement >> is not a problem since its a movement within arch/arm >> and does not end-up bringing new code in. > > I'm not sure I've actually seen this code? Is this the result of these > endless threads about MMC that I keep on getting spammed with - if it is > I did ask you guys to let me know when you'd actually figured out what > you were doing. Given that there's subsystem work on DMA within ASoC > recently I'd really rather this got reviewed. Mark, I will forward the patch to you separately (you were copied on it). This one is not really related to MMC. Its some (I think) trivial changes in sound/soc to go with the dma code movement. Thanks, Sekhar
On Fri, Jun 14, 2013 at 04:12:19PM +0530, Sekhar Nori wrote: > On 6/14/2013 3:03 PM, Mark Brown wrote: > > I'm not sure I've actually seen this code? Is this the result of these > > endless threads about MMC that I keep on getting spammed with - if it is > > I did ask you guys to let me know when you'd actually figured out what > > you were doing. Given that there's subsystem work on DMA within ASoC > > recently I'd really rather this got reviewed. > Mark, I will forward the patch to you separately (you were copied on > it). This one is not really related to MMC. Its some (I think) trivial > changes in sound/soc to go with the dma code movement. I know the patch itself wasn't but IIRC the bits that caused the endless repostings and discussions were - if it's the series I think it is it's been floating around for months.
On Fri, Jun 14, 2013 at 12:29:20PM +0530, Sekhar Nori wrote: > > Hi Arnd, Olof, > > Please pull the following DaVinci SoC updates for v3.10. > > On the DMA library movement, I was been waiting for acks > from Russell and Mark, but did not get them. OTOH, there > have been no objections as well. I assume the movement > is not a problem since its a movement within arch/arm > and does not end-up bringing new code in. Ok. I'll hold off since Mark was interested in seeing the code. Please resend once you have acks. -Olof
On Fri, Jun 14, 2013 at 03:25:19PM -0700, Olof Johansson wrote: > Ok. I'll hold off since Mark was interested in seeing the code. Please resend > once you have acks. It should be OK to go. I did actually find a bug when reviewing but the fix was to remove the offending driver since it's been broken for years.
Mark, On 6/17/2013 2:36 PM, Mark Brown wrote: > On Fri, Jun 14, 2013 at 03:25:19PM -0700, Olof Johansson wrote: > >> Ok. I'll hold off since Mark was interested in seeing the code. Please resend >> once you have acks. > > It should be OK to go. I did actually find a bug when reviewing but the > fix was to remove the offending driver since it's been broken for years. Since I have sent you a patch removing SFFSDR support, I will also re-spin this pull request to drop those hunks. I will send the patch out shortly, please do ack it. Thanks, Sekhar
Hi Arnd, Olof, Please pull the following DaVinci SoC updates for v3.10. On the DMA library movement, I was been waiting for acks from Russell and Mark, but did not get them. OTOH, there have been no objections as well. I assume the movement is not a problem since its a movement within arch/arm and does not end-up bringing new code in. Thanks, Sekhar The following changes since commit d683b96b072dc4680fc74964eca77e6a23d1fa6e: Linux 3.10-rc4 (2013-06-02 17:11:17 +0900) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git tags/davinci-for-v3.11/soc for you to fetch changes up to b50f23734903c8d33c9e08d93649a3ca806c90dc: ARM: edma: remove unused transfer controller handlers (2013-06-14 10:54:04 +0530) ---------------------------------------------------------------- This pull request moves DaVinci EDMA library to arch/arm/common so it can be used by OMAP based AM335x. This is a temporary step until all drivers are converted to use the dmaengine driver in drivers/dma/edma.c. Several drivers like SPI, MMC/SD have already been converted. Some line audio are pending. The other two patches in the pull request are cleanup in nature. ---------------------------------------------------------------- Lad, Prabhakar (1): ARM: davinci: remove __init atrribute from function declaration Matt Porter (2): ARM: davinci: move private EDMA API to arm/common ARM: edma: remove unused transfer controller handlers arch/arm/Kconfig | 1 + arch/arm/common/Kconfig | 3 + arch/arm/common/Makefile | 1 + arch/arm/{mach-davinci/dma.c => common/edma.c} | 39 +-------- arch/arm/mach-davinci/Makefile | 2 +- arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- arch/arm/mach-davinci/davinci.h | 30 +++---- arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- arch/arm/mach-davinci/devices.c | 6 +- arch/arm/mach-davinci/dm355.c | 2 +- arch/arm/mach-davinci/dm365.c | 2 +- arch/arm/mach-davinci/dm644x.c | 2 +- arch/arm/mach-davinci/dm646x.c | 2 +- arch/arm/mach-davinci/include/mach/cp_intc.h | 4 +- arch/arm/mach-davinci/include/mach/da8xx.h | 18 ++-- arch/arm/mach-davinci/include/mach/tnetv107x.h | 6 +- drivers/dma/edma.c | 2 +- drivers/mmc/host/davinci_mmc.c | 1 + include/linux/mfd/davinci_voicecodec.h | 3 +- .../mach => include/linux/platform_data}/edma.h | 89 +------------------- include/linux/platform_data/spi-davinci.h | 2 +- sound/soc/davinci/davinci-evm.c | 1 + sound/soc/davinci/davinci-pcm.c | 1 + sound/soc/davinci/davinci-pcm.h | 2 +- sound/soc/davinci/davinci-sffsdr.c | 5 +- 25 files changed, 60 insertions(+), 168 deletions(-) rename arch/arm/{mach-davinci/dma.c => common/edma.c} (97%) rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%)