ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Pavel Machek <pavel@ucw.cz>
Cc: "ksummit-discuss@lists.linuxfoundation.org"
	<ksummit-discuss@lists.linuxfoundation.org>,
	riverful.kim@samsung.com, kyungmin.park@samsung.com,
	John Stultz <john.stultz@linaro.org>,
	Bjorn Andersson <Bjorn.Andersson@sonymobile.com>
Subject: Re: [Ksummit-discuss] [CORE TOPIC] Mainline kernel on a cellphone
Date: Sun, 2 Aug 2015 22:33:01 -0700	[thread overview]
Message-ID: <20150803053300.GD10928@atomide.com> (raw)
In-Reply-To: <20150731220306.GA7867@amd>

* Pavel Machek <pavel@ucw.cz> [150731 15:06]:
> On Fri 2015-07-31 18:52:15, Mark Brown wrote:
> > On Fri, Jul 31, 2015 at 02:22:27PM +0200, Pavel Machek wrote:
> > > On Wed 2015-07-29 14:32:44, Mark Brown wrote:
> > > > On Thu, Jul 23, 2015 at 10:29:02PM +0200, Pavel Machek wrote:
> > 
> > > > ALSA is the right interface for this, this is going to be some
> > > > misunderstanding on their part about how to use it or limitations in
> > 
> > > That's what I was arguing, but see https://lkml.org/lkml/2015/3/5/606
> > 
> > I can't view that link since lkml.org was down earlier today when I was
> > working online and now I'm working offline.
> 
> It works for me now.
> 
> > > kai> Our take was that ALSA is not the right interface for cmt_speech. The
> > > kai> cmt_speech interface in the modem is _not_ a PCM interface as modelled
> > > kai> by
> > > kai> ALSA. Specifically:
> > > kai>
> > > kai>- the interface is lossy in both directions
> > 
> > Yay.

Well at the point when the audio gets played out of the speaker it's
a stream :) I think the issues above are related to reading raw data
from the modem before it gets processed further for noise cancellation
by the DSP and so on. Eventually it should be just audio stream though.

> > > kai>- data is sent in packets, not a stream of samples (could be other
> > > kai>- things
> > > kai>   than PCM samples), with timing and meta-data
> > 
> > This is what the copy() operation in the driver is for, there are other
> > DSP based devices out there as well as things like USB.
> > 
> > > kai>   - timing of uplink is of utmost importance
> > 
> > Well, ALSA doesn't impose any latency of its own so there should be no
> > issue there - the overheads should be minimal.

I believe the fact that that ALSA fifo size could not be changed
dynamically between what the modem needs and what should be used for
audio playback from PM point of view used to be a problem. Maybe
that's already fixed, now I don't know. Just FYI.
 
> The interface Nokia has is pretty strange. I believe you send
> timestamp with buffer, and it means "I'd like this packet of data to
> be sent at that time", or something like that.
> 
> Anyway, no need to argue with me, I believe ALSA should be suitable.

Yes ALSA is suitable for sure. It's the raw modem data to audio
stream processing part that's weird but that's not ALSA's problem.

Regards,

Tony

  parent reply	other threads:[~2015-08-03  5:33 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-23 10:57 Pavel Machek
2015-07-23 11:21 ` Linus Walleij
2015-07-23 12:14   ` Pavel Machek
2015-07-23 12:42     ` Steven Rostedt
2015-07-23 15:40       ` Mark Brown
2015-07-28 22:09         ` Tim Bird
2015-07-28 23:07           ` Bjorn Andersson
2015-07-31 16:18             ` Rob Herring
2015-07-31 16:56               ` Bjorn Andersson
2015-08-11  9:34                 ` Johannes Berg
2015-07-31 17:25               ` Tim Bird
2015-08-03 15:29                 ` John W. Linville
2015-08-03  7:42               ` Linus Walleij
2015-08-03 21:34                 ` Rob Herring
2015-08-03 22:36                   ` Marcel Holtmann
2015-08-05  8:40                     ` Linus Walleij
2015-08-05  8:46                       ` Linus Walleij
2015-08-05  9:11                         ` Samuel Ortiz
2015-08-05 11:54                         ` Pavel Machek
2015-08-05  9:09                       ` Samuel Ortiz
2015-08-05 17:19                       ` Marcel Holtmann
2015-08-05 16:02                     ` Rob Herring
2015-08-05 17:00                       ` Marcel Holtmann
2015-08-07 12:40                         ` Linus Walleij
2015-07-29  0:45           ` Krzysztof Kozlowski
2015-07-29  6:12             ` Laurent Pinchart
2015-07-29  7:40             ` Pavel Machek
2015-08-25 18:59               ` Tim Bird
2015-08-26  1:22                 ` Krzysztof Kozlowski
2015-08-26  4:25                   ` Sudip Mukherjee
2015-08-26  4:52                     ` Krzysztof Kozlowski
2015-08-26  5:30                       ` Laurent Pinchart
2015-08-26  5:33                         ` Krzysztof Kozlowski
2015-08-26  6:15                           ` Josh Triplett
2015-08-26  7:23                             ` Heiko Stuebner
2015-08-26  8:05                               ` Krzysztof Kozlowski
2015-08-28  8:20                                 ` Nicolas Ferre
2015-08-26 11:33                   ` Mark Brown
2015-08-26 12:56                   ` Jason Cooper
2015-08-26 13:35                     ` Geert Uytterhoeven
2015-08-26 13:58                     ` Sudip Mukherjee
2015-08-26 14:51                       ` Jason Cooper
2015-08-26 17:13                         ` Sudip Mukherjee
2015-08-26 20:09                           ` Greg Kroah-Hartman
2015-08-28  7:44                             ` Pavel Machek
2015-08-28  8:42                               ` Heiko Stuebner
2015-08-29 15:47                                 ` Sudip Mukherjee
2015-07-29  8:18             ` Heiko Stübner
2015-07-29  0:56           ` Rafael J. Wysocki
2015-07-29  6:43           ` Daniel Vetter
2015-07-29 17:42             ` Tim Bird
2015-07-29  7:14           ` Bintian
2015-07-29 18:07             ` Tim Bird
2015-07-31  1:50               ` Bintian
2015-07-23 20:29       ` Pavel Machek
2015-07-23 20:34         ` Pavel Machek
2015-07-24  4:34           ` NeilBrown
2015-07-24  6:00             ` Tony Lindgren
2015-07-24 22:26               ` Rafael J. Wysocki
2015-07-28 22:03                 ` NeilBrown
2015-07-29  0:51                   ` Rafael J. Wysocki
2015-07-29 23:23                     ` Rafael J. Wysocki
2015-07-29 23:59                       ` NeilBrown
2015-07-30  0:57                         ` Rafael J. Wysocki
2015-07-31 17:55                           ` Mark Brown
2015-08-01  0:08                             ` Rafael J. Wysocki
2015-07-23 21:00         ` josh
2015-07-23 21:29           ` Pavel Machek
2015-07-29 13:32         ` Mark Brown
2015-07-31 12:22           ` Pavel Machek
2015-07-31 17:52             ` Mark Brown
2015-07-31 22:03               ` Pavel Machek
2015-08-01 10:55                 ` Mark Brown
2015-08-01 19:03                   ` Pavel Machek
2015-08-04 17:17                     ` Mark Brown
2015-08-03  5:33                 ` Tony Lindgren [this message]
2015-07-23 13:23     ` Krzysztof Kozlowski
2015-07-23 21:56       ` Pavel Machek
2015-07-24  1:37         ` Krzysztof Kozlowski
2015-07-24  4:40       ` Kyungmin Park
2015-07-23 14:48 ` Shuah Khan
2015-07-23 15:08 ` Heiko Stübner
2015-08-04 19:39 ` Pavel Machek
2015-08-05  7:05   ` Bintian

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=20150803053300.GD10928@atomide.com \
    --to=tony@atomide.com \
    --cc=Bjorn.Andersson@sonymobile.com \
    --cc=john.stultz@linaro.org \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=kyungmin.park@samsung.com \
    --cc=pavel@ucw.cz \
    --cc=riverful.kim@samsung.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox