[PATCHv5 00/20] OMAP ASoC changes in DMA utilization

[PATCHv5 00/20] OMAP ASoC changes in DMA utilization

[Eduardo Valentin]

Hello guys,

I've updated this series after some previous confusion from my side.

Now that all dependencies are merged (Janusz's patches and Jarkko's patch),
I've based this series on Takashi's tree (master branch).

I've also talked with Tony and he agrees to have this changes merged
by alsa tree. And as pointed by Jarkko, the mcbsp code is currently
used only by Audio inside OMAP code. So, that should be safe.

It is the same series. It now includes patch description for
every patch. And also, as I already said, it is based on master
branch of sound-2.6.

Tested with beagle and rx51. Compiled tested for n770.

BR,

Eduardo Valentin (11):
  OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1
    registers.
  OMAP: McBSP: Use appropriate value for startup delay
  OMAP: McBSP: Add transmit/receive threshold handler
  OMAP: McBSP: Create and export max_(r|t)x_thres property
  OMAP: McBSP: Rename thres sysfs symbols
  OMAP: McBSP: Change wakeup signals
  OMAP: McBSP: Configure NO IDLE mode for DMA mode different of
    threshold
  ASoC: OMAP: Enable DMA burst mode
  ASoC: OMAP: Make DMA 64 aligned
  ASoC: OMAP: Use McBSP threshold to playback and capture
  ASoC: OMAP: Use DMA operating mode of McBSP

Eero Nurkkala (7):
  OMAP: McBSP: Provide functions for ASoC frame syncronization
  OMAP: McBSP: Wakeups utilized
  OMAP: McBSP: Retain McBSP FCLK clockactivity
  OMAP: McBSP: Do not enable wakeups for no-idle mode
  OMAP: McBSP: Let element DMA mode hit retention also
  ASoC: Add runtime check for RFIG and XFIG
  ASoC: Always syncronize audio transfers on frames

Peter Ujfalusi (2):
  OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5
  OMAP: McBSP: Add link DMA mode selection

 arch/arm/mach-omap2/mcbsp.c             |    5 +
 arch/arm/plat-omap/include/mach/mcbsp.h |   49 ++++
 arch/arm/plat-omap/mcbsp.c              |  377 ++++++++++++++++++++++++++++++-
 sound/soc/omap/omap-mcbsp.c             |   77 ++++++-
 sound/soc/omap/omap-pcm.c               |   14 +-
 sound/soc/omap/omap-pcm.h               |    2 +
 6 files changed, 511 insertions(+), 13 deletions(-)

Re: [PATCHv5 00/20] OMAP ASoC changes in DMA utilization

[Tony Lindgren]

* Eduardo Valentin <eduardo.valentin@nokia.com> [090820 16:36]:
> Hello guys,
> 
> I've updated this series after some previous confusion from my side.
> 
> Now that all dependencies are merged (Janusz's patches and Jarkko's patch),
> I've based this series on Takashi's tree (master branch).
> 
> I've also talked with Tony and he agrees to have this changes merged
> by alsa tree. And as pointed by Jarkko, the mcbsp code is currently
> used only by Audio inside OMAP code. So, that should be safe.

You might want to merge patches 5 and 6 together as 5 adds the
fifo, then 6 limits it's size. But in general, for patches
1 - 14: Acked-by: Tony Lindgren <tony@atomide.com>
 
> It is the same series. It now includes patch description for
> every patch. And also, as I already said, it is based on master
> branch of sound-2.6.
> 
> Tested with beagle and rx51. Compiled tested for n770.
> 
> BR,
> 
> Eduardo Valentin (11):
>   OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1
>     registers.
>   OMAP: McBSP: Use appropriate value for startup delay
>   OMAP: McBSP: Add transmit/receive threshold handler
>   OMAP: McBSP: Create and export max_(r|t)x_thres property
>   OMAP: McBSP: Rename thres sysfs symbols
>   OMAP: McBSP: Change wakeup signals
>   OMAP: McBSP: Configure NO IDLE mode for DMA mode different of
>     threshold
>   ASoC: OMAP: Enable DMA burst mode
>   ASoC: OMAP: Make DMA 64 aligned
>   ASoC: OMAP: Use McBSP threshold to playback and capture
>   ASoC: OMAP: Use DMA operating mode of McBSP
> 
> Eero Nurkkala (7):
>   OMAP: McBSP: Provide functions for ASoC frame syncronization
>   OMAP: McBSP: Wakeups utilized
>   OMAP: McBSP: Retain McBSP FCLK clockactivity
>   OMAP: McBSP: Do not enable wakeups for no-idle mode
>   OMAP: McBSP: Let element DMA mode hit retention also
>   ASoC: Add runtime check for RFIG and XFIG
>   ASoC: Always syncronize audio transfers on frames
> 
> Peter Ujfalusi (2):
>   OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5
>   OMAP: McBSP: Add link DMA mode selection
> 
>  arch/arm/mach-omap2/mcbsp.c             |    5 +
>  arch/arm/plat-omap/include/mach/mcbsp.h |   49 ++++
>  arch/arm/plat-omap/mcbsp.c              |  377 ++++++++++++++++++++++++++++++-
>  sound/soc/omap/omap-mcbsp.c             |   77 ++++++-
>  sound/soc/omap/omap-pcm.c               |   14 +-
>  sound/soc/omap/omap-pcm.h               |    2 +
>  6 files changed, 511 insertions(+), 13 deletions(-)
> 

Re: [PATCHv5 00/20] OMAP ASoC changes in DMA utilization

[Eduardo Valentin]

Hello Tony,

On Thu, Aug 20, 2009 at 5:30 PM, Tony Lindgren<tony@atomide.com> wrote:
> * Eduardo Valentin <eduardo.valentin@nokia.com> [090820 16:36]:
>> Hello guys,
>>
>> I've updated this series after some previous confusion from my side.
>>
>> Now that all dependencies are merged (Janusz's patches and Jarkko's patch),
>> I've based this series on Takashi's tree (master branch).
>>
>> I've also talked with Tony and he agrees to have this changes merged
>> by alsa tree. And as pointed by Jarkko, the mcbsp code is currently
>> used only by Audio inside OMAP code. So, that should be safe.
>
> You might want to merge patches 5 and 6 together as 5 adds the
> fifo, then 6 limits it's size. But in general, for patches

Right, I left it as it is just to keep track of what was done and to give
kudos to Peter.

> 1 - 14: Acked-by: Tony Lindgren <tony@atomide.com>

Okei.

>
>> It is the same series. It now includes patch description for
>> every patch. And also, as I already said, it is based on master
>> branch of sound-2.6.
>>
>> Tested with beagle and rx51. Compiled tested for n770.
>>
>> BR,
>>
>> Eduardo Valentin (11):
>>   OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1
>>     registers.
>>   OMAP: McBSP: Use appropriate value for startup delay
>>   OMAP: McBSP: Add transmit/receive threshold handler
>>   OMAP: McBSP: Create and export max_(r|t)x_thres property
>>   OMAP: McBSP: Rename thres sysfs symbols
>>   OMAP: McBSP: Change wakeup signals
>>   OMAP: McBSP: Configure NO IDLE mode for DMA mode different of
>>     threshold
>>   ASoC: OMAP: Enable DMA burst mode
>>   ASoC: OMAP: Make DMA 64 aligned
>>   ASoC: OMAP: Use McBSP threshold to playback and capture
>>   ASoC: OMAP: Use DMA operating mode of McBSP
>>
>> Eero Nurkkala (7):
>>   OMAP: McBSP: Provide functions for ASoC frame syncronization
>>   OMAP: McBSP: Wakeups utilized
>>   OMAP: McBSP: Retain McBSP FCLK clockactivity
>>   OMAP: McBSP: Do not enable wakeups for no-idle mode
>>   OMAP: McBSP: Let element DMA mode hit retention also
>>   ASoC: Add runtime check for RFIG and XFIG
>>   ASoC: Always syncronize audio transfers on frames
>>
>> Peter Ujfalusi (2):
>>   OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5
>>   OMAP: McBSP: Add link DMA mode selection
>>
>>  arch/arm/mach-omap2/mcbsp.c             |    5 +
>>  arch/arm/plat-omap/include/mach/mcbsp.h |   49 ++++
>>  arch/arm/plat-omap/mcbsp.c              |  377 ++++++++++++++++++++++++++++++-
>>  sound/soc/omap/omap-mcbsp.c             |   77 ++++++-
>>  sound/soc/omap/omap-pcm.c               |   14 +-
>>  sound/soc/omap/omap-pcm.h               |    2 +
>>  6 files changed, 511 insertions(+), 13 deletions(-)
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>



-- 
Eduardo Bezerra Valentin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCHv5 00/20] OMAP ASoC changes in DMA utilization

[Jarkko Nikula]

On Thu, 20 Aug 2009 17:30:18 +0300
Tony Lindgren <tony@atomide.com> wrote:

> * Eduardo Valentin <eduardo.valentin@nokia.com> [090820 16:36]:
> > Hello guys,
> > 
> > I've updated this series after some previous confusion from my side.
> > 
> > Now that all dependencies are merged (Janusz's patches and Jarkko's patch),
> > I've based this series on Takashi's tree (master branch).
> > 
Mark: Set applies and works (with Beagle) on top of both master and
topic/asoc branches.

> > I've also talked with Tony and he agrees to have this changes merged
> > by alsa tree. And as pointed by Jarkko, the mcbsp code is currently
> > used only by Audio inside OMAP code. So, that should be safe.
> 
> You might want to merge patches 5 and 6 together as 5 adds the
> fifo, then 6 limits it's size. But in general, for patches
> 1 - 14: Acked-by: Tony Lindgren <tony@atomide.com>
>  
15-20: Acked-by: Jarkko Nikula <jhnikula@gmail.com>


-- 
Jarkko

Re: [PATCHv5 00/20] OMAP ASoC changes in DMA utilization

[Mark Brown]

On Thu, Aug 20, 2009 at 07:15:42PM +0300, Jarkko Nikula wrote:
> Tony Lindgren <tony@atomide.com> wrote:

> > You might want to merge patches 5 and 6 together as 5 adds the
> > fifo, then 6 limits it's size. But in general, for patches
> > 1 - 14: Acked-by: Tony Lindgren <tony@atomide.com>

> 15-20: Acked-by: Jarkko Nikula <jhnikula@gmail.com>

OK, applied all of them - thanks guys!

One procedural issue, though, Eduardo - since you were sending out the
patches they should all have had your Signed-off-by.