All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
To: Danny Milosavljevic <dannym-bxPqe3T81XXwRsdMLXbzog@public.gmane.org>
Cc: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org,
	Jaroslav Kysela <perex-/Fr2/VpizcU@public.gmane.org>,
	Takashi Iwai <tiwai-IBi9RG/b67k@public.gmane.org>,
	Liam Girdwood <lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
Subject: Re: Re: [PATCH v8 2/2] ASoc: sun4i-codec: Add FM, Line and Mic inputs
Date: Mon, 14 Mar 2016 11:49:39 +0100	[thread overview]
Message-ID: <20160314104939.GL8418@lukather> (raw)
In-Reply-To: <20160312085222.226d89bf-bxPqe3T81XXwRsdMLXbzog@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 1143 bytes --]

On Sat, Mar 12, 2016 at 08:52:22AM +0100, Danny Milosavljevic wrote:
> Hi,
> 
> does anyone know the answer to the questions below?
> 
> > The other direction (making two controls which both do the same and update 
> > each other's value) doesn't seem to be easily available. 
> 
> > Should I write a _put handler that does it manually?
> 
> > (Or should that be handled by alsa-lib mixer modules instead?)
> 
> > Is it possible to merge the "Left Mixer" control and respective 
> > "Right Mixer" control into one selem in alsamixer?
> > 
> > Because alsamixer actually has support for one-sided muting.

Mark, do you know how to do it?

IIRC, in order to have shared controls, you just needed to share the
controls structure.

> If that's unknown, I can post v9 without doing the grouping-together
> of mixer controls and we can use it like that - I don't want the
> patch to stall indefinitely on mere usability issues...

Unfortunately, that is exposed to the user-space. Once it's in, you
cannot change it / fix it anymore.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

WARNING: multiple messages have this Message-ID (diff)
From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [linux-sunxi] Re: [PATCH v8 2/2] ASoc: sun4i-codec: Add FM, Line and Mic inputs
Date: Mon, 14 Mar 2016 11:49:39 +0100	[thread overview]
Message-ID: <20160314104939.GL8418@lukather> (raw)
In-Reply-To: <20160312085222.226d89bf@scratchpost.org>

On Sat, Mar 12, 2016 at 08:52:22AM +0100, Danny Milosavljevic wrote:
> Hi,
> 
> does anyone know the answer to the questions below?
> 
> > The other direction (making two controls which both do the same and update 
> > each other's value) doesn't seem to be easily available. 
> 
> > Should I write a _put handler that does it manually?
> 
> > (Or should that be handled by alsa-lib mixer modules instead?)
> 
> > Is it possible to merge the "Left Mixer" control and respective 
> > "Right Mixer" control into one selem in alsamixer?
> > 
> > Because alsamixer actually has support for one-sided muting.

Mark, do you know how to do it?

IIRC, in order to have shared controls, you just needed to share the
controls structure.

> If that's unknown, I can post v9 without doing the grouping-together
> of mixer controls and we can use it like that - I don't want the
> patch to stall indefinitely on mere usability issues...

Unfortunately, that is exposed to the user-space. Once it's in, you
cannot change it / fix it anymore.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160314/0a31e144/attachment.sig>

WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: Danny Milosavljevic <dannym@scratchpost.org>
Cc: Mark Brown <broonie@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	alsa-devel@alsa-project.org, Jaroslav Kysela <perex@perex.cz>,
	Takashi Iwai <tiwai@suse.com>,
	Liam Girdwood <lgirdwood@gmail.com>,
	linux-sunxi@googlegroups.com
Subject: Re: [linux-sunxi] Re: [PATCH v8 2/2] ASoc: sun4i-codec: Add FM, Line and Mic inputs
Date: Mon, 14 Mar 2016 11:49:39 +0100	[thread overview]
Message-ID: <20160314104939.GL8418@lukather> (raw)
In-Reply-To: <20160312085222.226d89bf@scratchpost.org>

[-- Attachment #1: Type: text/plain, Size: 1179 bytes --]

On Sat, Mar 12, 2016 at 08:52:22AM +0100, Danny Milosavljevic wrote:
> Hi,
> 
> does anyone know the answer to the questions below?
> 
> > The other direction (making two controls which both do the same and update 
> > each other's value) doesn't seem to be easily available. 
> 
> > Should I write a _put handler that does it manually?
> 
> > (Or should that be handled by alsa-lib mixer modules instead?)
> 
> > Is it possible to merge the "Left Mixer" control and respective 
> > "Right Mixer" control into one selem in alsamixer?
> > 
> > Because alsamixer actually has support for one-sided muting.

Mark, do you know how to do it?

IIRC, in order to have shared controls, you just needed to share the
controls structure.

> If that's unknown, I can post v9 without doing the grouping-together
> of mixer controls and we can use it like that - I don't want the
> patch to stall indefinitely on mere usability issues...

Unfortunately, that is exposed to the user-space. Once it's in, you
cannot change it / fix it anymore.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  parent reply	other threads:[~2016-03-14 10:49 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-21 11:31 [PATCH v8 0/2] ASoc: sun4i-codec: Add FM, Line and Mic inputs Danny Milosavljevic
2015-12-21 11:31 ` Danny Milosavljevic
2015-12-21 11:31 ` Danny Milosavljevic
2015-12-21 11:33 ` [PATCH v8 1/2] " Danny Milosavljevic
2015-12-21 11:33   ` [linux-sunxi] " Danny Milosavljevic
2015-12-21 11:33   ` Danny Milosavljevic
2015-12-27 17:34   ` Maxime Ripard
2015-12-27 17:34     ` [linux-sunxi] " Maxime Ripard
2015-12-27 17:34     ` Maxime Ripard
2015-12-21 11:34 ` [PATCH v8 2/2] " Danny Milosavljevic
2015-12-21 11:34   ` Danny Milosavljevic
2015-12-21 11:34   ` Danny Milosavljevic
2015-12-27 18:21   ` Maxime Ripard
2015-12-27 18:21     ` Maxime Ripard
2015-12-27 18:21     ` Maxime Ripard
2015-12-28  3:06     ` Danny Milosavljevic
2015-12-28  3:06       ` [linux-sunxi] " Danny Milosavljevic
2015-12-28  3:06       ` Danny Milosavljevic
2015-12-31 22:19       ` Mark Brown
2015-12-31 22:19         ` [linux-sunxi] " Mark Brown
2015-12-31 22:19         ` Mark Brown
2016-01-06 22:09       ` Maxime Ripard
2016-01-06 22:09         ` [linux-sunxi] " Maxime Ripard
2016-01-06 22:09         ` Maxime Ripard
2016-01-09 15:48         ` Danny Milosavljevic
2016-01-09 15:48           ` [linux-sunxi] " Danny Milosavljevic
2016-01-09 15:48           ` Danny Milosavljevic
2016-03-12  7:52           ` Danny Milosavljevic
2016-03-12  7:52             ` [linux-sunxi] " Danny Milosavljevic
2016-03-12  7:52             ` Danny Milosavljevic
     [not found]             ` <20160312085222.226d89bf-bxPqe3T81XXwRsdMLXbzog@public.gmane.org>
2016-03-12  8:31               ` Code Kipper
2016-03-12  8:31                 ` [linux-sunxi] " Code Kipper
2016-03-12  8:31                 ` Code Kipper
2016-03-14 10:49               ` Maxime Ripard [this message]
2016-03-14 10:49                 ` Maxime Ripard
2016-03-14 10:49                 ` Maxime Ripard
2016-03-15 10:58                 ` Mark Brown
2016-03-15 10:58                   ` [linux-sunxi] " Mark Brown
2016-03-15 10:58                   ` Mark Brown
     [not found]                   ` <20160315105802.GN2566-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2016-03-19 16:13                     ` Danny Milosavljevic
2016-03-19 16:13                       ` [linux-sunxi] " Danny Milosavljevic
2016-03-19 16:13                       ` Danny Milosavljevic
     [not found]                       ` <20160319171336.6e16eb5a-bxPqe3T81XXwRsdMLXbzog@public.gmane.org>
2016-03-21 14:24                         ` Mark Brown
2016-03-21 14:24                           ` [linux-sunxi] " Mark Brown
2016-03-21 14:24                           ` Mark Brown
2016-03-21 17:54                         ` Maxime Ripard
2016-03-21 17:54                           ` [linux-sunxi] " Maxime Ripard
2016-03-21 17:54                           ` Maxime Ripard
2016-04-21  8:55                           ` Danny Milosavljevic
2016-04-21  8:55                             ` Danny Milosavljevic
2016-04-21  8:55                             ` Danny Milosavljevic
2016-03-19 16:51                 ` Danny Milosavljevic
2016-03-19 16:51                   ` [linux-sunxi] " Danny Milosavljevic
2016-03-19 16:51                   ` Danny Milosavljevic
     [not found]                   ` <20160319175124.6bfe3617-bxPqe3T81XXwRsdMLXbzog@public.gmane.org>
2016-03-21 18:06                     ` Maxime Ripard
2016-03-21 18:06                       ` [linux-sunxi] " Maxime Ripard
2016-03-21 18:06                       ` Maxime Ripard
2016-03-21 18:19                       ` Mark Brown
2016-03-21 18:19                         ` Mark Brown
2016-03-22 12:20                         ` Stefan Monnier

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=20160314104939.GL8418@lukather \
    --to=maxime.ripard-wi1+55scjutkeb57/3fjtnbpr1lh4cv8@public.gmane.org \
    --cc=alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org \
    --cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=dannym-bxPqe3T81XXwRsdMLXbzog@public.gmane.org \
    --cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
    --cc=perex-/Fr2/VpizcU@public.gmane.org \
    --cc=tiwai-IBi9RG/b67k@public.gmane.org \
    --cc=wens-jdAy2FN1RRM@public.gmane.org \
    /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.