July 2022 - Alsa-devel - mailman.alsa-project.org

[PATCH v1 1/1] ASoC: Intel: catpt: remove duplicating driver data retrieval
by Andy Shevchenko 05 Jul '22

05 Jul '22

device_get_match_data() in ACPI case calls similar to acpi_match_device() API. Hence there is no need to duplicate the call. Just check what the former returns. Signed-off-by: Andy Shevchenko <andriy.shevchenko(a)linux.intel.com> --- sound/soc/intel/catpt/device.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/sound/soc/intel/catpt/device.c b/sound/soc/intel/catpt/device.c index 85a34e37316d..21856a394c3d 100644 --- a/sound/soc/intel/catpt/device.c +++ b/sound/soc/intel/catpt/device.c @@ -12,13 +12,14 @@ // helping backtrack its historical background // -#include <linux/acpi.h> #include <linux/dma-mapping.h> #include <linux/interrupt.h> +#include <linux/mod_devicetable.h> #include <linux/module.h> #include <linux/pci.h> #include <linux/platform_device.h> #include <linux/pm_runtime.h> +#include <linux/property.h> #include <sound/intel-dsp-config.h> #include <sound/soc.h> #include <sound/soc-acpi.h> @@ -244,8 +245,8 @@ static int catpt_acpi_probe(struct platform_device *pdev) struct resource *res; int ret; - id = acpi_match_device(dev->driver->acpi_match_table, dev); - if (!id) + spec = device_get_match_data(dev); + if (!spec) return -ENODEV; ret = snd_intel_acpi_dsp_driver_probe(dev, id->id); @@ -254,10 +255,6 @@ static int catpt_acpi_probe(struct platform_device *pdev) return -ENODEV; } - spec = device_get_match_data(dev); - if (!spec) - return -ENODEV; - cdev = devm_kzalloc(dev, sizeof(*cdev), GFP_KERNEL); if (!cdev) return -ENOMEM; -- 2.35.1

2 2

[PATCH v2] ASoC: dt-bindings: qcom,sm8250: add SDM845 sound
by Krzysztof Kozlowski 05 Jul '22

05 Jul '22

The Qualcomm SDM845 sound card bindings are almost the same as SM8250, except "pin-switches" and "widgets" properties. These were not documented in SDM845 text bindings but are actually valid for SDM845. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> --- Changes since v1: 1. Integrate into SM8250 instead of creating new file (Stephan). Cc: Stephan Gerhold <stephan(a)gerhold.net> --- .../devicetree/bindings/sound/qcom,sdm845.txt | 91 ------------------- .../bindings/sound/qcom,sm8250.yaml | 1 + 2 files changed, 1 insertion(+), 91 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/qcom,sdm845.txt diff --git a/Documentation/devicetree/bindings/sound/qcom,sdm845.txt b/Documentation/devicetree/bindings/sound/qcom,sdm845.txt deleted file mode 100644 index de4c604641da..000000000000 --- a/Documentation/devicetree/bindings/sound/qcom,sdm845.txt +++ /dev/null @@ -1,91 +0,0 @@ -* Qualcomm Technologies Inc. SDM845 ASoC sound card driver - -This binding describes the SDM845 sound card, which uses qdsp for audio. - -- compatible: - Usage: required - Value type: <stringlist> - Definition: must be one of this - "qcom,sdm845-sndcard" - "qcom,db845c-sndcard" - "lenovo,yoga-c630-sndcard" - -- audio-routing: - Usage: Optional - Value type: <stringlist> - Definition: A list of the connections between audio components. - Each entry is a pair of strings, the first being the - connection's sink, the second being the connection's - source. Valid names could be power supplies, MicBias - of codec and the jacks on the board. - -- model: - Usage: required - Value type: <stringlist> - Definition: The user-visible name of this sound card. - -- aux-devs - Usage: optional - Value type: <array of phandles> - Definition: A list of phandles for auxiliary devices (e.g. analog - amplifiers) that do not appear directly within the DAI - links. Should be connected to another audio component - using "audio-routing". - -= dailinks -Each subnode of sndcard represents either a dailink, and subnodes of each -dailinks would be cpu/codec/platform dais. - -- link-name: - Usage: required - Value type: <string> - Definition: User friendly name for dai link - -= CPU, PLATFORM, CODEC dais subnodes -- cpu: - Usage: required - Value type: <subnode> - Definition: cpu dai sub-node - -- codec: - Usage: required - Value type: <subnode> - Definition: codec dai sub-node - -- platform: - Usage: Optional - Value type: <subnode> - Definition: platform dai sub-node - -- sound-dai: - Usage: required - Value type: <phandle> - Definition: dai phandle/s and port of CPU/CODEC/PLATFORM node. - -Example: - -audio { - compatible = "qcom,sdm845-sndcard"; - model = "sdm845-snd-card"; - pinctrl-names = "default", "sleep"; - pinctrl-0 = <&pri_mi2s_active &pri_mi2s_ws_active>; - pinctrl-1 = <&pri_mi2s_sleep &pri_mi2s_ws_sleep>; - - mm1-dai-link { - link-name = "MultiMedia1"; - cpu { - sound-dai = <&q6asmdai MSM_FRONTEND_DAI_MULTIMEDIA1>; - }; - }; - - pri-mi2s-dai-link { - link-name = "PRI MI2S Playback"; - cpu { - sound-dai = <&q6afedai PRIMARY_MI2S_RX>; - }; - - platform { - sound-dai = <&q6routing>; - }; - }; -}; diff --git a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml index 4ecd4080bb96..7cdf9e9f4dd3 100644 --- a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml +++ b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml @@ -18,6 +18,7 @@ properties: enum: - qcom,apq8016-sbc-sndcard - qcom,msm8916-qdsp6-sndcard + - qcom,sdm845-sndcard - qcom,sm8250-sndcard - qcom,qrb5165-rb5-sndcard -- 2.34.1

2 2

[PATCH] ASoC: dt-bindings: qcom,sdm845: convert to dtschema
by Krzysztof Kozlowski 05 Jul '22

05 Jul '22

Convert the Samsung SDM845 sound card bindings to DT schema. Changes during conversion: do not require 'codec' under dai-links - not present in all nodes of examples and DTS; not required by the driver. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> --- .../devicetree/bindings/sound/qcom,sdm845.txt | 91 ---------- .../bindings/sound/qcom,sdm845.yaml | 166 ++++++++++++++++++ 2 files changed, 166 insertions(+), 91 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/qcom,sdm845.txt create mode 100644 Documentation/devicetree/bindings/sound/qcom,sdm845.yaml diff --git a/Documentation/devicetree/bindings/sound/qcom,sdm845.txt b/Documentation/devicetree/bindings/sound/qcom,sdm845.txt deleted file mode 100644 index de4c604641da..000000000000 --- a/Documentation/devicetree/bindings/sound/qcom,sdm845.txt +++ /dev/null @@ -1,91 +0,0 @@ -* Qualcomm Technologies Inc. SDM845 ASoC sound card driver - -This binding describes the SDM845 sound card, which uses qdsp for audio. - -- compatible: - Usage: required - Value type: <stringlist> - Definition: must be one of this - "qcom,sdm845-sndcard" - "qcom,db845c-sndcard" - "lenovo,yoga-c630-sndcard" - -- audio-routing: - Usage: Optional - Value type: <stringlist> - Definition: A list of the connections between audio components. - Each entry is a pair of strings, the first being the - connection's sink, the second being the connection's - source. Valid names could be power supplies, MicBias - of codec and the jacks on the board. - -- model: - Usage: required - Value type: <stringlist> - Definition: The user-visible name of this sound card. - -- aux-devs - Usage: optional - Value type: <array of phandles> - Definition: A list of phandles for auxiliary devices (e.g. analog - amplifiers) that do not appear directly within the DAI - links. Should be connected to another audio component - using "audio-routing". - -= dailinks -Each subnode of sndcard represents either a dailink, and subnodes of each -dailinks would be cpu/codec/platform dais. - -- link-name: - Usage: required - Value type: <string> - Definition: User friendly name for dai link - -= CPU, PLATFORM, CODEC dais subnodes -- cpu: - Usage: required - Value type: <subnode> - Definition: cpu dai sub-node - -- codec: - Usage: required - Value type: <subnode> - Definition: codec dai sub-node - -- platform: - Usage: Optional - Value type: <subnode> - Definition: platform dai sub-node - -- sound-dai: - Usage: required - Value type: <phandle> - Definition: dai phandle/s and port of CPU/CODEC/PLATFORM node. - -Example: - -audio { - compatible = "qcom,sdm845-sndcard"; - model = "sdm845-snd-card"; - pinctrl-names = "default", "sleep"; - pinctrl-0 = <&pri_mi2s_active &pri_mi2s_ws_active>; - pinctrl-1 = <&pri_mi2s_sleep &pri_mi2s_ws_sleep>; - - mm1-dai-link { - link-name = "MultiMedia1"; - cpu { - sound-dai = <&q6asmdai MSM_FRONTEND_DAI_MULTIMEDIA1>; - }; - }; - - pri-mi2s-dai-link { - link-name = "PRI MI2S Playback"; - cpu { - sound-dai = <&q6afedai PRIMARY_MI2S_RX>; - }; - - platform { - sound-dai = <&q6routing>; - }; - }; -}; diff --git a/Documentation/devicetree/bindings/sound/qcom,sdm845.yaml b/Documentation/devicetree/bindings/sound/qcom,sdm845.yaml new file mode 100644 index 000000000000..5e7e6b051cf0 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/qcom,sdm845.yaml @@ -0,0 +1,166 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/qcom,sdm845.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Qualcomm Technologies Inc. SDM845 ASoC sound card + +maintainers: + - Srinivas Kandagatla <srinivas.kandagatla(a)linaro.org> + +properties: + compatible: + enum: + - qcom,sdm845-sndcard + - qcom,db845c-sndcard + - lenovo,yoga-c630-sndcard + + audio-routing: + $ref: /schemas/types.yaml#/definitions/non-unique-string-array + description: + A list of the connections between audio components. Each entry is a pair + of strings, the first being the connection's sink, the second being the + connection's source. Valid names could be power supplies, MicBias of + codec and the jacks on the board. + + aux-devs: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: | + List of phandles for auxiliary devices (e.g. analog amplifiers) that do + not appear directly within the DAI links. Should be connected to another + audio component using "audio-routing". + + model: + $ref: /schemas/types.yaml#/definitions/string + description: User-visible sound card name + +patternProperties: + ".*-dai-link$": + type: object + description: + Each subnode represents a dai link. Subnodes of each dai links would be + cpu/codec dais. + + properties: + link-name: + $ref: /schemas/types.yaml#/definitions/string + description: User friendly name for dai link. + + cpu: + type: object + description: CPU DAI subnode. + properties: + sound-dai: + maxItems: 1 + + codec: + type: object + description: Codec DAI subnode. + properties: + sound-dai: + minItems: 1 + maxItems: 4 + + platform: + type: object + description: Platform DAI subnode. + properties: + sound-dai: + maxItems: 1 + + required: + - link-name + - cpu + + additionalProperties: false + +required: + - compatible + - model + +additionalProperties: false + +examples: + - | + #include <dt-bindings/sound/qcom,q6afe.h> + #include <dt-bindings/sound/qcom,q6asm.h> + + sound { + compatible = "qcom,sdm845-sndcard"; + model = "Samsung-W737"; + + audio-routing = "RX_BIAS", "MCLK", + "AMIC2", "MIC BIAS2", + "SpkrLeft IN", "SPK1 OUT", + "SpkrRight IN", "SPK2 OUT", + "MM_DL1", "MultiMedia1 Playback", + "MM_DL3", "MultiMedia3 Playback", + "MultiMedia2 Capture", "MM_UL2"; + + mm1-dai-link { + link-name = "MultiMedia1"; + cpu { + sound-dai = <&q6asmdai MSM_FRONTEND_DAI_MULTIMEDIA1>; + }; + }; + + mm2-dai-link { + link-name = "MultiMedia2"; + cpu { + sound-dai = <&q6asmdai MSM_FRONTEND_DAI_MULTIMEDIA2>; + }; + }; + + mm3-dai-link { + link-name = "MultiMedia3"; + cpu { + sound-dai = <&q6asmdai MSM_FRONTEND_DAI_MULTIMEDIA3>; + }; + }; + + slim-dai-link { + link-name = "SLIM Playback"; + cpu { + sound-dai = <&q6afedai SLIMBUS_0_RX>; + }; + + platform { + sound-dai = <&q6routing>; + }; + + codec { + sound-dai = <&left_spkr>, <&right_spkr>, <&swm 0>, <&wcd9340 0>; + }; + }; + + slimcap-dai-link { + link-name = "SLIM Capture"; + cpu { + sound-dai = <&q6afedai SLIMBUS_0_TX>; + }; + + platform { + sound-dai = <&q6routing>; + }; + + codec { + sound-dai = <&wcd9340 1>; + }; + }; + + slim-wcd-dai-link { + link-name = "SLIM WCD Playback"; + cpu { + sound-dai = <&q6afedai SLIMBUS_1_RX>; + }; + + platform { + sound-dai = <&q6routing>; + }; + + codec { + sound-dai = <&wcd9340 2>; + }; + }; + }; -- 2.34.1

2 3

equalizer cfg in max98088 driver
by Tommaso Merciai 05 Jul '22

05 Jul '22

Hi Mark, All, Just a question. Can you explain me the proper way to configure eq1 and eq2 of max98089 using sound/soc/codecs/max98088.c driver? Could be a valid solution fetching eq1, eq2 bands params from dts? Let me know. Thanks in advance Tommaso -- Tommaso Merciai Embedded Linux Engineer tommaso.merciai(a)amarulasolutions.com __________________________________ Amarula Solutions SRL Via Le Canevare 30, 31100 Treviso, Veneto, IT T. +39 042 243 5310 info(a)amarulasolutions.com www.amarulasolutions.com

2 3

[PATCH v1 1/1] ASoC: SOF: Intel: bdw: remove duplicating driver data retrieval
by Andy Shevchenko 05 Jul '22

05 Jul '22

device_get_match_data() in ACPI case calls similar to acpi_match_device(). Hence there is no need to duplicate the call. Just assign what is in the id->driver_data. Signed-off-by: Andy Shevchenko <andriy.shevchenko(a)linux.intel.com> --- sound/soc/sof/intel/bdw.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/sound/soc/sof/intel/bdw.c b/sound/soc/sof/intel/bdw.c index 26df780c702e..346813aa3768 100644 --- a/sound/soc/sof/intel/bdw.c +++ b/sound/soc/sof/intel/bdw.c @@ -674,6 +674,7 @@ static int sof_broadwell_probe(struct platform_device *pdev) id = acpi_match_device(dev->driver->acpi_match_table, dev); if (!id) return -ENODEV; + desc = (const struct sof_dev_desc *)id->driver_data; ret = snd_intel_acpi_dsp_driver_probe(dev, id->id); if (ret != SND_INTEL_DSP_DRIVER_ANY && ret != SND_INTEL_DSP_DRIVER_SOF) { @@ -681,11 +682,7 @@ static int sof_broadwell_probe(struct platform_device *pdev) return -ENODEV; } - desc = device_get_match_data(dev); - if (!desc) - return -ENODEV; - - return sof_acpi_probe(pdev, device_get_match_data(dev)); + return sof_acpi_probe(pdev, desc); } /* acpi_driver definition */ -- 2.35.1

4 6

[PATCH] ALSA: cs46xx: Fix missing snd_card_free() call at probe error
by Takashi Iwai 05 Jul '22

05 Jul '22

The previous cleanup with devres may lead to the incorrect release orders at the probe error handling due to the devres's nature. Until we register the card, snd_card_free() has to be called at first for releasing the stuff properly when the driver tries to manage and release the stuff via card->private_free(). This patch fixes it by calling snd_card_free() manually on the error from the probe callback. Fixes: 5bff69b3645d ("ALSA: cs46xx: Allocate resources with device-managed APIs") Cc: <stable(a)vger.kernel.org> Reported-and-tested-by: Jan Engelhardt <jengelh(a)inai.de> Link: https://lore.kernel.org/r/p2p1s96o-746-74p4-s95-61qo1p7782pn@vanv.qr Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/pci/cs46xx/cs46xx.c | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/sound/pci/cs46xx/cs46xx.c b/sound/pci/cs46xx/cs46xx.c index bd60308769ff..8634004a606b 100644 --- a/sound/pci/cs46xx/cs46xx.c +++ b/sound/pci/cs46xx/cs46xx.c @@ -74,36 +74,36 @@ static int snd_card_cs46xx_probe(struct pci_dev *pci, err = snd_cs46xx_create(card, pci, external_amp[dev], thinkpad[dev]); if (err < 0) - return err; + goto error; card->private_data = chip; chip->accept_valid = mmap_valid[dev]; err = snd_cs46xx_pcm(chip, 0); if (err < 0) - return err; + goto error; #ifdef CONFIG_SND_CS46XX_NEW_DSP err = snd_cs46xx_pcm_rear(chip, 1); if (err < 0) - return err; + goto error; err = snd_cs46xx_pcm_iec958(chip, 2); if (err < 0) - return err; + goto error; #endif err = snd_cs46xx_mixer(chip, 2); if (err < 0) - return err; + goto error; #ifdef CONFIG_SND_CS46XX_NEW_DSP if (chip->nr_ac97_codecs ==2) { err = snd_cs46xx_pcm_center_lfe(chip, 3); if (err < 0) - return err; + goto error; } #endif err = snd_cs46xx_midi(chip, 0); if (err < 0) - return err; + goto error; err = snd_cs46xx_start_dsp(chip); if (err < 0) - return err; + goto error; snd_cs46xx_gameport(chip); @@ -117,11 +117,15 @@ static int snd_card_cs46xx_probe(struct pci_dev *pci, err = snd_card_register(card); if (err < 0) - return err; + goto error; pci_set_drvdata(pci, card); dev++; return 0; + + error: + snd_card_free(card); + return err; } static struct pci_driver cs46xx_driver = { -- 2.35.3

1 0

snd_cs46xx regression, producing Oops
by Jan Engelhardt 05 Jul '22

05 Jul '22

Commit v5.14-rc1-39-g5bff69b3645d introduced a breakage into snd_cs46xx. When loading the module, an Oops is thrown. The system works fine afterwards, but the final reboot/poweroff message to ACPI has no effect after such Oops occurred. (Blacklisting snd_cs46xx works around the reboot problem.) bisected to: commit 5bff69b3645db7b3018ecbc26218d8866aeaf214 Author: Takashi Iwai <tiwai(a)suse.de> Date: Thu Jul 15 09:58:52 2021 +0200 ALSA: cs46xx: Allocate resources with device-managed APIs This patch converts the resource management in PCI cs46xx driver with devres as a clean up. Each manual resource management is converted with the corresponding devres helper, and the card object release is managed now via card->private_free instead of a lowlevel snd_device. trace log from modern kernel: [ 0.000000] Linux version 5.18.9 (jengelh@f3) (gcc (SUSE Linux) 12.1.0, GNU ld (GNU Binutils; openSUSE Tumbleweed) 2.38.20220525-6) #3 PREEMPT_DYNAMIC Tue Jul 5 10:43:37 CEST 2022 ... [ 32.475313] snd_intel8x0 0000:00:02.7: intel8x0_measure_ac97_clock: measured 59821 usecs (2877 samples) [ 32.475410] snd_intel8x0 0000:00:02.7: clocking to 48000 [ 33.683930] snd_cs46xx 0000:00:0d.0: Direct firmware load for cs46xx/cwc4630 failed with error -2 [ 33.684035] snd_cs46xx 0000:00:0d.0: firmware load error [cwc4630] [ 33.684092] snd_cs46xx: probe of 0000:00:0d.0 failed with error -2 [ 33.684772] BUG: unable to handle page fault for address: f7cd846c [ 33.684833] #PF: supervisor write access in kernel mode [ 33.684886] #PF: error_code(0x0002) - not-present page [ 33.684936] *pde = 010e9067 *pte = 00000000 [ 33.684992] Oops: 0002 [#1] PREEMPT [ 33.685045] CPU: 0 PID: 413 Comm: systemd-udevd Tainted: G E 5.18.9 #3 aa51fdb6e59156e58028f32c0abfef9c9b603d9b [ 33.685119] Hardware name: ECS L7S7A2/L7S7A2, BIOS 07.00T 04/02/01 [ 33.685172] EIP: snd_cs46xx_codec_write+0x3b/0xe0 [snd_cs46xx] [ 33.685265] Code: ec 18 83 7d 08 01 89 45 f0 0f 8f b0 00 00 00 89 f0 0f b7 f9 8b 8e 58 01 00 00 ba 01 00 00 00 e8 cb 83 af da 8b 46 28 8b 4d f0 <89> 88 6c 04 00 00 8b 46 28 89 b8 70 04 00 00 8b 46 28 05 60 04 00 [ 33.685356] EAX: f7cd8000 EBX: c232c548 ECX: 00000002 EDX: 00000001 [ 33.685410] ESI: c232c548 EDI: 00009f9f EBP: c1fd5b08 ESP: c1fd5ae4 [ 33.685463] DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 0068 EFLAGS: 00210297 [ 33.685528] CR0: 80050033 CR2: f7cd846c CR3: 0335d000 CR4: 00000690 [ 33.685581] Call Trace: [ 33.685637] ? snd_cs46xx_codec_write+0xe0/0xe0 [snd_cs46xx 58d1cdc18910e2717003e6be0e13f1fe4cf1ef4e] [ 33.685721] snd_cs46xx_ac97_write+0x2a/0x50 [snd_cs46xx 58d1cdc18910e2717003e6be0e13f1fe4cf1ef4e] [ 33.685804] snd_ac97_write+0x51/0x80 [snd_ac97_codec 9efa9e43a5e3a59d5b4ae0b33ee748e5f4f27a3a] [ 33.685908] snd_ac97_powerdown+0xc9/0x120 [snd_ac97_codec 9efa9e43a5e3a59d5b4ae0b33ee748e5f4f27a3a] [ 33.685990] snd_ac97_dev_free+0x13/0x30 [snd_ac97_codec 9efa9e43a5e3a59d5b4ae0b33ee748e5f4f27a3a] [ 33.686072] __snd_device_free+0x3d/0x80 [snd 6a4ed59569dfc8ba5a4beabb0b5a692067dfc833] [ 33.686178] snd_device_free_all+0x43/0x80 [snd 6a4ed59569dfc8ba5a4beabb0b5a692067dfc833] [ 33.686260] release_card_device+0x23/0x90 [snd 6a4ed59569dfc8ba5a4beabb0b5a692067dfc833] [ 33.686340] device_release+0x37/0xa0 [ 33.686399] kobject_put+0xb1/0x230 [ 33.686454] ? snd_card_disconnect+0x134/0x230 [snd 6a4ed59569dfc8ba5a4beabb0b5a692067dfc833] [ 33.686536] put_device+0x11/0x20 [ 33.686587] __snd_card_release+0x69/0x80 [snd 6a4ed59569dfc8ba5a4beabb0b5a692067dfc833] [ 33.686668] release_nodes+0x3d/0xb0 [ 33.686726] devres_release_all+0x77/0xd0 [ 33.686780] device_unbind_cleanup+0x10/0x60 [ 33.686833] really_probe+0x1ee/0x370 [ 33.686885] ? pm_runtime_barrier+0x44/0x90 [ 33.686945] __driver_probe_device+0x111/0x200 [ 33.686998] ? kernfs_create_dir_ns+0x79/0xc0 [ 33.687053] driver_probe_device+0x27/0xa0 [ 33.687106] __driver_attach+0xb7/0x1b0 [ 33.687158] ? __device_attach_driver+0x100/0x100 [ 33.687211] bus_for_each_dev+0x5a/0x90 [ 33.687263] driver_attach+0x1e/0x30 [ 33.687314] ? __device_attach_driver+0x100/0x100 [ 33.687366] bus_add_driver+0x14f/0x200 [ 33.687419] driver_register+0x7c/0xd0 [ 33.687469] ? sysfs_add_bin_file_mode_ns+0x67/0xd0 [ 33.687524] ? 0xf7f10000 [ 33.687574] __pci_register_driver+0x3c/0x40 [ 33.687628] cs46xx_driver_init+0x1c/0x1000 [snd_cs46xx 58d1cdc18910e2717003e6be0e13f1fe4cf1ef4e] [ 33.687710] do_one_initcall+0x3f/0x1a0 [ 33.687764] ? kmem_cache_alloc_trace+0x13f/0x2f0 [ 33.687822] ? do_init_module+0x24/0x240 [ 33.687882] do_init_module+0x46/0x240 [ 33.687935] ? __vfree+0x20/0x50 [ 33.687992] load_module+0x2393/0x2540 [ 33.691234] ? try_module_get+0xb0/0xb0 [ 33.691293] __ia32_sys_finit_module+0xb4/0x130 [ 33.691351] __do_fast_syscall_32+0x67/0xb0 [ 33.691411] ? syscall_exit_to_user_mode+0x1a/0x40 [ 33.691466] ? __do_fast_syscall_32+0x71/0xb0 [ 33.691519] ? irqentry_exit_to_user_mode+0x8/0x20 [ 33.691571] do_fast_syscall_32+0x31/0x70 [ 33.691624] do_SYSENTER_32+0x15/0x20 [ 33.691676] entry_SYSENTER_32+0x98/0xf0 [ 33.691732] EIP: 0xb7f27549 [ 33.691782] Code: 03 74 c0 01 10 05 03 74 b8 01 10 06 03 74 b4 01 10 07 03 74 b0 01 10 08 03 74 d8 01 00 00 00 00 00 51 52 55 89 e5 0f 34 cd 80 <5d> 5a 59 c3 90 90 90 90 8d 76 00 58 b8 77 00 00 00 cd 80 90 8d 76 [ 33.691873] EAX: ffffffda EBX: 00000012 ECX: b7bbe036 EDX: 00000000 [ 33.691926] ESI: b7bc5dd8 EDI: 01add640 EBP: ffffff08 ESP: bfee05ac [ 33.691979] DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 007b EFLAGS: 00200246 [ 33.692046] Modules linked in: drm_ttm_helper(E) snd_cs46xx(E+) ttm(E) snd_rawmidi(E) mxm_wmi(E) snd_intel8x0(E) wmi(E) pcspkr(E) snd_ac97_codec(E) video(E) ac97_bus(E) 8139too(E) snd_pcm(E) drm_dp_helper(E) 8139cp(E) sis900(E) snd_timer(E) parport_pc(E) mii(E) tiny_power_button(E) snd(E) soundcore(E) i2c_sis96x(E) button(E) parport(E) ext4(E) mbcache(E) jbd2(E) fuse(E) configfs(E) ip_tables(E) x_tables(E) xfs(E) libcrc32c(E) hid_generic(E) usbhid(E) sr_mod(E) cdrom(E) ata_generic(E) ohci_pci(E) serio_raw(E) sata_sil(E) ohci_hcd(E) ehci_pci(E) ehci_hcd(E) pata_sis(E) usbcore(E) libata(E) floppy(E) sg(E) [ 33.692314] CR2: 00000000f7cd846c [ 33.692365] ---[ end trace 0000000000000000 ]--- [ 33.692414] EIP: snd_cs46xx_codec_write+0x3b/0xe0 [snd_cs46xx] [ 33.692483] Code: ec 18 83 7d 08 01 89 45 f0 0f 8f b0 00 00 00 89 f0 0f b7 f9 8b 8e 58 01 00 00 ba 01 00 00 00 e8 cb 83 af da 8b 46 28 8b 4d f0 <89> 88 6c 04 00 00 8b 46 28 89 b8 70 04 00 00 8b 46 28 05 60 04 00 [ 33.692572] EAX: f7cd8000 EBX: c232c548 ECX: 00000002 EDX: 00000001 [ 33.692626] ESI: c232c548 EDI: 00009f9f EBP: c1fd5b08 ESP: c1fd5ae4 [ 33.692679] DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 0068 EFLAGS: 00210297 [ 33.692742] CR0: 80050033 CR2: f7cd846c CR3: 0335d000 CR4: 00000690 On a 5.14, it's just the three lines snd_cs46xx 0000:00:0d.0: Direct firmware load for cs46xx/cwc4630 failed with error -2 snd_cs46xx 0000:00:0d.0: firmware load error [cwc4630] snd_cs46xx: probe of 0000:00:0d.0 failed with error -2 and life goes on.

2 3

Scarlett 18i20 USB, USB Audio
by GitHub issues - edited 05 Jul '22

05 Jul '22

alsa-project/alsa-topology-conf issue #2 was edited from juanpc2018: Problems: $ alsamixer does Not allow to change SampleRate, only clock source. Focusrite Scarlett 18i20mk2 USB, USB Audio has a Control Software only for OSX and Windows. works in Wine/PlayOnMac, but does Not detect the USB device. sometimes even the Control Software also does Not detect the USB in OSX or Windows. anyway... i go to OSX, change the Sample Rate. 96khz, reboot and go back to Kubuntu 21.10, Kubuntu changes the SR to 44.1Khz at boot, problem #2. SampleRateConverter does Not work well Down, if i play a 48khz file in VLC, does Not play in 44.1Khz SR. jackd2 can change SampleRate No problems. VLC has jack output, but Firefox does Not, unless compied from source with the option in config anyway.. need to create: /etc/asound.conf $ aplay -L hw:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Direct hardware device without any conversions plughw:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Hardware device with all software conversions sysdefault:CARD=USB Scarlett 18i20 USB, USB Audio Default Audio Device front:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Front output / input surround21:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 2.1 Surround output to Front and Subwoofer speakers surround40:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 4.0 Surround output to Front and Rear speakers surround41:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 4.1 Surround output to Front, Rear and Subwoofer speakers surround50:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 5.0 Surround output to Front, Center and Rear speakers surround51:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 5.1 Surround output to Front, Center, Rear and Subwoofer speakers surround71:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 7.1 Surround output to Front, Center, Side, Rear and Woofer speakers iec958:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio IEC958 (S/PDIF) Digital Audio Output dmix:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Direct sample mixing device usbstream:CARD=USB Scarlett 18i20 USB USB Stream Output Issue URL : https://github.com/alsa-project/alsa-topology-conf/issues/2 Repository URL: https://github.com/alsa-project/alsa-topology-conf

1 0

Scarlett 18i20 USB, USB Audio
by GitHub issues - opened 05 Jul '22

05 Jul '22

alsa-project/alsa-topology-conf issue #2 was opened from juanpc2018: Problems: $ alsamixer does Not allow to change SampleRate, only clock source. Focusrite Scarlett 18i20mk2 USB, USB Audio has a Control Software only for OSX and Windows. works in Wine/PlayOnMac, but does Not detect the USB device. sometimes even the Control Software also does Not detect the USB in OSX or Windows. anyway... i go to OSX, change the Sample Rate. 96khz, reboot and go back to Kubuntu 21.10, Kubuntu changes the SR to 44.1Khz at boot, problems #2. SampleRateConverter does Not work well, if i play a 48khz file in VLC, does Not play in 44.1Khz SR. jackd2 can change SampleRate No problems. VLC has jack output, but Firefox does Not, unless compied from source with the option in config anyway.. need to create: /etc/asound.conf $ aplay -L hw:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Direct hardware device without any conversions plughw:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Hardware device with all software conversions sysdefault:CARD=USB Scarlett 18i20 USB, USB Audio Default Audio Device front:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Front output / input surround21:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 2.1 Surround output to Front and Subwoofer speakers surround40:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 4.0 Surround output to Front and Rear speakers surround41:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 4.1 Surround output to Front, Rear and Subwoofer speakers surround50:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 5.0 Surround output to Front, Center and Rear speakers surround51:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 5.1 Surround output to Front, Center, Rear and Subwoofer speakers surround71:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio 7.1 Surround output to Front, Center, Side, Rear and Woofer speakers iec958:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio IEC958 (S/PDIF) Digital Audio Output dmix:CARD=USB,DEV=0 Scarlett 18i20 USB, USB Audio Direct sample mixing device usbstream:CARD=USB Scarlett 18i20 USB USB Stream Output Issue URL : https://github.com/alsa-project/alsa-topology-conf/issues/2 Repository URL: https://github.com/alsa-project/alsa-topology-conf

1 0

[PATCH] ALSA: usb-audio: Add endianness annotations
by Jan Schär 05 Jul '22

05 Jul '22

Fixes: 4b8ea38fabab ("ALSA: usb-audio: Support jack detection on Dell dock") Reported-by: kernel test robot <lkp(a)intel.com> Link: https://lore.kernel.org/r/202207051932.qUilU0am-lkp@intel.com Signed-off-by: Jan Schär <jan(a)jschaer.ch> --- sound/usb/mixer_quirks.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c index 5a45822e60e7..c06d6dfa8139 100644 --- a/sound/usb/mixer_quirks.c +++ b/sound/usb/mixer_quirks.c @@ -1967,7 +1967,7 @@ static int snd_soundblaster_e1_switch_create(struct usb_mixer_interface *mixer) static int realtek_hda_set(struct snd_usb_audio *chip, u32 cmd) { struct usb_device *dev = chip->dev; - u32 buf = cpu_to_be32(cmd); + __be32 buf = cpu_to_be32(cmd); return snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), REALTEK_HDA_SET, USB_RECIP_DEVICE | USB_TYPE_VENDOR | USB_DIR_OUT, @@ -1978,7 +1978,7 @@ static int realtek_hda_get(struct snd_usb_audio *chip, u32 cmd, u32 *value) { struct usb_device *dev = chip->dev; int err; - u32 buf = cpu_to_be32(cmd); + __be32 buf = cpu_to_be32(cmd); err = snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0), REALTEK_HDA_GET_OUT, USB_RECIP_DEVICE | USB_TYPE_VENDOR | USB_DIR_OUT, base-commit: 2ef9868fff184a6246d2f71190be108a71aa2faa -- 2.32.0

2 1