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

[PATCH 0/8] iio: accel: bmc150: Add support for yoga's with dual accelerometers with an ACPI HID of DUAL250E
by Hans de Goede 23 May '21

23 May '21

Hi All, Some 360 degree hinges (yoga) style 2-in-1 devices use 2 bmc150 accels to allow the OS to determine the angle between the display and the base of the device, so that the OS can determine if the 2-in-1 is in laptop or in tablet-mode. We already support this setup on devices using a single ACPI node with a HID of "BOSC0200" to describe both accelerometers. This patch set extends this support to also support the same setup but then using a HID of "DUAL250E". While testing this I found some crashes on rmmod, patches 1-2 fix those patches, patch 3 does some refactoring and patch 4 adds support for the "DUAL250E" HID. Unfortunately we need some more special handling though, which the rest of the patches are for. On Windows both accelerometers are read (polled) by a special service and this service calls a DSM (Device Specific Method), which in turn translates the angles to one of laptop/tablet/tent/stand mode and then notifies the EC about the new mode and the EC then enables or disables the builtin keyboard and touchpad based in the mode. When the 2-in-1 is powered-on or resumed folded in tablet mode the EC senses this independent of the DSM by using a HALL effect sensor which senses that the keyboard has been folded away behind the display. At power-on or resume the EC disables the keyboard based on this and the only way to get the keyboard to work after this is to call the DSM to re-enable it (similar to how we also need to call a special DSM in the kxcjk-1013.c accel driver to re-enable the keyboard). Patches 5-7 deal with the DSM mess and patch 8 adds labels to the 2 accelerometers specifying which one is which. Regards, Hans Hans de Goede (8): iio: accel: bmc150: Fix dereferencing the wrong pointer in bmc150_get/set_second_device iio: accel: bmc150: Don't make the remove function of the second accelerometer unregister itself iio: accel: bmc150: Move check for second ACPI device into a separate function iio: accel: bmc150: Add support for dual-accelerometers with a DUAL250E HID iio: accel: bmc150: Move struct bmc150_accel_data definition to bmc150-accel.h iio: accel: bmc150: Remove bmc150_set/get_second_device() accessor functions iio: accel: bmc150: Add support for DUAL250E ACPI DSM for setting the hinge angle iio: accel: bmc150: Set label based on accel-location for ACPI DUAL250E fwnodes drivers/iio/accel/bmc150-accel-core.c | 87 ++---------- drivers/iio/accel/bmc150-accel-i2c.c | 192 +++++++++++++++++++++----- drivers/iio/accel/bmc150-accel.h | 66 ++++++++- 3 files changed, 239 insertions(+), 106 deletions(-) -- 2.31.1

3 28

[sound:for-linus 117/119] sound/usb/mixer_scarlett_gen2.c:2000:5: sparse: sparse: symbol 'snd_scarlett_gen2_controls_create' was not declared. Should it be static?
by kernel test robot 23 May '21

23 May '21

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-linus head: 9ebaef0540a981093bce5df15af32354d32391d9 commit: 265d1a90e4fb6d3264d8122fbd10760e5e733be6 [117/119] ALSA: usb-audio: scarlett2: Improve driver startup messages config: x86_64-randconfig-s021-20210522 (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.3-341-g8af24329-dirty # https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?id=… git remote add sound https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git git fetch --no-tags sound for-linus git checkout 265d1a90e4fb6d3264d8122fbd10760e5e733be6 # save the attached .config to linux build tree make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' W=1 ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com> sparse warnings: (new ones prefixed by >>) >> sound/usb/mixer_scarlett_gen2.c:2000:5: sparse: sparse: symbol 'snd_scarlett_gen2_controls_create' was not declared. Should it be static? Please review and possibly fold the followup patch. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

3 5

[sound:for-linus 117/119] sound/usb/mixer_scarlett_gen2.c:2000:5: warning: no previous prototype for function 'snd_scarlett_gen2_controls_create'
by kernel test robot 22 May '21

22 May '21

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-linus head: 9ebaef0540a981093bce5df15af32354d32391d9 commit: 265d1a90e4fb6d3264d8122fbd10760e5e733be6 [117/119] ALSA: usb-audio: scarlett2: Improve driver startup messages config: x86_64-randconfig-a014-20210522 (attached as .config) compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project e84a9b9bb3051c35dea993cdad7b3d2575638f85) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?id=… git remote add sound https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git git fetch --no-tags sound for-linus git checkout 265d1a90e4fb6d3264d8122fbd10760e5e733be6 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com> All warnings (new ones prefixed by >>): >> sound/usb/mixer_scarlett_gen2.c:2000:5: warning: no previous prototype for function 'snd_scarlett_gen2_controls_create' [-Wmissing-prototypes] int snd_scarlett_gen2_controls_create(struct usb_mixer_interface *mixer, ^ sound/usb/mixer_scarlett_gen2.c:2000:1: note: declare 'static' if the function is not intended to be used outside of this translation unit int snd_scarlett_gen2_controls_create(struct usb_mixer_interface *mixer, ^ static 1 warning generated. vim +/snd_scarlett_gen2_controls_create +2000 sound/usb/mixer_scarlett_gen2.c 1999 > 2000 int snd_scarlett_gen2_controls_create(struct usb_mixer_interface *mixer, 2001 const struct scarlett2_device_info *info) 2002 { 2003 int err; 2004 2005 /* Initialise private data, routing, sequence number */ 2006 err = scarlett2_init_private(mixer, info); 2007 if (err < 0) 2008 return err; 2009 2010 /* Read volume levels and controls from the interface */ 2011 err = scarlett2_read_configs(mixer); 2012 if (err < 0) 2013 return err; 2014 2015 /* Create the analogue output controls */ 2016 err = scarlett2_add_line_out_ctls(mixer); 2017 if (err < 0) 2018 return err; 2019 2020 /* Create the analogue input controls */ 2021 err = scarlett2_add_line_in_ctls(mixer); 2022 if (err < 0) 2023 return err; 2024 2025 /* Create the input, output, and mixer mux input selections */ 2026 err = scarlett2_add_mux_enums(mixer); 2027 if (err < 0) 2028 return err; 2029 2030 /* Create the matrix mixer controls */ 2031 err = scarlett2_add_mixer_ctls(mixer); 2032 if (err < 0) 2033 return err; 2034 2035 /* Create the level meter controls */ 2036 err = scarlett2_add_meter_ctl(mixer); 2037 if (err < 0) 2038 return err; 2039 2040 /* Set up the interrupt polling if there are hardware buttons */ 2041 if (info->button_count) { 2042 err = scarlett2_mixer_status_create(mixer); 2043 if (err < 0) 2044 return err; 2045 } 2046 2047 return 0; 2048 } 2049 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

1 0

[sound:for-linus 117/119] sound/usb/mixer_scarlett_gen2.c:2000:5: warning: no previous prototype for 'snd_scarlett_gen2_controls_create'
by kernel test robot 22 May '21

22 May '21

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-linus head: 9ebaef0540a981093bce5df15af32354d32391d9 commit: 265d1a90e4fb6d3264d8122fbd10760e5e733be6 [117/119] ALSA: usb-audio: scarlett2: Improve driver startup messages config: arc-allyesconfig (attached as .config) compiler: arceb-elf-gcc (GCC) 9.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?id=… git remote add sound https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git git fetch --no-tags sound for-linus git checkout 265d1a90e4fb6d3264d8122fbd10760e5e733be6 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com> All warnings (new ones prefixed by >>): >> sound/usb/mixer_scarlett_gen2.c:2000:5: warning: no previous prototype for 'snd_scarlett_gen2_controls_create' [-Wmissing-prototypes] 2000 | int snd_scarlett_gen2_controls_create(struct usb_mixer_interface *mixer, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ vim +/snd_scarlett_gen2_controls_create +2000 sound/usb/mixer_scarlett_gen2.c 1999 > 2000 int snd_scarlett_gen2_controls_create(struct usb_mixer_interface *mixer, 2001 const struct scarlett2_device_info *info) 2002 { 2003 int err; 2004 2005 /* Initialise private data, routing, sequence number */ 2006 err = scarlett2_init_private(mixer, info); 2007 if (err < 0) 2008 return err; 2009 2010 /* Read volume levels and controls from the interface */ 2011 err = scarlett2_read_configs(mixer); 2012 if (err < 0) 2013 return err; 2014 2015 /* Create the analogue output controls */ 2016 err = scarlett2_add_line_out_ctls(mixer); 2017 if (err < 0) 2018 return err; 2019 2020 /* Create the analogue input controls */ 2021 err = scarlett2_add_line_in_ctls(mixer); 2022 if (err < 0) 2023 return err; 2024 2025 /* Create the input, output, and mixer mux input selections */ 2026 err = scarlett2_add_mux_enums(mixer); 2027 if (err < 0) 2028 return err; 2029 2030 /* Create the matrix mixer controls */ 2031 err = scarlett2_add_mixer_ctls(mixer); 2032 if (err < 0) 2033 return err; 2034 2035 /* Create the level meter controls */ 2036 err = scarlett2_add_meter_ctl(mixer); 2037 if (err < 0) 2038 return err; 2039 2040 /* Set up the interrupt polling if there are hardware buttons */ 2041 if (info->button_count) { 2042 err = scarlett2_mixer_status_create(mixer); 2043 if (err < 0) 2044 return err; 2045 } 2046 2047 return 0; 2048 } 2049 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

1 0

[PATCH 0/6] ALSA: firewire-lib: code refactoring for processing rx packets
by Takashi Sakamoto 22 May '21

22 May '21

Hi, This patchset refactors mainly for processing rx packets. Except for devices handled by ALSA bebob and dice drivers, the devices doesn't take care of the value of syt field of CIP header for playback timing. To code it explicitly, the new flag is added in 1st patch. Some of devices based on DICE ASICs supports two pairs of packet streams. In the case, the sequence of rx packet in the streams should be different independently so that each of them deliver timing information from each peer. On the other hand, current implementation pools the sequence in AMDTP domain, then the rx packets are processed with the same pool. It is inconvenient and in 2nd patch the pool is prepared per streams. The other patches are for my future work. Regards Takashi Sakamoto (6): ALSA: firewire-lib: add flag to unaware of syt in CIP header ALSA: firewire-lib: pool sequence of packet in IT context independently ALSA: firewire-lib: code refactoring for generation of packet descriptors ALSA: firewire-lib: code refactoring for generation of syt sequence ALSA: firewire-lib: code refactoring for generation of data block sequence ALSA: firewire-lib: code refactoring for transfer delay sound/firewire/amdtp-stream.c | 267 +++++++++++++-------------- sound/firewire/amdtp-stream.h | 33 ++-- sound/firewire/motu/amdtp-motu.c | 4 +- sound/firewire/tascam/amdtp-tascam.c | 6 +- 4 files changed, 146 insertions(+), 164 deletions(-) -- 2.27.0

2 7

[PATCH] ALSA: hda/realtek: the bass speaker can't output sound on Yoga 9i
by Hui Wang 22 May '21

22 May '21

The Lenovo Yoga 9i has bass speaker, but the bass speaker can't work, that is because there is an i2s amplifier on that speaker, need to run ideapad_s740_coef() to initialize the amplifier. And also needs to apply ALC285_FIXUP_THINKPAD_HEADSET_JACK to rename the speaker's mixer control name, otherwise the PA can't handle them. BugLink: http://bugs.launchpad.net/bugs/1926165 Signed-off-by: Hui Wang <hui.wang(a)canonical.com> --- sound/pci/hda/patch_realtek.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 4eab336dc739..c6b5db831ed0 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -6552,6 +6552,7 @@ enum { ALC295_FIXUP_ASUS_DACS, ALC295_FIXUP_HP_OMEN, ALC285_FIXUP_HP_SPECTRE_X360, + ALC287_FIXUP_IDEAPAD_BASS_SPK_AMP, }; static const struct hda_fixup alc269_fixups[] = { @@ -8118,6 +8119,12 @@ static const struct hda_fixup alc269_fixups[] = { .chained = true, .chain_id = ALC285_FIXUP_SPEAKER2_TO_DAC1, }, + [ALC287_FIXUP_IDEAPAD_BASS_SPK_AMP] = { + .type = HDA_FIXUP_FUNC, + .v.func = alc285_fixup_ideapad_s740_coef, + .chained = true, + .chain_id = ALC285_FIXUP_THINKPAD_HEADSET_JACK, + }, }; static const struct snd_pci_quirk alc269_fixup_tbl[] = { @@ -8486,6 +8493,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), SND_PCI_QUIRK(0x17aa, 0x3818, "Lenovo C940", ALC298_FIXUP_LENOVO_SPK_VOLUME), SND_PCI_QUIRK(0x17aa, 0x3827, "Ideapad S740", ALC285_FIXUP_IDEAPAD_S740_COEF), + SND_PCI_QUIRK(0x17aa, 0x3843, "Yoga 9i", ALC287_FIXUP_IDEAPAD_BASS_SPK_AMP), SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", ALC269_FIXUP_DMIC_THINKPAD_ACPI), SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", ALC283_FIXUP_INT_MIC), SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -8701,6 +8709,7 @@ static const struct hda_model_fixup alc269_fixup_models[] = { {.id = ALC245_FIXUP_HP_X360_AMP, .name = "alc245-hp-x360-amp"}, {.id = ALC295_FIXUP_HP_OMEN, .name = "alc295-hp-omen"}, {.id = ALC285_FIXUP_HP_SPECTRE_X360, .name = "alc285-hp-spectre-x360"}, + {.id = ALC287_FIXUP_IDEAPAD_BASS_SPK_AMP, .name = "alc287-ideapad-bass-spk-amp"}, {} }; #define ALC225_STANDARD_PINS \ -- 2.25.1

2 1

[PATCH] ALSA: hda/realtek: Headphone volume is controlled by Front mixer
by Hui Wang 22 May '21

22 May '21

On some ASUS and MSI machines, the audio codec is alc1220 and the Headphone is connected to audio mixer 0xf and DAC 0x5, in theory the Headphone volume is controlled by DAC 0x5 (Heapdhone Playback Volume), but somehow it is controlled by DAC 0x2 (Front Playback Volume), maybe this is a defect on the codec alc1220. Because of this issue, the PA couldn't switch the headphone and Lineout correctly, If we apply the quirk CLEVO_P950 to those machines, the Lineout and Headphone will share the audio mixer 0xc and DAC 0x2, and generate Headphone+LO mixer, then PA could handle them when switching between them. BugLink: https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/1206 Cc: <stable(a)vger.kernel.org> Signed-off-by: Hui Wang <hui.wang(a)canonical.com> --- sound/pci/hda/patch_realtek.c | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index c6b5db831ed0..3d40d32ef3ba 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -2603,6 +2603,28 @@ static const struct hda_model_fixup alc882_fixup_models[] = { {} }; +static const struct snd_hda_pin_quirk alc882_pin_fixup_tbl[] = { + SND_HDA_PIN_QUIRK(0x10ec1220, 0x1043, "ASUS", ALC1220_FIXUP_CLEVO_P950, + {0x14, 0x01014010}, + {0x15, 0x01011012}, + {0x16, 0x01016011}, + {0x18, 0x01a19040}, + {0x19, 0x02a19050}, + {0x1a, 0x0181304f}, + {0x1b, 0x0221401f}, + {0x1e, 0x01456130}), + SND_HDA_PIN_QUIRK(0x10ec1220, 0x1462, "MS-7C35", ALC1220_FIXUP_CLEVO_P950, + {0x14, 0x01015010}, + {0x15, 0x01011012}, + {0x16, 0x01011011}, + {0x18, 0x01a11040}, + {0x19, 0x02a19050}, + {0x1a, 0x0181104f}, + {0x1b, 0x0221401f}, + {0x1e, 0x01451130}), + {} +}; + /* * BIOS auto configuration */ @@ -2644,6 +2666,7 @@ static int patch_alc882(struct hda_codec *codec) snd_hda_pick_fixup(codec, alc882_fixup_models, alc882_fixup_tbl, alc882_fixups); + snd_hda_pick_pin_fixup(codec, alc882_pin_fixup_tbl, alc882_fixups, true); snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_PRE_PROBE); alc_auto_parse_customize_define(codec); -- 2.25.1

2 1

[PATCH] ALSA: usb-audio: initialize the cur variable
by trix＠redhat.com 22 May '21

22 May '21

From: Tom Rix <trix(a)redhat.com> Static analysis reports this problem clock.c:338:10: warning: The right operand of '==' is a garbage value if (i == cur) ^ ~~~ In __uac_clock_find_source() the goto find_others skips over setting of the 'cur' variable. Since the pin loop variable starts with 1, it is safe to initialize cur to 0. Signed-off-by: Tom Rix <trix(a)redhat.com> --- sound/usb/clock.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/usb/clock.c b/sound/usb/clock.c index 772478c725c2e..edc458c50bf2c 100644 --- a/sound/usb/clock.c +++ b/sound/usb/clock.c @@ -258,9 +258,10 @@ static int __uac_clock_find_source(struct snd_usb_audio *chip, union uac23_clock_source_desc *source; union uac23_clock_selector_desc *selector; union uac23_clock_multiplier_desc *multiplier; - int ret, i, cur, err, pins, clock_id; + int ret, i, err, pins, clock_id; const u8 *sources; int proto = fmt->protocol; + int cur = 0; entity_id &= 0xff; -- 2.26.3

2 1

[PATCH][next] ALSA: firewire-lib: Fix uninitialized variable err issue
by Colin King 22 May '21

22 May '21

From: Colin Ian King <colin.king(a)canonical.com> Currently in the case where the payload_length is less than the cip_header_size the error return variable err is not being set and function parse_ir_ctx_header can return an uninitialized error return value. Fix this by setting err to zero. Addresses-Coverity: ("Uninitialized scalar variable") Fixes: c09010eeb373 ("ALSA: firewire-lib: handle the case that empty isochronous packet payload for CIP") Signed-off-by: Colin Ian King <colin.king(a)canonical.com> --- sound/firewire/amdtp-stream.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/firewire/amdtp-stream.c b/sound/firewire/amdtp-stream.c index af5c3629f1ac..242b1147d768 100644 --- a/sound/firewire/amdtp-stream.c +++ b/sound/firewire/amdtp-stream.c @@ -663,6 +663,7 @@ static int parse_ir_ctx_header(struct amdtp_stream *s, unsigned int cycle, } else { // Handle the cycle so that empty packet arrives. cip_header = NULL; + err = 0; *data_blocks = 0; *syt = 0; } -- 2.31.1

3 4

Motu M4 - where to report this issue?
by Georges Basile Stavracas Neto 21 May '21

21 May '21

Greetings ALSA community, I recently acquired a Motu M4 interface, and seems to be working well so far. Everyday usage is completely functional. However, I noticed an odd sound artifact that happens from time to time when using DAWs like Ardour and Waveform11. It's a windy, ghostly sound that happens whenever I open any DAWs (the simple act of opening them triggers this). Interestingly, audio apps that use PulseAudio do not suffer from this issue. It's hard to describe it, so I recorded a video reproducing this problem: https://youtu.be/bM8x-YuXLVI A few things I've discovered while trying to understand what's going on: * It only happens with apps that use JACK or ALSA APIs. Using PulseAudio (through PipeWire) does not trigger it. * Seems like this sound is not an artifact of the interface; it appears that this odd sound is sent to the interface through USB. (You can see this in the video.) * It's triggered per track; in the video, for example, the drums and keyboard tracks are affected independetly. I was oriented in #alsa at Freenode that the output of 'alsa-info' might be helpful, so here it is: http://alsa-project.org/db/?f=62f686710cc6c9344a8986707bf3d09248c514d1 (I'm running this on top of a 5.12.5 kernel, which should include this fix: https://bugzilla.kernel.org/show_bug.cgi?id=207023) I'd like to know where should I report this bug (kernel bugzilla?), and if there's any other information I can provide to help fixing it. With respect, Georges

2 1