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

[alsa-devel] [PATCH 0/5 v2] fbdev: sh_mobile_hdmi: bug fix patches V2
by Kuninori Morimoto 10 Sep '10

10 Sep '10

Dear Mark, Liam, Guennadi These are ALSA V2 bug fix patches which are reported by Guennadi. Kuninori Morimoto (5): fbdev: sh_mobile_hdmi: modify noisy comment out fbdev: sh_mobile_hdmi: modify flags name to more specific fbdev: sh_mobile_hdmi: modify snd_soc_dai_driver settings fbdev: sh_mobile_hdmi: add new label for sound error path ASoC: fsi-hdmi: remove unneeded header To Mark. Please care above patch independently from these patches. [alsa-devel] [PATCH 5/5] ASoC: fsi-ak4642: modiry platform_name These series are for ALSA side patches. I will send SH side V2 patches soon. Main diff v1 <==> v2 is I added Guennadi's report mail to log area on each patches, and care more. But these series still didn't care above. I added reasons. -- Guennadi ------------------------------------------ > +config SND_FSI_HDMI > + bool "FSI-HDMI sound support" > + depends on SND_SOC_SH4_FSI && FB_SH_MOBILE_HDMI > + help > + This option enables generic sound support for the > + FSI - HDMI unit > + "bool" means, if someone is linking the whole ASoC into the kernel, they will not be able to build this as a module. Not a big deal, but you're stealing some freedom from the user. ------------------------------------------------------- understand. But the config which use "bool" for FSI-XXX is not only FSI-HDMI. So, I will care your indication as "new function patch" in future. -- Guennadi ------------------------------------------ With this config option you will have 3 SND_SOC_SH4_FSI implementations in the Kconfig, all selectable independently. Do you really think it makes sense and would work, if someone were to select more than one of those options at the same time? ------------------------------------------------------- Yes. I created it for independently. For example, you can select FSI-AK4642 and FSI-HDMI in same time, and both works well for me on AP4 board now. But I guess, if you select FSI-DA7210 and FSI-HDMI, small patch which change FSIA <-> FSIB is needed. FSI-AK4642 : FSI-A FSI-DA7210 : FSI-B FSI-HDMI : FSI-B I think that it should be going to enable the selection of "FSI-A" and "FSI-B" in the future. But now, no way, no idea. It is my task By the way, about HDMI sound, I know that there are some HDMI monitors which can not use sound. Now I'm debuging about it. -- Guennadi ------------------------------------------ > + .formats = SNDRV_PCM_FMTBIT_S16_LE | SNDRV_PCM_FMTBIT_S24_LE, Again, this I am not sure about. The datasheet says 16 to 32 bit are possible, but then I only see configuration for 16 to 24 bits, but in any case, I think, you'd want to implement .hw_params to support non-default formats. ------------------------------------------------------- Yes. I should modify this .formats. But I can not select, test, support 32bit in current environment. So, please put it to my TODO list. And yes. .hw_params implementation is important for advanced support. But now, current HDMI sound support is still prototype (I didn't indicate though). So. please put it to my TODO list too. Best regards -- Kuninori Morimoto

3 12

[alsa-devel] [PATCH] ASoC: fsi codecs: modify menu attribute on Kconfig
by Kuninori Morimoto 10 Sep '10

10 Sep '10

Current SND_FSI_xxx menu attributes were bool, but it should be tristate. This patch solve below report from Guennadi "bool" means, if someone is linking the whole ASoC into the kernel, they will not be able to build this as a module. Not a big deal, but you're stealing some freedom from the user. Reported-by: Guennadi Liakhovetski <g.liakhovetski(a)gmx.de> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/sh/Kconfig | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/sh/Kconfig b/sound/soc/sh/Kconfig index 6b224d2..7f0a496 100644 --- a/sound/soc/sh/Kconfig +++ b/sound/soc/sh/Kconfig @@ -47,7 +47,7 @@ config SND_SH7760_AC97 AC97 unit of the SH7760. config SND_FSI_AK4642 - bool "FSI-AK4642 sound support" + tristate "FSI-AK4642 sound support" depends on SND_SOC_SH4_FSI && I2C_SH_MOBILE select SND_SOC_AK4642 help @@ -55,7 +55,7 @@ config SND_FSI_AK4642 FSI - AK4642 unit config SND_FSI_DA7210 - bool "FSI-DA7210 sound support" + tristate "FSI-DA7210 sound support" depends on SND_SOC_SH4_FSI && I2C_SH_MOBILE select SND_SOC_DA7210 help @@ -63,7 +63,7 @@ config SND_FSI_DA7210 FSI - DA7210 unit config SND_FSI_HDMI - bool "FSI-HDMI sound support" + tristate "FSI-HDMI sound support" depends on SND_SOC_SH4_FSI && FB_SH_MOBILE_HDMI help This option enables generic sound support for the -- 1.7.0.4

3 2

[alsa-devel] [PATCH 3/3] ASoC: Samsung: Debug PCM platform device name
by Jassi Brar 10 Sep '10

10 Sep '10

From: Jassi Brar <jassi.brar(a)samsung.com> The PCM controller platform devices are registered by the name 'samsung-pcm', so use the same in the CPU driver. Signed-off-by: Jassi Brar <jassi.brar(a)samsung.com> --- sound/soc/s3c24xx/s3c-pcm.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/s3c24xx/s3c-pcm.c b/sound/soc/s3c24xx/s3c-pcm.c index d404f80..6e9139f 100644 --- a/sound/soc/s3c24xx/s3c-pcm.c +++ b/sound/soc/s3c24xx/s3c-pcm.c @@ -525,7 +525,7 @@ static struct platform_driver s3c_pcm_driver = { .probe = s3c_pcm_dev_probe, .remove = s3c_pcm_dev_remove, .driver = { - .name = "samsung-pcm-audio", + .name = "samsung-pcm", .owner = THIS_MODULE, }, }; @@ -546,4 +546,4 @@ module_exit(s3c_pcm_exit); MODULE_AUTHOR("Jaswinder Singh, <jassi.brar(a)samsung.com>"); MODULE_DESCRIPTION("S3C PCM Controller Driver"); MODULE_LICENSE("GPL"); -MODULE_ALIAS("platform:samsung-pcm-audio"); +MODULE_ALIAS("platform:samsung-pcm"); -- 1.6.2.5

3 2

[alsa-devel] [PATCH 1/3] ASoC: S3C: AC97: Remove the -dai suffix
by Jassi Brar 10 Sep '10

10 Sep '10

From: Jassi Brar <jassi.brar(a)samsung.com> Drop the invalid -dai suffix appended to the Samsung AC97 CPU DAI. Signed-off-by: Jassi Brar <jassi.brar(a)samsung.com> --- sound/soc/s3c24xx/smdk2443_wm9710.c | 2 +- sound/soc/s3c24xx/smdk_wm9713.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/s3c24xx/smdk2443_wm9710.c b/sound/soc/s3c24xx/smdk2443_wm9710.c index 66f9e22..4613288 100644 --- a/sound/soc/s3c24xx/smdk2443_wm9710.c +++ b/sound/soc/s3c24xx/smdk2443_wm9710.c @@ -28,7 +28,7 @@ static struct snd_soc_dai_link smdk2443_dai[] = { { .name = "AC97", .stream_name = "AC97 HiFi", - .cpu_dai_name = "s3c-ac97-dai", + .cpu_dai_name = "s3c-ac97", .codec_dai_name = "ac97-hifi", .codec_name = "ac97-codec", .platform_name = "s3c24xx-pcm-audio", diff --git a/sound/soc/s3c24xx/smdk_wm9713.c b/sound/soc/s3c24xx/smdk_wm9713.c index 90108a7..11086f8 100644 --- a/sound/soc/s3c24xx/smdk_wm9713.c +++ b/sound/soc/s3c24xx/smdk_wm9713.c @@ -47,7 +47,7 @@ static struct snd_soc_dai_link smdk_dai = { .name = "AC97", .stream_name = "AC97 PCM", .platform_name = "s3c24xx-pcm-audio", - .cpu_dai_name = "s3c-ac97-dai", + .cpu_dai_name = "s3c-ac97", .codec_dai_name = "wm9713-hifi", .codec_name = "wm9713-codec", }; -- 1.6.2.5

3 2

[alsa-devel] [PATCH 0/5] ASoC: sh_mobile_hdmi: bug fix patches
by Kuninori Morimoto 10 Sep '10

10 Sep '10

Dear Mark, Liam, Guennadi These are bug fix patches for sh_mobile_hdmi. Kuninori Morimoto (5): fbdev: sh_mobile_hdmi: modify HDMI flags name to more specific fbdev: sh_mobile_hdmi: modify snd_soc_dai_driver settings fbdev: sh_mobile_hdmi: add new label for sound error path ASoC: fsi-hdmi: remove unneeded header ASoC: fsi-ak4642: modiry platform_name 1/5 - 4/5 bugs are reported by Guennadi. And 5/5 is needed if Paul's genesis tree are merged to Mark's "for-2.6.37" Best regards -- Kuninori Morimoto

4 14

[alsa-devel] M-Audio Fast Track Ultra (8r) mixer: How to verify dB information?
by Felix Homann 10 Sep '10

10 Sep '10

Hi list, after some struggle I managed to get the mixer in the M-Audio working. Before I submit a patch I would like to know if there's an way to test the dB information for the controls. I'm using the mixer_vol_tlv TLV callback right now but I'm not sure it produces correct data. Kind regards, Felix

2 3

[alsa-devel] [PATCH 1/2] ALSA: snd-usb-caiaq: drop version number
by Daniel Mack 10 Sep '10

10 Sep '10

Let git do the job. Signed-off-by: Daniel Mack <daniel(a)caiaq.de> --- sound/usb/caiaq/device.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sound/usb/caiaq/device.c b/sound/usb/caiaq/device.c index cdfb856..da9cb6d 100644 --- a/sound/usb/caiaq/device.c +++ b/sound/usb/caiaq/device.c @@ -36,7 +36,7 @@ #include "input.h" MODULE_AUTHOR("Daniel Mack <daniel(a)caiaq.de>"); -MODULE_DESCRIPTION("caiaq USB audio, version 1.3.21"); +MODULE_DESCRIPTION("caiaq USB audio"); MODULE_LICENSE("GPL"); MODULE_SUPPORTED_DEVICE("{{Native Instruments, RigKontrol2}," "{Native Instruments, RigKontrol3}," -- 1.7.0.4

2 5

[alsa-devel] [PATCH 1/3] ALSA: snd-usb-caiaq: drop version number
by Daniel Mack 10 Sep '10

10 Sep '10

Let git do the job. Signed-off-by: Daniel Mack <daniel(a)caiaq.de> --- sound/usb/caiaq/device.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sound/usb/caiaq/device.c b/sound/usb/caiaq/device.c index cdfb856..da9cb6d 100644 --- a/sound/usb/caiaq/device.c +++ b/sound/usb/caiaq/device.c @@ -36,7 +36,7 @@ #include "input.h" MODULE_AUTHOR("Daniel Mack <daniel(a)caiaq.de>"); -MODULE_DESCRIPTION("caiaq USB audio, version 1.3.21"); +MODULE_DESCRIPTION("caiaq USB audio"); MODULE_LICENSE("GPL"); MODULE_SUPPORTED_DEVICE("{{Native Instruments, RigKontrol2}," "{Native Instruments, RigKontrol3}," -- 1.7.0.4

2 4

[alsa-devel] cmipci surround sound front-left plays on lfe
by Tom Corner 10 Sep '10

10 Sep '10

I've been going around and around on this. The surround sound 5.1 is playing fine except channel 0 (front-left) is playing on the woofer. I have a Terratec Aureon fun - CMI8738-MC6. I can redirect the output on 0 to right-front successfully using .asoundrc and "ttable.0.1 1". I have gotten desperate and have tested this card in Windows and was able to get it working right there. I have been using the file www_lynnemusic_com_surround_test.wma through mplayer and speaker-test to to test the setup. I am using pulseaudio, but I reconfigured my system without pulseaudio and get the same result. Any help would be much appreciated. Tom Corner

2 1

[alsa-devel] [PATCH] ALSA: hda - fix cirrus dac/beep coefs setup
by Jan Nikitenko 10 Sep '10

10 Sep '10

Fix copy&paste bug in cirrus coefs setup, that caused bogus value in DAC_CFG that should have gone to BEEP_CFG. The problem could be noticed with 'Headphone' output getting audible volume ramp in the beginning of sound playback. Signed-off-by: Jan Nikitenko <jan.nikitenko(a)gmail.com> --- sound/pci/hda/patch_cirrus.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sound/pci/hda/patch_cirrus.c b/sound/pci/hda/patch_cirrus.c index 350ee8a..7d5c074 100644 --- a/sound/pci/hda/patch_cirrus.c +++ b/sound/pci/hda/patch_cirrus.c @@ -966,7 +966,7 @@ static struct hda_verb cs_coef_init_verbs[] = { | 0x0400 /* Disable Coefficient Auto increment */ )}, /* Beep */ - {0x11, AC_VERB_SET_COEF_INDEX, IDX_DAC_CFG}, + {0x11, AC_VERB_SET_COEF_INDEX, IDX_BEEP_CFG}, {0x11, AC_VERB_SET_PROC_COEF, 0x0007}, /* Enable Beep thru DAC1/2/3 */ {} /* terminator */ -- 1.7.1.1

1 0