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

[alsa-devel] [PATCH 1/2] hda-emu: Fix hda_i915 build failure
by David Henningsson 07 Sep '15

07 Sep '15

This allows building hda-emu even after recent i915 -> hda patches. Signed-off-by: David Henningsson <david.henningsson(a)canonical.com> --- include/drm/i915_component.h | 1 + include/sound/hda_i915.h | 1 + 2 files changed, 2 insertions(+) create mode 100644 include/sound/hda_i915.h diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h index e69de29..545c915 100644 --- a/include/drm/i915_component.h +++ b/include/drm/i915_component.h @@ -0,0 +1 @@ +#include "../../dist/include/drm/i915_component.h" diff --git a/include/sound/hda_i915.h b/include/sound/hda_i915.h new file mode 100644 index 0000000..1579430 --- /dev/null +++ b/include/sound/hda_i915.h @@ -0,0 +1 @@ +#include "../../dist/include/sound/hda_i915.h" -- 1.9.1

2 2

[alsa-devel] [PATCH 1/1] ALSA: usb-audio: Change internal PCM order
by johan＠oljud.se 07 Sep '15

07 Sep '15

From: Johan Rastén <johan(a)oljud.se> New PCMs will now be added to the end of the chip's PCM list instead of to the front. This changes the way streams are combined so that the first capture stream will now be merged with the first playback stream instead of the last. This fixes a problem with ASUS U7. Cards with one playback stream and cards without capture streams should be unaffected by this change. Exception added for M-Audio Audiophile USB (tm) since it seems to have a fix to swap capture stream numbering in alsa-lib conf/cards/USB-audio.conf Signed-off-by: Johan Rastén <johan(a)oljud.se> diff --git a/sound/usb/stream.c b/sound/usb/stream.c index 310a382..01f5a86 100644 --- a/sound/usb/stream.c +++ b/sound/usb/stream.c @@ -377,8 +377,15 @@ int snd_usb_add_audio_stream(struct snd_usb_audio *chip, snd_usb_init_substream(as, stream, fp); - list_add(&as->list, &chip->pcm_list); + /* + * Keep using head insertion for M-Audio Audiophile USB (tm) which has a + * fix to swap capture stream order in conf/cards/USB-audio.conf + */ + if (chip->usb_id == USB_ID(0x0763, 0x2003)) + list_add(&as->list, &chip->pcm_list); + else + list_add_tail(&as->list, &chip->pcm_list); + chip->pcm_devs++; snd_usb_proc_pcm_format_add(as); -- 2.1.4

2 1

[alsa-devel] [PATCH] ALSA: usb-audio: Recurse before saving terminal properties
by Julian Scheel 06 Sep '15

06 Sep '15

When the descriptor parser recurses into the clock source descriptor, the terminals properties, like name and type, are set to the values of the clock selector as check_input_term overwrites term->name and others in the recursive call. As we want to only fill missing values using the descriptors higher in the recursion tree the properties taken from the actual descriptor must be applied after recursing. Signed-off-by: Julian Scheel <julian(a)jusst.de> --- sound/usb/mixer.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index 6b3acba..4a49944 100644 --- a/sound/usb/mixer.c +++ b/sound/usb/mixer.c @@ -715,15 +715,16 @@ static int check_input_term(struct mixer_build *state, int id, term->name = d->iTerminal; } else { /* UAC_VERSION_2 */ struct uac2_input_terminal_descriptor *d = p1; - term->type = le16_to_cpu(d->wTerminalType); - term->channels = d->bNrChannels; - term->chconfig = le32_to_cpu(d->bmChannelConfig); - term->name = d->iTerminal; /* call recursively to get the clock selectors */ err = check_input_term(state, d->bCSourceID, term); if (err < 0) return err; + + term->type = le16_to_cpu(d->wTerminalType); + term->channels = d->bNrChannels; + term->chconfig = le32_to_cpu(d->bmChannelConfig); + term->name = d->iTerminal; } return 0; case UAC_FEATURE_UNIT: { -- 2.5.0

4 7

[alsa-devel] [PATCH] ASoC: au1x: psc-i2s: Fix unused variable 'ret' warning
by Axel Lin 06 Sep '15

06 Sep '15

Fix below build warning: sound/soc/au1x/psc-i2s.c: In function 'au1xpsc_i2s_drvprobe': sound/soc/au1x/psc-i2s.c:299:6: warning: unused variable 'ret' [-Wunused-variable] Reported-by: kbuild test robot <fengguang.wu(a)intel.com> Signed-off-by: Axel Lin <axel.lin(a)ingics.com> --- sound/soc/au1x/psc-i2s.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/au1x/psc-i2s.c b/sound/soc/au1x/psc-i2s.c index 38e853a..0bf9d62 100644 --- a/sound/soc/au1x/psc-i2s.c +++ b/sound/soc/au1x/psc-i2s.c @@ -296,7 +296,6 @@ static int au1xpsc_i2s_drvprobe(struct platform_device *pdev) { struct resource *iores, *dmares; unsigned long sel; - int ret; struct au1xpsc_audio_data *wd; wd = devm_kzalloc(&pdev->dev, sizeof(struct au1xpsc_audio_data), -- 2.1.0

2 1

[alsa-devel] [PATCH 1/2] ALSA: hda - Enable headphone jack detect on old Fujitsu laptops
by Takashi Iwai 05 Sep '15

05 Sep '15

According to the bug report, FSC Amilo laptops with ALC880 can detect the headphone jack but currently the driver disables it. It's partly intentionally, as non-working jack detect was reported in the past. Let's enable now. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=102501 Cc: <stable(a)vger.kernel.org> Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/pci/hda/patch_realtek.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 764dd5bba9ee..9b43cb95c047 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -1135,7 +1135,7 @@ static const struct hda_fixup alc880_fixups[] = { /* override all pins as BIOS on old Amilo is broken */ .type = HDA_FIXUP_PINS, .v.pins = (const struct hda_pintbl[]) { - { 0x14, 0x0121411f }, /* HP */ + { 0x14, 0x0121401f }, /* HP */ { 0x15, 0x99030120 }, /* speaker */ { 0x16, 0x99030130 }, /* bass speaker */ { 0x17, 0x411111f0 }, /* N/A */ @@ -1155,7 +1155,7 @@ static const struct hda_fixup alc880_fixups[] = { /* almost compatible with FUJITSU, but no bass and SPDIF */ .type = HDA_FIXUP_PINS, .v.pins = (const struct hda_pintbl[]) { - { 0x14, 0x0121411f }, /* HP */ + { 0x14, 0x0121401f }, /* HP */ { 0x15, 0x99030120 }, /* speaker */ { 0x16, 0x411111f0 }, /* N/A */ { 0x17, 0x411111f0 }, /* N/A */ -- 2.5.1

1 1

[alsa-devel] No sound with Odys Winpad v10 - byt-rt5640
by Moritz 05 Sep '15

05 Sep '15

Hi, I'm having trouble getting sound to work and it looks like it's a bug in the driver. Full description: When loading the kernel-module for Bay Trail SoC TCO device (Version=3, TCOBASE=0x0460) the kernel throws a kernel trace depending on the firmware-image. First I provided the firmware-image from kernel.org: fw_sst_0f28.bin-48kHz_i2s_master With this image the kernel loads the firmware but no sound-devices are found. /proc/asound/cards reports no cards. dmesg just tells [ 11.196018] byt-rt5640 byt-rt5640: ASoC: CPU DAI baytrail-pcm-audio not registered If I'm loading a firmware-image from Intel directly I get a sound-device in /proc but the module failes later with a kernel trace. The download-link to the intel-firmware: https://downloadcenter.intel.com/download/23903 The relevant parts of my dmesg for this problem: [ 11.120805] byt-rt5640 byt-rt5640: ASoC: CPU DAI baytrail-pcm-audio not registered [ 11.214831] iTCO_vendor_support: vendor-support=0 [ 11.215294] iTCO_wdt: Intel TCO WatchDog Timer Driver v1.11 [ 11.215391] iTCO_wdt: Found a Bay Trail SoC TCO device (Version=3, TCOBASE=0x0460) [ 11.215655] iTCO_wdt: initialized. heartbeat=30 sec (nowayout=0) [ 11.215997] byt-rt5640 byt-rt5640: ASoC: CPU DAI baytrail-pcm-audio not registered [ 11.232805] baytrail-pcm-audio baytrail-pcm-audio: FW version: 04.05.12.02 [ 11.232808] baytrail-pcm-audio baytrail-pcm-audio: Build type: 2 [ 11.232811] baytrail-pcm-audio baytrail-pcm-audio: Build date: Jan 16 2014 18:17:00 [ 11.259456] byt-rt5640 byt-rt5640: rt5640-aif1 <-> baytrail-pcm-audio mapping ok [ 11.970293] audit: type=1305 audit(1441466901.770:2): audit_pid=608 old=0 auid=4294967295 ses=4294967295 res=1 [ 12.291031] irq 6: nobody cared (try booting with the "irqpoll" option) [ 12.294817] CPU: 0 PID: 0 Comm: swapper/0 Tainted: G O 4.2.0-4-desktop #1 [ 12.294822] Hardware name: AXDIA International GmbH WINPAD V10/Type2 - Board Product Name, BIOS AD1012.1.03.017 11/21/2014 [ 12.294826] 00000000 00000000 f582df50 c080ecdd f5b3f500 f582df70 c02abab9 c09c9bfc [ 12.294839] 00000006 00000000 f582df6c f5b3f500 00000006 f582df94 c02abf49 f582df94 [ 12.294850] c05f400c fffb910a 00000000 f5b3f500 f5b3f570 00000000 f582dfd0 c02a9a53 [ 12.294861] Call Trace: [ 12.294874] [<c080ecdd>] dump_stack+0x48/0x69 [ 12.294882] [<c02abab9>] __report_bad_irq+0x29/0xd0 [ 12.294888] [<c02abf49>] note_interrupt+0x1c9/0x210 [ 12.294896] [<c05f400c>] ? add_interrupt_randomness+0x15c/0x190 [ 12.294902] [<c02a9a53>] handle_irq_event_percpu+0xa3/0x1c0 [ 12.294908] [<c02a9ba3>] handle_irq_event+0x33/0x50 [ 12.294914] [<c02ac3b5>] handle_fasteoi_irq+0x75/0x130 [ 12.294919] [<c02ac340>] ? handle_simple_irq+0x70/0x70 [ 12.294925] [<c0204b11>] handle_irq+0x71/0x90 [ 12.294929] <IRQ> [<c081603c>] do_IRQ+0x3c/0xd0 [ 12.294941] [<c08155b3>] common_interrupt+0x33/0x38 [ 12.294948] [<c06e753f>] ? cpuidle_enter_state+0xaf/0x2e0 [ 12.294954] [<c06e77a4>] cpuidle_enter+0x14/0x20 [ 12.294961] [<c02923f3>] call_cpuidle+0x33/0x70 [ 12.294967] [<c029268e>] cpu_startup_entry+0x25e/0x320 [ 12.294973] [<c080743b>] rest_init+0x7b/0x80 [ 12.294980] [<c0b34cd8>] start_kernel+0x456/0x45d [ 12.294986] [<c0b3464e>] ? set_init_arg+0x52/0x52 [ 12.294992] [<c0b3438c>] i386_start_kernel+0x13a/0x13d [ 12.294995] handlers: [ 12.298499] [<f8725c80>] sst_byt_irq [snd_soc_sst_baytrail_pcm] threaded [<f8724360>] sst_byt_irq_thread [snd_soc_sst_baytrail_pcm] [ 12.302073] Disabling IRQ #6 with my limited understanding it looks like the irq isn't handled correctly. System information: Linux klappstulle.XXX 4.2.0-4-desktop #1 SMP PREEMPT Mon Aug 31 12:34:37 UTC 2015 (efc468a) i686 i686 i386 GNU/Linux Modules Loaded fuse dm_crypt drbg ansi_cprng algif_skcipher af_alg loop nf_log_ipv6 af_packet xt_pkttype nf_log_ipv4 nf_log_common xt_LOG xt_limit ip6t_REJECT xt_tcpudp nf_conntrack_ipv6 nf_defrag_ipv6 ip6table_raw ipt_REJECT iptable_raw xt_CT iptable_filter ip6table_mangle nf_conntrack_netbios_ns nf_conntrack_broadcast nf_conntrack_ipv4 nf_defrag_ipv4 ip_tables xt_conntrack nf_conntrack ip6table_filter ip6_tables x_tables xfs libcrc32c nls_iso8859_1 nls_cp437 vfat fat joydev hid_multitouch iTCO_wdt iTCO_vendor_support r8723bs snd_soc_sst_baytrail_pcm snd_soc_sst_ipc snd_soc_sst_dsp dw_dmac_core snd_soc_sst_byt_rt5640_mach intel_powerclamp coretemp kvm_intel kvm crc32_pclmul aesni_intel aes_i586 xts lrw gf128mul ablk_helper cryptd pcspkr efivars cfg80211 snd_intel_sst_acpi snd_intel_sst_core snd_soc_rt5640 snd_soc_sst_mfld_platform snd_soc_rl6231 snd_soc_core snd_compress lpc_ich mfd_core snd_pcm thermal mei_txe xhci_pci mei rfkill_gpio xhci_hcd 8250_fintek jsa1212 industrialio i2c_hid regmap_i2c battery rfkill snd_timer snd acpi_pad int3402_thermal int3403_thermal processor_thermal_device int340x_thermal_zone intel_soc_dts_iosf ac int3400_thermal acpi_thermal_rel soundcore ac97_bus snd_soc_sst_acpi i2c_designware_platform i2c_designware_core 8250_dw pwm_lpss_platform pwm_lpss processor dm_mod efivarfs btrfs xor raid6_pq mmc_block crc32c_intel i915 i2c_algo_bit drm_kms_helper drm button video sdhci_acpi sdhci mmc_core sg Package versions gcc-4.8-7.1.2.i586 make-4.0-2.2.3.i586 binutils-2.24-6.1.7.i586 util-linux-2.25.1-17.1.i586 mount from util-linux 2.25.1 (libmount 2.25.0: selinux, assert, debug) glibc-2.19-16.15.1.i686 linux-glibc-devel-3.16-2.1.7.noarch /proc/cmdline BOOT_IMAGE=/boot/vmlinuz-4.2.0-4-desktop root=UUID=fc064bad-46f0-4947-9364-3a7864bb0b6b ro nomodeset \ sdhci.debug_quirks=0x8000 quiet showopts /proc/cpuinfo processor : *** 0 - 3 *** vendor_id : GenuineIntel cpu family : 6 model : 55 model name : Intel(R) Atom(TM) CPU Z3735F @ 1.33GHz stepping : 8 microcode : 0x82a cpu MHz : 1751.642 cache size : 1024 KB physical id : 0 siblings : 4 core id : 0 cpu cores : 4 apicid : 0 initial apicid : 0 fdiv_bug : no f00f_bug : no coma_bug : no fpu : yes fpu_exception : yes cpuid level : 11 wp : yes flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe nx rdtscp lm constant_tsc arch_perfmon pebs bts xtopology nonstop_tsc aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16 xtpr pdcm sse4_1 sse4_2 movbe popcnt tsc_deadline_timer aes rdrand lahf_lm 3dnowprefetch ida arat epb dtherm tpr_shadow vnmi flexpriority ept vpid tsc_adjust smep erms bugs : bogomips : 2662.40 clflush size : 64 cache_alignment : 64 address sizes : 36 bits physical, 48 bits virtual power management: /proc/asound/cards 0 [bytrt5640 ]: byt-rt5640 - byt-rt5640 byt-rt5640 /proc/asound/devices 2: [ 0] : control 3: [ 0- 0]: digital audio playback 4: [ 0- 0]: digital audio capture 33: : timer Please see attached for full dmesg. Kind regards Moritz

1 0

[alsa-devel] [RFC PATCH (alsa-lib)] pcm: Modify check condition in snd_pcm_sw_params_set_avail_min
by Koro Chen 04 Sep '15

04 Sep '15

When we use a ping-ping buffer in capture, and if hw_ptr reported at IRQ is a little smaller than period_size: |xxxxxxxxxxxxxxxxxxxxxxxxxxxx--|-----------------------------| hw_ptr < period_size This available buffer will not be read since its size is smaller than avail_min (which is set to be period_size), and read thread continues to sleep. If the next hw_ptr is just a little larger than buffer_size, overrun occurs. This could be resolved by setting a small avail_min to kernel, for example, 1, so read thread wakes up and reads every data at IRQ. But current alsa-lib only allows avail_min to be at least period_size. Remove the constraint and only check for zero case. Signed-off-by: Koro Chen <koro.chen(a)mediatek.com> --- src/pcm/pcm.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/pcm/pcm.c b/src/pcm/pcm.c index f5fc728..8492689 100644 --- a/src/pcm/pcm.c +++ b/src/pcm/pcm.c @@ -5958,12 +5958,8 @@ int snd_pcm_sw_params_set_avail_min(snd_pcm_t *pcm, snd_pcm_sw_params_t *params, #endif { assert(pcm && params); - /* Fix avail_min if it's below period size. The period_size - * defines the minimal wake-up timing accuracy, so it doesn't - * make sense to set below that. - */ - if (val < pcm->period_size) - val = pcm->period_size; + if (!val) + val = 1; params->avail_min = val; return 0; } -- 1.7.9.5

4 7

[alsa-devel] Tascam US-366 and Alsa Firmware
by Burton Samograd 04 Sep '15

04 Sep '15

Hi, I recently bought the Tascam US-366 and forgot to check for Linux support, which it doesn't have (yet! :). Looking through the Alsa firmware sources, there seems to be a firmware loader called usx2yloader which supports a number of Tascam cards, but the US-366 is not listed. It looks like really usx2yloader just needs the firmware and a `prepad` file. I`ve found the firmware for the card (I think) buried in the OSX firmware download package: http://tascam.com/content/downloads/products/789/us-322_us-366_fw_v103_mave… Now I think I just need the prepad file, but there's little documentation regarding exactly what that is (I'm assuming a number of 0s just to pad the firmware image to alignment, but I'm not sure of what alignment). Any ideas if usx2yloader will work at all with the US-366? Does anyone know what the pre-pad file should be/contain? Has anyone tried to get the US-366 working yet and failed? Reference: http://tascam.com/product/us-366/ Thanks. Burton Samograd

2 2

[alsa-devel] reconfigure cpu dai after resume from suspend
by maruthi srinivas 04 Sep '15

04 Sep '15

Hi, I came across a situation, where I need to reconfigure cpu dai ( similar to cpu dai's hw_params()) after system resumed from suspend. Should this be taken care by soc core or do I need to call the same logic(hw_params()) inside snd_soc_dai_driver's resume() ? Regards, Maruthi

1 0

[alsa-devel] [PATCH] drm/i915: Improve kernel-doc for i915_audio_component struct
by David Henningsson 04 Sep '15

04 Sep '15

To make kernel-doc happy, the i915_audio_component_audio_ops struct cannot be nested. Signed-off-by: David Henningsson <david.henningsson(a)canonical.com> --- Note that I didn't do the same un-nesting for i915_audio_component_ops. This is to make it easier to merge the pending sync_audio_rate patch set. It applies on top of my just sent patches so should probably be taken through Takashi's tree. Documentation/DocBook/drm.tmpl | 1 + include/drm/i915_component.h | 33 +++++++++++++++++++++++---------- 2 files changed, 24 insertions(+), 10 deletions(-) diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl index 2fb9a54..7554679 100644 --- a/Documentation/DocBook/drm.tmpl +++ b/Documentation/DocBook/drm.tmpl @@ -4048,6 +4048,7 @@ int num_ioctls;</synopsis> <title>High Definition Audio</title> !Pdrivers/gpu/drm/i915/intel_audio.c High Definition Audio over HDMI and Display Port !Idrivers/gpu/drm/i915/intel_audio.c +!Iinclude/drm/i915_component.h </sect2> <sect2> <title>Panel Self Refresh PSR (PSR/SRD)</title> diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h index b2d56dd..89d6362 100644 --- a/include/drm/i915_component.h +++ b/include/drm/i915_component.h @@ -24,6 +24,28 @@ #ifndef _I915_COMPONENT_H_ #define _I915_COMPONENT_H_ +/** + * struct i915_audio_component_audio_ops - i915 directly calling hda driver + * + * @audio_ptr: Pointer to pass when calling pin_eld_notify. + * @pin_eld_notify: Called from i915 driver, notifying the HDA driver that + * pin sense and/or ELD information has changed. + * + * These functions are implemented by hda driver and called by the i915 + * driver. + */ +struct i915_audio_component_audio_ops { + void *audio_ptr; + void (*pin_eld_notify)(void *audio_ptr, int port); +}; + +/** + * struct i915_audio_component - Used for direct communication between i915 and hda drivers + * + * @dev: i915 device, used as parameter for ops + * @ops: Ops implemented by i915 driver, called by hda driver + * @audio_ops: Ops implemented by hda driver, called by i915 driver + */ struct i915_audio_component { struct device *dev; @@ -35,16 +57,7 @@ struct i915_audio_component { int (*get_cdclk_freq)(struct device *); } *ops; - const struct i915_audio_component_audio_ops { - void *audio_ptr; - /** - * Call from i915 driver, notifying the HDA driver that - * pin sense and/or ELD information has changed. - * @audio_ptr: HDA driver object - * @port: Which port has changed (PORTA / PORTB / PORTC etc) - */ - void (*pin_eld_notify)(void *audio_ptr, int port); - } *audio_ops; + const struct i915_audio_component_audio_ops *audio_ops; }; #endif /* _I915_COMPONENT_H_ */ -- 1.9.1

3 3