[11/11] ucm: docs: Add DefaultNodeGain
diff mbox series

Message ID 20191107015808.26844-11-cujomalainey@chromium.org
State New
Headers show
Series
  • [01/11] ucm: docs: Add JackName value
Related show

Commit Message

Curtis Malainey Nov. 7, 2019, 1:58 a.m. UTC
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(+)

Comments

Jaroslav Kysela Nov. 11, 2019, 3:34 p.m. UTC | #1
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
Curtis Malainey Nov. 11, 2019, 6:44 p.m. UTC | #2
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.

Patch
diff mbox series

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
  */