All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jarkko Nikula <jhnikula@gmail.com>
To: ext-eero.nurkkala@nokia.com
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"Ujfalusi Peter (Nokia-D/Tampere)" <peter.ujfalusi@nokia.com>,
	"alsa-devel@vger.kernel.org" <alsa-devel@vger.kernel.org>
Subject: Re: [PATCH 11/20] OMAP: McBSP: Add link DMA mode selection
Date: Wed, 12 Aug 2009 14:45:47 +0300	[thread overview]
Message-ID: <20090812144547.eb4bf7f2.jhnikula@gmail.com> (raw)
In-Reply-To: <1249971481.16045.17.camel@eenurkka-desktop>

On Tue, 11 Aug 2009 09:18:01 +0300
Eero Nurkkala <ext-eero.nurkkala@nokia.com> wrote:

> > I would like to see this new threshold based transfer functionality to
> > be integrated so that projects can take the advantage of it and helps
> > generic PM development too but I don't want that it would cause any
> > regression now. Later on it is easy to switch threshold based transfer
> > to be the default for McBSP2 on OMAP3 but it's safer to keep current
> > mode default over one kernel release.
> > 
> 
> What regression are you addressing to? Of course in OMAP2s there could
> be issues I'm not aware of. But does this have any effect on them?
> (isn't this only OMAP3).
> 
A possible regression to a SW which is using McBSP2 on OMAP3.
There many of those boards.

The threshold based transfer will cause that omap_pcm_pointer will
loose a bit its accuracy. Probably irrelevant but still better to play
safe at least over one kernel release before making it default.

Return value prints from omap_pcm_pointer while playing
"aplay -f dat /dev/zero"

element:
614
669
691
712
734
755

threshold:
512
512
512
1024
1024
1536


-- 
Jarkko

  reply	other threads:[~2009-08-12 11:44 UTC|newest]

Thread overview: 115+ 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       ` [PATCH 04/20] OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1 registers Eduardo Valentin
2009-07-30 12:49       ` Eduardo Valentin
2009-07-30 12:49         ` [PATCH 05/20] OMAP: McBSP: Retain McBSP FCLK clockactivity 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           ` 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               ` 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 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                     ` 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-11  5:34                                     ` Jarkko Nikula
2009-08-11  6:22                                       ` Eduardo Valentin
2009-08-11  6:22                                       ` Eduardo Valentin
2009-08-10  8:53                                   ` 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-08-03 10:15                             ` Eduardo Valentin
2009-08-03 10:15                             ` Eduardo Valentin
2009-07-30 13:47                           ` Mark Brown
2009-07-30 13:28                         ` Eduardo Valentin
2009-07-30 13:04                       ` Mark Brown
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 [this message]
2009-08-12 11:48                                   ` 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-08-05  7:39                       ` [PATCH 11/20] OMAP: McBSP: Add link DMA mode selection Jarkko Nikula
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                     ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Eduardo Valentin
2009-07-31  7:58                     ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5 Eduardo Valentin
2009-07-31 17:26                       ` Jarkko Nikula
2009-08-03  8:11                         ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Eduardo Valentin
2009-08-03  8:11                         ` [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:36                           ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5 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-07-31 17:26                       ` [PATCH 09/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Jarkko Nikula
2009-08-06 12:10               ` [PATCH 07/20] OMAP: McBSP: Add transmit/receive threshold handler 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-08-06 12:10               ` Tony Lindgren
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  7:21               ` ext-Eero.Nurkkala
2009-07-31 17:25                 ` Jarkko Nikula
2009-07-31 17:25                 ` Jarkko Nikula
2009-07-30 12:49         ` [PATCH 05/20] OMAP: McBSP: Retain McBSP FCLK clockactivity Eduardo Valentin
2009-08-05  7:40       ` [PATCH 03/20] OMAP: McBSP: Change wakeup signals Peter Ujfalusi
2009-07-30 12:49     ` Eduardo Valentin
2009-07-30 18:56     ` [PATCH 02/20] OMAP: McBSP: Wakeups utilized 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-07-30 18:56     ` Jarkko Nikula
2009-08-06 12:00     ` Tony Lindgren
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-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
2009-07-30 19:11     ` Mark Brown
2009-08-05  7:23     ` Peter Ujfalusi

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=20090812144547.eb4bf7f2.jhnikula@gmail.com \
    --to=jhnikula@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=alsa-devel@vger.kernel.org \
    --cc=ext-eero.nurkkala@nokia.com \
    --cc=linux-omap@vger.kernel.org \
    --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.