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

[alsa-devel] [PATCH] ASoC: wm0010: Remove .bus field setting from wm0010_spi_driver
by Axel Lin 03 Jul '15

03 Jul '15

The .bus field will be set by spi_register_driver() so we don't need to set it in wm0010_spi_driver. Signed-off-by: Axel Lin <axel.lin(a)ingics.com> --- sound/soc/codecs/wm0010.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/wm0010.c b/sound/soc/codecs/wm0010.c index 6560a66..e1da49f 100644 --- a/sound/soc/codecs/wm0010.c +++ b/sound/soc/codecs/wm0010.c @@ -1003,7 +1003,6 @@ static int wm0010_spi_remove(struct spi_device *spi) static struct spi_driver wm0010_spi_driver = { .driver = { .name = "wm0010", - .bus = &spi_bus_type, .owner = THIS_MODULE, }, .probe = wm0010_spi_probe, -- 2.1.0

3 2

Re: [alsa-devel] Intel Baytrail Linux Audio
by Vinod Koul 03 Jul '15

03 Jul '15

On Tue, Jun 23, 2015 at 05:31:17PM -0400, Tyler Yeomans wrote: > Hello Vinod, Pls cc relevant mailing list, in this case alsa. I have cced Liam and Jarrko as well > > I am writing to you because unfortunately I have exhausted my search > and I am unable to get ALSA sound working on my Baytrail minipc. > My dmesg only reports that: byt-rt5640 byt-rt5640: ASoC: CPU DAI > baytrail-pcm-audio not registered > aplay -l rightfully says there are no sound cards as the driver is > not registered. Can you give more details of your, does this enable HDA audio or I2S audio? > I have acquired the firmware file from the chromium repo here (https://kernel.googlesource.com/pub/scm/linux/kernel/git/firmware/linux-fir…) > as was directed by the OP in this forum post: > http://ubuntuforums.org/showthread.php?t=2254631 > > But unfortunately it seems the driver is still not loading > correctly, even on Ubuntu 15.04 with ALSA 1.0.28 Do you have dmesg for us to look at so that we can see what might be going wrong > I see on gmane you are working on Skylake driver, could your work > there possibly be backported to the Baytrail/RT5640 chipset? No they are entirely different hw, so it will not work. We have uostream BYT driver and they should work -- ~Vinod

6 12

Re: [alsa-devel] [Intel-gfx] [PATCH 3/4] snd: add support for displayport multi-stream to hda codec.
by Dave Airlie 03 Jul '15

03 Jul '15

On 19 June 2015 at 19:54, Lin, Mengdong <mengdong.lin(a)intel.com> wrote: > Hi Takashi/Dave, > > Shall we move or cc this discussion on audio driver side to ALSA ML? Oops I thought I had cc'ed these patches to alsa-devel as well when I sent them. > I think we also need to decide how to manage PCM devices for DP MST. > Now the HD-A driver create a PCM device for each pin, and the substream > number is 1 for each PCM. Now with DP MST enabled, each pin can support > multiple streams (e.g. 3 on Intel HSW/BDW/SKL). > > There may be 2 options: > -#1: Let an HDMI codec specify number of substreams, same as the number > of device entries on a pin. We can specify 3 for HSW/BDW/SKL. Other > vendors can also specify a value according to actual HW capabilities. > > So for HSW, we have 3x3 subtreams totally. But we only have 3 convertors > (for 3 display pipelines), so we can open up to 3 substreams at the same > time. When the audio driver finds all 3 convertors are used when opening > a new substream, it will fail. One thing I noticed is the number of devices on a PIN is only updated when the MST device is plugged in so normally pins 5,6,7 have 0 devices, and when I plug in MST device, I get the 3 devices on port 6. So it seems dynamic enough at this point, though I guess it'll always be 0 or 3. > > - #2: Create PCM device dynamically. Only create a PCM devices for a device > entry which connects to monitor with audio support. When the monitor > is removed, the PCM device will be disconnected, closed and removed, > similar to the USB case. > > This will change ALSA core. But there will be less PCM devices and > substreams, since the number of connected monitors Is decided by the > actual GPU display pipeline. I like this option more, since I think it should be more like USB, but I've no idea how much work it would be from the alsa side, this patch was probably as deep into alsa as I've gone. Dave. >> Signed-off-by: Dave Airlie <airlied(a)redhat.com> >> --- >> include/sound/hda_verbs.h | 2 + >> sound/pci/hda/hda_codec.c | 1 + >> sound/pci/hda/hda_proc.c | 5 +- >> sound/pci/hda/patch_hdmi.c | 181 >> +++++++++++++++++++++++++++++++-------------- >> 4 files changed, 131 insertions(+), 58 deletions(-) >> >> diff --git a/include/sound/hda_verbs.h b/include/sound/hda_verbs.h index >> d0509db..3b62ac5 100644 >> --- a/include/sound/hda_verbs.h >> +++ b/include/sound/hda_verbs.h >> @@ -75,6 +75,7 @@ enum { >> #define AC_VERB_GET_HDMI_CHAN_SLOT 0x0f34 >> #define AC_VERB_GET_DEVICE_SEL 0xf35 >> #define AC_VERB_GET_DEVICE_LIST 0xf36 >> +#define AC_VERB_GET_DP_STREAM_ID 0xf3c >> >> /* >> * SET verbs >> @@ -115,6 +116,7 @@ enum { >> #define AC_VERB_SET_HDMI_CP_CTRL 0x733 >> #define AC_VERB_SET_HDMI_CHAN_SLOT 0x734 >> #define AC_VERB_SET_DEVICE_SEL 0x735 >> +#define AC_VERB_SET_DP_STREAM_ID 0x73C >> >> /* >> * Parameter IDs >> diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c index >> 5645481..3981c63 100644 >> --- a/sound/pci/hda/hda_codec.c >> +++ b/sound/pci/hda/hda_codec.c >> @@ -482,6 +482,7 @@ int snd_hda_get_devices(struct hda_codec *codec, >> hda_nid_t nid, >> } >> return devices; >> } >> +EXPORT_SYMBOL_GPL(snd_hda_get_devices); >> >> /* >> * destructor >> diff --git a/sound/pci/hda/hda_proc.c b/sound/pci/hda/hda_proc.c index >> baaf7ed0..39fac53 100644 >> --- a/sound/pci/hda/hda_proc.c >> +++ b/sound/pci/hda/hda_proc.c >> @@ -644,10 +644,13 @@ static void print_device_list(struct snd_info_buffer >> *buffer, >> int i, curr = -1; >> u8 dev_list[AC_MAX_DEV_LIST_LEN]; >> int devlist_len; >> + int dp_s_id; >> >> + dp_s_id = snd_hda_codec_read(codec, nid, 0, >> + AC_VERB_GET_DP_STREAM_ID, 0); >> devlist_len = snd_hda_get_devices(codec, nid, dev_list, >> AC_MAX_DEV_LIST_LEN); >> - snd_iprintf(buffer, " Devices: %d\n", devlist_len); >> + snd_iprintf(buffer, " Devices: %d: 0x%x\n", devlist_len, dp_s_id); >> if (devlist_len <= 0) >> return; >> >> diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c index >> 5f44f60..8272656 100644 >> --- a/sound/pci/hda/patch_hdmi.c >> +++ b/sound/pci/hda/patch_hdmi.c >> @@ -68,6 +68,17 @@ struct hdmi_spec_per_cvt { >> /* max. connections to a widget */ >> #define HDA_MAX_CONNECTIONS 32 >> >> +struct hdmi_spec_per_pin; >> +#define HDA_MAX_DEVICES 3 >> +struct hdmi_spec_per_device { >> + struct hdmi_spec_per_pin *pin; >> + int device_idx; >> + struct hdmi_eld sink_eld; >> +#ifdef CONFIG_PROC_FS >> + struct snd_info_entry *proc_entry; >> +#endif >> +}; >> + >> struct hdmi_spec_per_pin { >> hda_nid_t pin_nid; >> int num_mux_nids; >> @@ -76,7 +87,11 @@ struct hdmi_spec_per_pin { >> hda_nid_t cvt_nid; >> >> struct hda_codec *codec; >> - struct hdmi_eld sink_eld; >> + >> + int num_devices; >> + u8 dev_list[AC_MAX_DEV_LIST_LEN]; >> + struct hdmi_spec_per_device devices[HDA_MAX_DEVICES]; >> + >> struct mutex lock; >> struct delayed_work work; >> struct snd_kcontrol *eld_ctl; >> @@ -86,9 +101,6 @@ struct hdmi_spec_per_pin { >> bool non_pcm; >> bool chmap_set; /* channel-map override by ALSA API? */ >> unsigned char chmap[8]; /* ALSA API channel-map */ -#ifdef >> CONFIG_PROC_FS >> - struct snd_info_entry *proc_entry; >> -#endif >> }; >> >> struct cea_channel_speaker_allocation; >> @@ -409,7 +421,7 @@ static int hdmi_eld_ctl_info(struct snd_kcontrol >> *kcontrol, >> >> pin_idx = kcontrol->private_value; >> per_pin = get_pin(spec, pin_idx); >> - eld = &per_pin->sink_eld; >> + eld = &per_pin->devices[0].sink_eld; >> >> mutex_lock(&per_pin->lock); >> uinfo->count = eld->eld_valid ? eld->eld_size : 0; @@ -429,7 +441,7 @@ >> static int hdmi_eld_ctl_get(struct snd_kcontrol *kcontrol, >> >> pin_idx = kcontrol->private_value; >> per_pin = get_pin(spec, pin_idx); >> - eld = &per_pin->sink_eld; >> + eld = &per_pin->devices[0].sink_eld; >> >> mutex_lock(&per_pin->lock); >> if (eld->eld_size > ARRAY_SIZE(ucontrol->value.bytes.data)) { @@ -549,60 >> +561,63 @@ static void hdmi_set_channel_count(struct hda_codec *codec, >> */ >> >> #ifdef CONFIG_PROC_FS >> -static void print_eld_info(struct snd_info_entry *entry, >> - struct snd_info_buffer *buffer) >> +static void print_eld_info_device(struct snd_info_entry *entry, >> + struct snd_info_buffer *buffer) >> { >> - struct hdmi_spec_per_pin *per_pin = entry->private_data; >> + struct hdmi_spec_per_device *per_device = entry->private_data; >> >> - mutex_lock(&per_pin->lock); >> - snd_hdmi_print_eld_info(&per_pin->sink_eld, buffer); >> - mutex_unlock(&per_pin->lock); >> + mutex_lock(&per_device->pin->lock); >> + snd_hdmi_print_eld_info(&per_device->sink_eld, buffer); >> + mutex_unlock(&per_device->pin->lock); >> } >> >> -static void write_eld_info(struct snd_info_entry *entry, >> - struct snd_info_buffer *buffer) >> +static void write_eld_info_device(struct snd_info_entry *entry, >> + struct snd_info_buffer *buffer) >> { >> - struct hdmi_spec_per_pin *per_pin = entry->private_data; >> + struct hdmi_spec_per_device *per_device = entry->private_data; >> >> - mutex_lock(&per_pin->lock); >> - snd_hdmi_write_eld_info(&per_pin->sink_eld, buffer); >> - mutex_unlock(&per_pin->lock); >> + mutex_lock(&per_device->pin->lock); >> + snd_hdmi_write_eld_info(&per_device->sink_eld, buffer); >> + mutex_unlock(&per_device->pin->lock); >> } >> >> -static int eld_proc_new(struct hdmi_spec_per_pin *per_pin, int index) >> +static int eld_device_proc_new(struct hdmi_spec_per_device *per_device, >> +int pin_idx, int dev_idx) >> { >> char name[32]; >> - struct hda_codec *codec = per_pin->codec; >> + struct hda_codec *codec = per_device->pin->codec; >> struct snd_info_entry *entry; >> int err; >> >> - snprintf(name, sizeof(name), "eld#%d.%d", codec->addr, index); >> + if (dev_idx == -1) >> + snprintf(name, sizeof(name), "eld#%d.%d", codec->addr, pin_idx); >> + else >> + snprintf(name, sizeof(name), "eld#%d.%d.%d", codec->addr, >> pin_idx, >> +dev_idx); >> err = snd_card_proc_new(codec->card, name, &entry); >> if (err < 0) >> return err; >> >> - snd_info_set_text_ops(entry, per_pin, print_eld_info); >> - entry->c.text.write = write_eld_info; >> + snd_info_set_text_ops(entry, per_device, print_eld_info_device); >> + entry->c.text.write = write_eld_info_device; >> entry->mode |= S_IWUSR; >> - per_pin->proc_entry = entry; >> + per_device->proc_entry = entry; >> >> return 0; >> } >> >> -static void eld_proc_free(struct hdmi_spec_per_pin *per_pin) >> +static void eld_device_proc_free(struct hdmi_spec_per_device >> +*per_device) >> { >> - if (!per_pin->codec->bus->shutdown && per_pin->proc_entry) { >> - snd_device_free(per_pin->codec->card, per_pin->proc_entry); >> - per_pin->proc_entry = NULL; >> + if (!per_device->pin->codec->bus->shutdown && per_device->proc_entry) >> { >> + snd_device_free(per_device->pin->codec->card, >> per_device->proc_entry); >> + per_device->proc_entry = NULL; >> } >> } >> #else >> -static inline int eld_proc_new(struct hdmi_spec_per_pin *per_pin, >> - int index) >> +static inline int eld_device_proc_new(struct hdmi_spec_per_device >> *per_device, >> + int pin_idx, int dev_idx) >> { >> return 0; >> } >> -static inline void eld_proc_free(struct hdmi_spec_per_pin *per_pin) >> +static inline void eld_device_proc_free(struct hdmi_spec_per_device >> +*per_device) >> { >> } >> #endif >> @@ -1112,13 +1127,13 @@ static void hdmi_pin_setup_infoframe(struct >> hda_codec *codec, >> >> static void hdmi_setup_audio_infoframe(struct hda_codec *codec, >> struct hdmi_spec_per_pin *per_pin, >> + struct hdmi_eld *eld, >> bool non_pcm) >> { >> struct hdmi_spec *spec = codec->spec; >> hda_nid_t pin_nid = per_pin->pin_nid; >> int channels = per_pin->channels; >> int active_channels; >> - struct hdmi_eld *eld; >> int ca, ordered_ca; >> >> if (!channels) >> @@ -1129,7 +1144,8 @@ static void hdmi_setup_audio_infoframe(struct >> hda_codec *codec, >> AC_VERB_SET_AMP_GAIN_MUTE, >> AMP_OUT_UNMUTE); >> >> - eld = &per_pin->sink_eld; >> + if (!eld) >> + eld = &per_pin->devices[0].sink_eld; >> >> if (!non_pcm && per_pin->chmap_set) >> ca = hdmi_manual_channel_allocation(channels, per_pin->chmap); >> @@ -1191,7 +1207,7 @@ static void hdmi_intrinsic_event(struct hda_codec >> *codec, unsigned int res) >> return; >> jack->jack_dirty = 1; >> >> - codec_dbg(codec, >> + codec_info(codec, >> "HDMI hot plug event: Codec=%d Pin=%d Device=%d Inactive=%d >> Presence_Detect=%d ELD_Valid=%d\n", >> codec->addr, jack->nid, dev_entry, !!(res & AC_UNSOL_RES_IA), >> !!(res & AC_UNSOL_RES_PD), !!(res & AC_UNSOL_RES_ELDV)); @@ >> -1449,7 +1465,7 @@ static int hdmi_pcm_open(struct hda_pcm_stream >> *hinfo, >> if (snd_BUG_ON(pin_idx < 0)) >> return -EINVAL; >> per_pin = get_pin(spec, pin_idx); >> - eld = &per_pin->sink_eld; >> + eld = &per_pin->devices[0].sink_eld; >> >> err = hdmi_choose_cvt(codec, pin_idx, &cvt_idx, &mux_idx); >> if (err < 0) >> @@ -1530,7 +1546,7 @@ static bool hdmi_present_sense(struct >> hdmi_spec_per_pin *per_pin, int repoll) >> struct hda_codec *codec = per_pin->codec; >> struct hdmi_spec *spec = codec->spec; >> struct hdmi_eld *eld = &spec->temp_eld; >> - struct hdmi_eld *pin_eld = &per_pin->sink_eld; >> + struct hdmi_eld *pin_eld; >> hda_nid_t pin_nid = per_pin->pin_nid; >> /* >> * Always execute a GetPinSense verb here, even when called from @@ >> -1544,20 +1560,41 @@ static bool hdmi_present_sense(struct >> hdmi_spec_per_pin *per_pin, int repoll) >> bool update_eld = false; >> bool eld_changed = false; >> bool ret; >> + int device_num = 0; >> + bool need_repoll = false; >> + bool any_eld_valid = false; >> >> snd_hda_power_up_pm(codec); >> present = snd_hda_pin_sense(codec, pin_nid); >> >> mutex_lock(&per_pin->lock); >> - pin_eld->monitor_present = !!(present & AC_PINSENSE_PRESENCE); >> - if (pin_eld->monitor_present) >> - eld->eld_valid = !!(present & AC_PINSENSE_ELDV); >> - else >> - eld->eld_valid = false; >> >> - codec_dbg(codec, >> - "HDMI status: Codec=%d Pin=%d Presence_Detect=%d >> ELD_Valid=%d\n", >> - codec->addr, pin_nid, pin_eld->monitor_present, eld->eld_valid); >> + if (codec->dp_mst) >> + per_pin->num_devices = snd_hda_get_devices(codec, pin_nid, >> per_pin->dev_list, >> + AC_MAX_DEV_LIST_LEN); >> +next_device: >> + pin_eld = &per_pin->devices[device_num].sink_eld; >> + eld_changed = false; >> + update_eld = false; >> + if (per_pin->num_devices) { >> + pin_eld->monitor_present = !!(per_pin->dev_list[device_num] & >> AC_DE_PD); >> + if (pin_eld->monitor_present) >> + eld->eld_valid = !!(per_pin->dev_list[device_num] & >> AC_DE_ELDV); >> + else >> + eld->eld_valid = false; >> + if (eld->eld_valid) >> + snd_hda_codec_write(codec, pin_nid, 0, >> AC_VERB_SET_DEVICE_SEL, device_num); >> + } else { >> + pin_eld->monitor_present = !!(present & AC_PINSENSE_PRESENCE); >> + if (pin_eld->monitor_present) >> + eld->eld_valid = !!(present & AC_PINSENSE_ELDV); >> + else >> + eld->eld_valid = false; >> + } >> + >> + codec_info(codec, >> + "HDMI status: Codec=%d Pin=%d Device=%d Presence_Detect=%d >> ELD_Valid=%d\n", >> + codec->addr, pin_nid, device_num, pin_eld->monitor_present, >> +eld->eld_valid); >> >> if (eld->eld_valid) { >> if (spec->ops.pin_get_eld(codec, pin_nid, eld->eld_buffer, @@ >> -1573,11 +1610,11 @@ static bool hdmi_present_sense(struct >> hdmi_spec_per_pin *per_pin, int repoll) >> if (eld->eld_valid) { >> snd_hdmi_show_eld(codec, &eld->info); >> update_eld = true; >> + any_eld_valid = true; >> } >> else if (repoll) { >> - schedule_delayed_work(&per_pin->work, >> - msecs_to_jiffies(300)); >> - goto unlock; >> + need_repoll = true; >> + goto skip_to_next_device; >> } >> } >> >> @@ -1614,7 +1651,7 @@ static bool hdmi_present_sense(struct >> hdmi_spec_per_pin *per_pin, int repoll) >> per_pin->mux_idx); >> } >> >> - hdmi_setup_audio_infoframe(codec, per_pin, >> + hdmi_setup_audio_infoframe(codec, per_pin, eld, >> per_pin->non_pcm); >> } >> } >> @@ -1623,8 +1660,19 @@ static bool hdmi_present_sense(struct >> hdmi_spec_per_pin *per_pin, int repoll) >> snd_ctl_notify(codec->card, >> SNDRV_CTL_EVENT_MASK_VALUE | >> SNDRV_CTL_EVENT_MASK_INFO, >> &per_pin->eld_ctl->id); >> - unlock: >> - ret = !repoll || !pin_eld->monitor_present || pin_eld->eld_valid; >> +skip_to_next_device: >> + if (codec->dp_mst) { >> + device_num++; >> + if (device_num < per_pin->num_devices) >> + goto next_device; >> + } >> + >> + if (need_repoll) { >> + schedule_delayed_work(&per_pin->work, >> + msecs_to_jiffies(300)); >> + repoll = true; >> + } >> + ret = !repoll || any_eld_valid; >> >> jack = snd_hda_jack_tbl_get(codec, pin_nid); >> if (jack) >> @@ -1807,7 +1855,7 @@ static int >> generic_hdmi_playback_pcm_prepare(struct hda_pcm_stream *hinfo, >> per_pin->channels = substream->runtime->channels; >> per_pin->setup = true; >> >> - hdmi_setup_audio_infoframe(codec, per_pin, non_pcm); >> + hdmi_setup_audio_infoframe(codec, per_pin, NULL, non_pcm); >> mutex_unlock(&per_pin->lock); >> >> if (spec->dyn_pin_out) { >> @@ -2035,7 +2083,7 @@ static int hdmi_chmap_ctl_put(struct snd_kcontrol >> *kcontrol, >> per_pin->chmap_set = true; >> memcpy(per_pin->chmap, chmap, sizeof(chmap)); >> if (prepared) >> - hdmi_setup_audio_infoframe(codec, per_pin, per_pin->non_pcm); >> + hdmi_setup_audio_infoframe(codec, per_pin, NULL, >> per_pin->non_pcm); >> mutex_unlock(&per_pin->lock); >> >> return 0; >> @@ -2147,7 +2195,7 @@ static int generic_hdmi_build_controls(struct >> hda_codec *codec) static int generic_hdmi_init_per_pins(struct hda_codec >> *codec) { >> struct hdmi_spec *spec = codec->spec; >> - int pin_idx; >> + int pin_idx, dev_idx; >> >> for (pin_idx = 0; pin_idx < spec->num_pins; pin_idx++) { >> struct hdmi_spec_per_pin *per_pin = get_pin(spec, pin_idx); @@ >> -2155,7 +2203,20 @@ static int generic_hdmi_init_per_pins(struct hda_codec >> *codec) >> per_pin->codec = codec; >> mutex_init(&per_pin->lock); >> INIT_DELAYED_WORK(&per_pin->work, hdmi_repoll_eld); >> - eld_proc_new(per_pin, pin_idx); >> + >> + if (codec->dp_mst) { >> + for (dev_idx = 0; dev_idx < HDA_MAX_DEVICES; dev_idx++) { >> + per_pin->devices[dev_idx].device_idx = dev_idx; >> + per_pin->devices[dev_idx].pin = per_pin; >> + >> + eld_device_proc_new(&per_pin->devices[dev_idx], pin_idx, >> dev_idx); >> + } >> + } else { >> + per_pin->num_devices = 0; >> + per_pin->devices[0].device_idx = 0; >> + per_pin->devices[0].pin = per_pin; >> + eld_device_proc_new(&per_pin->devices[0], pin_idx, -1); >> + } >> } >> return 0; >> } >> @@ -2191,13 +2252,19 @@ static void hdmi_array_free(struct hdmi_spec >> *spec) static void generic_hdmi_free(struct hda_codec *codec) { >> struct hdmi_spec *spec = codec->spec; >> - int pin_idx; >> + int pin_idx, dev_idx; >> >> for (pin_idx = 0; pin_idx < spec->num_pins; pin_idx++) { >> struct hdmi_spec_per_pin *per_pin = get_pin(spec, pin_idx); >> >> cancel_delayed_work_sync(&per_pin->work); >> - eld_proc_free(per_pin); >> + if (per_pin->num_devices) { >> + for (dev_idx = 0; dev_idx < per_pin->num_devices; dev_idx++) { >> + struct hdmi_spec_per_device *per_device = >> &per_pin->devices[dev_idx]; >> + eld_device_proc_free(per_device); >> + } >> + } else >> + eld_device_proc_free(&per_pin->devices[0]); >> } >> >> hdmi_array_free(spec); >> @@ -2333,6 +2400,7 @@ static int patch_generic_hdmi(struct hda_codec >> *codec) >> if (is_haswell_plus(codec)) { >> intel_haswell_enable_all_pins(codec, true); >> intel_haswell_fixup_enable_dp12(codec); >> + codec->dp_mst = true; >> } >> >> if (is_haswell_plus(codec) || is_valleyview_plus(codec)) @@ -2346,7 >> +2414,6 @@ static int patch_generic_hdmi(struct hda_codec *codec) >> codec->patch_ops = generic_hdmi_patch_ops; >> if (is_haswell_plus(codec)) { >> codec->patch_ops.set_power_state = haswell_set_power_state; >> - codec->dp_mst = true; >> } >> >> generic_hdmi_init_per_pins(codec); >> -- >> 2.4.1 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx(a)lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

9 22

[alsa-devel] [PATCH] ASoC: dapm: Don't check dummy dai in soc_dapm_dai_stream_event
by Shengjiu Wang 03 Jul '15

03 Jul '15

Dummy dai can be used by multiple sound card. But it only belong to one card's dapm list. If another card use it, there will be dapm_assert_locked warning. [ 20.015782] WARNING: CPU: 1 PID: 661 at sound/soc/soc-dapm.c:124 dapm_assert_locked.isra.36+0x4c/0x58() [ 20.025249] Modules linked in: [ 20.028349] CPU: 1 PID: 661 Comm: aplay Not tainted 4.1.0-rc6-next-20150605-00004-gaee05d8-dirty #92 [ 20.037528] Hardware name: Freescale i.MX6 Quad/DualLite (Device Tree) [ 20.044110] Backtrace: [ 20.046614] [<80012e00>] (dump_backtrace) from [<80012fa0>] (show_stack+0x18/0x1c) [ 20.054229] r6:809e8060 r5:00000000 r4:00000000 r3:00000000 [ 20.060002] [<80012f88>] (show_stack) from [<807a0f74>] (dump_stack+0x80/0x9c) [ 20.067293] [<807a0ef4>] (dump_stack) from [<8002b144>] (warn_slowpath_common+0x7c/0xb4) [ 20.075427] r5:0000007c r4:00000000 [ 20.079065] [<8002b0c8>] (warn_slowpath_common) from [<8002b1a0>] (warn_slowpath_null+0x24/0x2c) [ 20.087898] r8:00000001 r7:88007c28 r6:ed94a680 r5:809e83e4 r4:ed83d6c0 [ 20.094747] [<8002b17c>] (warn_slowpath_null) from [<8058403c>] (dapm_assert_locked.isra.36+0x4c/0x58) [ 20.104101] [<80583ff0>] (dapm_assert_locked.isra.36) from [<805842ec>] (dapm_mark_dirty+0x64/0xa4) [ 20.113165] [<80584288>] (dapm_mark_dirty) from [<805853a8>] (soc_dapm_dai_stream_event.isra.42+0x30/0xc8) [ 20.122863] r8:ed9b5dbc r7:00000000 r6:00000001 r5:00000001 r4:ed83d6c0 [ 20.129706] [<80585378>] (soc_dapm_dai_stream_event.isra.42) from [<80587e28>] (snd_soc_dapm_stream_event+0x78/0xa0) [ 20.140264] r5:ee2ee62c r4:00000001 [ 20.143918] [<80587db0>] (snd_soc_dapm_stream_event) from [<8058957c>] (soc_pcm_prepare+0x138/0x21c) [ 20.153058] r8:ed8d9480 r7:00000000 r6:ed9b0e00 r5:00000001 r4:ee2ee62c r3:00000000 ... Signed-off-by: Shengjiu Wang <shengjiu.wang(a)freescale.com> --- sound/soc/soc-dapm.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c index aa327c9..b618541 100644 --- a/sound/soc/soc-dapm.c +++ b/sound/soc/soc-dapm.c @@ -3856,6 +3856,9 @@ static void soc_dapm_dai_stream_event(struct snd_soc_dai *dai, int stream, { struct snd_soc_dapm_widget *w; + if (snd_soc_dai_is_dummy(dai)) + return; + if (stream == SNDRV_PCM_STREAM_PLAYBACK) w = dai->playback_widget; else -- 1.7.9.5

2 8

Re: [alsa-devel] [pulseaudio-discuss] snd_pcm_htimestamp error
by Arun Raghavan 03 Jul '15

03 Jul '15

It looks like this might be more relevant on alsa-devel. Some more details on the platform might be useful. -- Arun On 2 July 2015 at 08:35, golden <magic.lixin(a)gmail.com> wrote: > Hi PulseAudio forks, > > snd_pcm_htimestamp is unstable after change the date time of OS by using > "date -s xxx-xxx-xx", > It is very strange that the time value get from snd_pcm_htimestamp is > always 1420588802151082308 > > and here is what I get when cat the alsa infomation: > > uid=0 gid=0@nutshell:/ # cat /proc/asound/card0/pcm0p/sub0/status > state: RUNNING > owner_pid : 1339 > trigger_time: 1420591411.800808521 > tstamp : 1420591457.486751372 > delay : 5040 > avail : 3152 > avail_max : 0 > ----- > hw_ptr : 2193408 > appl_ptr : 2198448 > > Does anyone met this problem before ? > > Thanks! > > > > BR, > Lixin > _______________________________________________ > pulseaudio-discuss mailing list > pulseaudio-discuss(a)lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

1 0

[alsa-devel] [PATCH] ALSA: usb-audio: Add MIDI support for Steinberg MI2/MI4
by Dominic Sacré 03 Jul '15

03 Jul '15

The Steinberg MI2 and MI4 interfaces are compatible with the USB class audio spec, but the MIDI part of the devices is reported as a vendor specific interface. This patch adds entries to quirks-table.h to recognize the MIDI endpoints. Audio functionality was already working and is unaffected by this change. Signed-off-by: Dominic Sacré <dominic.sacre(a)gmx.de> Signed-off-by: Albert Huitsing <albert(a)huitsing.nl> --- sound/usb/quirks-table.h | 43 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h index 2f6d3e9..304c02a 100644 --- a/sound/usb/quirks-table.h +++ b/sound/usb/quirks-table.h @@ -3152,6 +3152,49 @@ AU0828_DEVICE(0x2040, 0x7270, "Hauppauge", "HVR-950Q"), }, { + /* Steinberg MI2 */ + USB_DEVICE_VENDOR_SPEC(0x0a4e, 0x2040), + .driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) { + .ifnum = QUIRK_ANY_INTERFACE, + .type = QUIRK_COMPOSITE, + .data = & (const struct snd_usb_audio_quirk[]) { + { + .ifnum = 3, + .type = QUIRK_MIDI_FIXED_ENDPOINT, + .data = &(const struct snd_usb_midi_endpoint_info) { + .out_cables = 0x0001, + .in_cables = 0x0001 + } + }, + { + .ifnum = -1 + } + } + } +}, +{ + /* Steinberg MI4 */ + USB_DEVICE_VENDOR_SPEC(0x0a4e, 0x4040), + .driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) { + .ifnum = QUIRK_ANY_INTERFACE, + .type = QUIRK_COMPOSITE, + .data = & (const struct snd_usb_audio_quirk[]) { + { + .ifnum = 3, + .type = QUIRK_MIDI_FIXED_ENDPOINT, + .data = &(const struct snd_usb_midi_endpoint_info) { + .out_cables = 0x0001, + .in_cables = 0x0001 + } + }, + { + .ifnum = -1 + } + } + } +}, + +{ /* * Some USB MIDI devices don't have an audio control interface, * so we have to grab MIDI streaming interfaces here. -- 2.4.4

4 7

Re: [alsa-devel] Issues and/or possible bugs in alsa
by Yomi Ogunwumi 03 Jul '15

03 Jul '15

Whoops, I didn't realize I sent that to you and you alone, Raymond. I'm going to take a guess and conclude that there must be bugs in the interface patch written for the realtek codecs. I happened to test two other laptops and they worked just fine. (Intel...) ...I guess in should report a bug related to that realtek patch... • Yomi On Mar 22, 2015 11:11 AM, "Yomi Ogunwumi" <abyomi0(a)gmail.com> wrote: > Is there anything I can do to resolve it? > > On Sun, Mar 22, 2015 at 11:09 AM, Yomi Ogunwumi <abyomi0(a)gmail.com> wrote: > >> There is one log where that snippet looks correct. >> >> https://gist.github.com/Yomi0/9bfc3e72588cf8a35811#file-alsainfo-headphones… >> >> >> https://gist.githubusercontent.com/Yomi0/9bfc3e72588cf8a35811/raw/77ff724d2… >> >> Codec: Realtek ALC269VB >> Address: 0 >> AFG Function Id: 0x1 (unsol 1) >> Vendor Id: 0x10ec0269 >> Subsystem Id: 0x1179fa22 >> Revision Id: 0x100100 >> No Modem Function Group found >> Default PCM: >> rates [0x560]: 44100 48000 96000 192000 >> bits [0xe]: 16 20 24 >> formats [0x1]: PCM >> Default Amp-In caps: N/A >> Default Amp-Out caps: N/A >> State of AFG node 0x01: >> Power states: D0 D1 D2 D3 CLKSTOP EPSS >> Power: setting=D0, actual=D0 >> >> >> On Sun, Mar 15, 2015 at 11:58 AM, Raymond Yau < >> superquad.vortex2(a)gmail.com> wrote: >> >>> > >>> > Is there anything else I need to do? >>> >>> Codec: Realtek ALC269VB >>> Address: 0 >>> AFG Function Id: 0x1 (unsol 1) >>> Vendor Id: 0x10ec0269 >>> Subsystem Id: 0x1179fa22 >>> Revision Id: 0x100100 >>> No Modem Function Group found >>> Default PCM: >>> N/A >>> Default Amp-In caps: N/A >>> Default Amp-Out caps: N/A >>> State of AFG node 0x01: >>> Power: setting=UNKNOWN, actual=UNKNOWN, Error, Clock-stop-OK, >>> Setting-reset >>> Invalid AFG subtree >>> >>> Your problem seem related to power >>> >> -- >> *Yomi* >> > > > > -- > *Yomi* >

2 16

[alsa-devel] [PATCH v4 0/5] alsa-lib: audio timestamping evolutions
by Pierre-Louis Bossart 02 Jul '15

02 Jul '15

alsa-lib side of kernel patches already merged for 4.1. V4: added simple log to show timestamp report and accuracy Pierre-Louis Bossart (5): pcm: add helper functions to query timestamping capabilities pcm: add support for get/set_audio_htstamp_config pcm: add support for new STATUS_EXT ioctl test: fix audio_time with new get/set audio_tstamp_config test: audio_time: show report validity and accuracy include/pcm.h | 47 ++++- src/pcm/pcm.c | 73 +++++++- src/pcm/pcm_hw.c | 16 +- test/audio_time.c | 515 +++++++++++++++++++++++++++++++++++------------------- 4 files changed, 465 insertions(+), 186 deletions(-) -- 1.9.1

2 6

[alsa-devel] [PATCH] ASoC: omap: Fix typo in config option description
by Nik Nyby 02 Jul '15

02 Jul '15

This fixes a typo in the description for SND_OMAP_SOC_HDMI_AUDIO. Signed-off-by: Nik Nyby <nikolas(a)gnu.org> --- sound/soc/omap/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/omap/Kconfig b/sound/soc/omap/Kconfig index 30d0109..5185a38 100644 --- a/sound/soc/omap/Kconfig +++ b/sound/soc/omap/Kconfig @@ -24,7 +24,7 @@ config SND_OMAP_SOC_HDMI_AUDIO component also under DSS HDMI device. Dummy codec is used as as codec component. The hdmi audio driver implements also the card and registers it under its own platform device. - The device for the dirver is registered by OMAPDSS hdmi + The device for the driver is registered by OMAPDSS hdmi driver. config SND_OMAP_SOC_N810 -- 2.1.4

2 1

[alsa-devel] [PATCH v2 0/13] topology support for alsa-lib
by Liam Girdwood 01 Jul '15

01 Jul '15

This patch series adds the topology text parser support to alsa-lib. The topology parser converts text topology data to the binary ABI format read by the kernel topology core. Changes since V1 :- o Terminate all strncpy() strings with NULL. o Use snd_pcm_format_value() instead of local lookup. o Set a maximum size for private data files. o Set capture/playback stream direction for new PCMs o Only set TLV access flags for TLV CTLs. Liam

3 3