All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Andres Salomon <dilinger@queued.net>
Cc: cliff.cai@analog.com, Mike Frysinger <vapier.adi@gmail.com>,
	alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
	device-drivers-devel@blackfin.uclinux.org,
	akpm@linux-foundation.org, lrg@slimlogic.co.uk
Subject: Re: [Device-drivers-devel] [PATCH] Add driver for Analog Devices ADAU1701 SigmaDSP
Date: Tue, 19 Apr 2011 09:06:29 +0100	[thread overview]
Message-ID: <20110419080629.GA16357@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <20110418171505.4b1699f3@queued.net>

On Mon, Apr 18, 2011 at 05:15:05PM -0700, Andres Salomon wrote:
> Mark Brown <broonie@opensource.wolfsonmicro.com> wrote:

> > Given the number of people looking at this from various angles I'd
> > expect to see some sort of progress this year (probably in the next
> > quater or so), but obviously no guarantees.

> Out of curiosity, has any progress been made on this?

No, it's only been a month.

> What would actually cause the firmware to change?  Would the userspace
> application that's currently driving the codec know that it needs to
> load a new firmware, or will there be multiple applications potentially
> interacting with the codec (with only one driving it, and another
> deciding that the firmware should change)?

Clearly something in userspace will need to be able to do this.

> One can imagine two different types of APIs depending upon the answers
> to those questions.  If the application driving the codec is also
> changing the firmware, a simple 'echo new_firmware_name.bin >
> /sys/.../filter_coeff' (or equivalent ALSA function)

Using sysfs isn't going to fly for all the reasons discussed upthread -
it's not enumerable and it's not going to cope with large coefficient
sets.

> However, if multiple applications are going to be interacting with
> the codec, I'd imagine we'd need something more complex; perhaps a hook
> in snd_soc_dai_ops that's called anytime stream state changes?  That

Using the ops isn't going to work as there's no reason a coefficient set
has to be associated with a DAI.

WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Andres Salomon <dilinger@queued.net>
Cc: cliff.cai@analog.com, alsa-devel@alsa-project.org,
	linux-kernel@vger.kernel.org,
	device-drivers-devel@blackfin.uclinux.org,
	akpm@linux-foundation.org, lrg@slimlogic.co.uk,
	Mike Frysinger <vapier.adi@gmail.com>
Subject: Re: [Device-drivers-devel] [PATCH] Add driver for Analog Devices ADAU1701 SigmaDSP
Date: Tue, 19 Apr 2011 09:06:29 +0100	[thread overview]
Message-ID: <20110419080629.GA16357@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <20110418171505.4b1699f3@queued.net>

On Mon, Apr 18, 2011 at 05:15:05PM -0700, Andres Salomon wrote:
> Mark Brown <broonie@opensource.wolfsonmicro.com> wrote:

> > Given the number of people looking at this from various angles I'd
> > expect to see some sort of progress this year (probably in the next
> > quater or so), but obviously no guarantees.

> Out of curiosity, has any progress been made on this?

No, it's only been a month.

> What would actually cause the firmware to change?  Would the userspace
> application that's currently driving the codec know that it needs to
> load a new firmware, or will there be multiple applications potentially
> interacting with the codec (with only one driving it, and another
> deciding that the firmware should change)?

Clearly something in userspace will need to be able to do this.

> One can imagine two different types of APIs depending upon the answers
> to those questions.  If the application driving the codec is also
> changing the firmware, a simple 'echo new_firmware_name.bin >
> /sys/.../filter_coeff' (or equivalent ALSA function)

Using sysfs isn't going to fly for all the reasons discussed upthread -
it's not enumerable and it's not going to cope with large coefficient
sets.

> However, if multiple applications are going to be interacting with
> the codec, I'd imagine we'd need something more complex; perhaps a hook
> in snd_soc_dai_ops that's called anytime stream state changes?  That

Using the ops isn't going to work as there's no reason a coefficient set
has to be associated with a DAI.

  reply	other threads:[~2011-04-19  8:06 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-07  1:11 [PATCH] Add driver for Analog Devices ADAU1701 SigmaDSP cliff.cai
2011-03-07  1:11 ` cliff.cai
2011-03-07  2:44 ` [Device-drivers-devel] " Mike Frysinger
2011-03-07 11:44   ` Mark Brown
2011-03-07 11:44     ` [alsa-devel] " Mark Brown
2011-03-07 11:01 ` Liam Girdwood
2011-03-07 11:01   ` [alsa-devel] " Liam Girdwood
2011-03-07 11:41 ` Mark Brown
2011-03-07 11:41   ` Mark Brown
2011-03-07 11:50   ` [Device-drivers-devel] " Mike Frysinger
2011-03-07 11:50     ` Mike Frysinger
2011-03-07 12:15     ` Mark Brown
2011-03-07 12:15       ` Mark Brown
2011-03-07 12:29       ` Mike Frysinger
2011-03-07 12:29         ` Mike Frysinger
2011-03-07 14:59         ` Mark Brown
2011-03-07 15:33           ` Mike Frysinger
2011-03-07 15:33             ` Mike Frysinger
2011-03-07 15:55             ` Mark Brown
2011-03-07 15:55               ` Mark Brown
2011-03-09  6:08               ` Mike Frysinger
2011-03-09  6:08                 ` Mike Frysinger
2011-03-09  7:39                 ` Cliff Cai
2011-03-09  7:39                   ` [alsa-devel] " Cliff Cai
2011-03-09  9:55                 ` Mark Brown
2011-03-09  9:55                   ` Mark Brown
2011-04-19  0:15           ` Andres Salomon
2011-04-19  0:15             ` Andres Salomon
2011-04-19  8:06             ` Mark Brown [this message]
2011-04-19  8:06               ` Mark Brown
2011-03-09  7:25         ` Cliff Cai
2011-03-09  7:25           ` [alsa-devel] " Cliff Cai
2011-03-09 10:00           ` Mark Brown
2011-03-09 10:00             ` [alsa-devel] " Mark Brown
2011-03-10  0:35             ` Mike Frysinger
2011-03-10  0:35               ` [alsa-devel] " Mike Frysinger
2011-03-10  9:45             ` Cai, Cliff
2011-03-10  9:45               ` Cai, Cliff
2011-03-10 13:46               ` Mark Brown
2011-03-10 13:46                 ` [alsa-devel] " Mark Brown
2011-03-11  7:54                 ` Cai, Cliff
2011-03-11  7:54                   ` Cai, Cliff
2011-03-11 11:53                   ` Mark Brown
2011-03-11 11:53                     ` [alsa-devel] " Mark Brown
2011-03-14  2:23                     ` Cai, Cliff
2011-03-14  2:23                       ` Cai, Cliff
2011-03-14 11:29                       ` Mark Brown
2011-03-14 11:29                         ` [alsa-devel] " Mark Brown
2011-03-09  7:04   ` Cliff Cai
2011-03-09  7:04     ` [alsa-devel] " Cliff Cai
2011-03-09 10:12     ` Mark Brown
2011-03-09 10:12       ` [alsa-devel] " Mark Brown
2011-03-10 10:04       ` Cai, Cliff
2011-03-10 10:04         ` Cai, Cliff

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=20110419080629.GA16357@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=akpm@linux-foundation.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=cliff.cai@analog.com \
    --cc=device-drivers-devel@blackfin.uclinux.org \
    --cc=dilinger@queued.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lrg@slimlogic.co.uk \
    --cc=vapier.adi@gmail.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.