December 2021 - Alsa-devel - mailman.alsa-project.org

[PATCH] sound: ppc: beep: fix clang -Wimplicit-fallthrough
by Anders Roxell 07 Dec '21

07 Dec '21

Clang warns: sound/ppc/beep.c:103:2: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough] case SND_TONE: break; ^ sound/ppc/beep.c:103:2: note: insert 'break;' to avoid fall-through case SND_TONE: break; ^ break; 1 warning generated. Clang is more pedantic than GCC, which does not warn when failing through to a case that is just break or return. Clang's version is more in line with the kernel's own stance in deprecated.rst. Add athe missing break to silence the warning. Reported-by: Naresh Kamboju <naresh.kamboju(a)linaro.org> Signed-off-by: Anders Roxell <anders.roxell(a)linaro.org> --- sound/ppc/beep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/ppc/beep.c b/sound/ppc/beep.c index 0f4bce1c0d4f..bf289783eafd 100644 --- a/sound/ppc/beep.c +++ b/sound/ppc/beep.c @@ -99,7 +99,7 @@ static int snd_pmac_beep_event(struct input_dev *dev, unsigned int type, return -1; switch (code) { - case SND_BELL: if (hz) hz = 1000; + case SND_BELL: if (hz) hz = 1000; break; case SND_TONE: break; default: return -1; } -- 2.33.0

2 1

Logitech Z10 USB speakers need a volume change before audio works
by Hans de Goede 07 Dec '21

07 Dec '21

Hi, I've a set of Logitech Z10 USB speakers, which act as a USB soundcard. They have this weird glitch where after turning off my PC (and their power-supply as well) and then turning things back on, they are silent until I change the PCM volume control for the speakers inside alsa-mixer. It seems like they need some "set-volume" command to be send over the USB bus to unmute them when initially powered-up / turned on. Is their some existing usb-audio quirk which I can try to work around this? Regards, Hans

2 8

Add headset Mic supported for Lenovo platform
by Kailang 07 Dec '21

07 Dec '21

Hi Takashi, Attach patch will enable headset Mic support on Lenovo platform. Many Thanks. BR, Kailang

2 1

[PATCH] ALSA: mixart: Reduce size of mixart_timer_notify
by Kees Cook 07 Dec '21

07 Dec '21

The mixart_timer_notify structure was larger than could be represented by the mixart_msg_data array storage. Adjust the size to as large as possible to fix the warning seen with -Warray-bounds builds: sound/pci/mixart/mixart_core.c: In function 'snd_mixart_threaded_irq': sound/pci/mixart/mixart_core.c:447:50: error: array subscript 'struct mixart_timer_notify[0]' is partly outside array bounds of 'u32[128]' {aka 'unsigned int[128]'} [-Werror=array-bounds] 447 | for(i=0; i<notify->stream_count; i++) { | ^~ sound/pci/mixart/mixart_core.c:328:12: note: while referencing 'mixart_msg_data' 328 | static u32 mixart_msg_data[MSG_DEFAULT_SIZE / 4]; | ^~~~~~~~~~~~~~~ Signed-off-by: Kees Cook <keescook(a)chromium.org> --- sound/pci/mixart/mixart_core.c | 3 +-- sound/pci/mixart/mixart_core.h | 10 +++++++++- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/sound/pci/mixart/mixart_core.c b/sound/pci/mixart/mixart_core.c index fb8895af0363..853083dd4bad 100644 --- a/sound/pci/mixart/mixart_core.c +++ b/sound/pci/mixart/mixart_core.c @@ -23,8 +23,6 @@ #define MSG_DESCRIPTOR_SIZE 0x24 #define MSG_HEADER_SIZE (MSG_DESCRIPTOR_SIZE + 4) -#define MSG_DEFAULT_SIZE 512 - #define MSG_TYPE_MASK 0x00000003 /* mask for following types */ #define MSG_TYPE_NOTIFY 0 /* embedded -> driver (only notification, do not get_msg() !) */ #define MSG_TYPE_COMMAND 1 /* driver <-> embedded (a command has no answer) */ @@ -444,6 +442,7 @@ irqreturn_t snd_mixart_threaded_irq(int irq, void *dev_id) struct mixart_timer_notify *notify; notify = (struct mixart_timer_notify *)mixart_msg_data; + BUILD_BUG_ON(sizeof(notify) > sizeof(mixart_msg_data)); for(i=0; i<notify->stream_count; i++) { u32 buffer_id = notify->streams[i].buffer_id; diff --git a/sound/pci/mixart/mixart_core.h b/sound/pci/mixart/mixart_core.h index fbf4731a276d..2f0e29ed5d63 100644 --- a/sound/pci/mixart/mixart_core.h +++ b/sound/pci/mixart/mixart_core.h @@ -49,6 +49,7 @@ enum mixart_message_id { MSG_CLOCK_SET_PROPERTIES = 0x200002, }; +#define MSG_DEFAULT_SIZE 512 struct mixart_msg { @@ -251,10 +252,17 @@ struct mixart_sample_pos u32 sample_pos_low_part; } __attribute__((packed)); +/* + * This structure is limited by the size of MSG_DEFAULT_SIZE. Instead of + * having MIXART_MAX_STREAM_PER_CARD * MIXART_MAX_CARDS many streams, + * this is capped to have a total size below MSG_DEFAULT_SIZE. + */ +#define MIXART_MAX_TIMER_NOTIFY_STREAMS \ + ((MSG_DEFAULT_SIZE - sizeof(u32)) / sizeof(struct mixart_sample_pos)) struct mixart_timer_notify { u32 stream_count; - struct mixart_sample_pos streams[MIXART_MAX_STREAM_PER_CARD * MIXART_MAX_CARDS]; + struct mixart_sample_pos streams[MIXART_MAX_TIMER_NOTIFY_STREAMS]; } __attribute__((packed)); -- 2.30.2

2 1

[GIT PULL] ASoC fixes for v5.16-rc4
by Mark Brown 07 Dec '21

07 Dec '21

The following changes since commit 86f74ba3fef56dd1cee19b7a15ae27fc0da5bb61: ASoC: SOF: hda: reset DAI widget before reconfiguring it (2021-11-24 12:57:11 +0000) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v5.16-rc4 for you to fetch changes up to 3fc27e9a1f619b50700f020e6cd270c1b74755f0: ASoC: codecs: wsa881x: fix return values from kcontrol put (2021-12-01 14:13:53 +0000) ---------------------------------------------------------------- ASoC: Fixes for v5.16 A relatively large collection of updates, the size is increased quite a bit by there being some repetitive changes for similar issues that occur multiple times with both notifying control value changes and runtime PM. The Rockchip update looks at first glance like a cleanup but fixes instantiation of the hardware on some systems. ---------------------------------------------------------------- Dan Carpenter (1): ASoC: amd: fix uninitialized variable in snd_acp6x_probe() David Heidelberg (1): ASoC: dt-bindings: wlf,wm8962: add missing interrupt property Hui Wang (1): ASoC: SOF: Intel: Retry codec probing if it fails Mark Brown (1): Suspend related fixes on Tegra Nicolas Frattaroli (1): ASoC: rockchip: i2s_tdm: Dup static DAI template Rob Clark (2): ASoC: rt5682: Fix crash due to out of scope stack vars ASoC: rt5682s: Fix crash due to out of scope stack vars Sameer Pujar (6): ASoC: tegra: Balance runtime PM count ASoC: tegra: Use normal system sleep for SFC ASoC: tegra: Use normal system sleep for MVC ASoC: tegra: Use normal system sleep for Mixer ASoC: tegra: Use normal system sleep for AMX ASoC: tegra: Use normal system sleep for ADX Srinivas Kandagatla (4): ASoC: qdsp6: q6routing: Fix return value from msm_routing_put_audio_mixer ASoC: codecs: wcd934x: handle channel mappping list correctly ASoC: codecs: wcd934x: return correct value from mixer put ASoC: codecs: wsa881x: fix return values from kcontrol put .../devicetree/bindings/sound/wlf,wm8962.yaml | 3 + sound/soc/amd/yc/pci-acp6x.c | 3 +- sound/soc/codecs/rt5682.c | 10 +- sound/soc/codecs/rt5682s.c | 10 +- sound/soc/codecs/wcd934x.c | 126 +++++++++++++++------ sound/soc/codecs/wsa881x.c | 16 ++- sound/soc/qcom/qdsp6/q6routing.c | 8 +- sound/soc/rockchip/rockchip_i2s_tdm.c | 52 +++++---- sound/soc/sof/intel/hda-codec.c | 14 ++- sound/soc/tegra/tegra210_adx.c | 4 +- sound/soc/tegra/tegra210_amx.c | 4 +- sound/soc/tegra/tegra210_mixer.c | 4 +- sound/soc/tegra/tegra210_mvc.c | 8 +- sound/soc/tegra/tegra210_sfc.c | 4 +- 14 files changed, 179 insertions(+), 87 deletions(-)

2 1

Add support for some non-alphanumeric variable names in the math expr evaluator
by GitHub issues - edited 07 Dec '21

07 Dec '21

alsa-project/alsa-lib issue #197 was edited from ranj063: The math expression evaluator expects variable names to only be alpha-numeric. So if I have something like "[$DYNAMIC_PIPELINE + 2]", I get the error: ALSA lib confeval.c:263:(snd_config_evaluate_string) wrong expression '$[$DYNAMIC_PIPELINE + 2]' ALSA lib conf.c:5632:(snd_config_expand_custom) Expand error (walk): Invalid argument Failed to expand pre-processor definitions in input config Is it possible to add support for some non-alphanumeric characters in the variable names , esp '_'? I tried this change it seems to suffice: ``` diff --git a/src/confeval.c b/src/confeval.c index a971bf38..7330d67e 100644 --- a/src/confeval.c +++ b/src/confeval.c @@ -190,7 +190,7 @@ int _snd_eval_string(snd_config_t **dst, const char *s, } else { e = s + 1; while (*e) { - if (!isalnum(*e)) + if (!isalnum(*e) && (*e != '_')) break; e++; } ``` Issue URL : https://github.com/alsa-project/alsa-lib/issues/197 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Add support for some non-alphanumeric variable names in the math expr evaluator
by GitHub issues - edited 07 Dec '21

07 Dec '21

alsa-project/alsa-lib issue #197 was edited from ranj063: The math expression evaluator expects variable names to only be alpha-numeric. So if I have something like "[$DYNAMIC_PIPELINE + 2]", I get the error: ALSA lib confeval.c:263:(snd_config_evaluate_string) wrong expression '$[$DYNAMIC_PIPELINE + 77]' ALSA lib conf.c:5632:(snd_config_expand_custom) Expand error (walk): Invalid argument Failed to expand pre-processor definitions in input config Is it possible to add support for some non-alphanumeric characters in the variable names , esp '_'? I tried this change it seems to suffice: ``` diff --git a/src/confeval.c b/src/confeval.c index a971bf38..7330d67e 100644 --- a/src/confeval.c +++ b/src/confeval.c @@ -190,7 +190,7 @@ int _snd_eval_string(snd_config_t **dst, const char *s, } else { e = s + 1; while (*e) { - if (!isalnum(*e)) + if (!isalnum(*e) && (*e != '_')) break; e++; } ``` Issue URL : https://github.com/alsa-project/alsa-lib/issues/197 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Add support for some non-alphanumeric variable names in the math expr evaluator
by GitHub issues - opened 07 Dec '21

07 Dec '21

alsa-project/alsa-lib issue #197 was opened from ranj063: The math expression evaluator expects variable names to only be alpha-numeric. So if I have something like "[$DMIC_CH_COUNT + 2]", I get the error: Issue URL : https://github.com/alsa-project/alsa-lib/issues/197 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Issues with plughw working, however raw hw not on MAX98360A
by Dylanger Daly 07 Dec '21

07 Dec '21

Hi all, I'm currently having issues with my speakers on a Chromebook running the MAX98360A DAC. director@DRIFTWOOD:~$ aplay -D hw:sc7180rt5682max,1 --dump-hw-params /usr/share/sounds/alsa/Front_Center.wav XDG_RUNTIME_DIR (/run/user/0) is not owned by us (uid 1000), but by uid 0! (This could e.g. happen if you try to connect to a non-root PulseAudio as a root u) Playing WAVE '/usr/share/sounds/alsa/Front_Center.wav' : Signed 16 bit Little Endian, Rate 48000 Hz, Mono HW Params of device "hw:sc7180rt5682max,1": -------------------- ACCESS: MMAP_INTERLEAVED RW_INTERLEAVED FORMAT: S16_LE SUBFORMAT: STD SAMPLE_BITS: 16 FRAME_BITS: 64 CHANNELS: 4 RATE: 48000 PERIOD_TIME: 64000 PERIOD_SIZE: 3072 PERIOD_BYTES: 24576 PERIODS: 2 BUFFER_TIME: 128000 BUFFER_SIZE: 6144 BUFFER_BYTES: 49152 TICK_TIME: ALL -------------------- aplay: set_params:1358: Channels count non available director@DRIFTWOOD:~$ aplay -D plughw:sc7180rt5682max,1 --dump-hw-params /usr/share/sounds/alsa/Front_Center.wav XDG_RUNTIME_DIR (/run/user/0) is not owned by us (uid 1000), but by uid 0! (This could e.g. happen if you try to connect to a non-root PulseAudio as a root u) Playing WAVE '/usr/share/sounds/alsa/Front_Center.wav' : Signed 16 bit Little Endian, Rate 48000 Hz, Mono HW Params of device "plughw:sc7180rt5682max,1": -------------------- ACCESS: MMAP_INTERLEAVED MMAP_NONINTERLEAVED MMAP_COMPLEX RW_INTERLEAVED RW_NONINTERLEAVED FORMAT: S8 U8 S16_LE S16_BE U16_LE U16_BE S24_LE S24_BE U24_LE U24_BE S32_LE S32_BE U32_LE U32_BE FLOAT_LE FLOAT_BE FLOAT64_LE FLOAT64_BE MU_LAW A_LAW IMA_AE SUBFORMAT: STD SAMPLE_BITS: [4 64] FRAME_BITS: [4 640000] CHANNELS: [1 10000] RATE: [4000 4294967295) PERIOD_TIME: 64000 PERIOD_SIZE: [256 274877907) PERIOD_BYTES: [128 4294967295) PERIODS: (0 2147484) BUFFER_TIME: [1 4294967295] BUFFER_SIZE: [512 549755813] BUFFER_BYTES: [256 4294967295] TICK_TIME: ALL -------------------- When I specify plughw, meaning "add software processing", I can hear perfect output, however if I attempt to use the raw hw handle, I get significantly less format support and the channel count isn't even known, and there is no sound output. I believe this is happening because the driver isn't being picked up, and plughw simply adds what it can to the device making it work. The end result is neither PipeWire or PluseAudio detect the speaker device, I assume this is because they're expecting to use the raw hw device handle Here's my ucm2 config: https://github.com/hexdump0815/imagebuilder/blob/d02a71ace7b93b8efbff842662… Any help would be appreciated, thank you

1 0

Support for 8086:ae20 Intel Smart Sound Technology on Fujitsu Lifebook
by Norbert Preining 07 Dec '21

07 Dec '21

Dear all, (please Cc) today I realized that the built-in microphone (next to the webcam) of my Fujitsu Lifebook U-series is not detected. (I usully use a headset, but due to travelling was forced to use the internal). I tried hard to get this device working, but neither with my self-compiled kernel (5.15.6) nor the Debian kernel distribution kernel 5.15 the microphone seems to be detected. I booted into Windows and tried to find out what the device might be. My **guess** is what is mentioned as Intel Smart Sound Technology with Vendor ID 8086 Device ID ae20 I git grepped the kernel sources for this, and snd-hda-intel contains quite some mentioning about Vendor ID 8086, but ae20 is not appearing. Now the interesting thing is that booting into linux and using lspci and lsusb, I cannot see a single device with the above specifications: lspci: root@sakefilet /usr/src/git/linux/Documentation # lspci 00:00.0 Host bridge: Intel Corporation Comet Lake-U v1 4c Host Bridge/DRAM Controller (rev 0c) 00:02.0 VGA compatible controller: Intel Corporation CometLake-U GT2 [UHD Graphics] (rev 02) 00:04.0 Signal processing controller: Intel Corporation Xeon E3-1200 v5/E3-1500 v5/6th Gen Core Processor Thermal Subsystem (rev 0c) 00:12.0 Signal processing controller: Intel Corporation Comet Lake Thermal Subsytem 00:14.0 USB controller: Intel Corporation Comet Lake PCH-LP USB 3.1 xHCI Host Controller 00:14.2 RAM memory: Intel Corporation Comet Lake PCH-LP Shared SRAM 00:14.3 Network controller: Intel Corporation Comet Lake PCH-LP CNVi WiFi 00:15.0 Serial bus controller [0c80]: Intel Corporation Serial IO I2C Host Controller 00:15.1 Serial bus controller [0c80]: Intel Corporation Comet Lake Serial IO I2C Host Controller 00:15.3 Serial bus controller [0c80]: Intel Corporation Device 02eb 00:16.0 Communication controller: Intel Corporation Comet Lake Management Engine Interface 00:16.3 Serial controller: Intel Corporation Device 02e3 00:1c.0 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #1 (rev f0) 00:1c.4 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #5 (rev f0) 00:1d.0 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #9 (rev f0) 00:1d.4 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #13 (rev f0) 00:1e.0 Communication controller: Intel Corporation Device 02a8 00:1e.1 Communication controller: Intel Corporation Device 02a9 00:1f.0 ISA bridge: Intel Corporation Comet Lake PCH-LP LPC Premium Controller/eSPI Controller 00:1f.3 Multimedia audio controller: Intel Corporation Comet Lake PCH-LP cAVS 00:1f.4 SMBus: Intel Corporation Comet Lake PCH-LP SMBus Host Controller 00:1f.5 Serial bus controller [0c80]: Intel Corporation Comet Lake SPI (flash) Controller 00:1f.6 Ethernet controller: Intel Corporation Ethernet Connection (10) I219-LM 06:00.0 PCI bridge: Intel Corporation JHL7540 Thunderbolt 3 Bridge [Titan Ridge 4C 2018] (rev 06) 07:00.0 PCI bridge: Intel Corporation JHL7540 Thunderbolt 3 Bridge [Titan Ridge 4C 2018] (rev 06) 07:01.0 PCI bridge: Intel Corporation JHL7540 Thunderbolt 3 Bridge [Titan Ridge 4C 2018] (rev 06) 07:02.0 PCI bridge: Intel Corporation JHL7540 Thunderbolt 3 Bridge [Titan Ridge 4C 2018] (rev 06) 07:04.0 PCI bridge: Intel Corporation JHL7540 Thunderbolt 3 Bridge [Titan Ridge 4C 2018] (rev 06) 08:00.0 System peripheral: Intel Corporation JHL7540 Thunderbolt 3 NHI [Titan Ridge 4C 2018] (rev 06) 3d:00.0 USB controller: Intel Corporation JHL7540 Thunderbolt 3 USB Controller [Titan Ridge 4C 2018] (rev 06) 72:00.0 Non-Volatile memory controller: Samsung Electronics Co Ltd NVMe SSD Controller SM981/PM981/PM983 77:00.0 SD Host controller: O2 Micro, Inc. SD/MMC Card Reader Controller (rev 01) lsusb: Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 001 Device 004: ID 04f2:b6e1 Chicony Electronics Co., Ltd FJ Camera Bus 001 Device 003: ID 298d:2033 Next Biometrics NB-2033-U Bus 001 Device 002: ID 058f:9540 Alcor Micro Corp. AU9540 Smartcard Reader Bus 001 Device 005: ID 8087:0026 Intel Corp. AX201 Bluetooth Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub I attach the output of alsa-info for completeness. The kernel version is 5.15.6 with some patches from xannmod, but I could reproduce the same behaviour with standard Debian kernel 5.15. Am I missing some config option, or is the vendorid/devid not supported by now? Thanks for every suggestion, (Please CC!) Al the best Norbert -- PREINING Norbert https://www.preining.info Fujitsu Research + IFMGA Guide + TU Wien + TeX Live + Debian Dev GPG: 0x860CDC13 fp: F7D8 A928 26E3 16A1 9FA0 ACF0 6CAC A448 860C DC13

2 4