November 2024 - Alsa-devel - mailman.alsa-project.org

[PATCH 00/34] address all -Wunused-const warnings
by Arnd Bergmann 27 Jan '25

27 Jan '25

From: Arnd Bergmann <arnd(a)arndb.de> Compilers traditionally warn for unused 'static' variables, but not if they are constant. The reason here is a custom for C++ programmers to define named constants as 'static const' variables in header files instead of using macros or enums. In W=1 builds, we get warnings only static const variables in C files, but not in headers, which is a good compromise, but this still produces warning output in at least 30 files. These warnings are almost all harmless, but also trivial to fix, and there is no good reason to warn only about the non-const variables being unused. I've gone through all the files that I found using randconfig and allmodconfig builds and created patches to avoid these warnings, with the goal of retaining a clean build once the option is enabled by default. Unfortunately, there is one fairly large patch ("drivers: remove incorrect of_match_ptr/ACPI_PTR annotations") that touches 34 individual drivers that all need the same one-line change. If necessary, I can split it up by driver or by subsystem, but at least for reviewing I would keep it as one piece for the moment. Please merge the individual patches through subsystem trees. I expect that some of these will have to go through multiple revisions before they are picked up, so anything that gets applied early saves me from resending. Arnd Arnd Bergmann (31): powerpc/fsl-soc: hide unused const variable ubsan: fix unused variable warning in test module platform: goldfish: remove ACPI_PTR() annotations i2c: pxa: hide unused icr_bits[] variable 3c515: remove unused 'mtu' variable tracing: hide unused ftrace_event_id_fops Input: synaptics: hide unused smbus_pnp_ids[] array power: rt9455: hide unused rt9455_boost_voltage_values efi: sysfb: don't build when EFI is disabled clk: ti: dpll: fix incorrect #ifdef checks apm-emulation: hide an unused variable sisfb: hide unused variables dma/congiguous: avoid warning about unused size_bytes leds: apu: remove duplicate DMI lookup data iio: ad5755: hook up of_device_id lookup to platform driver greybus: arche-ctrl: move device table to its right location lib: checksum: hide unused expected_csum_ipv6_magic[] sunrpc: suppress warnings for unused procfs functions comedi: ni_atmio: avoid warning for unused device_ids[] table iwlegacy: don't warn for unused variables with DEBUG_FS=n drm/komeda: don't warn for unused debugfs files firmware: qcom_scm: mark qcom_scm_qseecom_allowlist as __maybe_unused crypto: ccp - drop platform ifdef checks usb: gadget: omap_udc: remove unused variable isdn: kcapi: don't build unused procfs code cpufreq: intel_pstate: hide unused intel_pstate_cpu_oob_ids[] net: xgbe: remove extraneous #ifdef checks Input: imagis - remove incorrect ifdef checks sata: mv: drop unnecessary #ifdef checks ASoC: remove incorrect of_match_ptr/ACPI_PTR annotations spi: remove incorrect of_match_ptr annotations drivers: remove incorrect of_match_ptr/ACPI_PTR annotations kbuild: always enable -Wunused-const-variable Krzysztof Kozlowski (1): Input: stmpe-ts - mark OF related data as maybe unused arch/powerpc/sysdev/fsl_msi.c | 2 + drivers/ata/sata_mv.c | 64 +++++++++---------- drivers/char/apm-emulation.c | 5 +- drivers/char/ipmi/ipmb_dev_int.c | 2 +- drivers/char/tpm/tpm_ftpm_tee.c | 2 +- drivers/clk/ti/dpll.c | 10 ++- drivers/comedi/drivers/ni_atmio.c | 2 +- drivers/cpufreq/intel_pstate.c | 2 + drivers/crypto/ccp/sp-platform.c | 14 +--- drivers/dma/img-mdc-dma.c | 2 +- drivers/firmware/efi/Makefile | 3 +- drivers/firmware/efi/sysfb_efi.c | 2 - drivers/firmware/qcom/qcom_scm.c | 2 +- drivers/fpga/versal-fpga.c | 2 +- .../gpu/drm/arm/display/komeda/komeda_dev.c | 8 --- drivers/hid/hid-google-hammer.c | 6 +- drivers/i2c/busses/i2c-pxa.c | 2 +- drivers/i2c/muxes/i2c-mux-ltc4306.c | 2 +- drivers/i2c/muxes/i2c-mux-reg.c | 2 +- drivers/iio/dac/ad5755.c | 1 + drivers/input/mouse/synaptics.c | 2 + drivers/input/touchscreen/imagis.c | 4 +- drivers/input/touchscreen/stmpe-ts.c | 2 +- drivers/input/touchscreen/wdt87xx_i2c.c | 2 +- drivers/isdn/capi/Makefile | 3 +- drivers/isdn/capi/kcapi.c | 7 +- drivers/leds/leds-apu.c | 3 +- drivers/mux/adg792a.c | 2 +- drivers/net/ethernet/3com/3c515.c | 3 - drivers/net/ethernet/amd/xgbe/xgbe-platform.c | 8 --- drivers/net/ethernet/apm/xgene-v2/main.c | 2 +- drivers/net/ethernet/hisilicon/hns_mdio.c | 2 +- drivers/net/wireless/intel/iwlegacy/4965-rs.c | 15 +---- drivers/net/wireless/intel/iwlegacy/common.h | 2 - drivers/platform/goldfish/goldfish_pipe.c | 2 +- drivers/power/supply/rt9455_charger.c | 2 + drivers/regulator/pbias-regulator.c | 2 +- drivers/regulator/twl-regulator.c | 2 +- drivers/regulator/twl6030-regulator.c | 2 +- drivers/rtc/rtc-fsl-ftm-alarm.c | 2 +- drivers/scsi/hisi_sas/hisi_sas_v1_hw.c | 2 +- drivers/scsi/hisi_sas/hisi_sas_v2_hw.c | 2 +- drivers/spi/spi-armada-3700.c | 2 +- drivers/spi/spi-img-spfi.c | 2 +- drivers/spi/spi-meson-spicc.c | 2 +- drivers/spi/spi-meson-spifc.c | 2 +- drivers/spi/spi-orion.c | 2 +- drivers/spi/spi-pic32-sqi.c | 2 +- drivers/spi/spi-pic32.c | 2 +- drivers/spi/spi-rockchip.c | 2 +- drivers/spi/spi-s3c64xx.c | 2 +- drivers/spi/spi-st-ssc4.c | 2 +- drivers/staging/greybus/arche-apb-ctrl.c | 1 + drivers/staging/greybus/arche-platform.c | 9 +-- drivers/staging/pi433/pi433_if.c | 2 +- drivers/tty/serial/amba-pl011.c | 6 +- drivers/tty/serial/ma35d1_serial.c | 2 +- drivers/usb/gadget/udc/omap_udc.c | 10 +-- drivers/video/fbdev/sis/init301.c | 3 +- kernel/dma/contiguous.c | 2 +- kernel/trace/trace_events.c | 4 ++ lib/checksum_kunit.c | 2 + lib/test_ubsan.c | 2 +- net/sunrpc/cache.c | 10 +-- scripts/Makefile.extrawarn | 1 - sound/soc/atmel/sam9x5_wm8731.c | 2 +- sound/soc/codecs/rt5514-spi.c | 2 +- sound/soc/qcom/lpass-sc7280.c | 2 +- sound/soc/samsung/aries_wm8994.c | 2 +- 69 files changed, 121 insertions(+), 169 deletions(-) -- 2.39.2 Cc: Michael Ellerman <mpe(a)ellerman.id.au> Cc: Christophe Leroy <christophe.leroy(a)csgroup.eu> Cc: Damien Le Moal <dlemoal(a)kernel.org> Cc: Jiri Kosina <jikos(a)kernel.org> Cc: Greg Kroah-Hartman <gregkh(a)linuxfoundation.org> Cc: Corey Minyard <minyard(a)acm.org> Cc: Peter Huewe <peterhuewe(a)gmx.de> Cc: Jarkko Sakkinen <jarkko(a)kernel.org> Cc: Tero Kristo <kristo(a)kernel.org> Cc: Stephen Boyd <sboyd(a)kernel.org> Cc: Ian Abbott <abbotti(a)mev.co.uk> Cc: H Hartley Sweeten <hsweeten(a)visionengravers.com> Cc: Srinivas Pandruvada <srinivas.pandruvada(a)linux.intel.com> Cc: Len Brown <lenb(a)kernel.org> Cc: "Rafael J. Wysocki" <rafael(a)kernel.org> Cc: John Allen <john.allen(a)amd.com> Cc: Herbert Xu <herbert(a)gondor.apana.org.au> Cc: Vinod Koul <vkoul(a)kernel.org> Cc: Ard Biesheuvel <ardb(a)kernel.org> Cc: Bjorn Andersson <andersson(a)kernel.org> Cc: Moritz Fischer <mdf(a)kernel.org> Cc: Liviu Dudau <liviu.dudau(a)arm.com> Cc: Benjamin Tissoires <benjamin.tissoires(a)redhat.com> Cc: Andi Shyti <andi.shyti(a)kernel.org> Cc: Michael Hennerich <michael.hennerich(a)analog.com> Cc: Peter Rosin <peda(a)axentia.se> Cc: Lars-Peter Clausen <lars(a)metafoo.de> Cc: Jonathan Cameron <jic23(a)kernel.org> Cc: Dmitry Torokhov <dmitry.torokhov(a)gmail.com> Cc: Markuss Broks <markuss.broks(a)gmail.com> Cc: Alexandre Torgue <alexandre.torgue(a)foss.st.com> Cc: Lee Jones <lee(a)kernel.org> Cc: Jakub Kicinski <kuba(a)kernel.org> Cc: Shyam Sundar S K <Shyam-sundar.S-k(a)amd.com> Cc: Iyappan Subramanian <iyappan(a)os.amperecomputing.com> Cc: Yisen Zhuang <yisen.zhuang(a)huawei.com> Cc: Stanislaw Gruszka <stf_xl(a)wp.pl> Cc: Kalle Valo <kvalo(a)kernel.org> Cc: Sebastian Reichel <sre(a)kernel.org> Cc: Tony Lindgren <tony(a)atomide.com> Cc: Mark Brown <broonie(a)kernel.org> Cc: Alexandre Belloni <alexandre.belloni(a)bootlin.com> Cc: Xiang Chen <chenxiang66(a)hisilicon.com> Cc: "Martin K. Petersen" <martin.petersen(a)oracle.com> Cc: Neil Armstrong <neil.armstrong(a)linaro.org> Cc: Heiko Stuebner <heiko(a)sntech.de> Cc: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> Cc: Vaibhav Hiremath <hvaibhav.linux(a)gmail.com> Cc: Alex Elder <elder(a)kernel.org> Cc: Jiri Slaby <jirislaby(a)kernel.org> Cc: Jacky Huang <ychuang3(a)nuvoton.com> Cc: Helge Deller <deller(a)gmx.de> Cc: Christoph Hellwig <hch(a)lst.de> Cc: Robin Murphy <robin.murphy(a)arm.com> Cc: Steven Rostedt <rostedt(a)goodmis.org> Cc: Masami Hiramatsu <mhiramat(a)kernel.org> Cc: Andrew Morton <akpm(a)linux-foundation.org> Cc: Kees Cook <keescook(a)chromium.org> Cc: Trond Myklebust <trond.myklebust(a)hammerspace.com> Cc: Anna Schumaker <anna(a)kernel.org> Cc: Masahiro Yamada <masahiroy(a)kernel.org> Cc: Nathan Chancellor <nathan(a)kernel.org> Cc: Takashi Iwai <tiwai(a)suse.com> Cc: linuxppc-dev(a)lists.ozlabs.org Cc: linux-kernel(a)vger.kernel.org Cc: linux-ide(a)vger.kernel.org Cc: openipmi-developer(a)lists.sourceforge.net Cc: linux-integrity(a)vger.kernel.org Cc: linux-omap(a)vger.kernel.org Cc: linux-clk(a)vger.kernel.org Cc: linux-pm(a)vger.kernel.org Cc: linux-crypto(a)vger.kernel.org Cc: dmaengine(a)vger.kernel.org Cc: linux-efi(a)vger.kernel.org Cc: linux-arm-msm(a)vger.kernel.org Cc: linux-fpga(a)vger.kernel.org Cc: dri-devel(a)lists.freedesktop.org Cc: linux-input(a)vger.kernel.org Cc: linux-i2c(a)vger.kernel.org Cc: linux-iio(a)vger.kernel.org Cc: linux-stm32(a)st-md-mailman.stormreply.com Cc: linux-arm-kernel(a)lists.infradead.org Cc: netdev(a)vger.kernel.org Cc: linux-leds(a)vger.kernel.org Cc: linux-wireless(a)vger.kernel.org Cc: linux-rtc(a)vger.kernel.org Cc: linux-scsi(a)vger.kernel.org Cc: linux-spi(a)vger.kernel.org Cc: linux-amlogic(a)lists.infradead.org Cc: linux-rockchip(a)lists.infradead.org Cc: linux-samsung-soc(a)vger.kernel.org Cc: greybus-dev(a)lists.linaro.org Cc: linux-staging(a)lists.linux.dev Cc: linux-serial(a)vger.kernel.org Cc: linux-usb(a)vger.kernel.org Cc: linux-fbdev(a)vger.kernel.org Cc: iommu(a)lists.linux.dev Cc: linux-trace-kernel(a)vger.kernel.org Cc: kasan-dev(a)googlegroups.com Cc: linux-hardening(a)vger.kernel.org Cc: linux-nfs(a)vger.kernel.org Cc: linux-kbuild(a)vger.kernel.org Cc: alsa-devel(a)alsa-project.org Cc: linux-sound(a)vger.kernel.org

7 7

[RESEND PATCH v2] drm/bridge: dw-hdmi-i2s: set insert_pcuv bit if hardware supports it
by Geraldo Nascimento 30 Dec '24

30 Dec '24

Hi Mark, resending this as it failed to apply in my last submission. Added Neil Armstrong to Cc: as hopefully he will be able to better review this. Thanks, Geraldo Nascimento --- Starting with version 2.10a of Synopsys DesignWare HDMI controller the insert_pcuv bit was introduced. On RK3399pro SoM (Radxa Rock Pi N10), for example, if we neglect to set this bit and proceed to enable hdmi_sound and i2s2 on the device tree there will be extreme clipping of sound output, to the point that music sounds like white noise. Problem could also manifest as just mild cracking depending of HDMI audio implementation of sink. Setting insert_pcuv bit (bit 2 of aud_conf2 Audio Sample register) fixes this. Signed-off-by: Geraldo Nascimento <geraldogabriel(a)gmail.com> --- v1->v2: SoC->SoM on description, better commenting, minor style changes, conditional application of fix for L-PCM only --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio-20221017.c +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio-20221017.c @@ -42,6 +42,7 @@ static int dw_hdmi_i2s_hw_params(struct device *dev, void *data, struct dw_hdmi *hdmi = audio->hdmi; u8 conf0 = 0; u8 conf1 = 0; + u8 conf2 = 0; u8 inputclkfs = 0; /* it cares I2S only */ @@ -101,6 +102,28 @@ static int dw_hdmi_i2s_hw_params(struct device *dev, void *data, return -EINVAL; } + /* + * dw-hdmi introduced insert_pcuv bit in + * version 2.10a. + * + * This single bit (bit 2 of HDMI_AUD_CONF2) + * when set to 1 will enable the insertion of the PCUV + * (Parity, Channel Status, User bit and Validity) + * bits on the incoming audio stream. + * + * Support is limited to Linear PCM audio. If + * neglected, the lack of valid PCUV bits + * on L-PCM streams will cause anything from + * mild cracking to full blown extreme + * clipping depending on the HDMI audio + * implementation of the sink. + * + */ + + if (hdmi_read(audio, HDMI_DESIGN_ID) >= 0x21 && + !(hparms->iec.status[0] & IEC958_AES0_NONAUDIO)) + conf2 = HDMI_AUD_CONF2_INSERT_PCUV; + dw_hdmi_set_sample_rate(hdmi, hparms->sample_rate); dw_hdmi_set_channel_status(hdmi, hparms->iec.status); dw_hdmi_set_channel_count(hdmi, hparms->channels); @@ -109,6 +120,7 @@ static int dw_hdmi_i2s_hw_params(struct device *dev, void *data, hdmi_write(audio, inputclkfs, HDMI_AUD_INPUTCLKFS); hdmi_write(audio, conf0, HDMI_AUD_CONF0); hdmi_write(audio, conf1, HDMI_AUD_CONF1); + hdmi_write(audio, conf2, HDMI_AUD_CONF2); return 0; } --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-20221017.h +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-20221017.h @@ -931,6 +931,11 @@ enum { HDMI_AUD_CONF1_WIDTH_16 = 0x10, HDMI_AUD_CONF1_WIDTH_24 = 0x18, +/* AUD_CONF2 field values */ + HDMI_AUD_CONF2_HBR = 0x01, + HDMI_AUD_CONF2_NLPCM = 0x02, + HDMI_AUD_CONF2_INSERT_PCUV = 0x04, + /* AUD_CTS3 field values */ HDMI_AUD_CTS3_N_SHIFT_OFFSET = 5, HDMI_AUD_CTS3_N_SHIFT_MASK = 0xe0,

3 6

[PATCH v4 0/9] ASoC: samsung: midas-audio: Add GPIO-based headset jack detection
by Artur Weber 29 Dec '24

29 Dec '24

Many of Samsung's Exynos 4 devices share the same midas-audio driver to handle the codec setup. While most of these devices, including the Midas itself, use the jack detection provided by the WM8994 driver, other devices such as the Samsung Galaxy Tab 3 8.0 (lt01) use two GPIOs and an ADC channel to determine jack insertion, the jack's type, and button presses (for headsets with volume up/down/play buttons). In the downstream kernel, this behavior is implemented in the sec-jack driver[1], and the per-device settings are configured in *-jack.c files in the mach folder (see e.g. the Tab 3's implementation[2]). This patchset implements this mechanism in the midas_wm1811.c driver, and adds new DTS options to allow for its configuration. It also enables jack detection for the Samsung Galaxy Tab 3 8.0. A very similar mechanism was already present in the aries_wm8994.c driver[3]; this implementation heavily borrows from it, though there are a few extra cleanups as well. Signed-off-by: Artur Weber <aweber.kernel(a)gmail.com> [1] https://github.com/gr8nole/android_kernel_samsung_smdk4x12/blob/lineage-14.… [2] https://github.com/gr8nole/android_kernel_samsung_smdk4x12/blob/lineage-14.… [3] https://github.com/torvalds/linux/blob/master/sound/soc/samsung/aries_wm899… --- Changes in v4: - Replaced manual regulator toggles with SND_SOC_DAPM_REGULATOR_SUPPLY, both for existing mic bias regulators and for the new headset mic bias regulator - Link to v3: https://lore.kernel.org/r/20240519-midas-wm1811-gpio-jack-v3-0-0c1736144c0e… Changes in v3: - Re-added pipe (|) character to samsung,headset-button-threshold-microvolt description to prevent it from being parsed as a mapping (fixes syntax error) - Split out "ASoC: dt-bindings: samsung,midas-audio: Add GPIO-based headset jack detection" into two patches - Link to v2: https://lore.kernel.org/r/20240508-midas-wm1811-gpio-jack-v2-0-b4d36cd02c6e… Changes in v2: - Added vendor prefix to threshold properties - Added separate headset mic bias regulator - Changed some cases of dev_err + return with return dev_err_probe - Added an extra patch to replace some previous dev_err + return cases with dev_err_probe - Moved tab3 DTS wm1811 codec config changes to separate commit --- Artur Weber (9): ASoC: dt-bindings: samsung,midas-audio: Add headset mic bias supply ASoC: dt-bindings: samsung,midas-audio: Add GPIO-based headset jack detection ASoC: samsung: midas_wm1811: Use SND_SOC_DAPM_REGULATOR_SUPPLY for bias regulators ASoC: samsung: midas_wm1811: Add headset mic bias supply support ASoC: samsung: midas_wm1811: Add GPIO-based headset jack detection ASoC: samsung: midas_wm1811: Use dev_err_probe where appropriate ARM: dts: samsung: exynos4212-tab3: Fix headset mic, add jack detection ARM: dts: samsung: exynos4212-tab3: Add MCLK2 clock to WM1811 codec config ARM: dts: samsung: exynos4212-tab3: Drop interrupt from WM1811 codec .../bindings/sound/samsung,midas-audio.yaml | 33 ++ arch/arm/boot/dts/samsung/exynos4212-tab3.dtsi | 31 +- sound/soc/samsung/Kconfig | 2 +- sound/soc/samsung/midas_wm1811.c | 348 +++++++++++++++++---- 4 files changed, 343 insertions(+), 71 deletions(-) --- base-commit: a38297e3fb012ddfa7ce0321a7e5a8daeb1872b6 change-id: 20240502-midas-wm1811-gpio-jack-b10226b17ecc Best regards, -- Artur Weber <aweber.kernel(a)gmail.com>

3 12

[PATCH] soundwire: amd: clear wake enable register for power off mode
by Vijendar Mukunda 23 Dec '24

23 Dec '24

As per design for power off mode, clear the wake enable register during resume sequence. Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda(a)amd.com> --- drivers/soundwire/amd_manager.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/soundwire/amd_manager.c b/drivers/soundwire/amd_manager.c index 5a4bfaef65fb..96a3aa6da711 100644 --- a/drivers/soundwire/amd_manager.c +++ b/drivers/soundwire/amd_manager.c @@ -1190,6 +1190,7 @@ static int __maybe_unused amd_resume_runtime(struct device *dev) if (amd_manager->power_mode_mask & AMD_SDW_CLK_STOP_MODE) { return amd_sdw_clock_stop_exit(amd_manager); } else if (amd_manager->power_mode_mask & AMD_SDW_POWER_OFF_MODE) { + writel(0x00, amd_manager->acp_mmio + ACP_SW_WAKE_EN(amd_manager->instance)); val = readl(amd_manager->mmio + ACP_SW_CLK_RESUME_CTRL); if (val) { val |= AMD_SDW_CLK_RESUME_REQ; -- 2.34.1

3 2

[PATCH 00/35] Reorganize kerneldoc parameter names
by Julia Lawall 20 Dec '24

20 Dec '24

Reorganize kerneldoc parameter names to match the parameter order in the function header. The misordered cases were identified using the following Coccinelle semantic patch: // <smpl> @initialize:ocaml@ @@ let parse_doc l = let pieces = List.map String.trim (String.split_on_char '*' l) in let l = String.concat " " pieces in match String.split_on_char ':' l with x::xs -> x | _ -> "" let params ps = List.rev (List.fold_left (fun prev (pm,_) -> let ty = String.trim(Pretty_print_c.string_of_fullType pm.Ast_c.p_type) in if ty = "void" && pm.Ast_c.p_namei = None then prev else let name = match pm.Ast_c.p_namei with Some name -> name | None -> failwith "function parameter has no name" in (String.trim (Pretty_print_c.string_of_name name),ty)::prev) [] ps) @r@ comments c; identifier fn; position p; parameter list ps; type T; @@ T@c fn@p(ps) { ... } @script:ocaml@ p << r.p; c << r.c; (_,ps) << r.ps; @@ let isdoc c ps = List.length ps > 1 && (let c = String.trim c in String.length c > 3 && String.sub c 0 3 = "/**" && String.get c 3 != '*') in let subset l1 l2 = List.for_all (fun x -> List.mem x l2) l1 in let (cb,cm,ca) = List.hd c in match List.rev cb with c::_ when isdoc c ps -> let pieces = String.split_on_char '@' c in (match pieces with _::tl -> let d_names = List.map parse_doc tl in (* check parameters *) let p_names = List.map fst (params ps) in if d_names <> [] && not(d_names = p_names) then begin if List.sort compare d_names = List.sort compare p_names then Coccilib.print_main "out of order" p else if subset d_names p_names then Coccilib.print_main "doc is missing a parameter" p else if subset d_names p_names then Coccilib.print_main "doc has an extra parameter" p end | _ -> ()) | _ -> () // </smpl> --- arch/arm/mach-omap2/prm2xxx_3xxx.c | 1 - arch/powerpc/platforms/ps3/interrupt.c | 2 +- arch/powerpc/platforms/ps3/repository.c | 2 +- drivers/base/firmware_loader/main.c | 2 +- drivers/comedi/drivers/comedi_8254.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +- drivers/gpu/drm/amd/display/dc/core/dc.c | 2 +- drivers/gpu/drm/drm_gem_framebuffer_helper.c | 3 +-- drivers/gpu/drm/drm_gpuvm.c | 2 +- drivers/gpu/drm/radeon/radeon_ib.c | 2 +- drivers/iommu/iommu.c | 2 +- drivers/leds/leds-gpio-register.c | 2 +- drivers/mfd/atmel-smc.c | 4 ++-- drivers/misc/mei/bus.c | 2 +- drivers/mtd/ubi/eba.c | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt_hwrm.c | 2 +- drivers/net/ethernet/intel/e1000/e1000_hw.c | 2 +- drivers/net/ethernet/intel/i40e/i40e_common.c | 7 +++---- drivers/net/ethernet/intel/ice/ice_common.c | 2 +- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 2 +- drivers/nvdimm/dimm_devs.c | 2 +- drivers/pci/hotplug/pci_hotplug_core.c | 2 +- drivers/pinctrl/pinmux.c | 2 +- drivers/slimbus/messaging.c | 2 +- drivers/soc/qcom/qmi_interface.c | 2 +- drivers/soundwire/stream.c | 2 +- drivers/usb/gadget/config.c | 4 ++-- fs/char_dev.c | 2 +- fs/dcache.c | 4 ++-- fs/seq_file.c | 2 +- kernel/audit.c | 2 +- kernel/resource.c | 2 +- kernel/sysctl.c | 1 - kernel/trace/ring_buffer.c | 2 +- lib/lru_cache.c | 2 +- lib/maple_tree.c | 2 +- mm/mmu_notifier.c | 2 +- net/dccp/feat.c | 2 +- net/mac80211/mesh_hwmp.c | 6 +++--- net/mac80211/mesh_pathtbl.c | 10 +++++----- net/socket.c | 2 +- net/sunrpc/xprt.c | 2 +- net/tipc/link.c | 14 +++++++------- net/tipc/msg.c | 2 +- sound/pci/hda/hda_codec.c | 2 +- 45 files changed, 60 insertions(+), 64 deletions(-)

4 5

hda: how to implement component master_unbind?
by Kai Vehmanen 20 Dec '24

20 Dec '24

Hi Takashi et al, I've been having multiple discussions with our i915 team w.r.t. audio driver behaviour when the aggregate component is unbound, triggered by i915 unbind. This came up again in the recent regression with devres allocations and I now dug into the topic again. In short, I'm puzzled how to really implement this. ALSA (or ASoC) don't really have support for individual components of a card disappearing in a hotplug fashion. At card level, we do have such support (USB, firewire and recent work for PCI hotplug). But not individual components of a card getting unplugged. In current code we have this: static void hdac_component_master_unbind(struct device *dev) { » struct drm_audio_component *acomp = hdac_get_acomp(dev); » if (acomp->audio_ops && acomp->audio_ops->master_unbind) » » acomp->audio_ops->master_unbind(dev, acomp); » module_put(acomp->ops->owner); » component_unbind_all(dev, acomp); » WARN_ON(acomp->ops || acomp->dev); } ... when e.g. i915 driver is unbound (but could be any driver using the component framework, not jus Intel hw), i915 calls component_del() and HDA gets call to the above. After this, acomp ops are null are audio no longer has ability to talk to i915 driver (which makes sense given it's unbound). If audio was runtime suspended, this kind of works (but relies on some good luck). Upon HDA controller resume, we note acomp ops are NULL and HDMI/DP operations (like snd_hdac_display_power()) silently become no-ops. PCM streaming will go to /dev/null, but this is ok'ish since no monitor can be connected anyways. If audio was active, we start to get warnings or worse. Audio expects hw display codec to be powered and programmed, but suddenly it mey lose state. If the audio controller is actually part of the display hardware (e.g. discrete GPUs), then controller registers can stop functioning (they should be still available, but given the main diplay driver is unbound, odds of suprising behaviour are high). So this is less than ideal. I've been looking at options: 1) prevent/block the unbind if audio device is busy The component framework does not allow individual components to return -EBUSY, so there's no way to let others know that unbind is not possible now. This would force anyone testing DRM driver unbind to first unbind the audio driver and stop any active audio use-cases if needed. 2) unbind the ALSA card I've experimented doing a device_unregister() from the above callback, but this has not really worked (onion peeling exercise of new probelms). The code is shared by multiple controllers, so getting a handle to an snd_card object is not straighforward (drvdata is differnet for SOF and snd-hda-intel for instance). But with some new work, maybe we could call snd_card_disconnect() to detach the card and inform user-space. 3) continue as-is and try to fix bugs If audio is active, maybe we could force a acomp->put_power() to ensure a clean unregister of the display part. But this leaves a big chunk of issues especially with HDA controllers that require the display to be powered on, to function. .. Any ideas, and/or has there been prior work? It seems Takashi it's been mostly you who has been active working on the acomp integration recently. I also included Chris, Daniel and Jani who've had patches to hdac_component.c. Br, Kai

3 5

[PATCH] dt-bindings: sound:atmel-at91sam9g20ek: convert bindings to json-schema
by Balakrishnan Sambath 10 Dec '24

10 Dec '24

Convert atmel-at91sam9g20ek-wm8731-audio DT binding to yaml based json-schema.Change file name to match json-scheme naming. Signed-off-by: Balakrishnan Sambath <balakrishnan.s(a)microchip.com> --- .../bindings/sound/atmel,at91sam9g20ek-wm8731.yaml | 60 ++++++++++++++++++++++ .../sound/atmel-at91sam9g20ek-wm8731-audio.txt | 26 ---------- 2 files changed, 60 insertions(+), 26 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/atmel,at91sam9g20ek-wm8731.yaml b/Documentation/devicetree/bindings/sound/atmel,at91sam9g20ek-wm8731.yaml new file mode 100644 index 000000000000..f6330707fe1b --- /dev/null +++ b/Documentation/devicetree/bindings/sound/atmel,at91sam9g20ek-wm8731.yaml @@ -0,0 +1,60 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/atmel,at91sam9g20ek-wm8731.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Atmel at91sam9g20ek wm8731 audio complex + +maintainers: + - Balakrishnan Sambath <balakrishnan.s(a)microchip.com> + +properties: + compatible: + const: atmel,at91sam9g20ek-wm8731-audio + + atmel,model: + $ref: /schemas/types.yaml#/definitions/string + description: The user-visible name of this sound complex. + atmel,audio-routing: + $ref: /schemas/types.yaml#/definitions/non-unique-string-array + description: A list of the connections between audio components. + minItems: 2 + items: + enum: + # Board Connectors + - "Ext Spk" + - "Int MIC" + + # CODEC Pins + - LHPOUT + - MICIN + atmel,ssc-controller: + $ref: /schemas/types.yaml#/definitions/phandle + description: The phandle of the SSC controller + atmel,audio-codec: + $ref: /schemas/types.yaml#/definitions/phandle + description: The phandle of WM8731 audio codec + +required: + - compatible + - atmel,model + - atmel,audio-routing + - atmel,ssc-controller + - atmel,audio-codec + +additionalProperties: false + +examples: + - | + sound { + compatible = "atmel,at91sam9g20ek-wm8731-audio"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_pck0_as_mck>; + atmel,model = "wm8731 @ AT91SAMG20EK"; + atmel,audio-routing = + "Ext Spk", "LHPOUT", + "Int MIC", "MICIN"; + atmel,ssc-controller = <&ssc0>; + atmel,audio-codec = <&wm8731>; + }; diff --git a/Documentation/devicetree/bindings/sound/atmel-at91sam9g20ek-wm8731-audio.txt b/Documentation/devicetree/bindings/sound/atmel-at91sam9g20ek-wm8731-audio.txt deleted file mode 100644 index 9c5a9947b64d..000000000000 --- a/Documentation/devicetree/bindings/sound/atmel-at91sam9g20ek-wm8731-audio.txt +++ /dev/null @@ -1,26 +0,0 @@ -* Atmel at91sam9g20ek wm8731 audio complex - -Required properties: - - compatible: "atmel,at91sam9g20ek-wm8731-audio" - - atmel,model: The user-visible name of this sound complex. - - atmel,audio-routing: A list of the connections between audio components. - - atmel,ssc-controller: The phandle of the SSC controller - - atmel,audio-codec: The phandle of the WM8731 audio codec -Optional properties: - - pinctrl-names, pinctrl-0: Please refer to pinctrl-bindings.txt - -Example: -sound { - compatible = "atmel,at91sam9g20ek-wm8731-audio"; - pinctrl-names = "default"; - pinctrl-0 = <&pinctrl_pck0_as_mck>; - - atmel,model = "wm8731 @ AT91SAMG20EK"; - - atmel,audio-routing = - "Ext Spk", "LHPOUT", - "Int MIC", "MICIN"; - - atmel,ssc-controller = <&ssc0>; - atmel,audio-codec = <&wm8731>; -}; --- base-commit: 0dd3ee31125508cd67f7e7172247f05b7fd1753a change-id: 20240214-at91sam9g20ek-wm8731-yaml-404ddddbb91a Best regards, -- Balakrishnan Sambath <balakrishnan.s(a)microchip.com>

5 5

[PATCH treewide 00/11] ASoC: Clean up {hp,mic}-det-gpio handling
by Geert Uytterhoeven 09 Dec '24

09 Dec '24

Hi all, The "gpio" suffix for GPIO consumers was deprecated a while ago, in favor of the "gpios" suffix. However, there are still several users of the "hp-det-gpio" and "mic-det-gpio" properties, in DT bindings, drivers, and DT source files. Drivers that use gpiolib handle both the deprecated and new properties transparently, but that is not the case when using of_property_*() calls directly. - The first patch fixes a bug, as the Freescale Generic ASoC Sound driver handles only the deprecated properties, while there is already a user of the new variants, - The second patch updates the Audio Graph and Simple Audio Card DT bindings, - Patches 3-9 converts various DTS files to use the new properties, - The last 2 patches convert Freescale sound device nodes to use the new properties. All patches can be applied independently, except for the last two, which depend on the first patch to avoid regressions. This series was compile-tested only. Thanks for your comments! Geert Uytterhoeven (11): ASoC: fsl-asoc-card: Add missing handling of {hp,mic}-dt-gpios ASoC: dt-bindings: Deprecate {hp,mic}-det-gpio ARM: dts: marvell: mmp2-olpc-xo-1-75: Switch to {hp,mic}-det-gpios arm64: dts: freescale: imx: Switch to simple-audio-card,hp-det-gpios arm64: dts: renesas: beacon-renesom: Switch to mic-det-gpios arm64: dts: rockchip: Switch to simple-audio-card,hp-det-gpios arm64: dts: rockchip: Switch to hp-det-gpios arm64: dts: uniphier: Switch to hp-det-gpios mips: dts: ingenic: Switch to simple-audio-card,hp-det-gpios ARM: dts: nxp: imx: Switch to {hp,mic}-det-gpios arm64: dts: freescale: imx: Switch to hp-det-gpios .../devicetree/bindings/sound/audio-graph.yaml | 6 ++++++ .../devicetree/bindings/sound/simple-card.yaml | 12 ++++++++++++ arch/arm/boot/dts/marvell/mmp2-olpc-xo-1-75.dts | 4 ++-- arch/arm/boot/dts/nxp/imx/imx6qdl-sabresd.dtsi | 4 ++-- arch/arm/boot/dts/nxp/imx/imx6sl-evk.dts | 2 +- arch/arm/boot/dts/nxp/imx/imx6sll-evk.dts | 2 +- arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtsi | 2 +- arch/arm/boot/dts/nxp/imx/imx6ul-14x14-evk.dtsi | 2 +- arch/arm/boot/dts/nxp/imx/imx7d-sdb.dts | 2 +- .../boot/dts/freescale/imx8mq-librem5-devkit.dts | 2 +- arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi | 2 +- arch/arm64/boot/dts/freescale/imx8qm-mek.dts | 2 +- arch/arm64/boot/dts/freescale/imx8qxp-mek.dts | 2 +- arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts | 2 +- .../boot/dts/renesas/beacon-renesom-baseboard.dtsi | 2 +- .../boot/dts/rockchip/rk3326-anbernic-rg351m.dtsi | 2 +- .../arm64/boot/dts/rockchip/rk3326-gameforce-chi.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3326-odroid-go.dtsi | 2 +- arch/arm64/boot/dts/rockchip/rk3368-lba3368.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3399-eaidk-610.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3399-roc-pc-plus.dts | 2 +- .../boot/dts/rockchip/rk3399-rock-pi-4a-plus.dts | 2 +- .../boot/dts/rockchip/rk3399-rock-pi-4b-plus.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3399-rock-pi-4c.dts | 2 +- .../boot/dts/rockchip/rk3399-sapphire-excavator.dts | 2 +- .../boot/dts/rockchip/rk3566-anbernic-rg-arc.dtsi | 2 +- .../boot/dts/rockchip/rk3566-anbernic-rg353p.dts | 2 +- .../boot/dts/rockchip/rk3566-anbernic-rg353ps.dts | 2 +- .../boot/dts/rockchip/rk3566-anbernic-rg353v.dts | 2 +- .../boot/dts/rockchip/rk3566-anbernic-rg353vs.dts | 2 +- .../boot/dts/rockchip/rk3566-anbernic-rg503.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3566-pinetab2.dtsi | 2 +- arch/arm64/boot/dts/rockchip/rk3566-powkiddy-x55.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3568-odroid-m1.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3588-armsom-sige7.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3588-evb1-v10.dts | 2 +- .../dts/rockchip/rk3588-friendlyelec-cm3588-nas.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3588-nanopc-t6.dtsi | 2 +- arch/arm64/boot/dts/rockchip/rk3588-ok3588-c.dts | 2 +- .../boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3588-rock-5-itx.dts | 2 +- arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 2 +- .../boot/dts/rockchip/rk3588s-gameforce-ace.dts | 2 +- .../boot/dts/socionext/uniphier-ld11-global.dts | 2 +- .../boot/dts/socionext/uniphier-ld20-global.dts | 2 +- arch/mips/boot/dts/ingenic/gcw0.dts | 2 +- arch/mips/boot/dts/ingenic/rs90.dts | 2 +- sound/soc/fsl/fsl-asoc-card.c | 8 +++++--- 50 files changed, 72 insertions(+), 52 deletions(-) -- 2.34.1 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert(a)linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

8 30

[PATCH v2 0/3] ASoC: mediatek:mt8186: fix both the speaker and hdmi
by Jiaxin Yu 06 Dec '24

06 Dec '24

The speaker and hdmi of mt8186 platform are shared the same port of I2S, when connect the external display, use build-in speakers to play audio, they both playback at the same time. So we want to manage the playback device through DAPM events. Change since v1: Support audio function for it6505 bridge to hdmi-codec Jiaxin Yu (3): ASoC: hdmi-codec: Add event handler for hdmi TX ASoC: mediatek: mt8186: correct the HDMI widgets drm/bridge: it6505: Add audio support drivers/gpu/drm/bridge/ite-it6505.c | 81 +++++++++++++++++-- include/sound/hdmi-codec.h | 6 ++ sound/soc/codecs/hdmi-codec.c | 37 ++++++++- .../mt8186/mt8186-mt6366-da7219-max98357.c | 2 +- .../mt8186/mt8186-mt6366-rt1019-rt5682s.c | 2 +- 5 files changed, 116 insertions(+), 12 deletions(-) -- 2.18.0

3 16

[PATCH] ASoC: tas2781: Fix redundant logical jump
by Tang Bin 02 Dec '24

02 Dec '24

In these functions, some logical jump of "goto" and variable are redundant, thus remove them. Signed-off-by: Tang Bin <tangbin(a)cmss.chinamobile.com> --- sound/soc/codecs/tas2781-i2c.c | 68 +++++++++++++--------------------- 1 file changed, 25 insertions(+), 43 deletions(-) diff --git a/sound/soc/codecs/tas2781-i2c.c b/sound/soc/codecs/tas2781-i2c.c index 68887799e..27f2f84a4 100644 --- a/sound/soc/codecs/tas2781-i2c.c +++ b/sound/soc/codecs/tas2781-i2c.c @@ -489,14 +489,11 @@ static int tas2563_calib_start_put(struct snd_kcontrol *kcontrol, struct snd_soc_component *comp = snd_soc_kcontrol_component(kcontrol); struct tasdevice_priv *tas_priv = snd_soc_component_get_drvdata(comp); const int sum = ARRAY_SIZE(tas2563_cali_start_reg); - int rc = 1; int i, j; guard(mutex)(&tas_priv->codec_lock); - if (tas_priv->chip_id != TAS2563) { - rc = -1; - goto out; - } + if (tas_priv->chip_id != TAS2563) + return -1; for (i = 0; i < tas_priv->ndev; i++) { struct tasdevice *tasdev = tas_priv->tasdevice; @@ -523,8 +520,8 @@ static int tas2563_calib_start_put(struct snd_kcontrol *kcontrol, q[j].val, 4); } } -out: - return rc; + + return 1; } static void tas2563_calib_stop_put(struct tasdevice_priv *tas_priv) @@ -576,7 +573,7 @@ static int tasdev_cali_data_put(struct snd_kcontrol *kcontrol, struct cali_reg *p = &cali_data->cali_reg_array; unsigned char *src = ucontrol->value.bytes.data; unsigned char *dst = cali_data->data; - int rc = 1, i = 0; + int i = 0; int j; guard(mutex)(&priv->codec_lock); @@ -605,7 +602,7 @@ static int tasdev_cali_data_put(struct snd_kcontrol *kcontrol, i += 3; memcpy(dst, &src[i], cali_data->total_sz); - return rc; + return 1; } static int tas2781_latch_reg_get(struct snd_kcontrol *kcontrol, @@ -1115,25 +1112,21 @@ static int tasdevice_dsp_create_ctrls(struct tasdevice_priv *tas_priv) char *conf_name, *prog_name; int nr_controls = 4; int mix_index = 0; - int ret; /* Alloc kcontrol via devm_kzalloc, which don't manually * free the kcontrol */ dsp_ctrls = devm_kcalloc(tas_priv->dev, nr_controls, sizeof(dsp_ctrls[0]), GFP_KERNEL); - if (!dsp_ctrls) { - ret = -ENOMEM; - goto out; - } + if (!dsp_ctrls) + return -ENOMEM; /* Create mixer items for selecting the active Program and Config */ prog_name = devm_kstrdup(tas_priv->dev, "Speaker Program Id", GFP_KERNEL); - if (!prog_name) { - ret = -ENOMEM; - goto out; - } + if (!prog_name) + return -ENOMEM; + dsp_ctrls[mix_index].name = prog_name; dsp_ctrls[mix_index].iface = SNDRV_CTL_ELEM_IFACE_MIXER; dsp_ctrls[mix_index].info = tasdevice_info_programs; @@ -1143,10 +1136,9 @@ static int tasdevice_dsp_create_ctrls(struct tasdevice_priv *tas_priv) conf_name = devm_kstrdup(tas_priv->dev, "Speaker Config Id", GFP_KERNEL); - if (!conf_name) { - ret = -ENOMEM; - goto out; - } + if (!conf_name) + return -ENOMEM; + dsp_ctrls[mix_index].name = conf_name; dsp_ctrls[mix_index].iface = SNDRV_CTL_ELEM_IFACE_MIXER; dsp_ctrls[mix_index].info = tasdevice_info_configurations; @@ -1156,10 +1148,9 @@ static int tasdevice_dsp_create_ctrls(struct tasdevice_priv *tas_priv) active_dev_num = devm_kstrdup(tas_priv->dev, "Activate Tasdevice Num", GFP_KERNEL); - if (!active_dev_num) { - ret = -ENOMEM; - goto out; - } + if (!active_dev_num) + return -ENOMEM; + dsp_ctrls[mix_index].name = active_dev_num; dsp_ctrls[mix_index].iface = SNDRV_CTL_ELEM_IFACE_MIXER; dsp_ctrls[mix_index].info = tasdevice_info_active_num; @@ -1168,21 +1159,17 @@ static int tasdevice_dsp_create_ctrls(struct tasdevice_priv *tas_priv) mix_index++; chip_id = devm_kstrdup(tas_priv->dev, "Tasdevice Chip Id", GFP_KERNEL); - if (!chip_id) { - ret = -ENOMEM; - goto out; - } + if (!chip_id) + return -ENOMEM; + dsp_ctrls[mix_index].name = chip_id; dsp_ctrls[mix_index].iface = SNDRV_CTL_ELEM_IFACE_MIXER; dsp_ctrls[mix_index].info = tasdevice_info_chip_id; dsp_ctrls[mix_index].get = tasdevice_get_chip_id; mix_index++; - ret = snd_soc_add_component_controls(tas_priv->codec, dsp_ctrls, + return snd_soc_add_component_controls(tas_priv->codec, dsp_ctrls, nr_controls < mix_index ? nr_controls : mix_index); - -out: - return ret; } static int tasdevice_create_cali_ctrls(struct tasdevice_priv *priv) @@ -1469,7 +1456,6 @@ static int tasdevice_hw_params(struct snd_pcm_substream *substream, unsigned int slot_width; unsigned int fsrate; int bclk_rate; - int rc = 0; fsrate = params_rate(params); switch (fsrate) { @@ -1479,8 +1465,7 @@ static int tasdevice_hw_params(struct snd_pcm_substream *substream, default: dev_err(tas_priv->dev, "%s: incorrect sample rate = %u\n", __func__, fsrate); - rc = -EINVAL; - goto out; + return -EINVAL; } slot_width = params_width(params); @@ -1493,20 +1478,17 @@ static int tasdevice_hw_params(struct snd_pcm_substream *substream, default: dev_err(tas_priv->dev, "%s: incorrect slot width = %u\n", __func__, slot_width); - rc = -EINVAL; - goto out; + return -EINVAL; } bclk_rate = snd_soc_params_to_bclk(params); if (bclk_rate < 0) { dev_err(tas_priv->dev, "%s: incorrect bclk rate = %d\n", __func__, bclk_rate); - rc = bclk_rate; - goto out; + return bclk_rate; } -out: - return rc; + return 0; } static int tasdevice_set_dai_sysclk(struct snd_soc_dai *codec_dai, -- 2.33.0

3 2

[PATCH v4] ASoC: Intel: avs: da7219: Remove suspend_pre() and resume_post()
by Marek Maslanka 02 Dec '24

02 Dec '24

The presence of a plugged jack is not detected after resuming the device if the jack was plugged before the device was suspended. This problem is caused by calling the sound/soc/codecs/da7219-aad.c:da7219_aad_jack_det() function on resume, which forces the jack insertion state to be unplugged. Signed-off-by: Marek Maslanka <mmaslanka(a)google.com> --- Changes in v4: - Remove redundant spaces from the commit message - Link to v3: https://lore.kernel.org/all/20241128155926.1774625-1-mmaslanka@google.com/ Changes in v3: - Fix line breaks in commit message - Link to v2: https://lore.kernel.org/all/20241128151239.1666582-1-mmaslanka@google.com/ Changes in v2: - Rephrase commit title and message as suggested in v1 - Link to v1: https://lore.kernel.org/all/20241128122732.1205732-1-mmaslanka@google.com/ --- --- sound/soc/intel/avs/boards/da7219.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/sound/soc/intel/avs/boards/da7219.c b/sound/soc/intel/avs/boards/da7219.c index 80c0a1a956542..daf53ca490a14 100644 --- a/sound/soc/intel/avs/boards/da7219.c +++ b/sound/soc/intel/avs/boards/da7219.c @@ -211,21 +211,6 @@ static int avs_create_dai_link(struct device *dev, const char *platform_name, in return 0; } -static int avs_card_suspend_pre(struct snd_soc_card *card) -{ - struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, DA7219_DAI_NAME); - - return snd_soc_component_set_jack(codec_dai->component, NULL, NULL); -} - -static int avs_card_resume_post(struct snd_soc_card *card) -{ - struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, DA7219_DAI_NAME); - struct snd_soc_jack *jack = snd_soc_card_get_drvdata(card); - - return snd_soc_component_set_jack(codec_dai->component, jack, NULL); -} - static int avs_da7219_probe(struct platform_device *pdev) { struct snd_soc_dai_link *dai_link; @@ -257,8 +242,6 @@ static int avs_da7219_probe(struct platform_device *pdev) card->name = "avs_da7219"; card->dev = dev; card->owner = THIS_MODULE; - card->suspend_pre = avs_card_suspend_pre; - card->resume_post = avs_card_resume_post; card->dai_link = dai_link; card->num_links = 1; card->controls = card_controls; -- 2.47.0.338.g60cca15819-goog

3 2

[PATCH v3] ASoC: Intel: avs: da7219: Remove suspend_pre() and resume_post()
by Marek Maslanka 02 Dec '24

02 Dec '24

The presence of a plugged jack is not detected after resuming the device if the jack was plugged before the device was suspended. This problem is caused by calling the sound/soc/codecs/da7219-aad.c:da7219_aad_jack_det() function on resume, which forces the jack insertion state to be unplugged. Signed-off-by: Marek Maslanka <mmaslanka(a)google.com> --- Changes in v3: - Fix line breaks in commit message - Link to v2: https://lore.kernel.org/all/20241128151239.1666582-1-mmaslanka@google.com/ Changes in v2: - Rephrase commit title and message as suggested in v1 - Link to v1: https://lore.kernel.org/all/20241128122732.1205732-1-mmaslanka@google.com/ --- --- sound/soc/intel/avs/boards/da7219.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/sound/soc/intel/avs/boards/da7219.c b/sound/soc/intel/avs/boards/da7219.c index 80c0a1a956542..daf53ca490a14 100644 --- a/sound/soc/intel/avs/boards/da7219.c +++ b/sound/soc/intel/avs/boards/da7219.c @@ -211,21 +211,6 @@ static int avs_create_dai_link(struct device *dev, const char *platform_name, in return 0; } -static int avs_card_suspend_pre(struct snd_soc_card *card) -{ - struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, DA7219_DAI_NAME); - - return snd_soc_component_set_jack(codec_dai->component, NULL, NULL); -} - -static int avs_card_resume_post(struct snd_soc_card *card) -{ - struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, DA7219_DAI_NAME); - struct snd_soc_jack *jack = snd_soc_card_get_drvdata(card); - - return snd_soc_component_set_jack(codec_dai->component, jack, NULL); -} - static int avs_da7219_probe(struct platform_device *pdev) { struct snd_soc_dai_link *dai_link; @@ -257,8 +242,6 @@ static int avs_da7219_probe(struct platform_device *pdev) card->name = "avs_da7219"; card->dev = dev; card->owner = THIS_MODULE; - card->suspend_pre = avs_card_suspend_pre; - card->resume_post = avs_card_resume_post; card->dai_link = dai_link; card->num_links = 1; card->controls = card_controls; -- 2.47.0.338.g60cca15819-goog

2 1

[PATCH] ASoC: Intel: avs: Remove component->set_jack() on suspend/resume
by Marek Maslanka 02 Dec '24

02 Dec '24

Removed calls to component->set_jack() during suspend/resume to fix an issue where an inserted jack is not detected after resuming from suspend if the jack was inserted before suspend. This problem is caused by calls to the sound/soc/codecs/da7219-aad.c:da7219_aad_jack_det() function during resume, which forces the jack insertion state to be unplugged. Signed-off-by: Marek Maslanka <mmaslanka(a)google.com> --- sound/soc/intel/avs/boards/da7219.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/sound/soc/intel/avs/boards/da7219.c b/sound/soc/intel/avs/boards/da7219.c index 80c0a1a956542..daf53ca490a14 100644 --- a/sound/soc/intel/avs/boards/da7219.c +++ b/sound/soc/intel/avs/boards/da7219.c @@ -211,21 +211,6 @@ static int avs_create_dai_link(struct device *dev, const char *platform_name, in return 0; } -static int avs_card_suspend_pre(struct snd_soc_card *card) -{ - struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, DA7219_DAI_NAME); - - return snd_soc_component_set_jack(codec_dai->component, NULL, NULL); -} - -static int avs_card_resume_post(struct snd_soc_card *card) -{ - struct snd_soc_dai *codec_dai = snd_soc_card_get_codec_dai(card, DA7219_DAI_NAME); - struct snd_soc_jack *jack = snd_soc_card_get_drvdata(card); - - return snd_soc_component_set_jack(codec_dai->component, jack, NULL); -} - static int avs_da7219_probe(struct platform_device *pdev) { struct snd_soc_dai_link *dai_link; @@ -257,8 +242,6 @@ static int avs_da7219_probe(struct platform_device *pdev) card->name = "avs_da7219"; card->dev = dev; card->owner = THIS_MODULE; - card->suspend_pre = avs_card_suspend_pre; - card->resume_post = avs_card_resume_post; card->dai_link = dai_link; card->num_links = 1; card->controls = card_controls; -- 2.47.0.338.g60cca15819-goog

3 4

[PATCH AUTOSEL 5.10 4/6] ASoC: stm: Prevent potential division by zero in stm32_sai_mclk_round_rate()
by Sasha Levin 02 Dec '24

02 Dec '24

From: Luo Yifan <luoyifan(a)cmss.chinamobile.com> [ Upstream commit 63c1c87993e0e5bb11bced3d8224446a2bc62338 ] This patch checks if div is less than or equal to zero (div <= 0). If div is zero or negative, the function returns -EINVAL, ensuring the division operation (*prate / div) is safe to perform. Signed-off-by: Luo Yifan <luoyifan(a)cmss.chinamobile.com> Link: https://patch.msgid.link/20241106014654.206860-1-luoyifan@cmss.chinamobile.… Signed-off-by: Mark Brown <broonie(a)kernel.org> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- sound/soc/stm/stm32_sai_sub.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/stm/stm32_sai_sub.c b/sound/soc/stm/stm32_sai_sub.c index 3aa1cf2624020..3a7f0102b4c5c 100644 --- a/sound/soc/stm/stm32_sai_sub.c +++ b/sound/soc/stm/stm32_sai_sub.c @@ -380,8 +380,8 @@ static long stm32_sai_mclk_round_rate(struct clk_hw *hw, unsigned long rate, int div; div = stm32_sai_get_clk_div(sai, *prate, rate); - if (div < 0) - return div; + if (div <= 0) + return -EINVAL; mclk->freq = *prate / div; -- 2.43.0

2 1

[PATCH] ALSA: hda/realtek: Enable mute and micmute LED on HP ProBook 430 G8
by Nazar Bilinskyi 01 Dec '24

01 Dec '24

HP ProBook 430 G8 has a mute and micmute LEDs that can be made to work using quirk ALC236_FIXUP_HP_GPIO_LED. Enable already existing quirk. Signed-off-by: Nazar Bilinskyi <nbilinskyi(a)gmail.com> --- sound/pci/hda/patch_realtek.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 2bf5c512e..877c5d20a 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -10340,6 +10340,7 @@ static const struct hda_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x103c, 0x87b7, "HP Laptop 14-fq0xxx", ALC236_FIXUP_HP_MUTE_LED_COEFBIT2), SND_PCI_QUIRK(0x103c, 0x87c8, "HP", ALC287_FIXUP_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x87d3, "HP Laptop 15-gw0xxx", ALC236_FIXUP_HP_MUTE_LED_COEFBIT2), + SND_PCI_QUIRK(0x103c, 0x87df, "HP ProBook 430 G8 Notebook PC", ALC236_FIXUP_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x87e5, "HP ProBook 440 G8 Notebook PC", ALC236_FIXUP_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x87e7, "HP ProBook 450 G8 Notebook PC", ALC236_FIXUP_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x87f1, "HP ProBook 630 G8 Notebook PC", ALC236_FIXUP_HP_GPIO_LED), -- 2.47.1

1 0

Linux Mint Cinnamon 22 Soundblaster AE-9 no sound
by GitHub issues - opened 30 Nov '24

30 Nov '24

alsa-project/alsa-lib issue #424 was opened from Starfoxfs: Hi, i have fresh installed Linux Mint Cinnamon 22 Kernel 6.8.0.49-generic I can´t get the Soundblaster AE-9 to work. Creative Labs has no Drivers for Linux and points to the alsa community. Is there any change to get support for the Soundblaster AE-9 in the future ? I think the big Problem is the external DAC for Headphones from the Soundblaster AE-9. In Windows this "clicks" on startup and activates the DAC, but in Linux this is all dead. Issue URL : https://github.com/alsa-project/alsa-lib/issues/424 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

arecordmidi.1: Some remarks and editorial changes for this man page
by Bjarni Ingi Gislason 30 Nov '24

30 Nov '24

Package: alsa-utils Version: 1.2.12-1 Severity: minor Tags: patch * What led up to the situation? Checking for defects with test-[g|n]roff -mandoc -t -K utf8 -rF0 -rHY=0 -ww -b -z < "man page" [Use "groff -e ' $' <file>" to find trailing spaces.] ["test-groff" is a script in the repository for "groff"; is not shipped] (local copy and "troff" slightly changed by me). [The fate of "test-nroff" was decided in groff bug #55941.] * What was the outcome of this action? troff: backtrace: file '<stdin>':69 troff:<stdin>:69: warning: trailing space in the line troff: backtrace: file '<stdin>':77 troff:<stdin>:77: warning: trailing space in the line Output from "test-nroff -mandoc -t -K utf8 -rF0 -rHY=0 -ww -b -z ": troff: backtrace: file '<stdin>':69 troff:<stdin>:69: warning: trailing space in the line troff: backtrace: file '<stdin>':77 troff:<stdin>:77: warning: trailing space in the line * What outcome did you expect instead? No output (no warnings). -.- General remarks and further material, if a diff-file exist, are in the attachments. -- System Information: Debian Release: trixie/sid APT prefers testing APT policy: (500, 'testing') Architecture: amd64 (x86_64) Kernel: Linux 6.11.10-amd64 (SMP w/2 CPU threads; PREEMPT) Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), LANGUAGE not set Shell: /bin/sh linked to /usr/bin/dash Init: sysvinit (via /sbin/init) Versions of packages alsa-utils depends on: ii kmod 33+20240816-2 ii libasound2t64 1.2.12-1+b1 ii libatopology2t64 1.2.12-1+b1 ii libc6 2.40-3 ii libfftw3-single3 3.3.10-1+b3 ii libncursesw6 6.5-2+b1 ii libsamplerate0 0.2.2-4+b2 ii libtinfo6 6.5-2+b1 alsa-utils recommends no packages. Versions of packages alsa-utils suggests: pn dialog <none> -- no debconf information

1 0

[tiwai-sound:for-next 6/10] sound/core/ump.c:1259:69: warning: '%s' directive output may be truncated writing up to 11 bytes into a region of size between 4 and 22
by kernel test robot 29 Nov '24

29 Nov '24

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: 9b5f8ee43e48c25fbe1a10163ec04343d750acd0 commit: e29e504e7890b9ee438ca6370d0180d607c473f9 [6/10] ALSA: ump: Indicate the inactive group in legacy substream names config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20241130/202411300103.FrGuTAYp-lkp@…) compiler: alpha-linux-gcc (GCC) 14.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241130/202411300103.FrGuTAYp-lkp@…) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202411300103.FrGuTAYp-lkp@intel.com/ All warnings (new ones prefixed by >>): sound/core/ump.c: In function 'fill_substream_names': >> sound/core/ump.c:1259:69: warning: '%s' directive output may be truncated writing up to 11 bytes into a region of size between 4 and 22 [-Wformat-truncation=] 1259 | snprintf(s->name, sizeof(s->name), "Group %d (%.16s)%s", | ^~ sound/core/ump.c:1259:17: note: 'snprintf' output between 11 and 40 bytes into a destination of size 32 1259 | snprintf(s->name, sizeof(s->name), "Group %d (%.16s)%s", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1260 | idx + 1, name, | ~~~~~~~~~~~~~~ 1261 | ump->groups[idx].active ? "" : " [Inactive]"); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ vim +1259 sound/core/ump.c 1246 1247 static void fill_substream_names(struct snd_ump_endpoint *ump, 1248 struct snd_rawmidi *rmidi, int dir) 1249 { 1250 struct snd_rawmidi_substream *s; 1251 const char *name; 1252 int idx; 1253 1254 list_for_each_entry(s, &rmidi->streams[dir].substreams, list) { 1255 idx = ump->legacy_mapping[s->number]; 1256 name = ump->groups[idx].name; 1257 if (!*name) 1258 name = ump->info.name; > 1259 snprintf(s->name, sizeof(s->name), "Group %d (%.16s)%s", 1260 idx + 1, name, 1261 ump->groups[idx].active ? "" : " [Inactive]"); 1262 } 1263 } 1264 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

amd-soundwire: add support for AMD generic legacy machine driver
by GitHub pull_request - opened 29 Nov '24

29 Nov '24

alsa-project/alsa-ucm-conf pull request #478 was opened from perexg: The card components string is similar to Intel's SOF driver like, so we can reuse sof-soundwire configuration and it is logical to reduce future maintenance, because the chips connected to soundwire bus are similar for multiple PCI bridges. Example of the components string: Components : ' cfg-amp:1 hs:rt722 spk:rt722 mic:rt722' Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/478 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/478.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[GIT PULL] ASoC fixes for v6.13-merge-window
by Mark Brown 28 Nov '24

28 Nov '24

The following changes since commit 82ff5abc2edcfba0c0f1a1be807795e2876f46e9: ASoC: hdmi-codec: reorder channel allocation list (2024-11-15 13:43:00 +0000) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.13-merge-window for you to fetch changes up to b682aa788e5f9f1ddacdfbb453e49fd3f4e83721: ASoC: amd: yc: Add a quirk for microfone on Lenovo ThinkPad P14s Gen 5 21MES00B00 (2024-11-27 13:45:45 +0000) ---------------------------------------------------------------- ASoC: Fixes for v6.13 A pile of driver specific quirks and fixes that came in since the merge window. One of the AMD fixes is a bit broken for some systems, I'm expecting an incremental change to fix that but it seems better overall to merge the rest of the fixes. There's also one small documentation update that seemed sensible to apply now, pointing to the dapm-graph tool. ---------------------------------------------------------------- Alex Far (1): ASoC: amd: yc: fix internal mic on Redmi G 2022 Bard Liao (1): ASoC: SOF: ipc3-topology: Convert the topology pin index to ALH dai index Charles Han (1): ASoC: imx-audmix: Add NULL check in imx_audmix_probe Ilya Zverev (1): ASoC: amd: yc: Add a quirk for microfone on Lenovo ThinkPad P14s Gen 5 21MES00B00 Mario Limonciello (1): ASoC: amd: Fix build dependencies for `SND_SOC_AMD_PS` Nícolas F. R. A. Prado (1): ASoC: mediatek: Check num_codecs is not zero to avoid panic during probe Uwe Kleine-König (1): ASoC: amd: yc: Add quirk for microphone on Lenovo Thinkpad T14s Gen 6 21M1CTO1WW Venkata Prasad Potturu (1): ASoC: amd: yc: Fix for enabling DMIC on acp6x via _DSD entry anish kumar (1): ASoC: doc: dapm: Add location information for dapm-graph tool liujing (1): ASoC: apple: Fix the wrong format specifier Documentation/sound/soc/dapm.rst | 3 ++ sound/soc/amd/Kconfig | 1 + sound/soc/amd/yc/acp6x-mach.c | 39 +++++++++++++++++++++- sound/soc/apple/mca.c | 2 +- sound/soc/fsl/imx-audmix.c | 3 ++ sound/soc/mediatek/mt8188/mt8188-mt6359.c | 9 +++-- .../mediatek/mt8192/mt8192-mt6359-rt1015-rt5682.c | 4 +-- sound/soc/mediatek/mt8195/mt8195-mt6359.c | 9 +++-- sound/soc/sof/ipc3-topology.c | 26 +++++++++++++-- 9 files changed, 86 insertions(+), 10 deletions(-)

2 1

aplay.1: Some remarks and editorial changes for this man page
by Bjarni Ingi Gislason 27 Nov '24

27 Nov '24

Package: alsa-utils Version: 1.2.12-1 (Debian testing) Severity: minor Tags: patch * What led up to the situation? Checking for defects with test-[g|n]roff -mandoc -t -K utf8 -rF0 -rHY=0 -ww -b -z < "man page" [Use "groff -e ' $' <file>" to find trailing spaces.] ["test-groff" is a script in the repository for "groff"; is not shipped] (local copy and "troff" slightly changed by me). [The fate of "test-nroff" was decided in groff bug #55941.] * What was the outcome of this action? troff: backtrace: file '<stdin>':3 troff:<stdin>:3: warning: trailing space in the line troff: backtrace: file '<stdin>':87 troff:<stdin>:87: warning: trailing space in the line troff: backtrace: file '<stdin>':91 troff:<stdin>:91: warning: trailing space in the line troff: backtrace: file '<stdin>':95 troff:<stdin>:95: warning: trailing space in the line troff: backtrace: file '<stdin>':99 troff:<stdin>:99: warning: trailing space in the line troff: backtrace: file '<stdin>':103 troff:<stdin>:103: warning: trailing space in the line troff: backtrace: file '<stdin>':107 troff:<stdin>:107: warning: trailing space in the line troff: backtrace: file '<stdin>':111 troff:<stdin>:111: warning: trailing space in the line troff: backtrace: file '<stdin>':114 troff:<stdin>:114: warning: trailing space in the line troff: backtrace: file '<stdin>':115 troff:<stdin>:115: warning: trailing space in the line troff: backtrace: file '<stdin>':118 troff:<stdin>:118: warning: trailing space in the line troff: backtrace: file '<stdin>':121 troff:<stdin>:121: warning: trailing space in the line troff: backtrace: file '<stdin>':131 troff:<stdin>:131: warning: trailing space in the line troff: backtrace: file '<stdin>':215 troff:<stdin>:215: warning: trailing space in the line troff: backtrace: file '<stdin>':225 troff:<stdin>:225: warning: trailing space in the line troff: backtrace: file '<stdin>':258 troff:<stdin>:258: warning: trailing space in the line * What outcome did you expect instead? No output (no warnings). -.- General remarks and further material, if a diff-file exist, are in the attachments. -- System Information: Debian Release: trixie/sid APT prefers testing APT policy: (500, 'testing') Architecture: amd64 (x86_64) Kernel: Linux 6.11.9-amd64 (SMP w/2 CPU threads; PREEMPT) Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), LANGUAGE not set Shell: /bin/sh linked to /usr/bin/dash Init: sysvinit (via /sbin/init) Versions of packages alsa-utils depends on: ii kmod 33+20240816-2 ii libasound2t64 1.2.12-1+b1 ii libatopology2t64 1.2.12-1+b1 ii libc6 2.40-3 ii libfftw3-single3 3.3.10-1+b3 ii libncursesw6 6.5-2+b1 ii libsamplerate0 0.2.2-4+b2 ii libtinfo6 6.5-2+b1 alsa-utils recommends no packages. Versions of packages alsa-utils suggests: pn dialog <none> -- no debconf information

1 0

[PATCH] ASoC: amd: yc: Fix for enabling DMIC on acp6x via _DSD entry
by Venkata Prasad Potturu 27 Nov '24

27 Nov '24

Add condition check to register ACP PDM sound card by reading _WOV acpi entry. Fixes: 5426f506b584 ("ASoC: amd: Add support for enabling DMIC on acp6x via _DSD") Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu(a)amd.com> --- sound/soc/amd/yc/acp6x-mach.c | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/sound/soc/amd/yc/acp6x-mach.c b/sound/soc/amd/yc/acp6x-mach.c index 438865d5e376..c301dac8b979 100644 --- a/sound/soc/amd/yc/acp6x-mach.c +++ b/sound/soc/amd/yc/acp6x-mach.c @@ -523,8 +523,14 @@ static int acp6x_probe(struct platform_device *pdev) struct acp6x_pdm *machine = NULL; struct snd_soc_card *card; struct acpi_device *adev; + acpi_handle handle; + acpi_integer dmic_status; int ret; + bool is_dmic_enable, wov_en; + /* IF WOV entry not found, enable dmic based on AcpDmicConnected entry*/ + is_dmic_enable = false; + wov_en = true; /* check the parent device's firmware node has _DSD or not */ adev = ACPI_COMPANION(pdev->dev.parent); if (adev) { @@ -532,9 +538,19 @@ static int acp6x_probe(struct platform_device *pdev) if (!acpi_dev_get_property(adev, "AcpDmicConnected", ACPI_TYPE_INTEGER, &obj) && obj->integer.value == 1) - platform_set_drvdata(pdev, &acp6x_card); + is_dmic_enable = true; } + handle = ACPI_HANDLE(pdev->dev.parent); + ret = acpi_evaluate_integer(handle, "_WOV", NULL, &dmic_status); + if (!ACPI_FAILURE(ret)) + wov_en = dmic_status; + + if (is_dmic_enable && wov_en) + platform_set_drvdata(pdev, &acp6x_card); + else + return 0; + /* check for any DMI overrides */ dmi_id = dmi_first_match(yc_acp_quirk_table); if (dmi_id) -- 2.39.2

4 5

[BUG] HDA Intel PCH, HDMI , HDA Intel PCH, ALC886 Digital, no IEC958 / HDMI output/passthrough to AVR, partly fixed but HDMI AC3 passthrough
by schorpp 27 Nov '24

27 Nov '24

>>> >>> [BUG] HDA Intel PCH, HDMI , HDA Intel PCH, ALC886 Digital, no IEC958 / HDMI output/passthrough to AVR >>> >>> ALSA versions 1.0.28 >>> >>> Kernel Driver versions 3.12.40 > Partly FIXED in Longterm Stable Kernel 4.19.324 and using HDMI DEV 1 instead of 0. > > 2 CH PCM HDMI to AVR works now but AC3/DTS passthrough still not or is not recognized by AVR or some wrong AES setting but iec958 ALC886 optical toslink AC3/DTS output to AVR works. > > New BUG: No more 5.1 Analog Output on the CM106 USB Box if AC3 stream is selected. No. Sorry, VDR xineliboutput config error, needs speaker setup 5.1 instead of pass through. y tom > > Your ALSA information is located at http://alsa-project.org/db/?f=5ef8b56450583ba8afbcc705afc02228da4a3072 > > y > tom > >>> >>> Hardware: >>> >>> DFI Mainboard MB330-CRM >>> >>> https://www.dfi.com/de/product/index/273#specification >>> >>> Reproducible Always >>> >>> Steps to reproduce and full report: >>> >>> https://www-vdr--portal-de.translate.goog/forum/index.php?thread/136519-gel… >> Output of alsa-info.sh is here >> >> http://alsa-project.org/db/?f=fc4734deddd9a92331692bf94b15cf84b7722cd4 >> >>> >>> Software crosscheck with CM106 USB Soundbox iec958: works OK. >>> >>> Where to start in the drivers source code to fix it? >>> >>> y >>> tom >>> >>>

1 0

alsa-ucm-conf update from 1.2.12 to 1.2.13 causes HDMI/DsiplayPort outputs to disappear
by GitHub issues - opened 27 Nov '24

27 Nov '24

alsa-project/alsa-ucm-conf issue #477 was opened from mwsltn: Basic Info: Linux Distro: Arch Kernel Version: 6.12.1 I have very reliably been using a USB-C connected external monitor that has internal speakers represented by the output device: HDMI / DisplayPort 1 After running a package update this AM all the HDMI/DisplayPort outputs disappeared. First I downgraded the kernel which didn't help, then checked the pacman logs to see what else had changed and the list included the following: ``` [2024-11-26T05:07:01-0600] [ALPM] upgraded alsa-ucm-conf (1.2.12-1 -> 1.2.13-1) [2024-11-26T05:07:01-0600] [ALPM] upgraded alsa-lib (1.2.12-1 -> 1.2.13-1) [2024-11-26T05:07:01-0600] [ALPM] upgraded alsa-utils (1.2.12-1 -> 1.2.13-1) ``` So I downgrade: ``` [2024-11-26T06:51:22-0600] [PACMAN] Running 'pacman -U file:///var/cache/pacman/pkg/alsa-ucm-conf-1.2.12-1-any.pkg.tar.zst file:///var/cache/pacman/pkg/alsa-lib-1.2.12-1-x86_64.pkg.tar.zst file:///var/cache/pacman/pkg/alsa-utils-1.2.12-1-x86_64.pkg.tar.zst' [2024-11-26T06:51:24-0600] [ALPM] downgraded alsa-ucm-conf (1.2.13-1 -> 1.2.12-1) [2024-11-26T06:51:24-0600] [ALPM] downgraded alsa-lib (1.2.13-1 -> 1.2.12-1) [2024-11-26T06:51:24-0600] [ALPM] downgraded alsa-utils (1.2.13-1 -> 1.2.12-1) ``` Problem fixed. Please let me know if I can provide any other info. Originally I had opened an issue on the `alsa-lib` project and others are reporting the same: https://github.com/alsa-project/alsa-lib/issues/423 Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/477 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

alsactl.1: Some remarks and editorial changes for this man page
by Bjarni Ingi Gislason 27 Nov '24

27 Nov '24

Package: alsa-utils Version: 1.2.12-1 Severity: minor Tags: patch * What led up to the situation? Checking for defects with test-[g|n]roff -mandoc -t -K utf8 -rF0 -rHY=0 -ww -b -z < "man page" [Use "groff -e ' $' <file>" to find trailing spaces.] ["test-groff" is a script in the repository for "groff"; is not shipped] (local copy and "troff" slightly changed by me). [The fate of "test-nroff" was decided in groff bug #55941.] * What was the outcome of this action? troff: backtrace: file '<stdin>':93 troff:<stdin>:93: warning: trailing space in the line troff: backtrace: file '<stdin>':224 troff:<stdin>:224: warning: trailing space in the line * What outcome did you expect instead? No output (no warnings). -.- General remarks and further material, if a diff-file exist, are in the attachments. -- System Information: Debian Release: trixie/sid APT prefers testing APT policy: (500, 'testing') Architecture: amd64 (x86_64) Kernel: Linux 6.11.9-amd64 (SMP w/2 CPU threads; PREEMPT) Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), LANGUAGE not set Shell: /bin/sh linked to /usr/bin/dash Init: sysvinit (via /sbin/init) Versions of packages alsa-utils depends on: ii kmod 33+20240816-2 ii libasound2t64 1.2.12-1+b1 ii libatopology2t64 1.2.12-1+b1 ii libc6 2.40-3 ii libfftw3-single3 3.3.10-1+b3 ii libncursesw6 6.5-2+b1 ii libsamplerate0 0.2.2-4+b2 ii libtinfo6 6.5-2+b1 alsa-utils recommends no packages. Versions of packages alsa-utils suggests: pn dialog <none> -- no debconf information

1 0

alsabat.1: Some remarks and editorial changes for this man page
by Bjarni Ingi Gislason 27 Nov '24

27 Nov '24

Package: alsa-utils Version: 1.2.12-1 (Debian testing) Severity: minor Tags: patch * What led up to the situation? Checking for defects with test-[g|n]roff -mandoc -t -K utf8 -rF0 -rHY=0 -ww -b -z < "man page" [Use "groff -e ' $' <file>" to find trailing spaces.] ["test-groff" is a script in the repository for "groff"; is not shipped] (local copy and "troff" slightly changed by me). [The fate of "test-nroff" was decided in groff bug #55941.] * What was the outcome of this action? troff: backtrace: file '<stdin>':175 troff:<stdin>:175: warning: trailing space in the line * What outcome did you expect instead? No output (no warnings). -.- General remarks and further material, if a diff-file exist, are in the attachments. -- System Information: Debian Release: trixie/sid APT prefers testing APT policy: (500, 'testing') Architecture: amd64 (x86_64) Kernel: Linux 6.11.9-amd64 (SMP w/2 CPU threads; PREEMPT) Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), LANGUAGE not set Shell: /bin/sh linked to /usr/bin/dash Init: sysvinit (via /sbin/init) Versions of packages alsa-utils depends on: ii kmod 33+20240816-2 ii libasound2t64 1.2.12-1+b1 ii libatopology2t64 1.2.12-1+b1 ii libc6 2.40-3 ii libfftw3-single3 3.3.10-1+b3 ii libncursesw6 6.5-2+b1 ii libsamplerate0 0.2.2-4+b2 ii libtinfo6 6.5-2+b1 alsa-utils recommends no packages. Versions of packages alsa-utils suggests: pn dialog <none> -- no debconf information

1 0

RE: Set pcbeep to default value
by Kailang 26 Nov '24

26 Nov '24

Sorry! Attach again. Add ALSA: hda/realtek: to Title. > -----Original Message----- > From: Kailang > Sent: Tuesday, November 26, 2024 10:36 AM > To: 'Takashi Iwai' <tiwai(a)suse.de> > Cc: (alsa-devel(a)alsa-project.org) <alsa-devel(a)alsa-project.org>; > 'linux-sound(a)vger.kernel.org' <linux-sound(a)vger.kernel.org> > Subject: RE: Set pcbeep to default value > > OK. > > > -----Original Message----- > > From: Takashi Iwai <tiwai(a)suse.de> > > Sent: Monday, November 25, 2024 10:45 PM > > To: Kailang <kailang(a)realtek.com> > > Cc: Takashi Iwai <tiwai(a)suse.de> > > Subject: Re: Set pcbeep to default value > > > > > > External mail. > > > > > > > > On Mon, 25 Nov 2024 03:00:26 +0100, > > Kailang wrote: > > > > > > > > > > > > > -----Original Message----- > > > > From: Takashi Iwai <tiwai(a)suse.de> > > > > Sent: Friday, November 22, 2024 5:34 PM > > > > To: Kailang <kailang(a)realtek.com> > > > > Cc: (alsa-devel(a)alsa-project.org) <alsa-devel(a)alsa-project.org> > > > > Subject: Re: Set pcbeep to default value > > > > > > > > > > > > External mail. > > > > > > > > > > > > > > > > On Fri, 22 Nov 2024 08:29:56 +0100, Kailang wrote: > > > > > > > > > > Hi Takashi, > > > > > > > > > > Attach patch will set pcbeep to default value for ALC274. > > > > > > > > Could you resubmit with your Signed-off-by tag? > > > > > > Sorry! Attach again. > > > > You seem to have dropped Cc to alsa-devel ML. > > Also, nowadays we use linux-sound(a)vger.kernel.org for the main ML for > > kernel patches. Care to resubmit to there? > > > > > > thanks, > > > > Takashi

2 1

alsa-lib update from 1.2.12 to 1.2.13 causes HDMI/DsiplayPort outputs to disappear
by GitHub issues - opened 26 Nov '24

26 Nov '24

alsa-project/alsa-lib issue #423 was opened from mwsltn: Basic Info: Linux Distro: Arch Kernel Version: 6.12.1 I have very reliably been using a USB-C connected external monitor that has internal speakers represented by the output device: `HDMI / DisplayPort 1` After running a package update this AM all the HDMI/DisplayPort outputs disappeared. First I downgraded the kernel which didn't help, then checked the pacman logs to see what else had changed and the list included the following: ``` [2024-11-26T05:07:01-0600] [ALPM] upgraded alsa-ucm-conf (1.2.12-1 -> 1.2.13-1) [2024-11-26T05:07:01-0600] [ALPM] upgraded alsa-lib (1.2.12-1 -> 1.2.13-1) [2024-11-26T05:07:01-0600] [ALPM] upgraded alsa-utils (1.2.12-1 -> 1.2.13-1) ``` So I downgrade: ``` [2024-11-26T06:51:22-0600] [PACMAN] Running 'pacman -U file:///var/cache/pacman/pkg/alsa-ucm-conf-1.2.12-1-any.pkg.tar.zst file:///var/cache/pacman/pkg/alsa-lib-1.2.12-1-x86_64.pkg.tar.zst file:///var/cache/pacman/pkg/alsa-utils-1.2.12-1-x86_64.pkg.tar.zst' [2024-11-26T06:51:24-0600] [ALPM] downgraded alsa-ucm-conf (1.2.13-1 -> 1.2.12-1) [2024-11-26T06:51:24-0600] [ALPM] downgraded alsa-lib (1.2.13-1 -> 1.2.12-1) [2024-11-26T06:51:24-0600] [ALPM] downgraded alsa-utils (1.2.13-1 -> 1.2.12-1) ``` Problem fixed. Please let me know if I can provide any other info. Issue URL : https://github.com/alsa-project/alsa-lib/issues/423 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

kernel snd seq bugs: SND_SEQ_EVENT_CLIENT_CHANGE & SND_SEQ_EVENT_PORT_CHANGE
by Mark Lentczner 26 Nov '24

26 Nov '24

I'm the author of amidiminder - a utility that keeps track of desired connections between ALSA Seq (MIDI) ports. I've found a bug in the kernel part of the ALSA Sequencer, and am not sure where or how to report it. Summary: The events: * SND_SEQ_EVENT_CLIENT_CHANGE is never sent * SND_SEQ_EVENT_PORT_CHANGE is not sent on common port changes (name and port info) I have a lot of detail on these bugs, what other software causes them to be a problem (PureData), and where the missing code in the kernel sound/core/seq files is. Before I dump all the details in this forum.... where is the right place for me to report and/or discuss this? - Mark

2 3

Build failures with GCC 15 (-std=gnu23)
by GitHub issues - edited 26 Nov '24

26 Nov '24

alsa-project/alsa-tools issue #25 was edited from kostadinsh: Some of the programs in alsa-tools fail to build with upcoming GCC 15 which uses `-std=gnu23` by default. This can be reproduced on older GCC and Clang versions by adding `-std=gnu23` to CFLAGS. Downstream Gentoo bug report: https://bugs.gentoo.org/943946 hdajackretask: ``` apply-changes.c:358:10: error: conflicting types for ‘reset_changes_boot’; have ‘gboolean(GError **)’ {aka ‘int(struct _GError **)’} 358 | gboolean reset_changes_boot(GError** err) | ^~~~~~~~~~~~~~~~~~ In file included from apply-changes.c:12: apply-changes.h:12:10: note: previous declaration of ‘reset_changes_boot’ with type ‘gboolean(void)’ {aka ‘int(void)’} 12 | gboolean reset_changes_boot(); | ^~~~~~~~~~~~~~~~~~ main-gtk.c: In function ‘reset_boot_clicked’: main-gtk.c:375:5: error: too many arguments to function ‘reset_changes_boot’ 375 | reset_changes_boot(&err); | ^~~~~~~~~~~~~~~~~~ In file included from main-gtk.c:11: apply-changes.h:12:10: note: declared here 12 | gboolean reset_changes_boot(); | ^~~~~~~~~~~~~~~~~~ ``` envy24control: ``` new_process.c: In function ‘new_process’: new_process.c:67:18: error: assignment to ‘void (*)(void)’ from incompatible pointer type ‘__sighandler_t’ {aka ‘void (*)(int)’} [-Wincompatible-pointer-types] 67 | int_stat = signal(SIGINT, SIG_IGN); | ^ new_process.c:68:19: error: assignment to ‘void (*)(void)’ from incompatible pointer type ‘__sighandler_t’ {aka ‘void (*)(int)’} [-Wincompatible-pointer-types] 68 | quit_stat = signal(SIGQUIT, SIG_IGN); | ^ new_process.c:69:19: error: assignment to ‘void (*)(void)’ from incompatible pointer type ‘__sighandler_t’ {aka ‘void (*)(int)’} [-Wincompatible-pointer-types] 69 | usr2_stat = signal(SIGUSR2, SIG_IGN); | ^ new_process.c:81:24: error: passing argument 2 of ‘signal’ from incompatible pointer type [-Wincompatible-pointer-types] 81 | signal(SIGINT, int_stat); | ^~~~~~~~ | | | void (*)(void) In file included from envy24control.h:4: /usr/include/signal.h:88:57: note: expected ‘__sighandler_t’ {aka ‘void (*)(int)’} but argument is of type ‘void (*)(void)’ 88 | extern __sighandler_t signal (int __sig, __sighandler_t __handler) | ~~~~~~~~~~~~~~~^~~~~~~~~ new_process.c:82:25: error: passing argument 2 of ‘signal’ from incompatible pointer type [-Wincompatible-pointer-types] 82 | signal(SIGQUIT, quit_stat); | ^~~~~~~~~ | | | void (*)(void) /usr/include/signal.h:88:57: note: expected ‘__sighandler_t’ {aka ‘void (*)(int)’} but argument is of type ‘void (*)(void)’ 88 | extern __sighandler_t signal (int __sig, __sighandler_t __handler) | ~~~~~~~~~~~~~~~^~~~~~~~~ new_process.c:83:25: error: passing argument 2 of ‘signal’ from incompatible pointer type [-Wincompatible-pointer-types] 83 | signal(SIGUSR2, usr2_stat); | ^~~~~~~~~ | | | void (*)(void) /usr/include/signal.h:88:57: note: expected ‘__sighandler_t’ {aka ‘void (*)(int)’} but argument is of type ‘void (*)(void)’ 88 | extern __sighandler_t signal (int __sig, __sighandler_t __handler) | ~~~~~~~~~~~~~~~^~~~~~~~~ ``` Issue URL : https://github.com/alsa-project/alsa-tools/issues/25 Repository URL: https://github.com/alsa-project/alsa-tools

1 0

Build failures with GCC 15 (-std=gnu23)
by GitHub issues - opened 26 Nov '24

26 Nov '24

alsa-project/alsa-tools issue #25 was opened from kostadinsh: Some of the programs in alsa-tools fail to build with upcoming GCC 15 which uses `-std=gnu23` by default. This can be reproduced on older GCC and Clang versions by adding `-std=gnu23` to CFLAGS. Downstream Gentoo bug report: https://bugs.gentoo.org/943946 hdajackretask: ``` main-gtk.c: In function ‘reset_boot_clicked’: main-gtk.c:375:5: error: too many arguments to function ‘reset_changes_boot’ 375 | reset_changes_boot(&err); | ^~~~~~~~~~~~~~~~~~ In file included from main-gtk.c:11: apply-changes.h:12:10: note: declared here 12 | gboolean reset_changes_boot(); | ^~~~~~~~~~~~~~~~~~ ``` envy24control: ``` new_process.c: In function ‘new_process’: new_process.c:67:18: error: assignment to ‘void (*)(void)’ from incompatible pointer type ‘__sighandler_t’ {aka ‘void (*)(int)’} [-Wincompatible-pointer-types] 67 | int_stat = signal(SIGINT, SIG_IGN); | ^ new_process.c:68:19: error: assignment to ‘void (*)(void)’ from incompatible pointer type ‘__sighandler_t’ {aka ‘void (*)(int)’} [-Wincompatible-pointer-types] 68 | quit_stat = signal(SIGQUIT, SIG_IGN); | ^ new_process.c:69:19: error: assignment to ‘void (*)(void)’ from incompatible pointer type ‘__sighandler_t’ {aka ‘void (*)(int)’} [-Wincompatible-pointer-types] 69 | usr2_stat = signal(SIGUSR2, SIG_IGN); | ^ new_process.c:81:24: error: passing argument 2 of ‘signal’ from incompatible pointer type [-Wincompatible-pointer-types] 81 | signal(SIGINT, int_stat); | ^~~~~~~~ | | | void (*)(void) In file included from envy24control.h:4: /usr/include/signal.h:88:57: note: expected ‘__sighandler_t’ {aka ‘void (*)(int)’} but argument is of type ‘void (*)(void)’ 88 | extern __sighandler_t signal (int __sig, __sighandler_t __handler) | ~~~~~~~~~~~~~~~^~~~~~~~~ new_process.c:82:25: error: passing argument 2 of ‘signal’ from incompatible pointer type [-Wincompatible-pointer-types] 82 | signal(SIGQUIT, quit_stat); | ^~~~~~~~~ | | | void (*)(void) /usr/include/signal.h:88:57: note: expected ‘__sighandler_t’ {aka ‘void (*)(int)’} but argument is of type ‘void (*)(void)’ 88 | extern __sighandler_t signal (int __sig, __sighandler_t __handler) | ~~~~~~~~~~~~~~~^~~~~~~~~ new_process.c:83:25: error: passing argument 2 of ‘signal’ from incompatible pointer type [-Wincompatible-pointer-types] 83 | signal(SIGUSR2, usr2_stat); | ^~~~~~~~~ | | | void (*)(void) /usr/include/signal.h:88:57: note: expected ‘__sighandler_t’ {aka ‘void (*)(int)’} but argument is of type ‘void (*)(void)’ 88 | extern __sighandler_t signal (int __sig, __sighandler_t __handler) | ~~~~~~~~~~~~~~~^~~~~~~~~ ``` Issue URL : https://github.com/alsa-project/alsa-tools/issues/25 Repository URL: https://github.com/alsa-project/alsa-tools

1 0

No built-in microphone in Thinkpad X1 Carbon (Gen 12) after update to v1.2.13
by GitHub issues - opened 26 Nov '24

26 Nov '24

alsa-project/alsa-ucm-conf issue #476 was opened from iyanmv: I think the title is self-explanatory. After updating from v1.2.12 to v1.2.13 in Arch Linux, the built-in microphone on my laptop is not working. Downgrading to previous versions fixes the issue. In addition, I also noticed that when attached to an external monitor using a docking station, the speakers of the monitor are not listed as before, and only the laptop's built-in speaker appears. I'm running linux 6.12.1 and KDE Plasma 6.2.3. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/476 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

how do I create configure a ucm2 conf file for my card just like an acp file
by GitHub issues - opened 26 Nov '24

26 Nov '24

alsa-project/alsa-ucm-conf issue #475 was opened from beh-10257: 1_ how do I know if my card is using acp or ucm profile 2_ can ucm config file be availble in user directories 3_ how do i test a ucm profile 4_ if my card is using ucm is there a way to cooverride that file 5_ for more context https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4421 6_ as a tldr I want to take advanatage of indep_hp=yes (IDK why this is not enabled by default its a great feature compared to other platforms after all) either way I want to make hw:0,0 as speaker and hw:0,2 as headphones can a ucm profile replace the indep_hp hint and udev rule and just porceed to seperate them directly also indep_hp seperates speaker and headphones output is there a way to seperate internal mic and headphone speaker also is there a way to put priority for headphones when the jack port is actually put or even disable it entirely and just default back to speaker automatically thats all thanks thats all and thanks further editing will be filled once I have more questions and thanks Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/475 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

ALC225 update depop procedure
by Kailang 26 Nov '24

26 Nov '24

Hi Takashi, ALC225 update depop procedure. Many Thanks. BR, Kailang

2 1

Enable speaker pins for Medion E15443 platform
by Kailang 26 Nov '24

26 Nov '24

Hi Takashi, Attach was fixed speaker no sound issue. BR, Kailang

2 1

Device selection regression in 1.2.13 for LNL
by GitHub issues - edited 26 Nov '24

26 Nov '24

alsa-project/alsa-ucm-conf issue #474 was edited from CarelessCaution: On Intel Lunar Lake, there seems to be a regression in alsa-ucm-conf that affects device selection in KDE Plasma and probably in other software. In 1.2.12, there are the options Play HiFi Quality Music and Pro Audio for both input and output. However, when upgrading to 1.2.13, there is only the option for Pro Audio. This problem is immediately remedied by downgrading just the alsa-ucm-conf package on Arch Linux. According to `aplay -l`, the audio card is sofhdadsp. Before (1.2.12): ![before](https://github.com/user-attachments/assets/c7fa81e0-97f1-45ae-a14e-8549f85cdd27) After (1.2.13): ![after](https://github.com/user-attachments/assets/5867886b-6686-4265-ad68-50c7f53e1a47) Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/474 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Device selection regression in 1.2.13 for LNL
by GitHub issues - edited 26 Nov '24

26 Nov '24

alsa-project/alsa-ucm-conf issue #474 was edited from CarelessCaution: On Intel Lunar Lake, there seems to be a regression in alsa-ucm-conf that affects device selection in KDE Plasma and probably in other software. In 1.2.12, there are the options Play HiFi Quality Music and Pro Audio for both input and output. However, when upgrading to 1.2.13, there is only the option for Pro Audio. This problem is immediately remedied by downgrading just the alsa-ucm-conf package on Arch Linux. According to `aplay -l`, the audio card is sofhdadsp . Before (1.2.12): ![before](https://github.com/user-attachments/assets/c7fa81e0-97f1-45ae-a14e-8549f85cdd27) After (1.2.13): ![after](https://github.com/user-attachments/assets/5867886b-6686-4265-ad68-50c7f53e1a47) Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/474 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Device selection regression in 1.2.13 for LNL
by GitHub issues - opened 26 Nov '24

26 Nov '24

alsa-project/alsa-ucm-conf issue #474 was opened from CarelessCaution: On Intel Lunar Lake, there seems to be a regression in alsa-ucm-conf that affects device selection in KDE Plasma and probably in other software. In 1.2.12, there are the options Play HiFi Quality Music and Pro Audio for both input and output. However, when upgrading to 1.2.13, there is only the option for Pro Audio. This problem is immediately remedied by downgrading just the alsa-ucm-conf package on Arch Linux. Before (1.2.12): ![before](https://github.com/user-attachments/assets/c7fa81e0-97f1-45ae-a14e-8549f85cdd27) After (1.2.13): ![after](https://github.com/user-attachments/assets/5867886b-6686-4265-ad68-50c7f53e1a47) Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/474 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

RE: Set pcbeep to default value
by Kailang 26 Nov '24

26 Nov '24

OK. > -----Original Message----- > From: Takashi Iwai <tiwai(a)suse.de> > Sent: Monday, November 25, 2024 10:45 PM > To: Kailang <kailang(a)realtek.com> > Cc: Takashi Iwai <tiwai(a)suse.de> > Subject: Re: Set pcbeep to default value > > > External mail. > > > > On Mon, 25 Nov 2024 03:00:26 +0100, > Kailang wrote: > > > > > > > > > -----Original Message----- > > > From: Takashi Iwai <tiwai(a)suse.de> > > > Sent: Friday, November 22, 2024 5:34 PM > > > To: Kailang <kailang(a)realtek.com> > > > Cc: (alsa-devel(a)alsa-project.org) <alsa-devel(a)alsa-project.org> > > > Subject: Re: Set pcbeep to default value > > > > > > > > > External mail. > > > > > > > > > > > > On Fri, 22 Nov 2024 08:29:56 +0100, > > > Kailang wrote: > > > > > > > > Hi Takashi, > > > > > > > > Attach patch will set pcbeep to default value for ALC274. > > > > > > Could you resubmit with your Signed-off-by tag? > > > > Sorry! Attach again. > > You seem to have dropped Cc to alsa-devel ML. > Also, nowadays we use linux-sound(a)vger.kernel.org for the main ML for kernel > patches. Care to resubmit to there? > > > thanks, > > Takashi

1 0

GoXLR not working in 1.2.13
by GitHub issues - opened 25 Nov '24

25 Nov '24

alsa-project/alsa-ucm-conf issue #473 was opened from AruAki: GoXLR not splitting into multiple channels. `alsaucm -c hw:1 dump text` returns ``` ALSA lib parser.c:270:(parse_syntax_field) Incompatible syntax 7 in USB-Audio.conf ALSA lib main.c:1554:(snd_use_case_mgr_open) error: failed to import hw:1 use case configuration -22 alsaucm: error failed to open sound card hw:1: Invalid argument ``` Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/473 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[RFC PATCH 0/2] Add support for stacked and parallel memories
by Amit Kumar Mahapatra 25 Nov '24

25 Nov '24

Hello Everyone, Following an email discussion with Miquel regarding the binding changes and overall architecture for implementing support for stacked and parallel memories, I’m sharing this RFC to initiate a discussion on the proposed updates to current bindings and to finalize the implementation architecture. Before diving into the main topic, here is some background on stacked and parallel memories. The AMD QSPI controller supports two advanced connection modes(Stacked and Parallel) which allow the controller to treat two different flashes as one storage. Stacked: Flashes share the same SPI bus, but different CS line, controller driver asserts the CS of the flash to which it needs to communicate. Stacked mode is a software abstraction rather than a controller feature or capability. At any given time, the controller communicates with one of the two connected flash devices, as determined by the requested address and data length. If an operation starts on one flash and ends on the other, the core needs to split it into two separate operations and adjust the data length accordingly. Parallel(Multi-CS): Both the flashes have their separate SPI bus, CS of both the flashes will be asserted/de-asserted at the same time. In this mode data will be split across both the flashes by enabling the STRIPE setting in the controller. Parallel mode is a controller feature where if the STRIPE bit is set then the controller internally handles the data split during data write to the flashes and while reading data from the flash the controller internally merges data from both the flashes before writing to the controller FIFO. If STRIPE is not enabled, then same data will be sent to both the devices. In parallel mode both the flashes should be identical. For more information on the modes please feel free to go through the controller flash interface below [1]. Mirochip QSPI controller[2] also supports "Dual Parallel 8-bit IO mode", but they call it "Twin Quad Mode". Initially in [3] [4] [5] Miquel had tried to extend MTD-CONCAT driver to support Stacked mode, but the bindings were not accepted. So, the MTD-CONCAT approach was dropped and the DT bindings that got accepted [6] [7] [8] that describes the two flash devices as being one. SPI core changes to support the above bindings were added [9]. While adding the support in SPI-NOR Tudor provided additional feedback, leading to a discussion on updating the current stacked and parallel DT bindings. Proposed Solution: The solution has two parts: 1. Update MTD-CONCAT Update MTD-CONCAT to create virtual concatinated mtd devices as defined in the device tree. 2. Add a New Layer Add a new layer between the SPI-NOR and MTD layers to support stacked and parallel configurations. This new layer will be part of spi-nor, located in mtd/spi-nor/, can be included/excluded via Kconfig, will be maintained by AMD and will: - During probing, store information from all connected flashes in stacked or parallel mode and present them as a single device to the MTD layer. - Register callbacks and manage MTD device registration within the new layer instead of spi-nor/core.c. - Make minimal changes in spi-nor/core.c, as stacked and parallel handling will be managed by the new layer on top of SPI-NOR. - Handle odd byte count requests from the MTD layer during flash operations in parallel mode. [1] https://docs.amd.com/r/en-US/am011-versal-acap-trm/QSPI-Flash-Device-Interf… [2] https://ww1.microchip.com/downloads/aemDocuments/documents/MPU32/ProductDoc… [3] https://lore.kernel.org/all/20191113171505.26128-4-miquel.raynal@bootlin.co… [4] https://lore.kernel.org/all/20191127105522.31445-5-miquel.raynal@bootlin.co… [5]https://lore.kernel.org/all/20211112152411.818321-1-miquel.raynal@bootlin… [6] https://github.com/torvalds/linux/commit/f89504300e94524d5d5846ff8b728592ac… [7] https://github.com/torvalds/linux/commit/eba5368503b4291db7819512600fa014ea… [8] https://github.com/torvalds/linux/commit/e2edd1b64f1c79e8abda365149ed62a2a9… [9]https://github.com/torvalds/linux/commit/4d8ff6b0991d5e86b17b235fc46ec62e… Thanks, Amit Amit Kumar Mahapatra (2): dt-bindings: mtd: Add bindings for describing concatinated MTD devices dt-bindings: spi: Update stacked and parallel bindings .../mtd/partitions/fixed-partitions.yaml | 18 +++++++++++++++ .../bindings/mtd/partitions/partitions.yaml | 6 +++++ .../bindings/spi/spi-controller.yaml | 23 +++++++++++++++++-- .../bindings/spi/spi-peripheral-props.yaml | 9 +++----- 4 files changed, 48 insertions(+), 8 deletions(-) -- 2.34.1

6 15

ABI break in 1.2.12 -> 1.2.13
by GitHub issues - edited 25 Nov '24

25 Nov '24

alsa-project/alsa-lib issue #422 was edited from bbhtt: This seems a bit unusual for a patch version bump. Any idea what this is about? Thanks! ``` ┌───────────────────────────┐ │ ABI Break: libasound.so.2 │ └───────────────────────────┘ Functions changes summary: 1 Removed, 3 Changed (250 filtered out), 35 Added functions Variables changes summary: 0 Removed, 0 Changed, 0 Added variable 1 Removed function: [D] 'function void snd_ump_block_info_set_block_id(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_block_id@@ALSA_1.2.10} 35 Added functions: [A] 'function const unsigned char* snd_pcm_hw_params_get_sync(const snd_pcm_hw_params_t*)' {snd_pcm_hw_params_get_sync@@ALSA_1.2.13} [A] 'function int snd_seq_create_ump_block(snd_seq_t*, int, const snd_ump_block_info_t*)' {snd_seq_create_ump_block@@ALSA_1.2.13} [A] 'function int snd_seq_create_ump_endpoint(snd_seq_t*, const snd_ump_endpoint_info_t*, unsigned int)' {snd_seq_create_ump_endpoint@@ALSA_1.2.13} [A] 'function int snd_seq_has_queue_tempo_base(snd_seq_t*)' {snd_seq_has_queue_tempo_base@@ALSA_0.9} [A] 'function int snd_seq_port_info_get_ump_is_midi1(const snd_seq_port_info_t*)' {snd_seq_port_info_get_ump_is_midi1@@ALSA_1.2.13} [A] 'function void snd_seq_port_info_set_ump_is_midi1(snd_seq_port_info_t*, int)' {snd_seq_port_info_set_ump_is_midi1@@ALSA_1.2.13} [A] 'function unsigned int snd_seq_queue_tempo_get_tempo_base(const snd_seq_queue_tempo_t*)' {snd_seq_queue_tempo_get_tempo_base@@ALSA_1.2.13} [A] 'function void snd_seq_queue_tempo_set_tempo_base(snd_seq_queue_tempo_t*, unsigned int)' {snd_seq_queue_tempo_set_tempo_base@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_clear(snd_ump_block_info_t*)' {snd_ump_block_info_clear@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_active(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_active@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_card(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_card@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_device(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_device@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_direction(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_direction@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_first_group(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_first_group@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_flags(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_flags@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_midi_ci_version(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_midi_ci_version@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_name(snd_ump_block_info_t*, const char*)' {snd_ump_block_info_set_name@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_num_groups(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_num_groups@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_sysex8_streams(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_sysex8_streams@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_ui_hint(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_ui_hint@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_clear(snd_ump_endpoint_info_t*)' {snd_ump_endpoint_info_clear@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_card(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_card@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_device(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_device@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_family_id(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_family_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_flags(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_flags@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_manufacturer_id(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_manufacturer_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_model_id(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_model_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_name(snd_ump_endpoint_info_t*, const char*)' {snd_ump_endpoint_info_set_name@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_num_blocks(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_num_blocks@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_product_id(snd_ump_endpoint_info_t*, const char*)' {snd_ump_endpoint_info_set_product_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_protocol(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_protocol@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_protocol_caps(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_protocol_caps@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_sw_revision(snd_ump_endpoint_info_t*, const unsigned char*)' {snd_ump_endpoint_info_set_sw_revision@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_version(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_version@@ALSA_1.2.13} [A] 'function int snd_ump_packet_length(unsigned int)' {snd_ump_packet_length@@ALSA_1.2.13} 3 functions with some indirect sub-type change: [C] 'function int __old_snd_pcm_hw_params_get_access(const snd_pcm_hw_params_t*)' at pcm.c:8062:1 has some indirect sub-type changes: Please note that the exported symbol of this function is snd_pcm_hw_params_get_access(a)ALSA_0.9 parameter 1 of type 'const snd_pcm_hw_params_t*' has sub-type changes: in pointed to type 'const snd_pcm_hw_params_t': in unqualified underlying type 'typedef snd_pcm_hw_params_t' at pcm.h:64:1: underlying type 'struct snd_pcm_hw_params' at asound.h:414:1 changed: type size hasn't changed 1 data member insertion: 'unsigned char sync[16]', at offset 4352 (in bits) at asound.h:421:1 1 data member change: type of 'unsigned char reserved[64]' changed: type name changed from 'unsigned char[64]' to 'unsigned char[48]' array type size changed from 512 to 384 array type subrange 1 changed length from 64 to 48 array subrange changed: upper bound of '<anonymous range>[48]' change from '63' to '47' and offset changed from 4352 to 4480 (in bits) (by +128 bits) [C] 'function int snd_ctl_pcm_info(snd_ctl_t*, snd_pcm_info_t*)' at control.c:1219:1 has some indirect sub-type changes: parameter 2 of type 'snd_pcm_info_t*' has sub-type changes: in pointed to type 'typedef snd_pcm_info_t' at pcm.h:48:1: underlying type 'struct snd_pcm_info' at asound.h:345:1 changed: type size hasn't changed 1 data member change: type of 'snd_pcm_sync_id sync' changed: entity changed from 'union snd_pcm_sync_id' to 'unsigned char[16]' and name of 'snd_pcm_info::sync' changed to 'snd_pcm_info::pad1' at asound.h:349:1 [C] 'function int snd_seq_get_queue_tempo(snd_seq_t*, int, snd_seq_queue_tempo_t*)' at seq.c:3880:1 has some indirect sub-type changes: parameter 3 of type 'snd_seq_queue_tempo_t*' has sub-type changes: in pointed to type 'typedef snd_seq_queue_tempo_t' at seq.h:441:1: underlying type 'struct snd_seq_queue_tempo' at asequencer.h:540:1 changed: type size hasn't changed 1 data member insertion: 'unsigned short int tempo_base', at offset 160 (in bits) at asequencer.h:532:1 1 data member change: type of 'char reserved[24]' changed: type name changed from 'char[24]' to 'char[22]' array type size changed from 192 to 176 array type subrange 1 changed length from 24 to 22 array subrange changed: upper bound of '<anonymous range>[22]' change from '23' to '21' and offset changed from 160 to 176 (in bits) (by +16 bits) ``` Issue URL : https://github.com/alsa-project/alsa-lib/issues/422 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

ABI break in 1.2.12 -> 1.2.13
by GitHub issues - opened 25 Nov '24

25 Nov '24

alsa-project/alsa-lib issue #422 was opened from bbhtt: This seems a bit unusual for a patch version bump. ``` ┌───────────────────────────┐ │ ABI Break: libasound.so.2 │ └───────────────────────────┘ Functions changes summary: 1 Removed, 3 Changed (250 filtered out), 35 Added functions Variables changes summary: 0 Removed, 0 Changed, 0 Added variable 1 Removed function: [D] 'function void snd_ump_block_info_set_block_id(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_block_id@@ALSA_1.2.10} 35 Added functions: [A] 'function const unsigned char* snd_pcm_hw_params_get_sync(const snd_pcm_hw_params_t*)' {snd_pcm_hw_params_get_sync@@ALSA_1.2.13} [A] 'function int snd_seq_create_ump_block(snd_seq_t*, int, const snd_ump_block_info_t*)' {snd_seq_create_ump_block@@ALSA_1.2.13} [A] 'function int snd_seq_create_ump_endpoint(snd_seq_t*, const snd_ump_endpoint_info_t*, unsigned int)' {snd_seq_create_ump_endpoint@@ALSA_1.2.13} [A] 'function int snd_seq_has_queue_tempo_base(snd_seq_t*)' {snd_seq_has_queue_tempo_base@@ALSA_0.9} [A] 'function int snd_seq_port_info_get_ump_is_midi1(const snd_seq_port_info_t*)' {snd_seq_port_info_get_ump_is_midi1@@ALSA_1.2.13} [A] 'function void snd_seq_port_info_set_ump_is_midi1(snd_seq_port_info_t*, int)' {snd_seq_port_info_set_ump_is_midi1@@ALSA_1.2.13} [A] 'function unsigned int snd_seq_queue_tempo_get_tempo_base(const snd_seq_queue_tempo_t*)' {snd_seq_queue_tempo_get_tempo_base@@ALSA_1.2.13} [A] 'function void snd_seq_queue_tempo_set_tempo_base(snd_seq_queue_tempo_t*, unsigned int)' {snd_seq_queue_tempo_set_tempo_base@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_clear(snd_ump_block_info_t*)' {snd_ump_block_info_clear@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_active(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_active@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_card(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_card@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_device(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_device@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_direction(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_direction@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_first_group(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_first_group@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_flags(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_flags@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_midi_ci_version(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_midi_ci_version@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_name(snd_ump_block_info_t*, const char*)' {snd_ump_block_info_set_name@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_num_groups(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_num_groups@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_sysex8_streams(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_sysex8_streams@@ALSA_1.2.13} [A] 'function void snd_ump_block_info_set_ui_hint(snd_ump_block_info_t*, unsigned int)' {snd_ump_block_info_set_ui_hint@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_clear(snd_ump_endpoint_info_t*)' {snd_ump_endpoint_info_clear@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_card(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_card@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_device(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_device@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_family_id(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_family_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_flags(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_flags@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_manufacturer_id(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_manufacturer_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_model_id(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_model_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_name(snd_ump_endpoint_info_t*, const char*)' {snd_ump_endpoint_info_set_name@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_num_blocks(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_num_blocks@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_product_id(snd_ump_endpoint_info_t*, const char*)' {snd_ump_endpoint_info_set_product_id@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_protocol(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_protocol@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_protocol_caps(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_protocol_caps@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_sw_revision(snd_ump_endpoint_info_t*, const unsigned char*)' {snd_ump_endpoint_info_set_sw_revision@@ALSA_1.2.13} [A] 'function void snd_ump_endpoint_info_set_version(snd_ump_endpoint_info_t*, unsigned int)' {snd_ump_endpoint_info_set_version@@ALSA_1.2.13} [A] 'function int snd_ump_packet_length(unsigned int)' {snd_ump_packet_length@@ALSA_1.2.13} 3 functions with some indirect sub-type change: [C] 'function int __old_snd_pcm_hw_params_get_access(const snd_pcm_hw_params_t*)' at pcm.c:8062:1 has some indirect sub-type changes: Please note that the exported symbol of this function is snd_pcm_hw_params_get_access(a)ALSA_0.9 parameter 1 of type 'const snd_pcm_hw_params_t*' has sub-type changes: in pointed to type 'const snd_pcm_hw_params_t': in unqualified underlying type 'typedef snd_pcm_hw_params_t' at pcm.h:64:1: underlying type 'struct snd_pcm_hw_params' at asound.h:414:1 changed: type size hasn't changed 1 data member insertion: 'unsigned char sync[16]', at offset 4352 (in bits) at asound.h:421:1 1 data member change: type of 'unsigned char reserved[64]' changed: type name changed from 'unsigned char[64]' to 'unsigned char[48]' array type size changed from 512 to 384 array type subrange 1 changed length from 64 to 48 array subrange changed: upper bound of '<anonymous range>[48]' change from '63' to '47' and offset changed from 4352 to 4480 (in bits) (by +128 bits) [C] 'function int snd_ctl_pcm_info(snd_ctl_t*, snd_pcm_info_t*)' at control.c:1219:1 has some indirect sub-type changes: parameter 2 of type 'snd_pcm_info_t*' has sub-type changes: in pointed to type 'typedef snd_pcm_info_t' at pcm.h:48:1: underlying type 'struct snd_pcm_info' at asound.h:345:1 changed: type size hasn't changed 1 data member change: type of 'snd_pcm_sync_id sync' changed: entity changed from 'union snd_pcm_sync_id' to 'unsigned char[16]' and name of 'snd_pcm_info::sync' changed to 'snd_pcm_info::pad1' at asound.h:349:1 [C] 'function int snd_seq_get_queue_tempo(snd_seq_t*, int, snd_seq_queue_tempo_t*)' at seq.c:3880:1 has some indirect sub-type changes: parameter 3 of type 'snd_seq_queue_tempo_t*' has sub-type changes: in pointed to type 'typedef snd_seq_queue_tempo_t' at seq.h:441:1: underlying type 'struct snd_seq_queue_tempo' at asequencer.h:540:1 changed: type size hasn't changed 1 data member insertion: 'unsigned short int tempo_base', at offset 160 (in bits) at asequencer.h:532:1 1 data member change: type of 'char reserved[24]' changed: type name changed from 'char[24]' to 'char[22]' array type size changed from 192 to 176 array type subrange 1 changed length from 24 to 22 array subrange changed: upper bound of '<anonymous range>[22]' change from '23' to '21' and offset changed from 160 to 176 (in bits) (by +16 bits) ``` Issue URL : https://github.com/alsa-project/alsa-lib/issues/422 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

1.2.13 will lead to mute of speakers and headphone jack on MTL laptop Lenovo Youga Pro 7 14IMH9
by GitHub issues - edited 25 Nov '24

25 Nov '24

alsa-project/alsa-ucm-conf issue #472 was edited from phush0: Changing version from 1.2.12 to 1.2.13 will lead to not working sound during plug/unplug of the head phone jack or mute during reboot on Arch install. Other ppl with similar laptops have reported same problem. On my version codec is ALC287 with bass speaker output. I can unmute speaker or headphones trough alsamixer, but plugin headphone and the unplugin will lead to both mute. Sometimes reboot will lead to same problem and I have to operate channels by hand again. Similar problems: [https://discuss.cachyos.org/t/sound-problems-kde/4350](https://discuss.cach… edit: reading more seems like problem is with the new HDMI macro, because other report problems with HDMI output. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/472 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

1.2.13 will lead to mute of speakers and headphone jack on MTL laptop Lenovo Youga Pro 7 14IMH9
by GitHub issues - edited 25 Nov '24

25 Nov '24

alsa-project/alsa-ucm-conf issue #472 was edited from phush0: Changing version from 1.2.12 to 1.2.13 will lead to not working sound during plug/unplug of the head phone jack or mute during reboot on Arch install. Other ppl with similar laptops have reported same problem. On my version codec is ALC287 with bass speaker output. I can unmute speaker or headphones trough alsamixer, but plugin headphone and the unplugin will lead to both mute. Sometimes reboot will lead to same problem and I have to operate channels by hand again. Similar problems: [https://discuss.cachyos.org/t/sound-problems-kde/4350](https://discuss.cach… Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/472 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

1.2.13 will lead to mute of speakers and head phone jack on MTL laptop Lenovo Youga Pro 7 14IMH9
by GitHub issues - opened 25 Nov '24

25 Nov '24

alsa-project/alsa-ucm-conf issue #472 was opened from phush0: Changing version from 1.2.12 to 1.2.13 will lead to not working sound during plug/unplug of the head phone jack or mute during reboot on Arch install. Other ppl with similar laptops have reported same problem. On my version codec is ALC287 with bass speaker output. I can unmute speaker or headphones trough alsamixer, but plugin headphone and the unplugin will lead to both mute. Sometimes reboot will lead to same problem and I have to operate channels by hand again. Similar problems: [https://discuss.cachyos.org/t/sound-problems-kde/4350](https://discuss.cach… Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/472 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[BUG] HDA Intel PCH, HDMI , HDA Intel PCH, ALC886 Digital, no IEC958 / HDMI output/passthrough to AVR, partly fixed but fix broke CM106
by schorpp 25 Nov '24

25 Nov '24

>> >> [BUG] HDA Intel PCH, HDMI , HDA Intel PCH, ALC886 Digital, no IEC958 / HDMI output/passthrough to AVR >> >> ALSA versions 1.0.28 >> >> Kernel Driver versions 3.12.40 Partly FIXED in Longterm Stable Kernel 4.19.324 and using HDMI DEV 1 instead of 0. 2 CH PCM HDMI to AVR works now but AC3/DTS passthrough still not or is not recognized by AVR or some wrong AES setting but iec958 optical out AC3/DTS output to AVR works. New BUG: No more 5.1 Analog Output on the CM106 USB Box if AC3 stream is selected. Your ALSA information is located at http://alsa-project.org/db/?f=5ef8b56450583ba8afbcc705afc02228da4a3072 y tom >> >> Hardware: >> >> DFI Mainboard MB330-CRM >> >> https://www.dfi.com/de/product/index/273#specification >> >> Reproducible Always >> >> Steps to reproduce and full report: >> >> https://www-vdr--portal-de.translate.goog/forum/index.php?thread/136519-gel… > Output of alsa-info.sh is here > > http://alsa-project.org/db/?f=fc4734deddd9a92331692bf94b15cf84b7722cd4 > >> >> Software crosscheck with CM106 USB Soundbox iec958: works OK. >> >> Where to start in the drivers source code to fix it? >> >> y >> tom >> >>

1 0

Motu D828 (2024) not working again
by GitHub issues - opened 25 Nov '24

25 Nov '24

alsa-project/alsa-ucm-conf issue #471 was opened from beatboxa: I had previously gotten my MOTU D828 working with UCM. I recently used Ubuntu 24.04's software updater, and now my UCM appears to have stopped working. The UCM files are still present in the directories where they previously worked. The output of spa-acp-tool -c 0 -vvvv info now begins with the following: ``` UCM is not supported for this USB device (MOTU 828 at usb-0000:59:00.0-6, high speed @ USB07fd:000e) error: failed to import hw:0 use case configuration -6 UCM is not supported for this USB device (MOTU 828 at usb-0000:59:00.0-6, high speed @ USB07fd:000e) error: failed to import 828 use case configuration -6 UCM not available for card 828 Check for file: /home/beatboxa/.config/alsa-card-profile/mixer/profile-sets/default.conf Check for file: /home/beatboxa/.config/alsa-card-profile/profile-sets/default.conf Check for file: /etc/alsa-card-profile/mixer/profile-sets/default.conf Check for file: /etc/alsa-card-profile/profile-sets/default.conf Loading profile set: /usr/share/alsa-card-profile/mixer/profile-sets/default.conf Parsing configuration file '/usr/share/alsa-card-profile/mixer/profile-sets/default.conf' Parsing configuration file '/usr/share/alsa-card-profile/mixer/profile-sets/9999-custom.conf' Looking at profile input:analog-stereo Checking for recording on Analog Stereo (analog-stereo) Trying front:0 with SND_PCM_NO_AUTO_FORMAT ... ALSA device open 'front:0' capture: 0x5804b63f3d90 snd_pcm_hw_params_set_format(Signed 16 bit Little Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Signed 16 bit Big Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Float 32 bit Little Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Float 32 bit Big Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Signed 32 bit Little Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Signed 32 bit Big Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Signed 24 bit Little Endian) failed: Invalid argument snd_pcm_hw_params_set_format(Signed 24 bit Big Endian) failed: Invalid argument snd_pcm_hw_params_set_channels(2) failed: Invalid argument ALSA device close 0x5804b63f3d90 ``` And goes on from there. Previously, my configuration worked, and is tracked in this thread: #https://github.com/alsa-project/alsa-ucm-conf/pull/416 Is there some little thing I need to change to get UCM working again? Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/471 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

No sound and alsactl init errors after updating to 1.2.13
by GitHub issues - edited 24 Nov '24

24 Nov '24

alsa-project/alsa-utils issue #283 was edited from wastlnd: I think I have kind of a regression after updating to the latest version. Running alsactl restore as a runit service, I have this error at startup: ``` Found hardware: "acp" "" "" "" "" Hardware is initialized using a generic method ``` And I have no sound at all. I have tried to apply the latest commit fixing the udev rule, but it didn't work. Downgrading to 1.2.12 fixes the issue. I am running an Arch based distro (kernel 6.6) on AMD Ryzen 7 5825U CPU with Radeon graphics. My soundcard is Realtek and I have an AMD microphone array. I don't run Pulseaudio or Pipewire, just plain ALSA. Issue URL : https://github.com/alsa-project/alsa-utils/issues/283 Repository URL: https://github.com/alsa-project/alsa-utils

1 0

No sound and alsactl init errors after updating to 1.2.13
by GitHub issues - edited 24 Nov '24

24 Nov '24

alsa-project/alsa-utils issue #283 was edited from wastlnd: I think I have kind of a regression after updating to the latest version. I have this error at startup: ``` Found hardware: "acp" "" "" "" "" Hardware is initialized using a generic method ``` And I have no sound at all. I have tried to apply the latest commit fixing the udev rule, but it didn't work. Downgrading to 1.2.12 fixes the issue. I am running an Arch based distro (kernel 6.6) on AMD Ryzen 7 5825U CPU with Radeon graphics. My soundcard is Realtek and I have an AMD microphone array. I don't run Pulseaudio or Pipewire, just plain ALSA. Issue URL : https://github.com/alsa-project/alsa-utils/issues/283 Repository URL: https://github.com/alsa-project/alsa-utils

1 0