From: Jarkko Nikula <jhnikula@gmail.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Peter Ujfalusi <peter.ujfalusi@nokia.com>,
alsa-devel@alsa-project.org,
Eduardo Valentin <eduardo.valentin@nokia.com>,
"Nurkkala Eero.An (EXT-Offcode/Oulu)"
<ext-Eero.Nurkkala@nokia.com>
Subject: Re: [PATCH 00/20] OMAP ASoC changes in DMA utilization
Date: Thu, 30 Jul 2009 21:55:46 +0300 [thread overview]
Message-ID: <20090730215546.4e184112.jhnikula@gmail.com> (raw)
In-Reply-To: <20090730135257.GE9975@sirena.org.uk>
On Thu, 30 Jul 2009 14:52:57 +0100
Mark Brown <broonie@opensource.wolfsonmicro.com> wrote:
> On Thu, Jul 30, 2009 at 03:49:23PM +0300, Eduardo Valentin wrote:
>
> > Patches 1-15 are McBSP changes and, for this reason, are intended to
> > be include in linux-omap tree.
> > Patches 16-20 are OMAP ASoC driver changes, so, they are for
> > alsa-devel.
>
> I'm OK with the ASoC changes. It looks like there are some
> dependencies on the OMAP changes in there so they should probably go
> via the OMAP tree?
Is Peter available or having a vacation? He would have better
possibilities to say and test the PM changes. Now omap_pcm_pointer is
very accurate and I would like that it remains accurate unless user
especially wants low-power over it.
I post few, more like minor notes than show stoppers to some of these
patches.
My wife and kids are having vacation so my ability to review, test and
understand those more fundamental changes is somewhat reduced.
--
Jarkko
next prev parent reply other threads:[~2009-07-30 18:54 UTC|newest]
Thread overview: 116+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-30 12:49 [PATCH 00/20] OMAP ASoC changes in DMA utilization Eduardo Valentin
2009-07-30 12:49 ` [PATCH 01/20] OMAP: McBSP: Provide functions for ASoC frame syncronization Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 02/20] OMAP: McBSP: Wakeups utilized Eduardo Valentin
2009-07-30 12:49 ` [PATCH 03/20] OMAP: McBSP: Change wakeup signals Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 04/20] OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1 registers Eduardo Valentin
2009-07-30 12:49 ` [PATCH 05/20] OMAP: McBSP: Retain McBSP FCLK clockactivity Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 06/20] OMAP: McBSP: Use appropriate value for startup delay Eduardo Valentin
2009-07-30 12:49 ` [PATCH 07/20] OMAP: McBSP: Add transmit/receive threshold handler Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 08/20] OMAP: McBSP: Create and export max_(r|t)x_thres property Eduardo Valentin
2009-07-30 12:49 ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5 Eduardo Valentin
2009-07-30 12:49 ` [PATCH 10/20] OMAP: McBSP: Rename thres sysfs symbols Eduardo Valentin
2009-07-30 12:49 ` [PATCH 11/20] OMAP: McBSP: Add link DMA mode selection Eduardo Valentin
2009-07-30 12:49 ` [PATCH 12/20] OMAP: McBSP: Configure NO IDLE mode for DMA mode different of threshold Eduardo Valentin
2009-07-30 12:49 ` [PATCH 13/20] OMAP: McBSP: Do not enable wakeups for no-idle mode Eduardo Valentin
2009-07-30 12:49 ` [PATCH 14/20] OMAP: McBSP: Disable all wakeups upon exit Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 15/20] OMAP: McBSP: Let element DMA mode hit retention also Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 16/20] ASoC: OMAP: Use McBSP threshold again Eduardo Valentin
2009-07-30 12:49 ` [PATCH 17/20] ASoC: OMAP: Make DMA 64 aligned Eduardo Valentin
2009-07-30 12:49 ` [PATCH 18/20] ASoC: OMAP: Enable DMA burst mode Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 19/20] ASoC: OMAP: Use DMA operating mode of McBSP Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 20/20] ASoC: Always syncronize audio transfers on frames Eduardo Valentin
2009-07-30 18:57 ` Jarkko Nikula
2009-07-30 18:57 ` Jarkko Nikula
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 17/20] ASoC: OMAP: Make DMA 64 aligned Eduardo Valentin
2009-08-05 7:48 ` [PATCH 16/20] ASoC: OMAP: Use McBSP threshold again Jarkko Nikula
2009-08-10 8:53 ` Eduardo Valentin
2009-08-10 8:53 ` Eduardo Valentin
2009-08-11 5:34 ` Jarkko Nikula
2009-08-11 6:22 ` Eduardo Valentin
2009-08-11 6:22 ` Eduardo Valentin
2009-08-05 7:48 ` Jarkko Nikula
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 13/20] OMAP: McBSP: Do not enable wakeups for no-idle mode Eduardo Valentin
2009-08-06 12:00 ` [PATCH 12/20] OMAP: McBSP: Configure NO IDLE mode for DMA mode different of threshold Eduardo Valentin
2009-08-06 12:00 ` Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 13:04 ` [PATCH 11/20] OMAP: McBSP: Add link DMA mode selection Mark Brown
2009-07-30 13:28 ` Eduardo Valentin
2009-07-30 13:47 ` Mark Brown
2009-07-30 13:47 ` Mark Brown
2009-08-03 10:15 ` Eduardo Valentin
2009-08-03 10:15 ` Eduardo Valentin
2009-07-30 13:28 ` Eduardo Valentin
2009-07-30 13:04 ` Mark Brown
2009-08-05 7:39 ` Jarkko Nikula
2009-08-05 7:39 ` Jarkko Nikula
2009-08-05 8:58 ` Peter Ujfalusi
2009-08-06 18:15 ` [alsa-devel] " ext-Eero.Nurkkala
2009-08-07 13:11 ` Eduardo Valentin
2009-08-11 6:04 ` Jarkko Nikula
2009-08-11 6:18 ` [alsa-devel] " Eero Nurkkala
2009-08-12 11:45 ` Jarkko Nikula
2009-08-12 11:48 ` [alsa-devel] " Eero Nurkkala
2009-08-13 6:01 ` Peter Ujfalusi
2009-08-13 20:46 ` snd_pcm_delay, hw buffers and driver api (was: Re: [PATCH 11/20] OMAP: McBSP: Add link DMA mode selection) Kai Vehmanen
2009-08-13 20:59 ` Mark Brown
2009-07-30 12:49 ` [PATCH 11/20] OMAP: McBSP: Add link DMA mode selection Eduardo Valentin
2009-07-30 12:49 ` [PATCH 10/20] OMAP: McBSP: Rename thres sysfs symbols Eduardo Valentin
2009-07-30 18:57 ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Jarkko Nikula
2009-07-30 18:57 ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5 Jarkko Nikula
2009-07-31 7:58 ` Eduardo Valentin
2009-07-31 17:26 ` Jarkko Nikula
2009-08-03 8:11 ` Eduardo Valentin
2009-08-03 8:36 ` Jarkko Nikula
2009-08-03 8:36 ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Eduardo Valentin
2009-08-03 8:36 ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5 Eduardo Valentin
2009-08-03 8:36 ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Jarkko Nikula
2009-08-03 8:11 ` Eduardo Valentin
2009-07-31 17:26 ` Jarkko Nikula
2009-07-31 7:58 ` Eduardo Valentin
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 08/20] OMAP: McBSP: Create and export max_(r|t)x_thres property Eduardo Valentin
2009-08-06 12:10 ` [PATCH 07/20] OMAP: McBSP: Add transmit/receive threshold handler Tony Lindgren
2009-08-06 12:10 ` Tony Lindgren
2009-08-06 12:20 ` ext-Eero.Nurkkala
2009-08-06 12:24 ` Tony Lindgren
2009-08-06 12:24 ` Tony Lindgren
2009-08-06 12:20 ` ext-Eero.Nurkkala
2009-07-30 18:56 ` [PATCH 06/20] OMAP: McBSP: Use appropriate value for startup delay Jarkko Nikula
2009-07-31 7:21 ` ext-Eero.Nurkkala
2009-07-31 17:25 ` Jarkko Nikula
2009-07-31 17:25 ` Jarkko Nikula
2009-07-31 7:21 ` ext-Eero.Nurkkala
2009-07-30 12:49 ` Eduardo Valentin
2009-07-30 12:49 ` [PATCH 04/20] OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1 registers Eduardo Valentin
2009-08-05 7:40 ` [PATCH 03/20] OMAP: McBSP: Change wakeup signals Peter Ujfalusi
2009-07-30 18:56 ` [PATCH 02/20] OMAP: McBSP: Wakeups utilized Jarkko Nikula
2009-07-30 18:56 ` Jarkko Nikula
2009-07-31 7:56 ` Eduardo Valentin
2009-08-06 12:05 ` Tony Lindgren
2009-08-06 12:00 ` Eduardo Valentin
2009-08-06 12:00 ` Eduardo Valentin
2009-08-06 12:05 ` Tony Lindgren
2009-07-31 7:56 ` Eduardo Valentin
2009-08-06 12:00 ` Tony Lindgren
2009-08-06 12:02 ` Tony Lindgren
2009-08-06 12:02 ` Tony Lindgren
2009-08-06 11:56 ` Eduardo Valentin
2009-08-06 11:56 ` Eduardo Valentin
2009-08-06 12:00 ` Tony Lindgren
2009-07-30 12:49 ` Eduardo Valentin
2009-08-05 7:18 ` [PATCH 01/20] OMAP: McBSP: Provide functions for ASoC frame syncronization Peter Ujfalusi
2009-07-30 13:52 ` [PATCH 00/20] OMAP ASoC changes in DMA utilization Mark Brown
2009-07-30 18:55 ` Jarkko Nikula [this message]
2009-07-30 19:11 ` Mark Brown
2009-08-05 7:23 ` Peter Ujfalusi
-- strict thread matches above, loose matches on Subject: below --
2009-07-30 12:49 Eduardo Valentin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090730215546.4e184112.jhnikula@gmail.com \
--to=jhnikula@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=eduardo.valentin@nokia.com \
--cc=ext-Eero.Nurkkala@nokia.com \
--cc=peter.ujfalusi@nokia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.