Re: [PATCH 4.19 267/346] ALSA: hda/ca0132 - Change Input Source enum strings.

[Takashi Iwai]

On Thu, 31 Dec 2020 19:15:42 +0100,
Pavel Machek wrote:
> 
> Hi!
> 
> > From: Connor McAdams <conmanx360@xxxxxxxxx>
> > 
> > commit 7079f785b50055a32b72eddcb7d9ba5688db24d0 upstream.
> > 
> > Change the Input Source enumerated control's strings to make it play
> > nice with pulseaudio.
> 
> > +++ b/sound/pci/hda/patch_ca0132.c
> > @@ -106,7 +106,7 @@ enum {
> >  };
> >  
> >  /* Strings for Input Source Enum Control */
> > -static const char *const in_src_str[3] = {"Rear Mic", "Line", "Front Mic" };
> > +static const char *const in_src_str[3] = { "Microphone", "Line In", "Front Microphone" };
> >  #define IN_SRC_NUM_OF_INPUTS 3
> 
> If pulseaudio expects the strings to be from small set, should we have
> defines for them?
> 
> If pulseaudio can't understand short versions, do these need fixing,
> too?

I don't believe so.  Even if PA really doesn't understand, it's a
problem of PA itself to be fixed there.

In this particular case, we changed the driver side since it's an
easier path and very specific to the certain model.  But if it's about
the generic parser that is applied to all models, it's a different
story.


thanks,

Takashi