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

[alsa-devel] [PATCH alsa-lib, alsa-plugins] Update the attributes.m4 macro file from xine
by David Ward 23 Dec '19

23 Dec '19

This file was imported from the xine project. Update it to the current revision, which resolves the "no AC_LANG_SOURCE call detected in body" warnings with Autoconf 2.68 or later. Signed-off-by: David Ward <david.ward(a)ll.mit.edu> --- m4/attributes.m4 | 33 ++++++++++++++++++++------------- 1 file changed, 20 insertions(+), 13 deletions(-) diff --git a/m4/attributes.m4 b/m4/attributes.m4 index e86456a..3d9c256 100644 --- a/m4/attributes.m4 +++ b/m4/attributes.m4 @@ -1,6 +1,6 @@ dnl Macros to check the presence of generic (non-typed) symbols. -dnl Copyright (c) 2006-2007 Diego Pettenò <flameeyes(a)gmail.com> -dnl Copyright (c) 2006-2007 xine project +dnl Copyright (c) 2006-2008 Diego Pettenò <flameeyes(a)gmail.com> +dnl Copyright (c) 2006-2008 xine project dnl dnl This program is free software; you can redistribute it and/or modify dnl it under the terms of the GNU General Public License as published by @@ -25,7 +25,7 @@ dnl License when using or distributing such scripts, even though portions dnl of the text of the Macro appear in them. The GNU General Public dnl License (GPL) does govern all other use of the material that dnl constitutes the Autoconf Macro. -dnl +dnl dnl This special exception to the GPL applies to versions of the dnl Autoconf Macro released by this project. When you make and dnl distribute a modified version of the Autoconf Macro, you may extend @@ -39,7 +39,7 @@ AC_DEFUN([CC_CHECK_CFLAGS_SILENT], [ AC_CACHE_VAL(AS_TR_SH([cc_cv_cflags_$1]), [ac_save_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS $1" - AC_COMPILE_IFELSE([int a;], + AC_COMPILE_IFELSE([AC_LANG_SOURCE([int a;])], [eval "AS_TR_SH([cc_cv_cflags_$1])='yes'"], [eval "AS_TR_SH([cc_cv_cflags_$1])='no'"]) CFLAGS="$ac_save_CFLAGS" @@ -71,7 +71,7 @@ AC_DEFUN([CC_CHECK_CFLAG_APPEND], [ ) AS_IF([eval test x$]AS_TR_SH([cc_cv_cflags_$1])[ = xyes], - [CFLAGS="$CFLAGS $1"; $2], [$3]) + [CFLAGS="$CFLAGS $1"; DEBUG_CFLAGS="$DEBUG_CFLAGS $1"; $2], [$3]) ]) dnl CC_CHECK_CFLAGS_APPEND([FLAG1 FLAG2], [action-if-found], [action-if-not]) @@ -89,7 +89,7 @@ AC_DEFUN([CC_CHECK_LDFLAGS], [ AS_TR_SH([cc_cv_ldflags_$1]), [ac_save_LDFLAGS="$LDFLAGS" LDFLAGS="$LDFLAGS $1" - AC_LINK_IFELSE([int main() { return 1; }], + AC_LINK_IFELSE([AC_LANG_SOURCE([int main() { return 1; }])], [eval "AS_TR_SH([cc_cv_ldflags_$1])='yes'"], [eval "AS_TR_SH([cc_cv_ldflags_$1])="]) LDFLAGS="$ac_save_LDFLAGS" @@ -109,14 +109,21 @@ AC_DEFUN([CC_NOUNDEFINED], [ dnl FreeBSD (et al.) does not complete linking for shared objects when pthreads dnl are requested, as different implementations are present; to avoid problems dnl use -Wl,-z,defs only for those platform not behaving this way. - *-freebsd*) ;; + dnl + dnl MinGW platforms: for libraries required -no-undefined, + dnl use it only for libraries in mingw32-w64 + + *-freebsd* | *-openbsd*) ;; + *-mingw*) + LDFLAGS_NOUNDEFINED="-no-undefined" + ;; *) dnl First of all check for the --no-undefined variant of GNU ld. This allows dnl for a much more readable commandline, so that people can understand what dnl it does without going to look for what the heck -z defs does. - for possible_flags in "-Wl,--no-undefined" "-Wl,-z,defs"; do + for possible_flags in "-Wl,--no-undefined" "-Wl,-z,defs"; do CC_CHECK_LDFLAGS([$possible_flags], [LDFLAGS_NOUNDEFINED="$possible_flags"]) - break + if test "x$LDFLAGS_NOUNDEFINED" = "x"; then break; fi done ;; esac @@ -147,7 +154,7 @@ AC_DEFUN([CC_CHECK_ATTRIBUTE], [ AS_TR_SH([cc_cv_attribute_$1]), [ac_save_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS $cc_cv_werror" - AC_COMPILE_IFELSE([$3], + AC_COMPILE_IFELSE([AC_LANG_SOURCE([$3])], [eval "AS_TR_SH([cc_cv_attribute_$1])='yes'"], [eval "AS_TR_SH([cc_cv_attribute_$1])='no'"]) CFLAGS="$ac_save_CFLAGS" @@ -257,7 +264,7 @@ AC_DEFUN([CC_FLAG_VISIBILITY], [ cc_cv_flag_visibility='yes', cc_cv_flag_visibility='no') CFLAGS="$cc_flag_visibility_save_CFLAGS"]) - + AS_IF([test "x$cc_cv_flag_visibility" = "xyes"], [AC_DEFINE([SUPPORT_FLAG_VISIBILITY], 1, [Define this if the compiler supports the -fvisibility flag]) @@ -295,11 +302,11 @@ AC_DEFUN([CC_ATTRIBUTE_ALIGNED], [ [ac_save_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS $cc_cv_werror" for cc_attribute_align_try in 64 32 16 8 4 2; do - AC_COMPILE_IFELSE([ + AC_COMPILE_IFELSE([AC_LANG_SOURCE([ int main() { static char c __attribute__ ((aligned($cc_attribute_align_try))) = 0; return c; - }], [cc_cv_attribute_aligned=$cc_attribute_align_try; break]) + }])], [cc_cv_attribute_aligned=$cc_attribute_align_try; break]) done CFLAGS="$ac_save_CFLAGS" ]) -- 1.8.3.1

1 0

[alsa-devel] [PATCH for-5.6 0/4] ALSA: ctl: UAPI updates for obsoleted members
by Takashi Sakamoto 23 Dec '19

23 Dec '19

Hi, As a result of applying patchset for Y2038 issue[1], 'snd_ctl_elem_value' structure is changed to lost 'tstamp' member. This change should corresponds to removal of an access flag of 'snd_ctl_elem_info' structure. These changes brings slight lose of backward compatibility, thus it's better to bump protocol version. Additionally, for recent few year I work to obsolete 'dimen' member from 'snd_ctl_elem_info' structure. It's good timing to attempt toward it. This patchset includes these changes and bump the version up to v2.1.0. [1] https://mailman.alsa-project.org/pipermail/alsa-devel/2019-December/160082.… Takashi Sakamoto (4): ALSA: ctl: remove unused macro for timestamping of elem_value ALSA: pci: echoaudio: remove usage of dimen menber of elem_value structure ALSA: ctl: remove dimen member from elem_info structure ALSA: ctl: bump protocol version up to v2.1.0 include/uapi/sound/asound.h | 10 +++------- sound/core/control.c | 32 -------------------------------- sound/pci/echoaudio/echoaudio.c | 11 ----------- 3 files changed, 3 insertions(+), 50 deletions(-) -- 2.20.1

2 5

[alsa-devel] [PATCH] ASoC: rt5682: Add option to select pulse IRQ in jack detect
by Akshu Agrawal 23 Dec '19

23 Dec '19

Some SoC need to set IRQ type as pulse along with other JD1 options. Signed-off-by: Akshu Agrawal <akshu.agrawal(a)amd.com> --- include/sound/rt5682.h | 1 + sound/soc/codecs/rt5682.c | 3 +++ sound/soc/codecs/rt5682.h | 2 ++ 3 files changed, 6 insertions(+) diff --git a/include/sound/rt5682.h b/include/sound/rt5682.h index bc2c31734df1..64cfa77ec9ee 100644 --- a/include/sound/rt5682.h +++ b/include/sound/rt5682.h @@ -22,6 +22,7 @@ enum rt5682_dmic1_clk_pin { enum rt5682_jd_src { RT5682_JD_NULL, RT5682_JD1, + RT5682_JD2, }; struct rt5682_platform_data { diff --git a/sound/soc/codecs/rt5682.c b/sound/soc/codecs/rt5682.c index ae6f6121bc1b..5135d7757361 100644 --- a/sound/soc/codecs/rt5682.c +++ b/sound/soc/codecs/rt5682.c @@ -1009,6 +1009,9 @@ static int rt5682_set_jack_detect(struct snd_soc_component *component, } switch (rt5682->pdata.jd_src) { + case RT5682_JD2: + regmap_update_bits(rt5682->regmap, RT5682_IRQ_CTRL_2, + RT5682_JD1_PULSE_MASK, RT5682_JD1_PULSE_EN); case RT5682_JD1: snd_soc_component_update_bits(component, RT5682_CBJ_CTRL_2, RT5682_EXT_JD_SRC, RT5682_EXT_JD_SRC_MANUAL); diff --git a/sound/soc/codecs/rt5682.h b/sound/soc/codecs/rt5682.h index 18faaa2a49a0..434b1c9778b2 100644 --- a/sound/soc/codecs/rt5682.h +++ b/sound/soc/codecs/rt5682.h @@ -1091,6 +1091,8 @@ #define RT5682_JD1_POL_MASK (0x1 << 13) #define RT5682_JD1_POL_NOR (0x0 << 13) #define RT5682_JD1_POL_INV (0x1 << 13) +#define RT5682_JD1_PULSE_MASK (0x1 << 10) +#define RT5682_JD1_PULSE_EN (0x1 << 10) /* IRQ Control 3 (0x00b8) */ #define RT5682_IL_IRQ_MASK (0x1 << 7) -- 2.17.1

2 1

[alsa-devel] How to install on debian buster?
by GitHub issues - opened 23 Dec '19

23 Dec '19

alsa-project/alsa-ucm-conf issue #8 was opened from kopax: I use kernel 5.3 on debian buster and I am following thos offical Lenovo tutorial to install the microphone on my x1c7: https://forums.lenovo.com/lnv/attachments/lnv/lx02_en/3199/1/Enabling%20aud… It says: ![image](https://user-images.githubusercontent.com/1866564/71328607-1a5e8180-251a-11ea-8621-1655d7746c60.png) Now the [link](https://github.com/alsa-project/alsa-ucm-conf/tree/master/ucm/sof-skl… is 404, what do I need to get now ? Thanks in advance and merry xmas! Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/8 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[alsa-devel] [PATCH v3 0/2] Add the broadmobi BM818
by Angus Ainslie (Purism) 22 Dec '19

22 Dec '19

The broadmobi uses slightly different parameters from the option modems so add the paramters and document them. Changes since v2: Use of_device_get_match_data to get the dai data. Updated subject styles. Changes since v1: Cut back the CC list. Use data from compatible match instead of explicit compatible match. Angus Ainslie (Purism) (2): ASoC: gtm601: add Broadmobi bm818 sound profile dt-bindings: sound: gtm601: add the broadmobi interface .../devicetree/bindings/sound/gtm601.txt | 10 ++++-- sound/soc/codecs/gtm601.c | 31 ++++++++++++++++--- 2 files changed, 35 insertions(+), 6 deletions(-) -- 2.17.1

3 5

[alsa-devel] [sound:for-next 163/166] ./usr/include/sound/hdspm.h:44:2: error: unknown type name '__u32'
by kbuild test robot 22 Dec '19

22 Dec '19

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: bfea224d9250e56ed5475bf72190783a7e1d8e43 commit: 4fa406caf950fd46ae06bccf9a4c72171401d203 [163/166] ALSA: hdspm: Drop linux/types.h inclusion in uapi header config: i386-allyesconfig (attached as .config) compiler: gcc-7 (Debian 7.5.0-3) 7.5.0 reproduce: git checkout 4fa406caf950fd46ae06bccf9a4c72171401d203 # save the attached .config to linux build tree make ARCH=i386 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp(a)intel.com> All errors (new ones prefixed by >>): In file included from <command-line>:32:0: >> ./usr/include/sound/hdspm.h:44:2: error: unknown type name '__u32' __u32 input_peaks[64]; ^~~~~ ./usr/include/sound/hdspm.h:45:2: error: unknown type name '__u32' __u32 playback_peaks[64]; ^~~~~ ./usr/include/sound/hdspm.h:46:2: error: unknown type name '__u32' __u32 output_peaks[64]; ^~~~~ >> ./usr/include/sound/hdspm.h:48:2: error: unknown type name '__u64' __u64 input_rms[64]; ^~~~~ ./usr/include/sound/hdspm.h:49:2: error: unknown type name '__u64' __u64 playback_rms[64]; ^~~~~ ./usr/include/sound/hdspm.h:50:2: error: unknown type name '__u64' __u64 output_rms[64]; ^~~~~ >> ./usr/include/sound/hdspm.h:52:2: error: unknown type name '__u8' __u8 speed; /* enum {ss, ds, qs} */ ^~~~ ./usr/include/sound/hdspm.h:153:2: error: unknown type name '__u8' __u8 card_type; /* enum hdspm_io_type */ ^~~~ ./usr/include/sound/hdspm.h:156:2: error: unknown type name '__u64' __u64 card_clock; ^~~~~ ./usr/include/sound/hdspm.h:157:2: error: unknown type name '__u32' __u32 master_period; ^~~~~ ./usr/include/sound/hdspm.h:161:4: error: unknown type name '__u8' __u8 sync_wc; /* enum hdspm_sync */ ^~~~ ./usr/include/sound/hdspm.h:162:4: error: unknown type name '__u8' __u8 sync_madi; /* enum hdspm_sync */ ^~~~ ./usr/include/sound/hdspm.h:163:4: error: unknown type name '__u8' __u8 sync_tco; /* enum hdspm_sync */ ^~~~ ./usr/include/sound/hdspm.h:164:4: error: unknown type name '__u8' __u8 sync_in; /* enum hdspm_sync */ ^~~~ ./usr/include/sound/hdspm.h:165:4: error: unknown type name '__u8' __u8 madi_input; /* enum hdspm_madi_input */ ^~~~ ./usr/include/sound/hdspm.h:166:4: error: unknown type name '__u8' __u8 channel_format; /* enum hdspm_madi_channel_format */ ^~~~ ./usr/include/sound/hdspm.h:167:4: error: unknown type name '__u8' __u8 frame_format; /* enum hdspm_madi_frame_format */ ^~~~ ./usr/include/sound/hdspm.h:182:2: error: unknown type name '__u8' __u8 card_type; /* enum hdspm_io_type */ ^~~~ --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation

1 0

[alsa-devel] ctl: elem_id/elem_value: add APIs to check whether two instances have the same meaning
by GitHub pull_request - opened 22 Dec '19

22 Dec '19

alsa-project/alsa-gobject pull request #2 was opened from takaswie: For some cases, it's convenient for users to check whether two instances of elem_id/elem_value have the same meaning. In glib, APIs for comparison of content of objects are named as equal. This commit adds equal APIs for this purpose. Request URL : https://github.com/alsa-project/alsa-gobject/pull/2 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/2.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

[alsa-devel] ctl: elem_info_int: fix wrong assignment to private value
by GitHub pull_request - opened 22 Dec '19

22 Dec '19

alsa-project/alsa-gobject pull request #1 was opened from takaswie: ALSACtl::ElemInfoInt has properties for 'min', 'max' and 'step' but their assignments are not proper. This commit fixes the bug. Fixes: aac870451a89 ("ctl: elem_info_int: add properties for type-dependent parameters and constructor") Signed-off-by: Takashi Sakamoto <o-takashi(a)sakamocchi.jp> Request URL : https://github.com/alsa-project/alsa-gobject/pull/1 Patch URL : https://github.com/alsa-project/alsa-gobject/pull/1.patch Repository URL: https://github.com/alsa-project/alsa-gobject

1 0

[alsa-devel] [sound:for-next 162/166] ./usr/include/sound/hdsp.h:35:2: error: unknown type name '__u32'
by kbuild test robot 22 Dec '19

22 Dec '19

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: bfea224d9250e56ed5475bf72190783a7e1d8e43 commit: d63e63d4210713ddc7eb1b2d2afc4a8d42ccf91e [162/166] ALSA: hdsp: Make uapi/hdsp.h compilable again config: i386-allyesconfig (attached as .config) compiler: gcc-7 (Debian 7.5.0-3) 7.5.0 reproduce: git checkout d63e63d4210713ddc7eb1b2d2afc4a8d42ccf91e # save the attached .config to linux build tree make ARCH=i386 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp(a)intel.com> All errors (new ones prefixed by >>): In file included from <command-line>:32:0: >> ./usr/include/sound/hdsp.h:35:2: error: unknown type name '__u32' __u32 input_peaks[26]; ^~~~~ ./usr/include/sound/hdsp.h:36:2: error: unknown type name '__u32' __u32 playback_peaks[26]; ^~~~~ ./usr/include/sound/hdsp.h:37:2: error: unknown type name '__u32' __u32 output_peaks[28]; ^~~~~ >> ./usr/include/sound/hdsp.h:38:2: error: unknown type name '__u64' __u64 input_rms[26]; ^~~~~ ./usr/include/sound/hdsp.h:39:2: error: unknown type name '__u64' __u64 playback_rms[26]; ^~~~~ ./usr/include/sound/hdsp.h:41:2: error: unknown type name '__u64' __u64 output_rms[26]; ^~~~~ --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation

1 0

[alsa-devel] [sound:for-next 161/166] usr/include/sound/emu10k1.h:322: found __[us]{8, 16, 32, 64} type without #include <linux/types.h>
by kbuild test robot 21 Dec '19

21 Dec '19

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: bfea224d9250e56ed5475bf72190783a7e1d8e43 commit: 2e4688676392767e16c1adeca4cc2c083e2db13f [161/166] ALSA: emu10k1: Make uapi/emu10k1.h compilable again config: i386-allyesconfig (attached as .config) compiler: gcc-7 (Debian 7.5.0-3) 7.5.0 reproduce: git checkout 2e4688676392767e16c1adeca4cc2c083e2db13f # save the attached .config to linux build tree make ARCH=i386 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp(a)intel.com> All warnings (new ones prefixed by >>): >> usr/include/sound/emu10k1.h:322: found __[us]{8,16,32,64} type without #include <linux/types.h> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation

1 0