Re: [PATCH 1/3] ucm: header - ALSA Use Case Manager

[Liam Girdwood <lrg@slimlogic.co.uk>]

On Tue, 2010-08-24 at 14:09 -0500, Chris Winter wrote:
> Hi Liam,
> 
> On Mon, Aug 23, 2010 at 11:17 AM, Liam Girdwood <lrg@slimlogic.co.uk> wrote:
> > This patch adds the API header for alsa-lib Use Case Management support.
> >
> > This file defines the public interface exported by UCM to client
> > applications.
> >
> > UCM development has been kindly sponsored by Texas Instruments Inc,
> > Wolfson Microelectronics PLC and Slimlogic Ltd.
> >
> > CC: Mark Brown <broonie@opensource.wolfsonmicro.com>
> > Signed-off-by: Stefan Schmidt <stefan@slimlogic.co.uk>
> > Signed-off-by: Jaroslav Kysela <perex@perex.cz>
> > Signed-off-by: Justin Xu <justinx@slimlogic.co.uk>
> > Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
> > ---
> >  include/use-case.h |  427 ++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  1 files changed, 427 insertions(+), 0 deletions(-)
> >  create mode 100644 include/use-case.h
> 
> [...]
> 
> > +/**
> > + * \brief Get ALSA volume/mute control names depending on use case device.
> > + * \param uc_mgr Use case manager
> > + * \param type The control type we are looking for
> > + * \param device The device we are interested in.
> > + * \return name if success, otherwise NULL
> > + *
> > + * Get the control name for common volume and mute controls that are aliased
> > + * in the current use case verb.
> > + */
> > +const char *snd_use_case_get_device_ctl_elem_id(snd_use_case_mgr_t *uc_mgr,
> > +               enum snd_use_case_control_alias type, const char *device);
> > +
> > +/**
> > + * \brief Get ALSA volume/mute control names depending on use case modifier.
> > + * \param uc_mgr Use case manager
> > + * \param type The control type we are looking for
> > + * \param modifier The modifier we are interested in.
> > + * \return name if success, otherwise NULL
> > + *
> > + * Get the control name for common volume and mute controls that are aliased
> > + * in the current use case modifier.
> > + */
> > +const char *snd_use_case_get_modifier_ctl_elem_id(snd_use_case_mgr_t *uc_mgr,
> > +               enum snd_use_case_control_alias type, const char *modifier);
> 
> 
> It would appear that the implementations of the above two functions
> are named differently in src/use-case.c:
> 
> const char *snd_use_case_get_device_ctl_elem_name(...)
> const char *snd_use_case_get_modifier_ctl_elem_name(...)
> 

Ah, they are indeed. I'll fix up. 

Thanks !

Liam
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk