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

[PATCH 1/2] ALSA: usb-audio: Inform the delayed registration more properly
by Takashi Iwai 31 Aug '22

31 Aug '22

The info message that was added in the commit a4aad5636c72 ("ALSA: usb-audio: Inform devices that need delayed registration") is actually useful to know the need for the delayed registration. However, it turned out that this doesn't catch the all cases; namely, this warned only when a PCM stream is attached onto the existing PCM instance, but it doesn't count for a newly created PCM instance. This made confusion as if there were no further delayed registration. This patch moves the check to the code path for either adding a stream or creating a PCM instance. Also, make it simpler by checking the card->registered flag instead of querying each snd_device state. Fixes: a4aad5636c72 ("ALSA: usb-audio: Inform devices that need delayed registration") Link: https://bugzilla.kernel.org/show_bug.cgi?id=216082 Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/usb/stream.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/sound/usb/stream.c b/sound/usb/stream.c index ceb93d798182..40b7821c6c99 100644 --- a/sound/usb/stream.c +++ b/sound/usb/stream.c @@ -495,6 +495,10 @@ static int __snd_usb_add_audio_stream(struct snd_usb_audio *chip, return 0; } } + + if (chip->card->registered) + chip->need_delayed_register = true; + /* look for an empty stream */ list_for_each_entry(as, &chip->pcm_list, list) { if (as->fmt_type != fp->fmt_type) @@ -502,9 +506,6 @@ static int __snd_usb_add_audio_stream(struct snd_usb_audio *chip, subs = &as->substream[stream]; if (subs->ep_num) continue; - if (snd_device_get_state(chip->card, as->pcm) != - SNDRV_DEV_BUILD) - chip->need_delayed_register = true; err = snd_pcm_new_stream(as->pcm, stream, 1); if (err < 0) return err; -- 2.35.3

1 1

[PATCH v3 1/2] ASoC: dt-bindings: max98396: Document data monitor properties
by Daniel Mack 31 Aug '22

31 Aug '22

This device features a data monitor that puts the device in software reset upon a configurable set of events. Signed-off-by: Daniel Mack <daniel(a)zonque.org> --- v1 -> v2: fix a typo and remove a stray blank line v2 -> v3: Rename device tree properties, drop $ref for standard unit suffix properties .../bindings/sound/adi,max98396.yaml | 34 +++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/Documentation/devicetree/bindings/sound/adi,max98396.yaml b/Documentation/devicetree/bindings/sound/adi,max98396.yaml index 8d2ef991db40..7985b1410017 100644 --- a/Documentation/devicetree/bindings/sound/adi,max98396.yaml +++ b/Documentation/devicetree/bindings/sound/adi,max98396.yaml @@ -78,6 +78,40 @@ properties: interleaved on a single output channel. type: boolean + adi,dmon-stuck-enable: + description: + Enables the "data monitor stuck" feature. Once the data monitor is + enabled, it actively monitors the selected input data (from DIN) to the + speaker amplifier. Once a data error is detected, the data monitor + automatically places the device into software shutdown. + type: boolean + + adi,dmon-stuck-threshold-bits: + description: + Sets the threshold for the "data monitor stuck" feature, in bits. + enum: [9, 11, 13, 15] + default: 15 + + adi,dmon-magnitude-enable: + description: + Enables the "data monitor magnitude" feature. Once the data monitor is + enabled, it actively monitors the selected input data (from DIN) to the + speaker amplifier. Once a data error is detected, the data monitor + automatically places the device into software shutdown. + type: boolean + + adi,dmon-magnitude-threshold-bits: + description: + Sets the threshold for the "data monitor magnitude" feature, in bits. + enum: [2, 3, 4, 5] + default: 5 + + adi,dmon-duration-ms: + description: + Sets the duration for the "data monitor" feature, in milliseconds. + enum: [64, 256, 1024, 4096] + default: 64 + reset-gpios: maxItems: 1 -- 2.37.2

3 3

[PATCH 0/5] ASoC: codecs: minor cppcheck cleanups
by Pierre-Louis Bossart 31 Aug '22

31 Aug '22

Small number of cleanups that were either missed in previous versions or detected by new cppcheck version. Pierre-Louis Bossart (5): ASoC: hdmi-codec: remove unused definitions ASoC: wcd-mbhc-v2: remove always-true condition ASoC: wcd9335: remove always-true condition ASoC: fsl: fsl-utils: remove useless assignment ASoC: ti: omap-mcbsp: remove useless assignment sound/soc/codecs/hdmi-codec.c | 4 ---- sound/soc/codecs/wcd-mbhc-v2.c | 2 +- sound/soc/codecs/wcd9335.c | 4 +--- sound/soc/fsl/fsl_utils.c | 2 +- sound/soc/ti/omap-mcbsp.c | 2 +- 5 files changed, 4 insertions(+), 10 deletions(-) -- 2.34.1

3 7

[PATCH 0/2] ASoC: SOF: compress: Add support for timestamp on capture
by Laurentiu Mihalcea 31 Aug '22

31 Aug '22

The purpose of this patch series is to add support for timestamping on capture direction using the compress API. This is simply done by splitting sof_compr_copy into 2 functions: sof_compr_copy_playback and sof_compr_copy_capture. Each of these functions handles one of the possible directions: capture or playback and is called in sof_compr_copy based on the stream's direction. The only difference between sof_compr_copy_playback and sof_compr_copy_capture is the fact that on playback case we need to copy data from user space and on capture we need to copy data to user space. Laurentiu Mihalcea (2): ASoC: SOF: compress: Move sof_compr_copy functionality ASoC: SOF: compress: Add copy function for capture case sound/soc/sof/compress.c | 47 +++++++++++++++++++++++++++++++++------- 1 file changed, 39 insertions(+), 8 deletions(-) -- 2.34.1

2 3

[PATCH v2] ASoC: amd: acp: Modify dai_id macros to be more generic
by Venkata Prasad Potturu 31 Aug '22

31 Aug '22

Change dai_id macros to make I2S instances in order. Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu(a)amd.com> --- Changes since v1: -- Change commit description. sound/soc/amd/acp/amd.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/amd/acp/amd.h b/sound/soc/amd/acp/amd.h index af9603724a68..60a7c95f134f 100644 --- a/sound/soc/amd/acp/amd.h +++ b/sound/soc/amd/acp/amd.h @@ -21,9 +21,9 @@ #define ACP3X_DEV 3 #define ACP6X_DEV 6 -#define I2S_SP_INSTANCE 0x00 -#define I2S_BT_INSTANCE 0x01 -#define DMIC_INSTANCE 0x02 +#define DMIC_INSTANCE 0x00 +#define I2S_SP_INSTANCE 0x01 +#define I2S_BT_INSTANCE 0x02 #define I2S_HS_INSTANCE 0x03 #define MEM_WINDOW_START 0x4080000 -- 2.25.1

2 1

[PATCH 0/6] ASoC: qcom: add support for sc8280xp machine
by Srinivas Kandagatla 31 Aug '22

31 Aug '22

THis patchset adds support for SC8280XP SoC machine driver. First patch moves some of the commonly used code to common from sm8250 machine driver and the follow on code adds minimal support for sc8280xp. Currently this driver is only tested with SmartSpeakers and Headset on Lenovo Thinkpad X13s. Support for sm8450 is tested and I will post the patches soon. Thanks, Srini Srinivas Kandagatla (3): ASoC: dt-bindings: qcom,sm8250: add compatibles for sm8450 and sm8250 ASoC: qcom: sm8250: move some code to common ASoC: qcom: add machine driver for sc8280xp .../bindings/sound/qcom,sm8250.yaml | 2 + sound/soc/qcom/Kconfig | 11 ++ sound/soc/qcom/Makefile | 2 + sound/soc/qcom/common.c | 169 ++++++++++++++++++ sound/soc/qcom/common.h | 12 ++ sound/soc/qcom/sc8280xp.c | 157 ++++++++++++++++ sound/soc/qcom/sm8250.c | 152 +--------------- 7 files changed, 360 insertions(+), 145 deletions(-) create mode 100644 sound/soc/qcom/sc8280xp.c -- 2.21.0

3 6

How does PlaybackChannelPos# Work?
by GitHub issues - opened 31 Aug '22

31 Aug '22

alsa-project/alsa-ucm-conf issue #209 was opened from 13r0ck: I found https://github.com/alsa-project/alsa-lib/commit/0ba999591b78e07a4b8e610ebce… but doesn't seem to work when setting audio jack output. I have a mostly working configuration for a motherboard's 3.5mm audio jack output. The only problem is that the C/Sub and Rear jacks are flipped. I am assuming setting the `PlaybackChannelPos` in the speaker should work, but it doesn't for some reason. ``` SectionDevice."Speaker" { Commant "Speakers" Value { PlaybackChannel2 "BL" PlaybackChannel3 "BR" PlaybackChannel4 "FC" PlaybackChannel5 "LFE" } } ``` Any help would be greatly appreciated! Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/209 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH] ASoC: amd: fix spelling mistake: "i.e" -> "i.e."
by Syed Saba Kareem 30 Aug '22

30 Aug '22

From: Syed Saba kareem <Syed.SabaKareem(a)amd.com> trivial fix to spelling mistake in Kconfig File. Reported by : Randy Dunlap <rdunlap(a)infradead.org> Signed-off-by: Syed Saba Kareem <Syed.SabaKareem(a)amd.com> --- sound/soc/amd/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/amd/Kconfig b/sound/soc/amd/Kconfig index 68837d42736d..150786279257 100644 --- a/sound/soc/amd/Kconfig +++ b/sound/soc/amd/Kconfig @@ -122,7 +122,7 @@ config SND_SOC_AMD_RPL_ACP6x tristate "AMD Audio Coprocessor-v6.2 RPL support" depends on X86 && PCI help - This option enables Audio Coprocessor i.e ACP v6.2 support on + This option enables Audio Coprocessor i.e. ACP v6.2 support on AMD RPL platform. By enabling this flag build will be triggered for ACP PCI driver. Say m if you have such a device. -- 2.25.1

3 2

[PATCH 1/3] slimbus: qcom-ngd: use correct error in message of pdr_add_lookup() failure
by Krzysztof Kozlowski 30 Aug '22

30 Aug '22

Use cprrect error code, instead of previous 'ret' value, when printing error from pdr_add_lookup() failure. Cc: <stable(a)vger.kernel.org> Fixes: e1ae85e1830e ("slimbus: qcom-ngd-ctrl: add Protection Domain Restart Support") Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> --- drivers/slimbus/qcom-ngd-ctrl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/slimbus/qcom-ngd-ctrl.c b/drivers/slimbus/qcom-ngd-ctrl.c index 0aa8408464ad..6fe6abb86061 100644 --- a/drivers/slimbus/qcom-ngd-ctrl.c +++ b/drivers/slimbus/qcom-ngd-ctrl.c @@ -1581,6 +1581,7 @@ static int qcom_slim_ngd_ctrl_probe(struct platform_device *pdev) pds = pdr_add_lookup(ctrl->pdr, "avs/audio", "msm/adsp/audio_pd"); if (IS_ERR(pds) && PTR_ERR(pds) != -EALREADY) { + ret = PTR_ERR(pds); dev_err(dev, "pdr add lookup failed: %d\n", ret); return PTR_ERR(pds); } -- 2.34.1

2 4

[PATCH] ASoC: codecs: allow compile testing without MFD drivers
by Krzysztof Kozlowski 30 Aug '22

30 Aug '22

Motorola CPCAP, Lochnagar Sound, Rockchip RK817 and Qualcomm WCD9340/WCD9341 do not depend on parent MFD driver in build time and can be compile tested without respective MFD part for increased build coverage. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski(a)linaro.org> --- sound/soc/codecs/Kconfig | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig index 5926b33ba09e..94b7bb85d236 100644 --- a/sound/soc/codecs/Kconfig +++ b/sound/soc/codecs/Kconfig @@ -610,7 +610,7 @@ config SND_SOC_BT_SCO config SND_SOC_CPCAP tristate "Motorola CPCAP codec" - depends on MFD_CPCAP + depends on MFD_CPCAP || COMPILE_TEST config SND_SOC_CQ0093VC tristate @@ -972,7 +972,7 @@ config SND_SOC_LM49453 config SND_SOC_LOCHNAGAR_SC tristate "Lochnagar Sound Card" - depends on MFD_LOCHNAGAR + depends on MFD_LOCHNAGAR || COMPILE_TEST help This driver support the sound card functionality of the Cirrus Logic Lochnagar audio development board. @@ -1197,7 +1197,7 @@ config SND_SOC_RK3328 config SND_SOC_RK817 tristate "Rockchip RK817 audio CODEC" - depends on MFD_RK808 + depends on MFD_RK808 || COMPILE_TEST select REGMAP_I2C config SND_SOC_RL6231 @@ -1745,7 +1745,7 @@ config SND_SOC_WCD934X tristate "WCD9340/WCD9341 Codec" depends on COMMON_CLK select SND_SOC_WCD_MBHC - depends on MFD_WCD934X + depends on MFD_WCD934X || COMPILE_TEST help The WCD9340/9341 is a audio codec IC Integrated in Qualcomm SoCs like SDM845. -- 2.34.1

2 5