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

[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

GoXLR - Dynamic Channel Count
by GitHub issues - edited 26 Sep '24

26 Sep '24

alsa-project/alsa-ucm-conf issue #444 was edited from FrostyCoolSlug: The next GoXLR firmware release is going to add two new output channels for the full sized device, labeled a 'Stream Mix 2'. Outside of general usage, there's no way to determine whether these channels exist or not outside of the UCM2 definitions as the device is booting up. Is there currently the facility to configure the devices virtual channels based on the number of channels reported by the hardware (so if there are 23 hardware channels, use the current configuration, if there are 25 hardware channels use an adjusted configuration), and if not, could you suggest the best way to handle this implementation? Regards, Craig McLure Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/444 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

GoXLR - Dynamic Output Channel Count
by GitHub issues - opened 26 Sep '24

26 Sep '24

alsa-project/alsa-ucm-conf issue #444 was opened from FrostyCoolSlug: The next GoXLR firmware release is going to add two new output channels for the full sized device, labeled a 'Stream Mix 2'. Outside of general usage, there's no way to determine whether these channels exist or not outside of the UCM2 definitions as the device is booting up. Is there currently the facility to configure the devices virtual channels based on the number of channels reported by the hardware (so if there are 23 hardware channels, use the current configuration, if there are 25 hardware channels use an adjusted configuration), and if not, could you suggest the best way to handle this implementation? Regards, Craig McLure Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/444 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH 00/13] ASoC: remove dpcm_xxx flags
by Kuninori Morimoto 26 Sep '24

26 Sep '24

Hi Mark Now, we don't need to use dpcm_playback/capture flags for DPCM since v6.11, and thus we can use playback/capture_only flags instead too for DPCM, which is same as non-DPCM. Let's remove dpcm_playback/capture flags, and use playback/capture_only flags instead. [Before] [After] dpcm_playback = 1; /* no setting is needed */ dpcm_capture = 1; dpcm_playback = 1; playback_only = 1; dpcm_capture = 0; dpcm_playback = 0; capture_only = 1; dpcm_capture = 1; And then, because no-one uses dpcm_xxx flags, we don't need to have the code for both DPCM and non-DPCM. These can be handled by same code. This patch-set is based on latest linus/master branch. Kuninori Morimoto (13): ASoC: amd: remove dpcm_xxx flags ASoC: fsl: remove dpcm_xxx flags ASoC: sof: remove dpcm_xxx flags ASoC: intel: remove dpcm_xxx flags ASoC: samsung: remove dpcm_xxx flags ASoC: mediatek: remove dpcm_xxx flags ASoC: soc-core: remove dpcm_xxx flags ASoC: soc-compress: remove dpcm_xxx flags ASoC: soc-topology: remove dpcm_xxx flags ASoC: intel: boards: remove dpcm_xxx flags ASoC: soc-pcm: remove dpcm_xxx flags ASoC: doc: remove dpcm_xxx flags ASoC: soc-pcm: merge DPCM and non-DPCM validation check Documentation/sound/soc/dpcm.rst | 11 +-- include/sound/soc.h | 5 - sound/soc/amd/acp-da7219-max98357a.c | 20 ++-- sound/soc/amd/acp-es8336.c | 2 - sound/soc/amd/acp/acp-mach-common.c | 24 ++--- sound/soc/amd/acp3x-rt5682-max9836.c | 6 +- sound/soc/amd/vangogh/acp5x-mach.c | 6 -- sound/soc/fsl/fsl-asoc-card.c | 24 ++--- sound/soc/fsl/imx-audmix.c | 18 ++-- sound/soc/fsl/imx-card.c | 4 +- sound/soc/intel/avs/boards/da7219.c | 2 - sound/soc/intel/avs/boards/dmic.c | 4 +- sound/soc/intel/avs/boards/es8336.c | 2 - sound/soc/intel/avs/boards/hdaudio.c | 4 - sound/soc/intel/avs/boards/i2s_test.c | 2 - sound/soc/intel/avs/boards/max98357a.c | 2 +- sound/soc/intel/avs/boards/max98373.c | 2 - sound/soc/intel/avs/boards/max98927.c | 2 - sound/soc/intel/avs/boards/nau8825.c | 2 - sound/soc/intel/avs/boards/rt274.c | 2 - sound/soc/intel/avs/boards/rt286.c | 2 - sound/soc/intel/avs/boards/rt298.c | 2 - sound/soc/intel/avs/boards/rt5514.c | 2 +- sound/soc/intel/avs/boards/rt5663.c | 2 - sound/soc/intel/avs/boards/rt5682.c | 2 - sound/soc/intel/avs/boards/ssm4567.c | 2 - sound/soc/intel/boards/bdw-rt5650.c | 4 - sound/soc/intel/boards/bdw-rt5677.c | 4 - sound/soc/intel/boards/bdw_rt286.c | 10 +- sound/soc/intel/boards/bytcht_cx2072x.c | 6 +- sound/soc/intel/boards/bytcht_da7213.c | 6 +- sound/soc/intel/boards/bytcht_es8316.c | 6 +- sound/soc/intel/boards/bytcht_nocodec.c | 6 +- sound/soc/intel/boards/bytcr_rt5640.c | 6 +- sound/soc/intel/boards/bytcr_rt5651.c | 6 +- sound/soc/intel/boards/bytcr_wm5102.c | 6 +- sound/soc/intel/boards/cht_bsw_max98090_ti.c | 6 +- sound/soc/intel/boards/cht_bsw_nau8824.c | 6 +- sound/soc/intel/boards/cht_bsw_rt5645.c | 6 +- sound/soc/intel/boards/cht_bsw_rt5672.c | 6 +- sound/soc/intel/boards/ehl_rt5660.c | 14 ++- sound/soc/intel/boards/hsw_rt5640.c | 10 +- sound/soc/intel/boards/sof_board_helpers.c | 15 +-- sound/soc/intel/boards/sof_es8336.c | 8 +- sound/soc/intel/boards/sof_pcm512x.c | 9 +- sound/soc/intel/boards/sof_wm8804.c | 2 - sound/soc/mediatek/mt2701/mt2701-cs42448.c | 20 +--- sound/soc/mediatek/mt2701/mt2701-wm8960.c | 6 +- sound/soc/mediatek/mt6797/mt6797-mt6351.c | 24 ++--- sound/soc/mediatek/mt7986/mt7986-wm8960.c | 6 +- sound/soc/mediatek/mt8173/mt8173-max98090.c | 6 +- .../mediatek/mt8173/mt8173-rt5650-rt5514.c | 6 +- .../mediatek/mt8173/mt8173-rt5650-rt5676.c | 10 +- sound/soc/mediatek/mt8173/mt8173-rt5650.c | 10 +- .../mediatek/mt8183/mt8183-da7219-max98357.c | 34 +++---- .../mt8183/mt8183-mt6358-ts3a227-max98357.c | 34 +++---- sound/soc/mediatek/mt8186/mt8186-mt6366.c | 86 +++++++---------- sound/soc/mediatek/mt8188/mt8188-mt6359.c | 58 ++++++------ .../mt8192/mt8192-mt6359-rt1015-rt5682.c | 78 +++++++--------- sound/soc/mediatek/mt8195/mt8195-mt6359.c | 60 ++++++------ sound/soc/mediatek/mt8365/mt8365-mt6357.c | 14 +-- sound/soc/samsung/odroid.c | 10 +- sound/soc/soc-compress.c | 15 ++- sound/soc/soc-core.c | 20 +--- sound/soc/soc-pcm.c | 93 +++++++------------ sound/soc/soc-topology-test.c | 2 - sound/soc/soc-topology.c | 4 +- sound/soc/sof/nocodec.c | 4 - 68 files changed, 315 insertions(+), 583 deletions(-) -- 2.43.0 Thank you for your help !! Best regards --- Kuninori Morimoto

1 14

Re: [PATCH v2] MAINTAINERS: Update maintainer list for MICROCHIP ASOC, SSC and MCP16502 drivers
by Nicolas Ferre 25 Sep '24

25 Sep '24

(adding Mark and alsa-devel to the thread) On 25/09/2024 at 15:07, Andrei Simion wrote: > To help Claudiu and offload the work, add myself to the maintainer list for > those drivers. > > Acked-by: Claudiu Beznea <claudiu.beznea(a)tuxon.dev> Acked-by: Nicolas Ferre <nicolas.ferre(a)microchip.com> Thanks a lot to both of you to continue this work. Best regards, Nicolas > Signed-off-by: Andrei Simion <andrei.simion(a)microchip.com> > --- > v1 -> v2: > - add acked-by tag > - adjust commit title > - update maintainer list for MICROCHIP SSC DRIVER > --- > MAINTAINERS | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 09d0f09c36cc..12e6777bb6b4 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -15088,6 +15088,7 @@ F: drivers/spi/spi-at91-usart.c > > MICROCHIP AUDIO ASOC DRIVERS > M: Claudiu Beznea <claudiu.beznea(a)tuxon.dev> > +M: Andrei Simion <andrei.simion(a)microchip.com> > L: alsa-devel(a)alsa-project.org (moderated for non-subscribers) > S: Supported > F: Documentation/devicetree/bindings/sound/atmel* > @@ -15196,6 +15197,7 @@ F: include/video/atmel_lcdc.h > > MICROCHIP MCP16502 PMIC DRIVER > M: Claudiu Beznea <claudiu.beznea(a)tuxon.dev> > +M: Andrei Simion <andrei.simion(a)microchip.com> > L: linux-arm-kernel(a)lists.infradead.org (moderated for non-subscribers) > S: Supported > F: Documentation/devicetree/bindings/regulator/microchip,mcp16502.yaml > @@ -15327,6 +15329,7 @@ F: drivers/spi/spi-atmel.* > > MICROCHIP SSC DRIVER > M: Claudiu Beznea <claudiu.beznea(a)tuxon.dev> > +M: Andrei Simion <andrei.simion(a)microchip.com> > L: linux-arm-kernel(a)lists.infradead.org (moderated for non-subscribers) > S: Supported > F: Documentation/devicetree/bindings/misc/atmel-ssc.txt > > base-commit: 2b7275670032a98cba266bd1b8905f755b3e650f

1 0