Alsa-Devel Archive on lore.kernel.org
 help / color / Atom feed
* [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value
@ 2019-11-07  1:57 Curtis Malainey
  2019-11-07  1:57 ` [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value Curtis Malainey
                   ` (11 more replies)
  0 siblings, 12 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:57 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

This field represents the input device name for a jack detection line

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 5cb4f31c..2051bd40 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -320,6 +320,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *        trick upper software layers to e.g. automatically mute speakers when
  *        headphones are plugged in, but that's application policy
  *        configuration that doesn't belong to UCM configuration files.
+ *   - JackName
+ *      - Input name is the input device name for the jack
  */
 int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
                      const char *identifier,
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
@ 2019-11-07  1:57 ` Curtis Malainey
  2019-11-11 13:39   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 03/11] ucm: docs: add value JackSwitch Curtis Malainey
                   ` (10 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:57 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Identifies the type of jack and how it should be accessed

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 2051bd40..3208cc30 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -322,6 +322,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *        configuration that doesn't belong to UCM configuration files.
  *   - JackName
  *      - Input name is the input device name for the jack
+ *   - JackType
+ *      - Specifies whether the jack is accessed via hctl or gpio and therefore
+ *        only carries the possible values of "gpio" or "hctl"
  */
 int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
                      const char *identifier,
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 03/11] ucm: docs: add value JackSwitch
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
  2019-11-07  1:57 ` [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 13:30   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 04/11] ucm: docs: Add EDID File value Curtis Malainey
                   ` (9 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Identifies event code that will be sent by the device on an jack detect
event

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 3208cc30..a79e5b91 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -322,6 +322,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *        configuration that doesn't belong to UCM configuration files.
  *   - JackName
  *      - Input name is the input device name for the jack
+ *   - JackSwitch
+ *      - The type of event code the jack will send, corresponds to
+ *        Switch Events in include/uapi/linux/input-event-codes.h in the kernel
  *   - JackType
  *      - Specifies whether the jack is accessed via hctl or gpio and therefore
  *        only carries the possible values of "gpio" or "hctl"
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 04/11] ucm: docs: Add EDID File value
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
  2019-11-07  1:57 ` [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value Curtis Malainey
  2019-11-07  1:58 ` [alsa-devel] [PATCH 03/11] ucm: docs: add value JackSwitch Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 15:18   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value Curtis Malainey
                   ` (8 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Add a sys path to the EDID file for the corresponding HDMI device so
userspace can use if needed

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index a79e5b91..28e92d17 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -301,6 +301,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *      - name of capture mixer
  *   - CaptureMixerID
  *      - mixer capture ID
+ *   - EDIDFile
+ *      - Path to EDID file for HDMI devices
  *   - JackControl, JackDev, JackHWMute
  *      - Jack information for a device. The jack status can be reported via
  *        a kcontrol and/or via an input device. **JackControl** is the
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (2 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 04/11] ucm: docs: Add EDID File value Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 14:55   ` Jaroslav Kysela
  2019-11-11 15:19   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume Curtis Malainey
                   ` (7 subsequent siblings)
  11 siblings, 2 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Allows ucm to override minimum buffer level for platforms that
incorrectly report their buffer level.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 28e92d17..05628303 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -330,6 +330,10 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *   - JackType
  *      - Specifies whether the jack is accessed via hctl or gpio and therefore
  *        only carries the possible values of "gpio" or "hctl"
+ *   - MinBufferLevel
+ *      - This is used on platform where reported buffer level is not accurate.
+ *        E.g. "512", which holds 512 samples in device buffer. Note: this will
+ *        increase latency.
  */
 int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
                      const char *identifier,
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (3 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 15:22   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap Curtis Malainey
                   ` (6 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Some userspace audio servers might default to software volume, this
allows the ucm to force its behaviour

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 05628303..1aeaf9d4 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -334,6 +334,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *      - This is used on platform where reported buffer level is not accurate.
  *        E.g. "512", which holds 512 samples in device buffer. Note: this will
  *        increase latency.
+ *   - DisableSoftwareVolume
+ *      - Set to "1" to force userspace to use hardware volume controls
  */
 int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
                      const char *identifier,
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (4 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 15:30   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 08/11] " Curtis Malainey
                   ` (5 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Add ucm value to tell userspace to remap channels. This is useful for
systems with DMICs where there may be more channels than active
channels.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 1aeaf9d4..31f9e4be 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -301,6 +301,10 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *      - name of capture mixer
  *   - CaptureMixerID
  *      - mixer capture ID
+ *   - CaptureChannelMap
+ *      - Remap channels using ALSA PCM channel mapping API notation
+ *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
+ *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
  *   - EDIDFile
  *      - Path to EDID file for HDMI devices
  *   - JackControl, JackDev, JackHWMute
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 08/11] ucm: docs: Add CaptureChannelMap
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (5 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap Curtis Malainey
@ 2019-11-07  1:58 ` " Curtis Malainey
  2019-11-07  2:02   ` Curtis Malainey
  2019-11-11 15:31   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev Curtis Malainey
                   ` (4 subsequent siblings)
  11 siblings, 2 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Explains to userspace how some channels may need to be rearranged. This
is useful for systems that may have DMIC pcms with dead channels that
will need remapping.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 31f9e4be..9a10f390 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -305,6 +305,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *      - Remap channels using ALSA PCM channel mapping API notation
  *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
  *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
+ *   - CoupledMixers
+ *      - Mixer controls that should be changed together.
+ *        E.g. "Left Master,Right Master".
  *   - EDIDFile
  *      - Path to EDID file for HDMI devices
  *   - JackControl, JackDev, JackHWMute
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (6 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 08/11] " Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 15:36   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain Curtis Malainey
                   ` (3 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Sometimes userspace may want to use a reference channel to cancel echos
when using video chat, this value identifies the device which carries
that channel.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index 9a10f390..e84f3f8f 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -308,6 +308,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *   - CoupledMixers
  *      - Mixer controls that should be changed together.
  *        E.g. "Left Master,Right Master".
+ *   - EchoReferenceDev
+ *      - If userspace wants an echo refence channel this value identifies the
+ *        device to be used.
  *   - EDIDFile
  *      - Path to EDID file for HDMI devices
  *   - JackControl, JackDev, JackHWMute
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (7 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 15:33   ` Jaroslav Kysela
  2019-11-07  1:58 ` [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain Curtis Malainey
                   ` (2 subsequent siblings)
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Add software gain ucm definition.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index e84f3f8f..ee6ee5e0 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -344,6 +344,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *      - This is used on platform where reported buffer level is not accurate.
  *        E.g. "512", which holds 512 samples in device buffer. Note: this will
  *        increase latency.
+ *   - SoftwareGain
+ *      - Specify the software gain in 0.01dB. E.g. "20" means there will be
+ *        0.2 dB software gain on this node.
  *   - DisableSoftwareVolume
  *      - Set to "1" to force userspace to use hardware volume controls
  */
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (8 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain Curtis Malainey
@ 2019-11-07  1:58 ` Curtis Malainey
  2019-11-11 15:34   ` Jaroslav Kysela
  2019-11-11  9:44 ` [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Kai Vehmanen
  2019-11-11 13:26 ` Jaroslav Kysela
  11 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  1:58 UTC (permalink / raw)
  To: alsa-devel; +Cc: Curtis Malainey, broonie

Allow ucm to specify a default hardware gain to overide on the respective node.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
---
 include/use-case.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/use-case.h b/include/use-case.h
index ee6ee5e0..d0e821d5 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -347,6 +347,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
  *   - SoftwareGain
  *      - Specify the software gain in 0.01dB. E.g. "20" means there will be
  *        0.2 dB software gain on this node.
+ *   - DefaultNodeGain
+ *      - Change the default node gain. E.g: “-700” to set -7 dB on this node
  *   - DisableSoftwareVolume
  *      - Set to "1" to force userspace to use hardware volume controls
  */
-- 
2.24.0.432.g9d3f5f5b63-goog

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 08/11] ucm: docs: Add CaptureChannelMap
  2019-11-07  1:58 ` [alsa-devel] [PATCH 08/11] " Curtis Malainey
@ 2019-11-07  2:02   ` Curtis Malainey
  2019-11-11 15:31   ` Jaroslav Kysela
  1 sibling, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-07  2:02 UTC (permalink / raw)
  To: Curtis Malainey; +Cc: ALSA development, Mark Brown

On Wed, Nov 6, 2019 at 5:58 PM Curtis Malainey
<cujomalainey@chromium.org> wrote:
>
> Explains to userspace how some channels may need to be rearranged. This
> is useful for systems that may have DMIC pcms with dead channels that
> will need remapping.
>
Apologies this is the coupled mixer patch, this is an error.
I will resend this with anything else that needs to be corrected
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>  include/use-case.h | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/include/use-case.h b/include/use-case.h
> index 31f9e4be..9a10f390 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -305,6 +305,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>   *      - Remap channels using ALSA PCM channel mapping API notation
>   *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
>   *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
> + *   - CoupledMixers
> + *      - Mixer controls that should be changed together.
> + *        E.g. "Left Master,Right Master".
>   *   - EDIDFile
>   *      - Path to EDID file for HDMI devices
>   *   - JackControl, JackDev, JackHWMute
> --
> 2.24.0.432.g9d3f5f5b63-goog
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (9 preceding siblings ...)
  2019-11-07  1:58 ` [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain Curtis Malainey
@ 2019-11-11  9:44 ` Kai Vehmanen
  2019-11-11 13:26 ` Jaroslav Kysela
  11 siblings, 0 replies; 32+ messages in thread
From: Kai Vehmanen @ 2019-11-11  9:44 UTC (permalink / raw)
  To: Curtis Malainey; +Cc: alsa-devel, broonie

Hi Curtis,

On Wed, 6 Nov 2019, Curtis Malainey wrote:

> This field represents the input device name for a jack detection line
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>

this series looks good to me and I think it's a very good idea to bring 
extensions that have proven useful in Chromium to alsa-lib.

For reviewing, a bit of background context and pointers would probably 
help the wider audience. E.g. I found this very useful:

https://www.chromium.org/chromium-os/chromiumos-design-docs/cras-chromeos-audio-server
-> 
https://docs.google.com/document/d/1AcXZI9dvJBW0Vy6h9vraD7VP9X_MpwHx0eJ6hNc4G_s/edit?usp=sharing

Not directly related to this series, but it would be nice if the 
versioning would be aligned as well (especially now as ucmv2 is being 
proposed).

Br, Kai
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value
  2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
                   ` (10 preceding siblings ...)
  2019-11-11  9:44 ` [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Kai Vehmanen
@ 2019-11-11 13:26 ` Jaroslav Kysela
  11 siblings, 0 replies; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 13:26 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:57 Curtis Malainey napsal(a):
> This field represents the input device name for a jack detection line
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 5cb4f31c..2051bd40 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -320,6 +320,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *        trick upper software layers to e.g. automatically mute speakers when
>    *        headphones are plugged in, but that's application policy
>    *        configuration that doesn't belong to UCM configuration files.
> + *   - JackName
> + *      - Input name is the input device name for the jack
>    */
>   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
>                        const char *identifier,
> 
What is the difference from JackDev ?

				Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 03/11] ucm: docs: add value JackSwitch
  2019-11-07  1:58 ` [alsa-devel] [PATCH 03/11] ucm: docs: add value JackSwitch Curtis Malainey
@ 2019-11-11 13:30   ` Jaroslav Kysela
  0 siblings, 0 replies; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 13:30 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Identifies event code that will be sent by the device on an jack detect
> event
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 3208cc30..a79e5b91 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -322,6 +322,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *        configuration that doesn't belong to UCM configuration files.
>    *   - JackName
>    *      - Input name is the input device name for the jack
> + *   - JackSwitch
> + *      - The type of event code the jack will send, corresponds to
> + *        Switch Events in include/uapi/linux/input-event-codes.h in the kernel
>    *   - JackType
>    *      - Specifies whether the jack is accessed via hctl or gpio and therefore
>    *        only carries the possible values of "gpio" or "hctl"
> 

It's a string or hexa value or so? It requires a better description.

				Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value
  2019-11-07  1:57 ` [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value Curtis Malainey
@ 2019-11-11 13:39   ` Jaroslav Kysela
  0 siblings, 0 replies; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 13:39 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:57 Curtis Malainey napsal(a):
> Identifies the type of jack and how it should be accessed
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 2051bd40..3208cc30 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -322,6 +322,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *        configuration that doesn't belong to UCM configuration files.
>    *   - JackName
>    *      - Input name is the input device name for the jack
> + *   - JackType
> + *      - Specifies whether the jack is accessed via hctl or gpio and therefore
> + *        only carries the possible values of "gpio" or "hctl"
>    */
>   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
>                        const char *identifier,
> 

What is meant with the "gpio" type? The standard input device interface? I 
believe it should be "inputdev" and "ctl" (hctl is just interface on top of 
ctl and the application can access the jack through snd_ctl functions, too.

					Jaroslav



-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value
  2019-11-07  1:58 ` [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value Curtis Malainey
@ 2019-11-11 14:55   ` Jaroslav Kysela
  2019-11-11 15:19   ` Jaroslav Kysela
  1 sibling, 0 replies; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 14:55 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: Takashi Iwai, broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Allows ucm to override minimum buffer level for platforms that
> incorrectly report their buffer level.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 28e92d17..05628303 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -330,6 +330,10 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *   - JackType
>    *      - Specifies whether the jack is accessed via hctl or gpio and therefore
>    *        only carries the possible values of "gpio" or "hctl"
> + *   - MinBufferLevel
> + *      - This is used on platform where reported buffer level is not accurate.
> + *        E.g. "512", which holds 512 samples in device buffer. Note: this will
> + *        increase latency.
>    */
>   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
>                        const char *identifier,
> 

This is something like the BATCH flag we have in the PCM API. It would be 
probably nice to add this to the PCM API, too. Takashi?

				Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 04/11] ucm: docs: Add EDID File value
  2019-11-07  1:58 ` [alsa-devel] [PATCH 04/11] ucm: docs: Add EDID File value Curtis Malainey
@ 2019-11-11 15:18   ` Jaroslav Kysela
  0 siblings, 0 replies; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:18 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Add a sys path to the EDID file for the corresponding HDMI device so
> userspace can use if needed
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index a79e5b91..28e92d17 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -301,6 +301,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *      - name of capture mixer
>    *   - CaptureMixerID
>    *      - mixer capture ID
> + *   - EDIDFile
> + *      - Path to EDID file for HDMI devices
>    *   - JackControl, JackDev, JackHWMute
>    *      - Jack information for a device. The jack status can be reported via
>    *        a kcontrol and/or via an input device. **JackControl** is the
> 

Applied to alsa-lib's git tree:

https://git.alsa-project.org/?p=alsa-lib.git;a=commit;h=ef085a55c488f64834fca233355b9881489be10c

			Thanks,
				Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value
  2019-11-07  1:58 ` [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value Curtis Malainey
  2019-11-11 14:55   ` Jaroslav Kysela
@ 2019-11-11 15:19   ` Jaroslav Kysela
  1 sibling, 0 replies; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:19 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Allows ucm to override minimum buffer level for platforms that
> incorrectly report their buffer level.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 28e92d17..05628303 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -330,6 +330,10 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *   - JackType
>    *      - Specifies whether the jack is accessed via hctl or gpio and therefore
>    *        only carries the possible values of "gpio" or "hctl"
> + *   - MinBufferLevel
> + *      - This is used on platform where reported buffer level is not accurate.
> + *        E.g. "512", which holds 512 samples in device buffer. Note: this will
> + *        increase latency.
>    */
>   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
>                        const char *identifier,
> 

Applied to alsa-lib's git tree:

https://git.alsa-project.org/?p=alsa-lib.git;a=commit;h=88c69ed2c865f6b857c26c04f5fc56d9396437f8

				Thanks,
					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume
  2019-11-07  1:58 ` [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume Curtis Malainey
@ 2019-11-11 15:22   ` Jaroslav Kysela
  2019-11-11 18:51     ` Curtis Malainey
  0 siblings, 1 reply; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:22 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Some userspace audio servers might default to software volume, this
> allows the ucm to force its behaviour
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 05628303..1aeaf9d4 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -334,6 +334,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *      - This is used on platform where reported buffer level is not accurate.
>    *        E.g. "512", which holds 512 samples in device buffer. Note: this will
>    *        increase latency.
> + *   - DisableSoftwareVolume
> + *      - Set to "1" to force userspace to use hardware volume controls
>    */
>   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
>                        const char *identifier,
> 


My understanding is that if the hardware volume controls are defined in UCM, 
then the UCM user (audio server) should use them. Otherwise, the software 
volume control should be applied. I don't see the reason to have this info 
doubled.

					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap
  2019-11-07  1:58 ` [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap Curtis Malainey
@ 2019-11-11 15:30   ` Jaroslav Kysela
  2019-11-11 22:08     ` Curtis Malainey
  0 siblings, 1 reply; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:30 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Add ucm value to tell userspace to remap channels. This is useful for
> systems with DMICs where there may be more channels than active
> channels.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 1aeaf9d4..31f9e4be 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -301,6 +301,10 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *      - name of capture mixer
>    *   - CaptureMixerID
>    *      - mixer capture ID
> + *   - CaptureChannelMap
> + *      - Remap channels using ALSA PCM channel mapping API notation
> + *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
> + *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
>    *   - EDIDFile
>    *      - Path to EDID file for HDMI devices
>    *   - JackControl, JackDev, JackHWMute
> 

I see a room for the improvement to have a better and complete mapping 
description like:

CaptureChannelMap "FR=0 RL=1 TC=2"

The location names should follow SND_CHMAP defines.

					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 08/11] ucm: docs: Add CaptureChannelMap
  2019-11-07  1:58 ` [alsa-devel] [PATCH 08/11] " Curtis Malainey
  2019-11-07  2:02   ` Curtis Malainey
@ 2019-11-11 15:31   ` Jaroslav Kysela
  2019-11-11 22:09     ` Curtis Malainey
  1 sibling, 1 reply; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:31 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Explains to userspace how some channels may need to be rearranged. This
> is useful for systems that may have DMIC pcms with dead channels that
> will need remapping.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 31f9e4be..9a10f390 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -305,6 +305,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *      - Remap channels using ALSA PCM channel mapping API notation
>    *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
>    *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
> + *   - CoupledMixers
> + *      - Mixer controls that should be changed together.
> + *        E.g. "Left Master,Right Master".
>    *   - EDIDFile
>    *      - Path to EDID file for HDMI devices
>    *   - JackControl, JackDev, JackHWMute
> 

I do not see which names are referred by this. It's for the snd_ctl interface 
or for the simple mixer ?

						Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain
  2019-11-07  1:58 ` [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain Curtis Malainey
@ 2019-11-11 15:33   ` Jaroslav Kysela
  2019-11-11 18:46     ` Curtis Malainey
  0 siblings, 1 reply; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:33 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Add software gain ucm definition.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index e84f3f8f..ee6ee5e0 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -344,6 +344,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *      - This is used on platform where reported buffer level is not accurate.
>    *        E.g. "512", which holds 512 samples in device buffer. Note: this will
>    *        increase latency.
> + *   - SoftwareGain
> + *      - Specify the software gain in 0.01dB. E.g. "20" means there will be
> + *        0.2 dB software gain on this node.
>    *   - DisableSoftwareVolume
>    *      - Set to "1" to force userspace to use hardware volume controls
>    */
> 

In other words, it's the recommended software gain control? The description 
should be improved then.

						Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain
  2019-11-07  1:58 ` [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain Curtis Malainey
@ 2019-11-11 15:34   ` Jaroslav Kysela
  2019-11-11 18:44     ` Curtis Malainey
  0 siblings, 1 reply; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:34 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Allow ucm to specify a default hardware gain to overide on the respective node.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index ee6ee5e0..d0e821d5 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -347,6 +347,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *   - SoftwareGain
>    *      - Specify the software gain in 0.01dB. E.g. "20" means there will be
>    *        0.2 dB software gain on this node.
> + *   - DefaultNodeGain
> + *      - Change the default node gain. E.g: “-700” to set -7 dB on this node
>    *   - DisableSoftwareVolume
>    *      - Set to "1" to force userspace to use hardware volume controls
>    */
> 

I don't see the usage. Could you elaborate a bit which purpose is for this value?

					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev
  2019-11-07  1:58 ` [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev Curtis Malainey
@ 2019-11-11 15:36   ` Jaroslav Kysela
  2019-11-11 18:42     ` Curtis Malainey
  0 siblings, 1 reply; 32+ messages in thread
From: Jaroslav Kysela @ 2019-11-11 15:36 UTC (permalink / raw)
  To: Curtis Malainey, alsa-devel; +Cc: broonie

Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> Sometimes userspace may want to use a reference channel to cancel echos
> when using video chat, this value identifies the device which carries
> that channel.
> 
> Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> ---
>   include/use-case.h | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 9a10f390..e84f3f8f 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -308,6 +308,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
>    *   - CoupledMixers
>    *      - Mixer controls that should be changed together.
>    *        E.g. "Left Master,Right Master".
> + *   - EchoReferenceDev
> + *      - If userspace wants an echo refence channel this value identifies the
> + *        device to be used.
>    *   - EDIDFile
>    *      - Path to EDID file for HDMI devices
>    *   - JackControl, JackDev, JackHWMute
> 

It's PCM device name like PlaybackPCM ? I would rename this to 
EchoReferencePCM then.

					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev
  2019-11-11 15:36   ` Jaroslav Kysela
@ 2019-11-11 18:42     ` Curtis Malainey
  0 siblings, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 18:42 UTC (permalink / raw)
  To: Jaroslav Kysela; +Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 7:36 AM Jaroslav Kysela <perex@perex.cz> wrote:
>
> Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > Sometimes userspace may want to use a reference channel to cancel echos
> > when using video chat, this value identifies the device which carries
> > that channel.
> >
> > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > ---
> >   include/use-case.h | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/include/use-case.h b/include/use-case.h
> > index 9a10f390..e84f3f8f 100644
> > --- a/include/use-case.h
> > +++ b/include/use-case.h
> > @@ -308,6 +308,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> >    *   - CoupledMixers
> >    *      - Mixer controls that should be changed together.
> >    *        E.g. "Left Master,Right Master".
> > + *   - EchoReferenceDev
> > + *      - If userspace wants an echo refence channel this value identifies the
> > + *        device to be used.
> >    *   - EDIDFile
> >    *      - Path to EDID file for HDMI devices
> >    *   - JackControl, JackDev, JackHWMute
> >
>
> It's PCM device name like PlaybackPCM ? I would rename this to
> EchoReferencePCM then.
>
>                                         Jaroslav
Not quite. The name associated is another SectionDevice.
Although I will have to send a v2 for this patch as I noticed a typo "refence"
>
> --
> Jaroslav Kysela <perex@perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain
  2019-11-11 15:34   ` Jaroslav Kysela
@ 2019-11-11 18:44     ` Curtis Malainey
  0 siblings, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 18:44 UTC (permalink / raw)
  To: Jaroslav Kysela; +Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 7:34 AM Jaroslav Kysela <perex@perex.cz> wrote:
>
> Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > Allow ucm to specify a default hardware gain to overide on the respective node.
> >
> > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > ---
> >   include/use-case.h | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/include/use-case.h b/include/use-case.h
> > index ee6ee5e0..d0e821d5 100644
> > --- a/include/use-case.h
> > +++ b/include/use-case.h
> > @@ -347,6 +347,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> >    *   - SoftwareGain
> >    *      - Specify the software gain in 0.01dB. E.g. "20" means there will be
> >    *        0.2 dB software gain on this node.
> > + *   - DefaultNodeGain
> > + *      - Change the default node gain. E.g: “-700” to set -7 dB on this node
> >    *   - DisableSoftwareVolume
> >    *      - Set to "1" to force userspace to use hardware volume controls
> >    */
> >
>
> I don't see the usage. Could you elaborate a bit which purpose is for this value?
>
>                                         Jaroslav
>
This along with software gain actually should be dropped from the
patchset. We are modifying our use case to make it more configurable
for multi-stream use. The original use case is for systems like
hotword capture streams.
> --
> Jaroslav Kysela <perex@perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain
  2019-11-11 15:33   ` Jaroslav Kysela
@ 2019-11-11 18:46     ` Curtis Malainey
  0 siblings, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 18:46 UTC (permalink / raw)
  To: Jaroslav Kysela; +Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 7:33 AM Jaroslav Kysela <perex@perex.cz> wrote:
>
> Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > Add software gain ucm definition.
> >
> > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > ---
> >   include/use-case.h | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/include/use-case.h b/include/use-case.h
> > index e84f3f8f..ee6ee5e0 100644
> > --- a/include/use-case.h
> > +++ b/include/use-case.h
> > @@ -344,6 +344,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> >    *      - This is used on platform where reported buffer level is not accurate.
> >    *        E.g. "512", which holds 512 samples in device buffer. Note: this will
> >    *        increase latency.
> > + *   - SoftwareGain
> > + *      - Specify the software gain in 0.01dB. E.g. "20" means there will be
> > + *        0.2 dB software gain on this node.
> >    *   - DisableSoftwareVolume
> >    *      - Set to "1" to force userspace to use hardware volume controls
> >    */
> >
>
> In other words, it's the recommended software gain control? The description
> should be improved then.
>
>                                                 Jaroslav
>
Not the control/mixer, in this case it is a gain value that is
expected to be applied by the audio server (should the server parse
it)
> --
> Jaroslav Kysela <perex@perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume
  2019-11-11 15:22   ` Jaroslav Kysela
@ 2019-11-11 18:51     ` Curtis Malainey
  2019-11-11 19:11       ` Curtis Malainey
  0 siblings, 1 reply; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 18:51 UTC (permalink / raw)
  To: Jaroslav Kysela; +Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 7:22 AM Jaroslav Kysela <perex@perex.cz> wrote:
>
> Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > Some userspace audio servers might default to software volume, this
> > allows the ucm to force its behaviour
> >
> > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > ---
> >   include/use-case.h | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/include/use-case.h b/include/use-case.h
> > index 05628303..1aeaf9d4 100644
> > --- a/include/use-case.h
> > +++ b/include/use-case.h
> > @@ -334,6 +334,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> >    *      - This is used on platform where reported buffer level is not accurate.
> >    *        E.g. "512", which holds 512 samples in device buffer. Note: this will
> >    *        increase latency.
> > + *   - DisableSoftwareVolume
> > + *      - Set to "1" to force userspace to use hardware volume controls
> >    */
> >   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
> >                        const char *identifier,
> >
>
>
> My understanding is that if the hardware volume controls are defined in UCM,
> then the UCM user (audio server) should use them. Otherwise, the software
> volume control should be applied. I don't see the reason to have this info
> doubled.
>
>                                         Jaroslav
>
+Jimmy Cheng-Yi Chiang can you clarify in what context CRAS might need
this value? The only cases I see are for Chat 150 C and Jabra SPEAK
810. There are no volume mixers associated with these files.
> --
> Jaroslav Kysela <perex@perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume
  2019-11-11 18:51     ` Curtis Malainey
@ 2019-11-11 19:11       ` Curtis Malainey
  0 siblings, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 19:11 UTC (permalink / raw)
  To: Jaroslav Kysela, Jimmy Cheng-Yi Chiang
  Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 10:51 AM Curtis Malainey
<cujomalainey@google.com> wrote:
>
> On Mon, Nov 11, 2019 at 7:22 AM Jaroslav Kysela <perex@perex.cz> wrote:
> >
> > Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > > Some userspace audio servers might default to software volume, this
> > > allows the ucm to force its behaviour
> > >
> > > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > > ---
> > >   include/use-case.h | 2 ++
> > >   1 file changed, 2 insertions(+)
> > >
> > > diff --git a/include/use-case.h b/include/use-case.h
> > > index 05628303..1aeaf9d4 100644
> > > --- a/include/use-case.h
> > > +++ b/include/use-case.h
> > > @@ -334,6 +334,8 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> > >    *      - This is used on platform where reported buffer level is not accurate.
> > >    *        E.g. "512", which holds 512 samples in device buffer. Note: this will
> > >    *        increase latency.
> > > + *   - DisableSoftwareVolume
> > > + *      - Set to "1" to force userspace to use hardware volume controls
> > >    */
> > >   int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
> > >                        const char *identifier,
> > >
> >
> >
> > My understanding is that if the hardware volume controls are defined in UCM,
> > then the UCM user (audio server) should use them. Otherwise, the software
> > volume control should be applied. I don't see the reason to have this info
> > doubled.
> >
> >                                         Jaroslav
> >
> +Jimmy Cheng-Yi Chiang can you clarify in what context CRAS might need
> this value? The only cases I see are for Chat 150 C and Jabra SPEAK
> 810. There are no volume mixers associated with these files.
+Jimmy Cheng-Yi Chiang retry adding Jimmy to email chain
> > --
> > Jaroslav Kysela <perex@perex.cz>
> > Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap
  2019-11-11 15:30   ` Jaroslav Kysela
@ 2019-11-11 22:08     ` Curtis Malainey
  0 siblings, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 22:08 UTC (permalink / raw)
  To: Jaroslav Kysela; +Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 7:30 AM Jaroslav Kysela <perex@perex.cz> wrote:
>
> Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > Add ucm value to tell userspace to remap channels. This is useful for
> > systems with DMICs where there may be more channels than active
> > channels.
> >
> > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > ---
> >   include/use-case.h | 4 ++++
> >   1 file changed, 4 insertions(+)
> >
> > diff --git a/include/use-case.h b/include/use-case.h
> > index 1aeaf9d4..31f9e4be 100644
> > --- a/include/use-case.h
> > +++ b/include/use-case.h
> > @@ -301,6 +301,10 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> >    *      - name of capture mixer
> >    *   - CaptureMixerID
> >    *      - mixer capture ID
> > + *   - CaptureChannelMap
> > + *      - Remap channels using ALSA PCM channel mapping API notation
> > + *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
> > + *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
> >    *   - EDIDFile
> >    *      - Path to EDID file for HDMI devices
> >    *   - JackControl, JackDev, JackHWMute
> >
>
> I see a room for the improvement to have a better and complete mapping
> description like:
>
> CaptureChannelMap "FR=0 RL=1 TC=2"
>
> The location names should follow SND_CHMAP defines.
>
>                                         Jaroslav
>
I am definitely for that modification as the mapping for the plugin is
not easy to read, that being said, do we have a tool to handle
parsing/converting that from a string already? I feel like we should
add one if we are going to make that the spec.
> --
> Jaroslav Kysela <perex@perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [alsa-devel] [PATCH 08/11] ucm: docs: Add CaptureChannelMap
  2019-11-11 15:31   ` Jaroslav Kysela
@ 2019-11-11 22:09     ` Curtis Malainey
  0 siblings, 0 replies; 32+ messages in thread
From: Curtis Malainey @ 2019-11-11 22:09 UTC (permalink / raw)
  To: Jaroslav Kysela; +Cc: Curtis Malainey, Mark Brown, ALSA development

On Mon, Nov 11, 2019 at 7:32 AM Jaroslav Kysela <perex@perex.cz> wrote:
>
> Dne 07. 11. 19 v 2:58 Curtis Malainey napsal(a):
> > Explains to userspace how some channels may need to be rearranged. This
> > is useful for systems that may have DMIC pcms with dead channels that
> > will need remapping.
> >
> > Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
> > ---
> >   include/use-case.h | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/include/use-case.h b/include/use-case.h
> > index 31f9e4be..9a10f390 100644
> > --- a/include/use-case.h
> > +++ b/include/use-case.h
> > @@ -305,6 +305,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
> >    *      - Remap channels using ALSA PCM channel mapping API notation
> >    *        E.g. "2 3 0 1 -1 -1 -1 -1 -1 -1 -1" means, FL takes channel 2,
> >    *        FR takes channel 3, RL takes channel 0, RL takes channel 1.
> > + *   - CoupledMixers
> > + *      - Mixer controls that should be changed together.
> > + *        E.g. "Left Master,Right Master".
> >    *   - EDIDFile
> >    *      - Path to EDID file for HDMI devices
> >    *   - JackControl, JackDev, JackHWMute
> >
>
> I do not see which names are referred by this. It's for the snd_ctl interface
> or for the simple mixer ?
>
>                                                 Jaroslav
>
Yes, this would be referring to volume mixers in amixer.
> --
> Jaroslav Kysela <perex@perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, back to index

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-07  1:57 [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Curtis Malainey
2019-11-07  1:57 ` [alsa-devel] [PATCH 02/11] ucm: docs: Add JackType value Curtis Malainey
2019-11-11 13:39   ` Jaroslav Kysela
2019-11-07  1:58 ` [alsa-devel] [PATCH 03/11] ucm: docs: add value JackSwitch Curtis Malainey
2019-11-11 13:30   ` Jaroslav Kysela
2019-11-07  1:58 ` [alsa-devel] [PATCH 04/11] ucm: docs: Add EDID File value Curtis Malainey
2019-11-11 15:18   ` Jaroslav Kysela
2019-11-07  1:58 ` [alsa-devel] [PATCH 05/11] ucm: docs: Add MinBufferLevel value Curtis Malainey
2019-11-11 14:55   ` Jaroslav Kysela
2019-11-11 15:19   ` Jaroslav Kysela
2019-11-07  1:58 ` [alsa-devel] [PATCH 06/11] ucm: docs: Add DisableSoftwareVolume Curtis Malainey
2019-11-11 15:22   ` Jaroslav Kysela
2019-11-11 18:51     ` Curtis Malainey
2019-11-11 19:11       ` Curtis Malainey
2019-11-07  1:58 ` [alsa-devel] [PATCH 07/11] ucm: docs: Add CaptureChannelMap Curtis Malainey
2019-11-11 15:30   ` Jaroslav Kysela
2019-11-11 22:08     ` Curtis Malainey
2019-11-07  1:58 ` [alsa-devel] [PATCH 08/11] " Curtis Malainey
2019-11-07  2:02   ` Curtis Malainey
2019-11-11 15:31   ` Jaroslav Kysela
2019-11-11 22:09     ` Curtis Malainey
2019-11-07  1:58 ` [alsa-devel] [PATCH 09/11] ucm: docs: Add EchoReferenceDev Curtis Malainey
2019-11-11 15:36   ` Jaroslav Kysela
2019-11-11 18:42     ` Curtis Malainey
2019-11-07  1:58 ` [alsa-devel] [PATCH 10/11] ucm: docs: Add SoftwareGain Curtis Malainey
2019-11-11 15:33   ` Jaroslav Kysela
2019-11-11 18:46     ` Curtis Malainey
2019-11-07  1:58 ` [alsa-devel] [PATCH 11/11] ucm: docs: Add DefaultNodeGain Curtis Malainey
2019-11-11 15:34   ` Jaroslav Kysela
2019-11-11 18:44     ` Curtis Malainey
2019-11-11  9:44 ` [alsa-devel] [PATCH 01/11] ucm: docs: Add JackName value Kai Vehmanen
2019-11-11 13:26 ` Jaroslav Kysela

Alsa-Devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/alsa-devel/0 alsa-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 alsa-devel alsa-devel/ https://lore.kernel.org/alsa-devel \
		alsa-devel@alsa-project.org
	public-inbox-index alsa-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.alsa-project.alsa-devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git