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

[PATCH v2] ALSA: hda/conexant: fix some typos
by Oldherl Oh 30 Sep '24

30 Sep '24

Fix some typos in patch_conexant.c Signed-off-by: Oldherl Oh <me(a)oldherl.one> --- sound/pci/hda/patch_conexant.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c index e851785ff058..ade42a8209c2 100644 --- a/sound/pci/hda/patch_conexant.c +++ b/sound/pci/hda/patch_conexant.c @@ -166,18 +166,18 @@ static void cxt_init_gpio_led(struct hda_codec *codec) static void cx_fixup_headset_recog(struct hda_codec *codec) { - unsigned int mic_persent; + unsigned int mic_present; /* fix some headset type recognize fail issue, such as EDIFIER headset */ - /* set micbiasd output current comparator threshold from 66% to 55%. */ + /* set micbias output current comparator threshold from 66% to 55%. */ snd_hda_codec_write(codec, 0x1c, 0, 0x320, 0x010); - /* set OFF voltage for DFET from -1.2V to -0.8V, set headset micbias registor + /* set OFF voltage for DFET from -1.2V to -0.8V, set headset micbias register * value adjustment trim from 2.2K ohms to 2.0K ohms. */ snd_hda_codec_write(codec, 0x1c, 0, 0x3b0, 0xe10); /* fix reboot headset type recognize fail issue */ - mic_persent = snd_hda_codec_read(codec, 0x19, 0, AC_VERB_GET_PIN_SENSE, 0x0); - if (mic_persent & AC_PINSENSE_PRESENCE) + mic_present = snd_hda_codec_read(codec, 0x19, 0, AC_VERB_GET_PIN_SENSE, 0x0); + if (mic_present & AC_PINSENSE_PRESENCE) /* enable headset mic VREF */ snd_hda_codec_write(codec, 0x19, 0, AC_VERB_SET_PIN_WIDGET_CONTROL, 0x24); else @@ -249,9 +249,9 @@ static void cx_update_headset_mic_vref(struct hda_codec *codec, struct hda_jack_ { unsigned int mic_present; - /* In cx8070 and sn6140, the node 16 can only be config to headphone or disabled, - * the node 19 can only be config to microphone or disabled. - * Check hp&mic tag to process headset pulgin&plugout. + /* In cx8070 and sn6140, the node 16 can only be configured to headphone or disabled, + * the node 19 can only be configured to microphone or disabled. + * Check hp&mic tag to process headset plugin & plugout. */ mic_present = snd_hda_codec_read(codec, 0x19, 0, AC_VERB_GET_PIN_SENSE, 0x0); if (!(mic_present & AC_PINSENSE_PRESENCE)) /* mic plugout */ -- 2.46.2

2 1

ucm2: USB-Audio: add Steinberg UR22C (USB0499:172f)
by GitHub pull_request - opened 30 Sep '24

30 Sep '24

alsa-project/alsa-ucm-conf pull request #446 was opened from CoelacanthusHex: Similar to UR24C and UR44C, but with 2 inputs 2 outputs. Signed-off-by: Celeste Liu <CoelacanthusHex(a)gmail.com> Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/446 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/446.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH] ALSA: hda/realtek: Fix the push button function for the ALC257
by Oder Chiou 30 Sep '24

30 Sep '24

The patch "Fix noise from speakers on Lenovo IdeaPad 3 15IAU7" caused that the headset push button cannot work properly in case of ALC257. This patch reverted it to correct the side effect. Signed-off-by: Oder Chiou <oder_chiou(a)realtek.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 f787ff4182d4..86db1a4ec24c 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -587,6 +587,7 @@ static void alc_shutup_pins(struct hda_codec *codec) switch (codec->core.vendor_id) { case 0x10ec0236: case 0x10ec0256: + case 0x10ec0257: case 0x19e58326: case 0x10ec0283: case 0x10ec0285: -- 2.34.1

2 1

Fix some typos in hda/conexant
by Oldherl Oh 30 Sep '24

30 Sep '24

I'm trying to understand Conexant driver code in order to solve my microphone detection problem. As I am new to kernel development, I have made this simple typo-fixing-only patch as an exercise. Hope it is not disturbing.

2 2

[PATCH v2] ALSA: Fix typos in comments across various files
by Yu Jiaoliang 30 Sep '24

30 Sep '24

This patch fixes typos in comments within the ALSA subsystem. These changes improve code readability without affecting functionality. Signed-off-by: Yu Jiaoliang <yujiaoliang(a)vivo.com> --- v2: Update commit message. --- sound/core/compress_offload.c | 2 +- sound/core/oss/rate.c | 2 +- sound/core/pcm_native.c | 2 +- sound/core/sound.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c index b8c0d6edbdd1..bdf1d78de833 100644 --- a/sound/core/compress_offload.c +++ b/sound/core/compress_offload.c @@ -288,7 +288,7 @@ static ssize_t snd_compr_write(struct file *f, const char __user *buf, stream = &data->stream; guard(mutex)(&stream->device->lock); - /* write is allowed when stream is running or has been steup */ + /* write is allowed when stream is running or has been setup */ switch (stream->runtime->state) { case SNDRV_PCM_STATE_SETUP: case SNDRV_PCM_STATE_PREPARED: diff --git a/sound/core/oss/rate.c b/sound/core/oss/rate.c index 98269119347f..b56eeda5e30e 100644 --- a/sound/core/oss/rate.c +++ b/sound/core/oss/rate.c @@ -294,7 +294,7 @@ static int rate_action(struct snd_pcm_plugin *plugin, default: break; } - return 0; /* silenty ignore other actions */ + return 0; /* silently ignore other actions */ } int snd_pcm_plugin_build_rate(struct snd_pcm_substream *plug, diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c index 99e39b5359cc..b49bc89c0758 100644 --- a/sound/core/pcm_native.c +++ b/sound/core/pcm_native.c @@ -3115,7 +3115,7 @@ struct snd_pcm_sync_ptr32 { } c; } __packed; -/* recalcuate the boundary within 32bit */ +/* recalculate the boundary within 32bit */ static snd_pcm_uframes_t recalculate_boundary(struct snd_pcm_runtime *runtime) { snd_pcm_uframes_t boundary; diff --git a/sound/core/sound.c b/sound/core/sound.c index b9db9aa0bfcb..6531a67f13b3 100644 --- a/sound/core/sound.c +++ b/sound/core/sound.c @@ -133,7 +133,7 @@ static struct snd_minor *autoload_device(unsigned int minor) /* /dev/aloadSEQ */ snd_request_other(minor); } - mutex_lock(&sound_mutex); /* reacuire lock */ + mutex_lock(&sound_mutex); /* reacquire lock */ return snd_minors[minor]; } #else /* !CONFIG_MODULES */ -- 2.34.1

2 1

[PATCH v1] slimbus: Correct some typos in comments
by Shen Lichuan 29 Sep '24

29 Sep '24

Fixed some confusing typos that were currently identified with codespell, the details are as follows: -in the code comments: drivers/slimbus/messaging.c:226: transfered ==> transferred drivers/slimbus/messaging.c:230: vlaue ==> value drivers/slimbus/qcom-ctrl.c:127: Oder ==> Order drivers/slimbus/slimbus.h:263: Presense ==> Presence drivers/slimbus/slimbus.h:371: resposible ==> responsible Signed-off-by: Shen Lichuan <shenlichuan(a)vivo.com> --- drivers/slimbus/messaging.c | 4 ++-- drivers/slimbus/qcom-ctrl.c | 2 +- drivers/slimbus/slimbus.h | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/slimbus/messaging.c b/drivers/slimbus/messaging.c index 242570a5e565..b7dd2badc9d1 100644 --- a/drivers/slimbus/messaging.c +++ b/drivers/slimbus/messaging.c @@ -223,11 +223,11 @@ static u16 slim_slicesize(int code) /** * slim_xfer_msg() - Transfer a value info message on slim device * - * @sbdev: slim device to which this msg has to be transfered + * @sbdev: slim device to which this msg has to be transferred * @msg: value info message pointer * @mc: message code of the message * - * Called by drivers which want to transfer a vlaue or info elements. + * Called by drivers which want to transfer a value or info elements. * * Return: -ETIMEDOUT: If transmission of this message timed out */ diff --git a/drivers/slimbus/qcom-ctrl.c b/drivers/slimbus/qcom-ctrl.c index e25f9bdd9b23..039384d85e7b 100644 --- a/drivers/slimbus/qcom-ctrl.c +++ b/drivers/slimbus/qcom-ctrl.c @@ -124,7 +124,7 @@ static void qcom_slim_queue_tx(struct qcom_slim_ctrl *ctrl, void *buf, __iowrite32_copy(ctrl->base + tx_reg, buf, count); - /* Ensure Oder of subsequent writes */ + /* Ensure Order of subsequent writes */ mb(); } diff --git a/drivers/slimbus/slimbus.h b/drivers/slimbus/slimbus.h index 00a7f112574b..2067d47541f2 100644 --- a/drivers/slimbus/slimbus.h +++ b/drivers/slimbus/slimbus.h @@ -260,7 +260,7 @@ enum slim_ch_aux_bit_fmt { * struct slim_channel - SLIMbus channel, used for state machine * * @id: ID of channel - * @prrate: Presense rate of channel from Table 66 of SLIMbus 2.0 Specs + * @prrate: Presence rate of channel from Table 66 of SLIMbus 2.0 Specs * @seg_dist: segment distribution code from Table 20 of SLIMbus 2.0 Specs * @data_fmt: Data format of channel. * @aux_fmt: Aux format for this channel. @@ -368,7 +368,7 @@ struct slim_stream_runtime { * @get_laddr: It is possible that controller needs to set fixed logical * address table and get_laddr can be used in that case so that controller * can do this assignment. Use case is when the master is on the remote - * processor side, who is resposible for allocating laddr. + * processor side, who is responsible for allocating laddr. * @wakeup: This function pointer implements controller-specific procedure * to wake it up from clock-pause. Framework will call this to bring * the controller out of clock pause. -- 2.17.1

1 0

ucm2: Add Roland Bridge Cast V2
by GitHub pull_request - edited 28 Sep '24

28 Sep '24

alsa-project/alsa-ucm-conf pull request #445 was edited from Fumler: Roland Bridge Cast recently had a firmware update which brings in e.g. virtual surround and is dubbed as "V2" and has a separate usb device id than the existing Bridge Cast in the repo. This PR adds the changes needed for the new firmware to be mapped correctly, as a new USB audio device. Changelog: https://www.roland.com/global/support/by_product/bridge_cast/updates_driver… Thanks to [geoffreybennett](https://old.reddit.com/user/geoffreybennett) on reddit for helping me understand how to debug this stuff. Thread: https://old.reddit.com/r/linuxaudio/comments/1fq06z8/roland_bridge_cast_no_… Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/445 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/445.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Add Roland Bridge Cast V2
by GitHub pull_request - opened 28 Sep '24

28 Sep '24

alsa-project/alsa-ucm-conf pull request #445 was opened from Fumler: Roland Bridge Cast recently had a firmware update which brings in e.g. virtual surround and is dubbed as "V2" and has a separate usb device id than the existing Bridge Cast in the repo. This PR adds the changes needed for the new firmware to be mapped correctly, as a new USB audio device. Changelog: https://www.roland.com/global/support/by_product/bridge_cast/updates_driver… Thanks to [geoffreybennett](https://old.reddit.com/user/geoffreybennett) on reddit for helping me understand how to debug this stuff. Thread: https://old.reddit.com/r/linuxaudio/comments/1fq06z8/roland_bridge_cast_no_… Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/445 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/445.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

No sound from XonarDX
by GitHub issues - reopened 28 Sep '24

28 Sep '24

alsa-project/alsa-lib issue #408 was reopened from 3bitbrain: No matter what wiki or advice I try, no sound at all from a known good Xonar DX card. Distro : Gentoo, fresh install. Always un-muted everything in the console alsamixer. alsa-lib version 1.2.11 [alsa.txt](https://github.com/user-attachments/files/16697602/alsa.txt) Issue URL : https://github.com/alsa-project/alsa-lib/issues/408 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

sound: soc: uniphier: Handle regmap_write errors in aio_src_set_param()
by (학생) 장인규 (컴퓨터공학과) 26 Sep '24

26 Sep '24

From 791716bf359b8540c519810848fd1f8006d7c3c5 Mon Sep 17 00:00:00 2001 From: Ingyu Jang <ingyujang25(a)unist.ac.kr> Date: Thu, 26 Sep 2024 19:40:04 +0900 Subject: [PATCH] sound: soc: uniphier: Handle regmap_write errors in aio_src_set_param() The aio_src_set_param() function did not previously check the return values of regmap_write() and regmap_update_bits(). If these functions fail, it could lead to silent failures when configuring the sample rate converter (SRC), causing improper behavior in audio processing without any indication of an error. This patch modifies aio_src_set_param to check the return values of regmap_write() and regmap_update_bits(). If either function returns an error, the error code is propagated back to the caller to ensure proper error handling. This change aligns with the existing error-handling behavior in functions like uniphier_aio_prepare(), where a failure in a sub-function should result in an immediate return of the error. Signed-off-by: Ingyu Jang <ingyujang25(a)unist.ac.kr> --- sound/soc/uniphier/aio-core.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/sound/soc/uniphier/aio-core.c b/sound/soc/uniphier/aio-core.c index 0eba60758134..2c4e8b873253 100644 --- a/sound/soc/uniphier/aio-core.c +++ b/sound/soc/uniphier/aio-core.c @@ -921,16 +921,19 @@ int aio_src_set_param(struct uniphier_aio_sub *sub, { struct regmap *r = sub->aio->chip->regmap; u32 v; + int ret; if (sub->swm->dir != PORT_DIR_OUTPUT) return 0; - regmap_write(r, OPORTMXSRC1CTR(sub->swm->oport.map), + ret = regmap_write(r, OPORTMXSRC1CTR(sub->swm->oport.map), OPORTMXSRC1CTR_THMODE_SRC | OPORTMXSRC1CTR_SRCPATH_CALC | OPORTMXSRC1CTR_SYNC_ASYNC | OPORTMXSRC1CTR_FSIIPSEL_INNER | OPORTMXSRC1CTR_FSISEL_ACLK); + if (ret) + return ret; switch (params_rate(params)) { default: @@ -951,12 +954,18 @@ int aio_src_set_param(struct uniphier_aio_sub *sub, break; } - regmap_write(r, OPORTMXRATE_I(sub->swm->oport.map), + + ret = regmap_write(r, OPORTMXRATE_I(sub->swm->oport.map), v | OPORTMXRATE_I_ACLKSRC_APLL | OPORTMXRATE_I_LRCKSTP_STOP); - regmap_update_bits(r, OPORTMXRATE_I(sub->swm->oport.map), + if (ret) + return ret; + + ret = regmap_update_bits(r, OPORTMXRATE_I(sub->swm->oport.map), OPORTMXRATE_I_LRCKSTP_MASK, OPORTMXRATE_I_LRCKSTP_START); + if (ret) + return ret; return 0; } -- 2.34.1

1 0