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

[PATCH v4] ASoC: ti: omap-mcbsp: duplicate sysfs error
by David Owens 27 Jun '22

27 Jun '22

From: David Owens <daowens01(a)gmail.com> Convert to managed versions of sysfs and clk allocation to simplify unbinding and error handling in probe. Managed sysfs node creation specifically addresses the following error seen the second time probe is attempted after sdma_pcm_platform_register() previously requsted probe deferral: sysfs: cannot create duplicate filename '/devices/platform/68000000.ocp/49022000.mcbsp/max_tx_thres' Signed-off-by: David Owens <dowens(a)precisionplanting.com> --- Changes in v4: * Reverted change that added use ATTRIBUTE_GROUP and left the attribute structs as-is to fix compilation error and lessen the area of impact. Changes in v3: * Whitespace changes only to allow clean apply Changes in v2: * Improved error handling --- sound/soc/ti/omap-mcbsp-priv.h | 2 -- sound/soc/ti/omap-mcbsp-st.c | 14 ++------------ sound/soc/ti/omap-mcbsp.c | 19 ++----------------- 3 files changed, 4 insertions(+), 31 deletions(-) diff --git a/sound/soc/ti/omap-mcbsp-priv.h b/sound/soc/ti/omap-mcbsp-priv.h index 7865cda4bf0a..da519ea1f303 100644 --- a/sound/soc/ti/omap-mcbsp-priv.h +++ b/sound/soc/ti/omap-mcbsp-priv.h @@ -316,8 +316,6 @@ static inline int omap_mcbsp_read(struct omap_mcbsp *mcbsp, u16 reg, /* Sidetone specific API */ int omap_mcbsp_st_init(struct platform_device *pdev); -void omap_mcbsp_st_cleanup(struct platform_device *pdev); - int omap_mcbsp_st_start(struct omap_mcbsp *mcbsp); int omap_mcbsp_st_stop(struct omap_mcbsp *mcbsp); diff --git a/sound/soc/ti/omap-mcbsp-st.c b/sound/soc/ti/omap-mcbsp-st.c index 0bc7d26c660a..7e8179cae92e 100644 --- a/sound/soc/ti/omap-mcbsp-st.c +++ b/sound/soc/ti/omap-mcbsp-st.c @@ -347,7 +347,7 @@ int omap_mcbsp_st_init(struct platform_device *pdev) if (!st_data) return -ENOMEM; - st_data->mcbsp_iclk = clk_get(mcbsp->dev, "ick"); + st_data->mcbsp_iclk = devm_clk_get(mcbsp->dev, "ick"); if (IS_ERR(st_data->mcbsp_iclk)) { dev_warn(mcbsp->dev, "Failed to get ick, sidetone might be broken\n"); @@ -359,7 +359,7 @@ int omap_mcbsp_st_init(struct platform_device *pdev) if (!st_data->io_base_st) return -ENOMEM; - ret = sysfs_create_group(&mcbsp->dev->kobj, &sidetone_attr_group); + ret = devm_device_add_group(mcbsp->dev, &sidetone_attr_group); if (ret) return ret; @@ -368,16 +368,6 @@ int omap_mcbsp_st_init(struct platform_device *pdev) return 0; } -void omap_mcbsp_st_cleanup(struct platform_device *pdev) -{ - struct omap_mcbsp *mcbsp = platform_get_drvdata(pdev); - - if (mcbsp->st_data) { - sysfs_remove_group(&mcbsp->dev->kobj, &sidetone_attr_group); - clk_put(mcbsp->st_data->mcbsp_iclk); - } -} - static int omap_mcbsp_st_info_volsw(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_info *uinfo) { diff --git a/sound/soc/ti/omap-mcbsp.c b/sound/soc/ti/omap-mcbsp.c index 58d8e200a7b9..9fb7cf0c9f88 100644 --- a/sound/soc/ti/omap-mcbsp.c +++ b/sound/soc/ti/omap-mcbsp.c @@ -702,8 +702,7 @@ static int omap_mcbsp_init(struct platform_device *pdev) mcbsp->max_tx_thres = max_thres(mcbsp) - 0x10; mcbsp->max_rx_thres = max_thres(mcbsp) - 0x10; - ret = sysfs_create_group(&mcbsp->dev->kobj, - &additional_attr_group); + ret = devm_device_add_group(mcbsp->dev, &additional_attr_group); if (ret) { dev_err(mcbsp->dev, "Unable to create additional controls\n"); @@ -711,16 +710,7 @@ static int omap_mcbsp_init(struct platform_device *pdev) } } - ret = omap_mcbsp_st_init(pdev); - if (ret) - goto err_st; - - return 0; - -err_st: - if (mcbsp->pdata->buffer_size) - sysfs_remove_group(&mcbsp->dev->kobj, &additional_attr_group); - return ret; + return omap_mcbsp_st_init(pdev); } /* @@ -1431,11 +1421,6 @@ static int asoc_mcbsp_remove(struct platform_device *pdev) if (cpu_latency_qos_request_active(&mcbsp->pm_qos_req)) cpu_latency_qos_remove_request(&mcbsp->pm_qos_req); - if (mcbsp->pdata->buffer_size) - sysfs_remove_group(&mcbsp->dev->kobj, &additional_attr_group); - - omap_mcbsp_st_cleanup(pdev); - return 0; } -- 2.34.1

3 2

[PATCH v3] ASoC: ti: omap-mcbsp: duplicate sysfs error
by David Owens 27 Jun '22

27 Jun '22

Convert to managed versions of sysfs and clk allocation to simplify unbinding and error handling in probe. Managed sysfs node creation specifically addresses the following error seen the second time probe is attempted after sdma_pcm_platform_register() previously requested probe deferral: sysfs: cannot create duplicate filename '/devices/platform/68000000.ocp/49022000.mcbsp/max_tx_thres' Signed-off-by: David Owens <dowens(a)precisionplanting.com> --- Changes in v3: * Whitespace changes only to allow clean applly Changes in v2: * Improved error handling --- sound/soc/ti/omap-mcbsp-priv.h | 2 -- sound/soc/ti/omap-mcbsp-st.c | 21 ++++----------------- sound/soc/ti/omap-mcbsp.c | 26 ++++---------------------- 3 files changed, 8 insertions(+), 41 deletions(-) diff --git a/sound/soc/ti/omap-mcbsp-priv.h b/sound/soc/ti/omap-mcbsp-priv.h index 7865cda4bf0a..da519ea1f303 100644 --- a/sound/soc/ti/omap-mcbsp-priv.h +++ b/sound/soc/ti/omap-mcbsp-priv.h @@ -316,8 +316,6 @@ static inline int omap_mcbsp_read(struct omap_mcbsp *mcbsp, u16 reg, /* Sidetone specific API */ int omap_mcbsp_st_init(struct platform_device *pdev); -void omap_mcbsp_st_cleanup(struct platform_device *pdev); - int omap_mcbsp_st_start(struct omap_mcbsp *mcbsp); int omap_mcbsp_st_stop(struct omap_mcbsp *mcbsp); diff --git a/sound/soc/ti/omap-mcbsp-st.c b/sound/soc/ti/omap-mcbsp-st.c index 0bc7d26c660a..402a57a502e6 100644 --- a/sound/soc/ti/omap-mcbsp-st.c +++ b/sound/soc/ti/omap-mcbsp-st.c @@ -292,14 +292,11 @@ static ssize_t st_taps_store(struct device *dev, static DEVICE_ATTR_RW(st_taps); -static const struct attribute *sidetone_attrs[] = { +static struct attribute *sidetone_attrs[] = { &dev_attr_st_taps.attr, NULL, }; - -static const struct attribute_group sidetone_attr_group = { - .attrs = (struct attribute **)sidetone_attrs, -}; +ATTRIBUTE_GROUPS(sidetone); int omap_mcbsp_st_start(struct omap_mcbsp *mcbsp) { @@ -347,7 +344,7 @@ int omap_mcbsp_st_init(struct platform_device *pdev) if (!st_data) return -ENOMEM; - st_data->mcbsp_iclk = clk_get(mcbsp->dev, "ick"); + st_data->mcbsp_iclk = devm_clk_get(mcbsp->dev, "ick"); if (IS_ERR(st_data->mcbsp_iclk)) { dev_warn(mcbsp->dev, "Failed to get ick, sidetone might be broken\n"); @@ -359,7 +356,7 @@ int omap_mcbsp_st_init(struct platform_device *pdev) if (!st_data->io_base_st) return -ENOMEM; - ret = sysfs_create_group(&mcbsp->dev->kobj, &sidetone_attr_group); + ret = devm_device_add_group(mcbsp->dev, &sidetone_group); if (ret) return ret; @@ -368,16 +365,6 @@ int omap_mcbsp_st_init(struct platform_device *pdev) return 0; } -void omap_mcbsp_st_cleanup(struct platform_device *pdev) -{ - struct omap_mcbsp *mcbsp = platform_get_drvdata(pdev); - - if (mcbsp->st_data) { - sysfs_remove_group(&mcbsp->dev->kobj, &sidetone_attr_group); - clk_put(mcbsp->st_data->mcbsp_iclk); - } -} - static int omap_mcbsp_st_info_volsw(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_info *uinfo) { diff --git a/sound/soc/ti/omap-mcbsp.c b/sound/soc/ti/omap-mcbsp.c index 4479d74f0a45..395493a2d965 100644 --- a/sound/soc/ti/omap-mcbsp.c +++ b/sound/soc/ti/omap-mcbsp.c @@ -595,16 +595,13 @@ static ssize_t dma_op_mode_store(struct device *dev, static DEVICE_ATTR_RW(dma_op_mode); -static const struct attribute *additional_attrs[] = { +static struct attribute *additional_attrs[] = { &dev_attr_max_tx_thres.attr, &dev_attr_max_rx_thres.attr, &dev_attr_dma_op_mode.attr, NULL, }; - -static const struct attribute_group additional_attr_group = { - .attrs = (struct attribute **)additional_attrs, -}; +ATTRIBUTE_GROUPS(additional); /* * McBSP1 and McBSP3 are directly mapped on 1610 and 1510. @@ -702,8 +699,7 @@ static int omap_mcbsp_init(struct platform_device *pdev) mcbsp->max_tx_thres = max_thres(mcbsp) - 0x10; mcbsp->max_rx_thres = max_thres(mcbsp) - 0x10; - ret = sysfs_create_group(&mcbsp->dev->kobj, - &additional_attr_group); + ret = devm_device_add_group(mcbsp->dev, &additional_group); if (ret) { dev_err(mcbsp->dev, "Unable to create additional controls\n"); @@ -711,16 +707,7 @@ static int omap_mcbsp_init(struct platform_device *pdev) } } - ret = omap_mcbsp_st_init(pdev); - if (ret) - goto err_st; - - return 0; - -err_st: - if (mcbsp->pdata->buffer_size) - sysfs_remove_group(&mcbsp->dev->kobj, &additional_attr_group); - return ret; + return omap_mcbsp_st_init(pdev); } /* @@ -1431,11 +1418,6 @@ static int asoc_mcbsp_remove(struct platform_device *pdev) if (cpu_latency_qos_request_active(&mcbsp->pm_qos_req)) cpu_latency_qos_remove_request(&mcbsp->pm_qos_req); - if (mcbsp->pdata->buffer_size) - sysfs_remove_group(&mcbsp->dev->kobj, &additional_attr_group); - - omap_mcbsp_st_cleanup(pdev); - return 0; } -- 2.34.1

4 6

Re: [PATCH v2] ASoC: ti: omap-mcbsp: duplicate sysfs error
by Mark Brown 27 Jun '22

27 Jun '22

On Wed, 15 Jun 2022 10:09:55 -0500, David Owens wrote: > Convert to managed versions of sysfs and clk allocation to simplify > unbinding and error handling in probe. Managed sysfs node > creation specifically addresses the following error seen the second time > probe is attempted after sdma_pcm_platform_register() previously requsted > probe deferral: > > sysfs: cannot create duplicate filename '/devices/platform/68000000.ocp/49022000.mcbsp/max_tx_thres' > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [1/1] ASoC: ti: omap-mcbsp: duplicate sysfs error commit: 6d31e225742a1955db8a0c6f6f52beb748a2895e All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark

1 0

[PATCH] drm: amd: amdgpu: fix checkpatch warnings
by Vijendar Mukunda 27 Jun '22

27 Jun '22

From: vijendar <vijendar.mukunda(a)amd.com> Fixed below checkpatch warnings and errors drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:131: CHECK: Comparison to NULL could be written "apd" drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:150: CHECK: Comparison to NULL could be written "apd" drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:196: CHECK: Prefer kernel type 'u64' over 'uint64_t' drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:224: CHECK: Please don't use multiple blank lines drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:226: CHECK: Comparison to NULL could be written "!adev->acp.acp_genpd" drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:233: CHECK: Please don't use multiple blank lines drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:239: CHECK: Alignment should match open parenthesis drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:241: CHECK: Comparison to NULL could be written "!adev->acp.acp_cell" drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:247: CHECK: Comparison to NULL could be written "!adev->acp.acp_res" drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:253: CHECK: Comparison to NULL could be written "!i2s_pdata" drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:350: CHECK: Alignment should match open parenthesis drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c:550: ERROR: that open brace { should be on the previous line Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda(a)amd.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c | 27 +++++++++---------------- 1 file changed, 10 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c index cc9c9f8b23b2..ba1605ff521f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acp.c @@ -128,7 +128,7 @@ static int acp_poweroff(struct generic_pm_domain *genpd) struct amdgpu_device *adev; apd = container_of(genpd, struct acp_pm_domain, gpd); - if (apd != NULL) { + if (apd) { adev = apd->adev; /* call smu to POWER GATE ACP block * smu will @@ -147,7 +147,7 @@ static int acp_poweron(struct generic_pm_domain *genpd) struct amdgpu_device *adev; apd = container_of(genpd, struct acp_pm_domain, gpd); - if (apd != NULL) { + if (apd) { adev = apd->adev; /* call smu to UNGATE ACP block * smu will @@ -193,7 +193,7 @@ static int acp_genpd_remove_device(struct device *dev, void *data) static int acp_hw_init(void *handle) { int r; - uint64_t acp_base; + u64 acp_base; u32 val = 0; u32 count = 0; struct i2s_platform_data *i2s_pdata = NULL; @@ -220,37 +220,32 @@ static int acp_hw_init(void *handle) return -EINVAL; acp_base = adev->rmmio_base; - - adev->acp.acp_genpd = kzalloc(sizeof(struct acp_pm_domain), GFP_KERNEL); - if (adev->acp.acp_genpd == NULL) + if (!adev->acp.acp_genpd) return -ENOMEM; adev->acp.acp_genpd->gpd.name = "ACP_AUDIO"; adev->acp.acp_genpd->gpd.power_off = acp_poweroff; adev->acp.acp_genpd->gpd.power_on = acp_poweron; - - adev->acp.acp_genpd->adev = adev; pm_genpd_init(&adev->acp.acp_genpd->gpd, NULL, false); - adev->acp.acp_cell = kcalloc(ACP_DEVS, sizeof(struct mfd_cell), - GFP_KERNEL); + adev->acp.acp_cell = kcalloc(ACP_DEVS, sizeof(struct mfd_cell), GFP_KERNEL); - if (adev->acp.acp_cell == NULL) { + if (!adev->acp.acp_cell) { r = -ENOMEM; goto failure; } adev->acp.acp_res = kcalloc(5, sizeof(struct resource), GFP_KERNEL); - if (adev->acp.acp_res == NULL) { + if (!adev->acp.acp_res) { r = -ENOMEM; goto failure; } i2s_pdata = kcalloc(3, sizeof(struct i2s_platform_data), GFP_KERNEL); - if (i2s_pdata == NULL) { + if (!i2s_pdata) { r = -ENOMEM; goto failure; } @@ -346,8 +341,7 @@ static int acp_hw_init(void *handle) adev->acp.acp_cell[3].platform_data = &i2s_pdata[2]; adev->acp.acp_cell[3].pdata_size = sizeof(struct i2s_platform_data); - r = mfd_add_hotplug_devices(adev->acp.parent, adev->acp.acp_cell, - ACP_DEVS); + r = mfd_add_hotplug_devices(adev->acp.parent, adev->acp.acp_cell, ACP_DEVS); if (r) goto failure; @@ -546,8 +540,7 @@ static const struct amd_ip_funcs acp_ip_funcs = { .set_powergating_state = acp_set_powergating_state, }; -const struct amdgpu_ip_block_version acp_ip_block = -{ +const struct amdgpu_ip_block_version acp_ip_block = { .type = AMD_IP_BLOCK_TYPE_ACP, .major = 2, .minor = 2, -- 2.25.1

2 2

Re: Sound not working after commit bbf7d3b1c4f40eb02dd1dffb500ba00b0bff0303 on Amlogic A311D device
by Pierre-Louis Bossart 27 Jun '22

27 Jun '22

[Adding alsa-devel] On 6/26/22 04:29, Alex Natalsson wrote: > Hello friends. > I have Khadas VIM3 SBC with Amlogic A311D SOC device. > After update kernel my media software (alsamixer, mplayer, ffplay and > etc.) not responding after launch and not terminating throught kill > -9. > > The log after alsamixer launch: > [ 103.786358] Internal error: Oops: 96000004 [#1] SMP > [ 103.786509] Modules linked in: snd_seq_dummy snd_hrtimer snd_seq > snd_seq_device bnep nls_utf8 mt7601u mac80211 hci_uart libarc4 joydev > brcmfmac btbcm panfrost bluetooth meson_vdec(C) brcmutil gpu_sched > cfg80211 dwmac_generic videobuf2_dma_contig v4l2_mem2mem > videobuf2_memops videobuf2_v4l2 videobuf2_common drm_shmem_helper > ecdh_generic dwmac_meson8b stmmac_platform dw_hdmi_i2s_audio stmmac > ecc ir_nec_decoder videodev pcs_xpcs rc_khadas phylink ptp > meson_saradc meson_ir rc_core snd_soc_meson_axg_frddr > snd_soc_meson_axg_toddr snd_soc_meson_axg_sound_card > snd_soc_meson_card_utils snd_soc_meson_axg_fifo ao_cec_g12a > snd_soc_meson_axg_tdmin snd_soc_meson_axg_tdm_interface > snd_soc_meson_axg_tdmout snd_soc_meson_axg_tdm_formatter rfkill > meson_gxbb_wdt fuse crypto_user uas usb_storage adc_keys > gpio_keys_polled industrialio > [ 103.857778] CPU: 3 PID: 530 Comm: pipewire-media- Tainted: G > C 5.16.0-ARCH+ #8 Any chance you could try with an updated kernel? It's not clear to me if this is a real issue or just a warning that's been fixed since by commit. bbf7d3b1c4f40 "ASoC: soc-pcm: Fix DPCM lockdep warning due to nested stream locks" You may also want to open a bugzilla issue to attach the full dmesg log and your configuration reported by 'alsa-info' > [ 103.857791] Hardware name: Khadas VIM3 (DT) > [ 103.857795] pstate: 60000005 (nZCv daif -PAN -UAO -TCO -DIT -SSBS BTYPE=--) > [ 103.857801] pc : dpcm_be_connect+0xd0/0x2d0 > [ 103.881758] lr : dpcm_add_paths.isra.0+0xcc/0x1e0 > [ 103.881763] sp : ffff800009f4b870 > [ 103.881765] x29: ffff800009f4b870 x28: ffff000003b944c0 x27: 0000000000000001 > [ 103.881772] x26: 0000000000000001 x25: 0000000000000340 x24: 00000000000002a0 > [ 103.881777] x23: 0000000000000001 x22: 0000000000000000 x21: ffff000005b18080 > [ 103.881782] x20: ffff000005b183c0 x19: ffff000005b1c080 x18: 0000000000000000 > [ 103.881787] x17: 0000000000000000 x16: 0000000000000000 x15: 0000000000000000 > [ 103.881792] x14: 0000000000000000 x13: 0000000000000000 x12: 0000000000000000 > [ 103.881797] x11: 0000000000000000 x10: 0000000000000000 x9 : 0000000000000000 > [ 103.881801] x8 : ffff000012020400 x7 : 0000000000000000 x6 : 000000000000003f > [ 103.881806] x5 : 0000000000000040 x4 : 0000000000000000 x3 : ffff000002822000 > [ 103.881811] x2 : ffff000002861b50 x1 : ffff00000282f800 x0 : ffff000002861800 > [ 103.881817] Call trace: > [ 103.881819] dpcm_be_connect+0xd0/0x2d0 > [ 103.881823] dpcm_add_paths.isra.0+0xcc/0x1e0 > [ 103.881826] dpcm_fe_dai_open+0x80/0x194 > [ 103.881831] snd_pcm_open_substream+0xa4/0x174 > [ 103.881838] snd_pcm_open.part.0+0xd8/0x1dc > [ 103.881843] snd_pcm_capture_open+0x64/0x94 > [ 103.992730] snd_open+0xac/0x1d0 > [ 103.992739] chrdev_open+0xe0/0x2d0 > [ 103.999389] do_dentry_open+0x12c/0x3a0 > [ 103.999397] vfs_open+0x30/0x3c > [ 103.999401] do_open+0x204/0x414 > [ 103.999405] path_openat+0x10c/0x27c > [ 103.999408] do_filp_open+0x80/0x130 > [ 103.999412] do_sys_openat2+0xb4/0x170 > [ 103.999414] __arm64_sys_openat+0x64/0xb0 > [ 103.999416] invoke_syscall+0x48/0x114 > [ 103.999426] el0_svc_common.constprop.0+0xd4/0xfc > [ 103.999429] do_el0_svc+0x28/0x90 > [ 103.999432] el0_svc+0x28/0x80 > > After I removed pipewire software the error message was > disappear. But the sound and media still doesn't working and alsamixer > cannot change sound card to my usb audio device.

2 1

[PATCH v2 0/2] Read _SUB from ACPI to be able to identify firmware
by Stefan Binding 27 Jun '22

27 Jun '22

CS35L41 has a DSP which is able to run firmware, as well as a tuning file. Different systems may want to use different firmwares and tuning files, and some firmwares/tunings may not be compatible with other systems. To allow a system to select the correct fimware/tuning, we can read an _SUB from the ACPI. This _SUB can then be used to uniquely identify the system in the firmware/tuning file name. Add a helper function which reads the _SUB, so this can be used by other parts in the future. Add support inside the CS35L41 ASoC driver to read this _SUB, and save it appropriately. Changes since v1: - Add length validation for SSID String - Rename API - Allocate memory inside API - Use ACPI_HANDLE macro instead of ACPI_COMPANION - Improve error handling Stefan Binding (2): ACPI: utils: Add api to read _SUB from ACPI ASoC: cs35l41: Read System Name from ACPI _SUB to identify firmware drivers/acpi/utils.c | 38 ++++++++++++++++++++++++++++++++++++++ include/linux/acpi.h | 6 ++++++ sound/soc/codecs/cs35l41.c | 24 ++++++++++++++++++++++++ 3 files changed, 68 insertions(+) -- 2.25.1

2 3

Pandora audio using machine_is_omap3_pandora() check
by Tony Lindgren 27 Jun '22

27 Jun '22

Hi Grazvydas, Arnd noticed that pandora audio is using machine_is_omap3_pandora() check that never succeeds for devicetree booting machines. Looks like this has been broken at some point many years ago with the devicetree conversion. Does anybody have an interest in fixing this driver? Regards, Tony

2 2

[PATCH 0/5] seq: code change for cell helper functionalities
by GitHub pull_request - opened 27 Jun '22

27 Jun '22

alsa-project/alsa-gobject pull request #80 was opened from takaswie: In design of ALSA Sequencer, cell is a unit to deliver events. This patchset adds code change for helper functionalities. ``` Takashi Sakamoto (5): event: add public API to compute pool consumption when scheduling event client-pool: change signedness for size properties user-client/client-pool: update function comments about event scheduling all: replace 'represent' with 'express' in comments ci: separate libudev-dev installation from the other .github/workflows/build.yml | 3 +- doc/ctl/overview.md | 2 +- doc/rawmidi/overview.md | 2 +- doc/seq/overview.md | 6 +-- doc/timer/overview.md | 2 +- src/ctl/card-info.c | 2 +- src/ctl/card.c | 6 +-- src/ctl/elem-id.c | 4 +- src/hwdep/device-info.c | 4 +- src/rawmidi/stream-pair.c | 6 +-- src/rawmidi/substream-info.c | 4 +- src/rawmidi/substream-params.c | 4 +- src/rawmidi/substream-status.c | 4 +- src/seq/addr.c | 4 +- src/seq/alsaseq.map | 1 + src/seq/client-info.c | 4 +- src/seq/client-pool.c | 91 ++++++++++++++++++---------------- src/seq/event-data-connect.c | 4 +- src/seq/event-data-ctl.c | 4 +- src/seq/event-data-note.c | 4 +- src/seq/event-data-queue.c | 4 +- src/seq/event-data-result.c | 4 +- src/seq/event.c | 16 ++++++ src/seq/event.h | 2 + src/seq/port-info.c | 4 +- src/seq/queue-info.c | 4 +- src/seq/queue-status.c | 4 +- src/seq/queue-tempo.c | 4 +- src/seq/remove-filter.c | 4 +- src/seq/subscribe-data.c | 4 +- src/seq/system-info.c | 4 +- src/seq/user-client.c | 16 ++++-- src/timer/device-id.c | 4 +- src/timer/device-info.c | 4 +- src/timer/device-params.c | 4 +- src/timer/device-status.c | 4 +- src/timer/instance-info.c | 4 +- src/timer/instance-params.c | 4 +- src/timer/instance-status.c | 2 +- src/timer/user-instance.c | 6 +-- tests/alsaseq-event | 1 + 41 files changed, 148 insertions(+), 116 deletions(-) ``` Request URL : https://github.com/alsa-project/alsa-gobject/pull/80 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/80.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

[PATCH v5 0/2] Add pinctrl support adsp bypass platforms
by Srinivasa Rao Mandadapu 27 Jun '22

27 Jun '22

This patch set is to make clock voting optinal for adsp bypass sc7280 platforms. Changes Since V4: -- Fix the improper device node variable usage. -- Optimize the code by removing redundant private variant data structure variable "is_clk_optional". Changes Since V3: -- Revert compatible type change in dt bindings. -- Update boolean flag description in dt bindings. -- Drop redundant clock optional flag initialization. -- Remove redundant if check. Changes Since V2: -- Remove redundant lpi pincontrol variant data structure and compatible entry. -- Add adsp bypass mode boolean param check. -- Remove compatible name in dt bindings. -- Update dt binding bypass mode boolean param. Changes Since V1: -- Update commit message. Srinivasa Rao Mandadapu (2): dt-bindings: pinctrl: qcom: sc7280: Add boolean param for ADSP bypass platforms pinctrl: qcom: sc7280: Add clock optional check for ADSP bypass targets .../devicetree/bindings/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml | 5 +++++ drivers/pinctrl/qcom/pinctrl-lpass-lpi.c | 2 +- drivers/pinctrl/qcom/pinctrl-lpass-lpi.h | 1 - drivers/pinctrl/qcom/pinctrl-sc7280-lpass-lpi.c | 1 - 4 files changed, 6 insertions(+), 3 deletions(-) -- 2.7.4

1 2

[PATCH 0/2] Update machine driver for audioreach solution
by Srinivasa Rao Mandadapu 27 Jun '22

27 Jun '22

This patch set is to update sc7280 machine driver for audioreach support. Srinivasa Rao Mandadapu (2): ASoC: google: dt-bindings: Update sc7280-audioreach-herobrine compatible name ASoC: qcom: Add driver support for audioreach solution .../bindings/sound/google,sc7280-herobrine.yaml | 1 + sound/soc/qcom/sc7280.c | 34 ++++++++++++++++++++++ 2 files changed, 35 insertions(+) -- 2.7.4

2 4