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

Re: [alsa-devel] mmotm 2019-08-27-20-39 uploaded (sound/hda/intel-nhlt.c)
by Randy Dunlap 29 Aug '19

29 Aug '19

On 8/27/19 8:40 PM, akpm(a)linux-foundation.org wrote: > The mm-of-the-moment snapshot 2019-08-27-20-39 has been uploaded to > > http://www.ozlabs.org/~akpm/mmotm/ > > mmotm-readme.txt says > > README for mm-of-the-moment: > > http://www.ozlabs.org/~akpm/mmotm/ > > This is a snapshot of my -mm patch queue. Uploaded at random hopefully > more than once a week. > > You will need quilt to apply these patches to the latest Linus release (5.x > or 5.x-rcY). The series file is in broken-out.tar.gz and is duplicated in > http://ozlabs.org/~akpm/mmotm/series > > The file broken-out.tar.gz contains two datestamp files: .DATE and > .DATE-yyyy-mm-dd-hh-mm-ss. Both contain the string yyyy-mm-dd-hh-mm-ss, > followed by the base kernel version against which this patch series is to > be applied. (from linux-next tree, but problem found/seen in mmotm) Sorry, I don't know who is responsible for this driver. ~~~~~~~~~~~~~~~~~~~~~~ on i386: CC sound/hda/intel-nhlt.o ../sound/hda/intel-nhlt.c:14:25: error: redefinition of ‘intel_nhlt_init’ struct nhlt_acpi_table *intel_nhlt_init(struct device *dev) ^~~~~~~~~~~~~~~ In file included from ../sound/hda/intel-nhlt.c:5:0: ../include/sound/intel-nhlt.h:134:39: note: previous definition of ‘intel_nhlt_init’ was here static inline struct nhlt_acpi_table *intel_nhlt_init(struct device *dev) ^~~~~~~~~~~~~~~ ../sound/hda/intel-nhlt.c: In function ‘intel_nhlt_init’: ../sound/hda/intel-nhlt.c:39:14: error: dereferencing pointer to incomplete type ‘struct nhlt_resource_desc’ if (nhlt_ptr->length) ^~ ../sound/hda/intel-nhlt.c:41:4: error: implicit declaration of function ‘memremap’; did you mean ‘ioremap’? [-Werror=implicit-function-declaration] memremap(nhlt_ptr->min_addr, nhlt_ptr->length, ^~~~~~~~ ioremap ../sound/hda/intel-nhlt.c:42:6: error: ‘MEMREMAP_WB’ undeclared (first use in this function) MEMREMAP_WB); ^~~~~~~~~~~ ../sound/hda/intel-nhlt.c:42:6: note: each undeclared identifier is reported only once for each function it appears in ../sound/hda/intel-nhlt.c:45:25: error: dereferencing pointer to incomplete type ‘struct nhlt_acpi_table’ (strncmp(nhlt_table->header.signature, ^~ ../sound/hda/intel-nhlt.c:48:3: error: implicit declaration of function ‘memunmap’; did you mean ‘vunmap’? [-Werror=implicit-function-declaration] memunmap(nhlt_table); ^~~~~~~~ vunmap ../sound/hda/intel-nhlt.c: At top level: ../sound/hda/intel-nhlt.c:56:6: error: redefinition of ‘intel_nhlt_free’ void intel_nhlt_free(struct nhlt_acpi_table *nhlt) ^~~~~~~~~~~~~~~ In file included from ../sound/hda/intel-nhlt.c:5:0: ../include/sound/intel-nhlt.h:139:20: note: previous definition of ‘intel_nhlt_free’ was here static inline void intel_nhlt_free(struct nhlt_acpi_table *addr) ^~~~~~~~~~~~~~~ ../sound/hda/intel-nhlt.c:62:5: error: redefinition of ‘intel_nhlt_get_dmic_geo’ int intel_nhlt_get_dmic_geo(struct device *dev, struct nhlt_acpi_table *nhlt) ^~~~~~~~~~~~~~~~~~~~~~~ In file included from ../sound/hda/intel-nhlt.c:5:0: ../include/sound/intel-nhlt.h:143:19: note: previous definition of ‘intel_nhlt_get_dmic_geo’ was here static inline int intel_nhlt_get_dmic_geo(struct device *dev, ^~~~~~~~~~~~~~~~~~~~~~~ ../sound/hda/intel-nhlt.c: In function ‘intel_nhlt_get_dmic_geo’: ../sound/hda/intel-nhlt.c:76:11: error: dereferencing pointer to incomplete type ‘struct nhlt_endpoint’ if (epnt->linktype == NHLT_LINK_DMIC) { ^~ ../sound/hda/intel-nhlt.c:76:25: error: ‘NHLT_LINK_DMIC’ undeclared (first use in this function) if (epnt->linktype == NHLT_LINK_DMIC) { ^~~~~~~~~~~~~~ ../sound/hda/intel-nhlt.c:79:15: error: dereferencing pointer to incomplete type ‘struct nhlt_dmic_array_config’ switch (cfg->array_type) { ^~ ../sound/hda/intel-nhlt.c:80:9: error: ‘NHLT_MIC_ARRAY_2CH_SMALL’ undeclared (first use in this function) case NHLT_MIC_ARRAY_2CH_SMALL: ^~~~~~~~~~~~~~~~~~~~~~~~ ../sound/hda/intel-nhlt.c:81:9: error: ‘NHLT_MIC_ARRAY_2CH_BIG’ undeclared (first use in this function); did you mean ‘NHLT_MIC_ARRAY_2CH_SMALL’? case NHLT_MIC_ARRAY_2CH_BIG: ^~~~~~~~~~~~~~~~~~~~~~ NHLT_MIC_ARRAY_2CH_SMALL ../sound/hda/intel-nhlt.c:82:16: error: ‘MIC_ARRAY_2CH’ undeclared (first use in this function); did you mean ‘NHLT_MIC_ARRAY_2CH_BIG’? dmic_geo = MIC_ARRAY_2CH; ^~~~~~~~~~~~~ NHLT_MIC_ARRAY_2CH_BIG ../sound/hda/intel-nhlt.c:85:9: error: ‘NHLT_MIC_ARRAY_4CH_1ST_GEOM’ undeclared (first use in this function); did you mean ‘NHLT_MIC_ARRAY_2CH_BIG’? case NHLT_MIC_ARRAY_4CH_1ST_GEOM: ^~~~~~~~~~~~~~~~~~~~~~~~~~~ NHLT_MIC_ARRAY_2CH_BIG ../sound/hda/intel-nhlt.c:86:9: error: ‘NHLT_MIC_ARRAY_4CH_L_SHAPED’ undeclared (first use in this function); did you mean ‘NHLT_MIC_ARRAY_4CH_1ST_GEOM’? case NHLT_MIC_ARRAY_4CH_L_SHAPED: ^~~~~~~~~~~~~~~~~~~~~~~~~~~ NHLT_MIC_ARRAY_4CH_1ST_GEOM AR sound/i2c/other/built-in.a ../sound/hda/intel-nhlt.c:87:9: error: ‘NHLT_MIC_ARRAY_4CH_2ND_GEOM’ undeclared (first use in this function); did you mean ‘NHLT_MIC_ARRAY_4CH_1ST_GEOM’? case NHLT_MIC_ARRAY_4CH_2ND_GEOM: ^~~~~~~~~~~~~~~~~~~~~~~~~~~ NHLT_MIC_ARRAY_4CH_1ST_GEOM ../sound/hda/intel-nhlt.c:88:16: error: ‘MIC_ARRAY_4CH’ undeclared (first use in this function); did you mean ‘MIC_ARRAY_2CH’? dmic_geo = MIC_ARRAY_4CH; ^~~~~~~~~~~~~ MIC_ARRAY_2CH AR sound/i2c/built-in.a CC drivers/bluetooth/btmtksdio.o ../sound/hda/intel-nhlt.c:90:9: error: ‘NHLT_MIC_ARRAY_VENDOR_DEFINED’ undeclared (first use in this function); did you mean ‘NHLT_MIC_ARRAY_4CH_L_SHAPED’? case NHLT_MIC_ARRAY_VENDOR_DEFINED: ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ NHLT_MIC_ARRAY_4CH_L_SHAPED ../sound/hda/intel-nhlt.c:92:26: error: dereferencing pointer to incomplete type ‘struct nhlt_vendor_dmic_array_config’ dmic_geo = cfg_vendor->nb_mics; ^~ ../sound/hda/intel-nhlt.c: At top level: ../sound/hda/intel-nhlt.c:106:16: error: expected declaration specifiers or ‘...’ before string constant MODULE_LICENSE("GPL v2"); ^~~~~~~~ ../sound/hda/intel-nhlt.c:107:20: error: expected declaration specifiers or ‘...’ before string constant MODULE_DESCRIPTION("Intel NHLT driver"); ^~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors make[3]: *** [../scripts/Makefile.build:266: sound/hda/intel-nhlt.o] Error 1 -- ~Randy

3 8

[alsa-devel] [PATCH] ALSA: firewire-lib: fix isoc cycle count to which rx packet is scheduled
by Takashi Sakamoto 29 Aug '19

29 Aug '19

When introducing the list of packet descriptor, for rx packet, the calculation of scheduled isoc cycle is omitted. This commit fixes the bug. Fixes: f4f6ae7b7c1f ("ALSA: firewire-lib: use packet descriptor for IT context") Signed-off-by: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> --- sound/firewire/amdtp-stream.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/firewire/amdtp-stream.c b/sound/firewire/amdtp-stream.c index 1a92855c7647..88270257e896 100644 --- a/sound/firewire/amdtp-stream.c +++ b/sound/firewire/amdtp-stream.c @@ -732,7 +732,7 @@ static void generate_ideal_pkt_descs(struct amdtp_stream *s, struct pkt_desc *desc = descs + i; unsigned int index = (s->packet_index + i) % QUEUE_LENGTH; - desc->cycle = compute_cycle_count(*ctx_header); + desc->cycle = compute_it_cycle(*ctx_header); desc->syt = calculate_syt(s, desc->cycle); desc->data_blocks = calculate_data_blocks(s, desc->syt); -- 2.20.1

2 1

[alsa-devel] [PATCH] ALSA: dice: fix error path for initialization of stream structures
by Takashi Sakamoto 29 Aug '19

29 Aug '19

When introducing AMDTP domain to ALSA dice driver, error path does not handle error correctly. This commit fixes the bug. Fixes: e9f21129b8d8 ("ALSA: dice: support AMDTP domain") Signed-off-by: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> --- sound/firewire/dice/dice-stream.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/firewire/dice/dice-stream.c b/sound/firewire/dice/dice-stream.c index af8c5a2c28f3..f6a8627ae5a2 100644 --- a/sound/firewire/dice/dice-stream.c +++ b/sound/firewire/dice/dice-stream.c @@ -570,7 +570,7 @@ int snd_dice_stream_init_duplex(struct snd_dice *dice) destroy_stream(dice, AMDTP_OUT_STREAM, i); for (i = 0; i < MAX_STREAMS; i++) destroy_stream(dice, AMDTP_IN_STREAM, i); - break; + goto end; } } -- 2.20.1

2 1

[alsa-devel] mailing list outage
by Jaroslav Kysela 29 Aug '19

29 Aug '19

Hi, I am really sorry, but the e-mail delivery was paused for more than one day for this alsa-devel mailing list. The problem is fixed right now (messages from the queue are just being delivered). No messages should be lost. Jaroslav -- Jaroslav Kysela <perex(a)perex.cz> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.

1 0

[alsa-devel] [PATCH v2 0/4] wm8904: adapt driver for use with audio-graph-card
by Michał Mirosław 29 Aug '19

29 Aug '19

This series allows to use WM8904 codec as audio-graph-card component. It starts with rework of FLL handling in the codec's driver, and as an example includes (untested) rework for codec with similar FLL: WM8994. Series based on tiwai/sound/for-next tree. You can also pull from: https://rere.qmqm.pl/git/linux branch: wm8904 (branch includes two fixes already sent to alsa-devel, but not merged yet). Michał Mirosław (4): ASoC: wm_fll: extract common code for Wolfson FLLs ASoC: wm8904: use common FLL code ASoC: wm8904: automatically choose clock source [RFT] ASoC: wm8994: use common FLL code sound/soc/atmel/atmel_wm8904.c | 11 +- sound/soc/codecs/Kconfig | 9 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/wm8904.c | 516 +++++++++++--------------------- sound/soc/codecs/wm8904.h | 5 - sound/soc/codecs/wm8994.c | 281 +++++------------- sound/soc/codecs/wm8994.h | 4 +- sound/soc/codecs/wm_fll.c | 518 +++++++++++++++++++++++++++++++++ sound/soc/codecs/wm_fll.h | 60 ++++ 9 files changed, 849 insertions(+), 557 deletions(-) create mode 100644 sound/soc/codecs/wm_fll.c create mode 100644 sound/soc/codecs/wm_fll.h -- 2.20.1

3 6

[alsa-devel] [PATCH] ASoC: cs42xx8: Force suspend/resume during system suspend/resume
by Shengjiu Wang 29 Aug '19

29 Aug '19

Use force_suspend/resume to make sure clocks are disabled/enabled accordingly during system suspend/resume. Signed-off-by: Dong Aisheng <aisheng.dong(a)nxp.com> Signed-off-by: Shengjiu Wang <shengjiu.wang(a)nxp.com> --- sound/soc/codecs/cs42xx8.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/soc/codecs/cs42xx8.c b/sound/soc/codecs/cs42xx8.c index 5b049fcdba20..94b1adb088fd 100644 --- a/sound/soc/codecs/cs42xx8.c +++ b/sound/soc/codecs/cs42xx8.c @@ -684,6 +684,8 @@ static int cs42xx8_runtime_suspend(struct device *dev) #endif const struct dev_pm_ops cs42xx8_pm = { + SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, + pm_runtime_force_resume) SET_RUNTIME_PM_OPS(cs42xx8_runtime_suspend, cs42xx8_runtime_resume, NULL) }; EXPORT_SYMBOL_GPL(cs42xx8_pm); -- 2.21.0

4 3

[alsa-devel] [PATCH v2 00/25] ASoC: cleanup patches for soc-core
by Kuninori Morimoto 29 Aug '19

29 Aug '19

Hi Mark These are v2 of soc-core cleanup patches. I removed 3 patches which are reviewd from Ranjani / Pierre-Louis. Kuninori Morimoto (25): ASoC: soc-core: use device_register() ASoC: soc-core: set component->debugfs_root NULL ASoC: soc-core: add comment for for_each_xxx ASoC: soc-core: check return value of snd_soc_add_dai_link() ASoC: soc-core: don't use for_each_card_links_safe() at snd_soc_find_dai_link() ASoC: soc-core: reuse rtdcom at snd_soc_rtdcom_add() ASoC: soc-core: tidyup for snd_soc_dapm_new_controls() ASoC: soc-core: tidyup for snd_soc_add_component_controls() ASoC: soc-core: tidyup for snd_soc_dapm_add_routes() ASoC: soc-core: tidyup for snd_soc_add_card_controls() ASoC: soc-core: call snd_soc_dapm_debugfs_init() at soc_init_card_debugfs() ASoC: soc-core: remove unneeded list_empty() check for snd_soc_try_rebind_card() ASoC: soc-core: tidyup for card->deferred_resume_work ASoC: soc-core: define soc_dpcm_debugfs_add() for non CONFIG_DEBUG_FS ASoC: soc-core: dai_link check under soc_dpcm_debugfs_add() ASoC: soc-core: merge snd_soc_initialize_card_lists() ASoC: soc-core: remove unneeded dai_link check from snd_soc_remove_dai_link() ASoC: soc-core: add NOTE to snd_soc_rtdcom_lookup() ASoC: soc-core: don't call snd_soc_component_set_jack() ASoC: soc-core: don't alloc memory carelessly when creating debugfs ASoC: soc-core: soc_cleanup_card_resources() become void ASoC: soc-core: initialize component list ASoC: soc-core: initialize list at one place ASoC: soc-dai: use bit field for bus_control ASoC: soc-topology: use for_each_component_dais() at remove_dai() include/sound/soc-dai.h | 3 +- include/sound/soc-dpcm.h | 9 ++- include/sound/soc.h | 15 +--- sound/soc/soc-core.c | 197 ++++++++++++++++++++++++----------------------- sound/soc/soc-pcm.c | 3 + sound/soc/soc-topology.c | 2 +- 6 files changed, 118 insertions(+), 111 deletions(-) -- 2.7.4

6 53

[alsa-devel] [PATCH] ALSA: hda - Fix potential endless loop at applying quirks
by Takashi Iwai 29 Aug '19

29 Aug '19

Since the chained quirks via chained_before flag is applied before the depth check, it may lead to the endless recursive calls, when the chain were set up incorrectly. Fix it by moving the depth check at the beginning of the loop. Fixes: 1f57825077dc ("ALSA: hda - Add chained_before flag to the fixup entry") Cc: <stable(a)vger.kernel.org> Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/pci/hda/hda_auto_parser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/pci/hda/hda_auto_parser.c b/sound/pci/hda/hda_auto_parser.c index 8a134f750bae..2c6d2becfe1a 100644 --- a/sound/pci/hda/hda_auto_parser.c +++ b/sound/pci/hda/hda_auto_parser.c @@ -824,6 +824,8 @@ static void apply_fixup(struct hda_codec *codec, int id, int action, int depth) while (id >= 0) { const struct hda_fixup *fix = codec->fixup_list + id; + if (++depth > 10) + break; if (fix->chained_before) apply_fixup(codec, fix->chain_id, action, depth + 1); @@ -863,8 +865,6 @@ static void apply_fixup(struct hda_codec *codec, int id, int action, int depth) } if (!fix->chained || fix->chained_before) break; - if (++depth > 10) - break; id = fix->chain_id; } } -- 2.16.4

1 0

[alsa-devel] Options for ASYNC feedback source in USB-audio gadget (USB OTG)?
by Pavel Hofman 28 Aug '19

28 Aug '19

Hi, The existing usb-audio v.2 gadget defines its EP OUT as asynchronous https://github.com/torvalds/linux/blob/master/drivers/usb/gadget/function/f… , but does not define any feedback IN EP (which is against USB audio class 2.0 specs). Windows 10 USB audio class 2 driver requires presence of the feedback EP. By changing the EP OUT to adaptive (which in fact it is), we could fix the driver issue and windows plays through such device (RPi4) correctly now (linux/OSX already worked OK before). We will post a patch soon. RPi can now be used as an "intelligent" USB soundcard, allowing further processing in linux on RPi, before outputting to some final sound device. However, there are two clock domains involved - the host USB controller sending stream to the USB gadget, and the output soundcard of the gadget. While adaptive resampling is possible (jackd, PA do that), IMO the correct solution is going back to ASYNC EP OUT in the audio gadget and implementing a proper feedback EP. But the feedback EP needs input data. The whole chain behind the virtual capture alsa device defined by the gadget driver is clocked by the playback soundcard. May I ask for your opinions on possible options for 1) gathering the feedback data from the playback chain 2) passing the gathered samplerate-diff data to the gadget module for the feedback EP to send to the USB host A general solution would be very useful, at best without using alsa-lib defined PCM devices (e.g. Java still supports only hardware devices (hw:X), not PCM devices defined by alsa-lib). I very much appreciate your suggestions and help. Best regards, Pavel.

1 2

[alsa-devel] [PATCH] ALSA: hda - Update descriptions about new position_fix values
by Takashi Iwai 28 Aug '19

28 Aug '19

The new values for the recent Intel and AMD chips are missing in the documentation. Add the new descriptions. Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- Documentation/sound/alsa-configuration.rst | 2 ++ Documentation/sound/hd-audio/notes.rst | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/Documentation/sound/alsa-configuration.rst b/Documentation/sound/alsa-configuration.rst index 4a3cecc8ad38..02aacd69ab96 100644 --- a/Documentation/sound/alsa-configuration.rst +++ b/Documentation/sound/alsa-configuration.rst @@ -1001,6 +1001,8 @@ position_fix 2 = POSBUF: use position buffer, 3 = VIACOMBO: VIA-specific workaround for capture, 4 = COMBO: use LPIB for playback, auto for capture stream + 5 = SKL+: apply the delay calculation available on recent Intel chips + 6 = FIFO: correct the position with the fixed FIFO size, for recent AMD chips probe_mask Bitmask to probe codecs (default = -1, meaning all slots); When the bit 8 (0x100) is set, the lower 8 bits are used diff --git a/Documentation/sound/hd-audio/notes.rst b/Documentation/sound/hd-audio/notes.rst index 9f7347830ba4..0f3109d9abc8 100644 --- a/Documentation/sound/hd-audio/notes.rst +++ b/Documentation/sound/hd-audio/notes.rst @@ -66,6 +66,11 @@ by comparing both LPIB and position-buffer values. ``position_fix=4`` is another combination available for all controllers, and uses LPIB for the playback and the position-buffer for the capture streams. +``position_fix=5`` is specific to Intel platforms, so far, for Skylake +and onward. It applies the delay calculation for the precise position +reporting. +``position_fix=6`` is to correct the position with the fixed FIFO +size, mainly targeted for the recent AMD controllers. 0 is the default value for all other controllers, the automatic check and fallback to LPIB as described in the above. If you get a problem of repeated sounds, this option might -- 2.16.4

1 0