Re: [PATCH 2/2 v2] ARM: configs: remove all CONFIG_RCAR_AUDMAC_PP from ARM defconfigs

[Simon Horman <horms@verge.net.au>]

On Wed, Mar 18, 2015 at 11:54:02PM +0000, Kuninori Morimoto wrote:
> 
> Hi Vinod, and Simon
> 
> > > Renesas R-Car sound (= rsnd) needs 2 DMAC which are called as
> > > Audio DMAC (= 1st DMAC) and Audio DMAC peri peri (2nd DMAC).
> > > And rsnd had assumed that 1st / 2nd DMACs are implemented as DMAEngine.
> > > But, in result of DMA ML discussion, 2nd DMAC was concluded that it is
> > > not a general purpose DMAC (2nd DMAC is for Device to Device inside
> > > sound system). Additionally, current DMAEngine can't support Device to
> > > Device, and we don't have correct DT bindings for it at this point.
> > > So the easiest solution for it is that move it from DMAEngine to rsnd
> > > driver.
> > > Audio DMAC peri peri DMAEngine is no longer needed.
> > > Remove all CONFIG_RCAR_AUDMAC_PP from ARM defconfigs
> > 
> > This fails for me, I noticed the removal is not present in topc/sh.
> > What was this patch based on?
> 
> Thank you for your help.
> I think [1/2] (= remove driver) is controlled by Vinod,
> [2/2] (= remove setting from defconfigs) is controlled by Simon is good idea.
> These are no relationship anymore.

This patch is probably best taken by the ARM SoC maintainers
as it is for a generic defconfig rather than a Renesas-specific one.

> > -- 
> > ~Vinod
> > > 
> > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> > > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > ---
> > > v1 -> v2
> > > 
> > >  - new patch
> > > 
> > >  arch/arm/configs/multi_v7_defconfig | 1 -
> > >  1 file changed, 1 deletion(-)
> > > 
> > > diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
> > > index 06075b6..e805984 100644
> > > --- a/arch/arm/configs/multi_v7_defconfig
> > > +++ b/arch/arm/configs/multi_v7_defconfig
> > > @@ -505,7 +505,6 @@ CONFIG_DW_DMAC=y
> > >  CONFIG_MV_XOR=y
> > >  CONFIG_TEGRA20_APB_DMA=y
> > >  CONFIG_SH_DMAE=y
> > > -CONFIG_RCAR_AUDMAC_PP=m
> > >  CONFIG_RCAR_DMAC=y
> > >  CONFIG_STE_DMA40=y
> > >  CONFIG_SIRF_DMA=y
> > > -- 
> > > 1.9.1
> > > 
> > 
> > -- 
> 
> 
> Best regards
> ---
> Kuninori Morimoto
>