lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 01 Jan 2021 09:02:16 +0100 From: Takashi Iwai <tiwai@...e.de> To: Pavel Machek <pavel@....cz> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-kernel@...r.kernel.org, stable@...r.kernel.org, stable@...nel.org, Connor McAdams <conmanx360@...il.com>, Takashi Iwai <tiwai@...e.de> Subject: Re: [PATCH 4.19 267/346] ALSA: hda/ca0132 - Change Input Source enum strings. On Thu, 31 Dec 2020 19:15:42 +0100, Pavel Machek wrote: > > Hi! > > > From: Connor McAdams <conmanx360@...il.com> > > > > 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
Powered by blists - more mailing lists