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

[tiwai-sound:test/pcm-iov_iter 3/25] sound/core/memory.c:108:53: warning: passing argument 2 of 'copy_from_iter' makes integer from pointer without a cast
by kernel test robot 12 Aug '23

12 Aug '23

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git test/pcm-iov_iter head: da1be13e050f24d088d24fd65517d56f27a2eeb1 commit: 516e284c8b17a5129fcfbfb1f2f4e11093e13d3f [3/25] ALSA: core: Add memory copy helpers between iov_iter and iomem config: i386-randconfig-r033-20230812 (https://download.01.org/0day-ci/archive/20230812/202308120931.Aj1gaY7K-lkp@…) compiler: gcc-12 (Debian 12.2.0-14) 12.2.0 reproduce: (https://download.01.org/0day-ci/archive/20230812/202308120931.Aj1gaY7K-lkp@…) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202308120931.Aj1gaY7K-lkp@intel.com/ All warnings (new ones prefixed by >>): sound/core/memory.c: In function 'copy_from_iter_toio': >> sound/core/memory.c:108:53: warning: passing argument 2 of 'copy_from_iter' makes integer from pointer without a cast [-Wint-conversion] 108 | return !copy_from_iter((void __force *)dst, src, count) ? -EFAULT : 0; | ^~~ | | | struct iov_iter * In file included from include/sound/pcm.h:19, from sound/core/memory.c:12: include/linux/uio.h:199:42: note: expected 'size_t' {aka 'unsigned int'} but argument is of type 'struct iov_iter *' 199 | size_t copy_from_iter(void *addr, size_t bytes, struct iov_iter *i) | ~~~~~~~^~~~~ >> sound/core/memory.c:108:58: warning: passing argument 3 of 'copy_from_iter' makes pointer from integer without a cast [-Wint-conversion] 108 | return !copy_from_iter((void __force *)dst, src, count) ? -EFAULT : 0; | ^~~~~ | | | size_t {aka unsigned int} include/linux/uio.h:199:66: note: expected 'struct iov_iter *' but argument is of type 'size_t' {aka 'unsigned int'} 199 | size_t copy_from_iter(void *addr, size_t bytes, struct iov_iter *i) | ~~~~~~~~~~~~~~~~~^ vim +/copy_from_iter +108 sound/core/memory.c 93 94 /** 95 * copy_from_iter_toio - copy data from iov_iter to mmio-space 96 * @dst: the destination pointer on mmio-space 97 * @src: the source iov_iter 98 * @count: the data size to copy in bytes 99 * 100 * Copies the data from iov_iter to mmio-space. 101 * Only the single segment iov_iter is assumed for now. 102 * 103 * Return: Zero if successful, or non-zero on failure. 104 */ 105 int copy_from_iter_toio(void __iomem *dst, struct iov_iter *src, size_t count) 106 { 107 #if defined(__i386__) || defined(CONFIG_SPARC32) > 108 return !copy_from_iter((void __force *)dst, src, count) ? -EFAULT : 0; -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

MSI x570s Carbon Max WiFi ALC 4080
by GitHub issues - edited 11 Aug '23

11 Aug '23

alsa-project/alsa-ucm-conf issue #338 was edited from zpackrat: Fedora 38 patched to current, issue has existed for months. Rear output jacks for Center/Sub and Rear Speakers are reversed. I use a 5.1 speaker setup. When I run sound tests in the gnome audio settings the issue is verifiable. Also audio that requires center channel for dialogue has issues. This is specifically the rear audio I/O ports. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/338 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

MSI x570s Carbon Max WiFi ALC 4080
by GitHub issues - opened 11 Aug '23

11 Aug '23

alsa-project/alsa-ucm-conf issue #338 was opened from zpackrat: Fedora 38 patched to current, issue has existed for months. Rear output jacks for Center/Sub and Rear Speakers are reversed. I use a 5.1 speaker setup. When I run sound tests in the gnome audio settings the issue is verifiable. Also audio that requires center channel for dialogue has issues. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/338 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[RFC] pcm: Add MSBITS subformat options
by GitHub pull_request - edited 11 Aug '23

11 Aug '23

alsa-project/alsa-lib pull request #342 was edited from crojewsk-intel: Improve granularity of format selection for S32/U32 formats by adding masks representing 20, 24 and 32 most significant bits. As a preparation step introduce `snd_pcm_subformat_value()` for easy name-to-subformat conversions. While at it, fix few warnings that pop up in subformat-related code during compilation. Request URL : https://github.com/alsa-project/alsa-lib/pull/342 Patch URL : https://github.com/alsa-project/alsa-lib/pull/342.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[RFC] pcm: Add MSBITS subformat options
by GitHub pull_request - edited 11 Aug '23

11 Aug '23

alsa-project/alsa-lib pull request #342 was edited from crojewsk-intel: Improve granularity of format selection for S32/U32 formats by adding masks representing 20 and 24 most significant bits. As a preparation step introduce `snd_pcm_subformat_value()` for easy name-to-subformat conversions. While at it, fix few warnings that pop up in subformat-related code during compilation. Request URL : https://github.com/alsa-project/alsa-lib/pull/342 Patch URL : https://github.com/alsa-project/alsa-lib/pull/342.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[PATCH v1] ALSA: hda/cs8409: Support new Dell Dolphin Variants
by Stefan Binding 11 Aug '23

11 Aug '23

Add 4 new Dell Dolphin Systems, same configuration as older systems. Signed-off-by: Stefan Binding <sbinding(a)opensource.cirrus.com> --- sound/pci/hda/patch_cs8409-tables.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sound/pci/hda/patch_cs8409-tables.c b/sound/pci/hda/patch_cs8409-tables.c index b288874e401e..36b411d1a960 100644 --- a/sound/pci/hda/patch_cs8409-tables.c +++ b/sound/pci/hda/patch_cs8409-tables.c @@ -550,6 +550,10 @@ const struct snd_pci_quirk cs8409_fixup_tbl[] = { SND_PCI_QUIRK(0x1028, 0x0C50, "Dolphin", CS8409_DOLPHIN), SND_PCI_QUIRK(0x1028, 0x0C51, "Dolphin", CS8409_DOLPHIN), SND_PCI_QUIRK(0x1028, 0x0C52, "Dolphin", CS8409_DOLPHIN), + SND_PCI_QUIRK(0x1028, 0x0C73, "Dolphin", CS8409_DOLPHIN), + SND_PCI_QUIRK(0x1028, 0x0C75, "Dolphin", CS8409_DOLPHIN), + SND_PCI_QUIRK(0x1028, 0x0C7D, "Dolphin", CS8409_DOLPHIN), + SND_PCI_QUIRK(0x1028, 0x0C7F, "Dolphin", CS8409_DOLPHIN), {} /* terminator */ }; -- 2.34.1

2 1

[PATCH v2 0/3] soundwire: allow for more than 8 devices, keep IDA for wake-capable devices
by Bard Liao 11 Aug '23

11 Aug '23

This series suggests a hybrid strategy for device number allocation, where only wake-capable devices use a system-unique Device Number which will be used on LunarLake to handle wake-ups using the HDaudio WAKEEN and WAKESTS. Pierre-Louis Bossart (3): soundwire: extend parameters of new_peripheral_assigned() callback soundwire: bus: add callbacks for device_number allocation soundwire: intel_auxdevice: add hybrid IDA-based device_number allocation v2: - Move the device numver allocation from bus.c to intel_auxdevice.c - List the codecs with wake capability in a quirk table, and assign device number according to that quirk table. drivers/soundwire/bus.c | 18 +++--- drivers/soundwire/intel_auxdevice.c | 85 ++++++++++++++++++++++++++--- include/linux/soundwire/sdw.h | 12 ++-- include/linux/soundwire/sdw_intel.h | 7 +++ 4 files changed, 99 insertions(+), 23 deletions(-) -- 2.25.1

2 4

[PATCH 0/2] soundwire: improve pm_runtime handling
by Bard Liao 11 Aug '23

11 Aug '23

This patchset improves the pm_runtime behavior in rare corner cases identified by the Intel CI in the last 6 months. a) in stress-tests, it's not uncommon to see the following type of warnings when the codec reports as ATTACHED "rt711 sdw:0:025d:0711:00: runtime PM trying to activate child device sdw:0:025d:0711:00 but parent (sdw-master-0) is not active" This warning was not correlated with any functional issue, but it exposed a design issue on when to enable pm_runtime. The recommended practice in the pm_runtime documentation is to keep the devices in 'suspended' mode and mark them as 'active' when they are really functional. Pierre-Louis Bossart (2): soundwire: intel_auxdevice: enable pm_runtime earlier on startup soundWire: intel_auxdevice: resume 'sdw-master' on startup and system resume drivers/soundwire/intel_auxdevice.c | 27 +++++++++++++++++++-------- 1 file changed, 19 insertions(+), 8 deletions(-) -- 2.25.1

3 5

[PATCH -next] ASoC: SOF: Intel: Remove duplicated include in lnl.c
by Yang Li 10 Aug '23

10 Aug '23

./sound/soc/sof/intel/lnl.c: hda.h is included more than once. Signed-off-by: Yang Li <yang.lee(a)linux.alibaba.com> --- sound/soc/sof/intel/lnl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/sof/intel/lnl.c b/sound/soc/sof/intel/lnl.c index 3d919b0b6891..db94b45e53af 100644 --- a/sound/soc/sof/intel/lnl.c +++ b/sound/soc/sof/intel/lnl.c @@ -16,7 +16,6 @@ #include "hda-ipc.h" #include "../sof-audio.h" #include "mtl.h" -#include "hda.h" #include <sound/hda-mlink.h> /* LunarLake ops */ -- 2.20.1.7.g153144c

3 2

[PATCH v1 0/5] Device tree and config updates for the Librem 5 devkit
by Guido Günther 10 Aug '23

10 Aug '23

The device tree updates ensure the A53 don't get powered off and fix a DT warning. This isn't testable with the arm64 default config unless we enable the rsi wifi modules too so do this as well. While at that include two binding file updates. This is against next-20230809 that include David's option,gtm601 conversion. Guido Günther (5): dt-bindings: sound: gtm601: Add description dt-bindings: mmc: Fix reference to pwr-seq-simple arm64: dts: imx8mq-librem5-devkit: Mark buck2 as always on arm64: dts: imx8mq-librem5-devkit: Drop power-supply arm64: defconfig: Enable Redpine 91X wlan driver .../devicetree/bindings/mmc/mmc-controller.yaml | 2 +- .../devicetree/bindings/sound/option,gtm601.yaml | 6 ++++-- .../boot/dts/freescale/imx8mq-librem5-devkit.dts | 14 +++++--------- arch/arm64/configs/defconfig | 1 + 4 files changed, 11 insertions(+), 12 deletions(-) -- 2.40.1

6 10