Re: [PATCH][resend] dmaengine: rcar-audmapp: independent from SH_DMAE_BASE

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

 



Hi Vinod

> > > > Current Renesas Audio DMAC Peri Peri driver is for Renesas R-Car series,
> > > > but it is based on SH_DMAE_BASE driver which is used for Renesas
> > > > SH-Mobile. But it is not fully cared about DT.
> > > > 
> > > > For example, current SH_DMAE_BASE base driver will return non-matching
> > > > DMA channel if some non-SH_DMAE_BASE drivers are probed.
> > > > So, sound driver will get wrong DMA channel if Audio DMAC (= rcar-dma)
> > > > was not probed and Audio DMAC Peri Peri (= SH_DMAE_BASE) was probed.
> > > > 
> > > > OTOH, many SH-Mobile series drivers are using SH_DMAE_BASE driver, and
> > > > Renesas R-Car series will not use it anymore. Maintenance cost for fully
> > > > cared DT support on SH_DMAE_BASE will be very high
> > > > (and keeping compatibility will be very complex).
> > > > 
> > > > In addition, Audio DMAC Peri Peri itself is very simple device, and,
> > > > no SoC/board is usign it from non-DT environment.
> > > > 
> > > > This patch remakes rcar-audmapp driver simply independents
> > > > from SH_DMAE_BASE.
> > > 
> > > Hi Kuninori
> > > 
> > > Can you please split these changes into multiple smaller changes so that we
> > > can review this propely...
> Since this seems to be complet rewrite then remove old and add new driver
> would be easier too

Yes, this is complete rewrite patch.
Ok, I will send like this

 [1/2]: remove old driver
 [2/2]: add new driver

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