Re: ALSA patch Nvidia 8 channel hdmi audio

Re: ALSA patch Nvidia 8 channel hdmi audio

[Marek Vasut]

I'm not the author of the patch, CCing alsa-devel and the original author.

On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> Hi Marek !
>
> I just read your emailing with nvidia devs about alsa supporting 8 channel
> lpcm over hdmi.
>
> I checked it out an i have to tell it is working awsome, except for a few
> bugs:
>
> The channels are wrong mapped. Running a speakertest -Dhdmi -c6 gives the
> following result (first where it should sound)
> FL - FL
> C - RL
> FR - FR
> RR - LFE
> RL - C
>
> The other thing i realized that playback is limited to 16 bits 48 khz. I
> think the minimum which is supported by nvidia hardware is 24bits 96 (or
> 192) khz.
> Why not ?
>
> Could you contact the devs as i don't know how to reach them.
>
> ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with the
> oatched nvidia.c file. Anyway the amplifier shows multichannel LPCM and
> that's great news.

Re: ALSA patch Nvidia 8 channel hdmi audio

[Marek Vasut]

On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> Thank's for the info.
> How can i contact him ?

He and alsa-devel mailing list are both in CC already, please preserve them 
there or fix your mailer.
>
>
> 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
>
> > I'm not the author of the patch, CCing alsa-devel and the original
> > author.
> >
> > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > Hi Marek !
> > >
> > > I just read your emailing with nvidia devs about alsa supporting 8
> >
> > channel
> >
> > > lpcm over hdmi.
> > >
> > > I checked it out an i have to tell it is working awsome, except for a
> > > few bugs:
> > >
> > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6 gives
> > > the following result (first where it should sound)
> > > FL - FL
> > > C - RL
> > > FR - FR
> > > RR - LFE
> > > RL - C
> > >
> > > The other thing i realized that playback is limited to 16 bits 48 khz.
> > > I think the minimum which is supported by nvidia hardware is 24bits 96
> > > (or 192) khz.
> > > Why not ?
> > >
> > > Could you contact the devs as i don't know how to reach them.
> > >
> > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with the
> > > oatched nvidia.c file. Anyway the amplifier shows multichannel LPCM and
> > > that's great news.

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wei Ni]

Hi, Marek and Alan
I'm a Nvidia engineer and develop this patch file.

I used mplayer to test. The channels are mapped good.
Running mplayer -ao alsa:device=hw=0.3 6ch.wav

I tried to use speaker-test to test, but I can't get the hdmi device name.
I used speaker-test -Dsurround51:NVidia -c6, the sound output from the analog jack.
Which device name should I use?

I use aplay -l, it show
**** List of PLAYBACK Hardware Devices ****
card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200 Digital]
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

Thanks
Wei.

-----Original Message-----
From: Marek Vasut [mailto:marek.vasut@gmail.com] 
Sent: Wednesday, June 03, 2009 8:01 AM
To: Alan Alan
Cc: alsa-devel@alsa-project.org; Wei Ni
Subject: Re: ALSA patch Nvidia 8 channel hdmi audio

On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> Thank's for the info.
> How can i contact him ?

He and alsa-devel mailing list are both in CC already, please preserve them 
there or fix your mailer.
>
>
> 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
>
> > I'm not the author of the patch, CCing alsa-devel and the original
> > author.
> >
> > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > Hi Marek !
> > >
> > > I just read your emailing with nvidia devs about alsa supporting 8
> >
> > channel
> >
> > > lpcm over hdmi.
> > >
> > > I checked it out an i have to tell it is working awsome, except for a
> > > few bugs:
> > >
> > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6 gives
> > > the following result (first where it should sound)
> > > FL - FL
> > > C - RL
> > > FR - FR
> > > RR - LFE
> > > RL - C
> > >
> > > The other thing i realized that playback is limited to 16 bits 48 khz.
> > > I think the minimum which is supported by nvidia hardware is 24bits 96
> > > (or 192) khz.
> > > Why not ?
> > >
> > > Could you contact the devs as i don't know how to reach them.
> > >
> > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with the
> > > oatched nvidia.c file. Anyway the amplifier shows multichannel LPCM and
> > > that's great news.



-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

Re: ALSA patch Nvidia 8 channel hdmi audio

[Clemens Ladisch]

Wei Ni wrote:
> I used mplayer to test. The channels are mapped good.
> Running mplayer -ao alsa:device=hw=0.3 6ch.wav
> 
> I tried to use speaker-test to test, but I can't get the hdmi device name.
> I used speaker-test -Dsurround51:NVidia -c6, the sound output from the analog jack.

speaker-test -Dhw:0,3 -c6

Or put a line
  pcm.hdmi = "hw:0,3"
into your ~/.asoundrc or the /etc/asound.conf file, then you can just
use the device name "hdmi" everywhere.


HTH
Clemens

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Wed, 3 Jun 2009 15:14:19 +0800,
Wei Ni wrote:
> 
> Hi, Marek and Alan
> I'm a Nvidia engineer and develop this patch file.
> 
> I used mplayer to test. The channels are mapped good.
> Running mplayer -ao alsa:device=hw=0.3 6ch.wav
> 
> I tried to use speaker-test to test, but I can't get the hdmi device name.

It's in the recent alsa-lib, but maybe not in the older version.
You can see it in "aplay -L" output.

The latest version should work like

	% speaker-test -Dhdmi:NVidia -c6


Takashi

> I used speaker-test -Dsurround51:NVidia -c6, the sound output from the analog jack.
> Which device name should I use?
> 
> I use aplay -l, it show
> **** List of PLAYBACK Hardware Devices ****
> card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
>   Subdevices: 1/1
>   Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200 Digital]
>   Subdevices: 1/1
>   Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
>   Subdevices: 1/1
>   Subdevice #0: subdevice #0
> 
> Thanks
> Wei.
> 
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut@gmail.com] 
> Sent: Wednesday, June 03, 2009 8:01 AM
> To: Alan Alan
> Cc: alsa-devel@alsa-project.org; Wei Ni
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> 
> On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > Thank's for the info.
> > How can i contact him ?
> 
> He and alsa-devel mailing list are both in CC already, please preserve them 
> there or fix your mailer.
> >
> >
> > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> >
> > > I'm not the author of the patch, CCing alsa-devel and the original
> > > author.
> > >
> > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > Hi Marek !
> > > >
> > > > I just read your emailing with nvidia devs about alsa supporting 8
> > >
> > > channel
> > >
> > > > lpcm over hdmi.
> > > >
> > > > I checked it out an i have to tell it is working awsome, except for a
> > > > few bugs:
> > > >
> > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6 gives
> > > > the following result (first where it should sound)
> > > > FL - FL
> > > > C - RL
> > > > FR - FR
> > > > RR - LFE
> > > > RL - C
> > > >
> > > > The other thing i realized that playback is limited to 16 bits 48 khz.
> > > > I think the minimum which is supported by nvidia hardware is 24bits 96
> > > > (or 192) khz.
> > > > Why not ?
> > > >
> > > > Could you contact the devs as i don't know how to reach them.
> > > >
> > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with the
> > > > oatched nvidia.c file. Anyway the amplifier shows multichannel LPCM and
> > > > that's great news.
> 
> 
> 
> -----------------------------------------------------------------------------------
> This email message is for the sole use of the intended recipient(s) and may contain
> confidential information.  Any unauthorized review, use, disclosure or distribution
> is prohibited.  If you are not the intended recipient, please contact the sender by
> reply email and destroy all copies of the original message.
> -----------------------------------------------------------------------------------
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Alan Alan]

Hello Wei !

Thanks for replying.
I tested it with speaker-test and xbmc babylon. Both had problems with
channel mapping.
I used this command:

speaker-test -Dhdmi -c6
or
speaker-test -Dhdmi -c8

alternatively you could try

speaker-test -Dhw:0,3 -c6


I had to rund Xorg first and have the command from an xterm, becasue from a
normal terminal it would not work.
I think that is normal as a limitation of hdmi.

What about the support for 24bits 192khz sound ?

speaker-test -Dhdmi -c6  -r192000 does not work.

Thanks for working on this.




2009/6/3 Wei Ni <wni@nvidia.com>

> Hi, Marek and Alan
> I'm a Nvidia engineer and develop this patch file.
>
> I used mplayer to test. The channels are mapped good.
> Running mplayer -ao alsa:device=hw=0.3 6ch.wav
>
> I tried to use speaker-test to test, but I can't get the hdmi device name.
> I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
> analog jack.
> Which device name should I use?
>
> I use aplay -l, it show
> **** List of PLAYBACK Hardware Devices ****
> card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200 Digital]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
>
> Thanks
> Wei.
>
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut@gmail.com]
> Sent: Wednesday, June 03, 2009 8:01 AM
> To: Alan Alan
> Cc: alsa-devel@alsa-project.org; Wei Ni
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
>
> On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > Thank's for the info.
> > How can i contact him ?
>
> He and alsa-devel mailing list are both in CC already, please preserve them
> there or fix your mailer.
> >
> >
> > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> >
> > > I'm not the author of the patch, CCing alsa-devel and the original
> > > author.
> > >
> > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > Hi Marek !
> > > >
> > > > I just read your emailing with nvidia devs about alsa supporting 8
> > >
> > > channel
> > >
> > > > lpcm over hdmi.
> > > >
> > > > I checked it out an i have to tell it is working awsome, except for a
> > > > few bugs:
> > > >
> > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6 gives
> > > > the following result (first where it should sound)
> > > > FL - FL
> > > > C - RL
> > > > FR - FR
> > > > RR - LFE
> > > > RL - C
> > > >
> > > > The other thing i realized that playback is limited to 16 bits 48
> khz.
> > > > I think the minimum which is supported by nvidia hardware is 24bits
> 96
> > > > (or 192) khz.
> > > > Why not ?
> > > >
> > > > Could you contact the devs as i don't know how to reach them.
> > > >
> > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with
> the
> > > > oatched nvidia.c file. Anyway the amplifier shows multichannel LPCM
> and
> > > > that's great news.
>
>
>
>
> -----------------------------------------------------------------------------------
> This email message is for the sole use of the intended recipient(s) and may
> contain
> confidential information.  Any unauthorized review, use, disclosure or
> distribution
> is prohibited.  If you are not the intended recipient, please contact the
> sender by
> reply email and destroy all copies of the original message.
>
> -----------------------------------------------------------------------------------
>

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Wed, 3 Jun 2009 10:00:24 +0200,
Alan Alan wrote:
> 
> Hello Wei !
> 
> Thanks for replying.
> I tested it with speaker-test and xbmc babylon. Both had problems with
> channel mapping.
> I used this command:
> 
> speaker-test -Dhdmi -c6
> or
> speaker-test -Dhdmi -c8
> 
> alternatively you could try
> 
> speaker-test -Dhw:0,3 -c6
> 
> 
> I had to rund Xorg first and have the command from an xterm, becasue from a
> normal terminal it would not work.
> I think that is normal as a limitation of hdmi.
> 
> What about the support for 24bits 192khz sound ?
> 
> speaker-test -Dhdmi -c6  -r192000 does not work.

How about the patch below?

Also, could you give alsa-info.sh output?


Takashi

---
diff --git a/sound/pci/hda/patch_nvhdmi.c b/sound/pci/hda/patch_nvhdmi.c
index f5792e2..48443de 100644
--- a/sound/pci/hda/patch_nvhdmi.c
+++ b/sound/pci/hda/patch_nvhdmi.c
@@ -258,9 +258,11 @@ static struct hda_pcm_stream nvhdmi_pcm_digital_playback_8ch = {
 	.channels_min = 2,
 	.channels_max = 8,
 	.nid = Nv_Master_Convert_nid,
+#if 0
 	.rates = SNDRV_PCM_RATE_48000,
 	.maxbps = 16,
 	.formats = SNDRV_PCM_FMTBIT_S16_LE,
+#endif
 	.ops = {
 		.open = nvhdmi_dig_playback_pcm_open,
 		.close = nvhdmi_dig_playback_pcm_close_8ch,
@@ -273,9 +275,11 @@ static struct hda_pcm_stream nvhdmi_pcm_digital_playback_2ch = {
 	.channels_min = 2,
 	.channels_max = 2,
 	.nid = Nv_Master_Convert_nid,
+#if 0
 	.rates = SNDRV_PCM_RATE_48000,
 	.maxbps = 16,
 	.formats = SNDRV_PCM_FMTBIT_S16_LE,
+#endif
 	.ops = {
 		.open = nvhdmi_dig_playback_pcm_open,
 		.close = nvhdmi_dig_playback_pcm_close_2ch,

Re: ALSA patch Nvidia 8 channel hdmi audio

[Alan Alan]

Hello Takashi !

What should this patch do ?
Is it a cure for the 96, 192khz or 24bit sample playback problem ?
If so, will it resample or natievly pass the stream to the amplifier ?

Thank you

Alan

2009/6/3 Takashi Iwai <tiwai@suse.de>

> At Wed, 3 Jun 2009 10:00:24 +0200,
> Alan Alan wrote:
> >
> > Hello Wei !
> >
> > Thanks for replying.
> > I tested it with speaker-test and xbmc babylon. Both had problems with
> > channel mapping.
> > I used this command:
> >
> > speaker-test -Dhdmi -c6
> > or
> > speaker-test -Dhdmi -c8
> >
> > alternatively you could try
> >
> > speaker-test -Dhw:0,3 -c6
> >
> >
> > I had to rund Xorg first and have the command from an xterm, becasue from
> a
> > normal terminal it would not work.
> > I think that is normal as a limitation of hdmi.
> >
> > What about the support for 24bits 192khz sound ?
> >
> > speaker-test -Dhdmi -c6  -r192000 does not work.
>
> How about the patch below?
>
> Also, could you give alsa-info.sh output?
>
>
> Takashi
>
> ---
> diff --git a/sound/pci/hda/patch_nvhdmi.c b/sound/pci/hda/patch_nvhdmi.c
> index f5792e2..48443de 100644
> --- a/sound/pci/hda/patch_nvhdmi.c
> +++ b/sound/pci/hda/patch_nvhdmi.c
> @@ -258,9 +258,11 @@ static struct hda_pcm_stream
> nvhdmi_pcm_digital_playback_8ch = {
>        .channels_min = 2,
>        .channels_max = 8,
>        .nid = Nv_Master_Convert_nid,
> +#if 0
>        .rates = SNDRV_PCM_RATE_48000,
>        .maxbps = 16,
>        .formats = SNDRV_PCM_FMTBIT_S16_LE,
> +#endif
>        .ops = {
>                .open = nvhdmi_dig_playback_pcm_open,
>                .close = nvhdmi_dig_playback_pcm_close_8ch,
> @@ -273,9 +275,11 @@ static struct hda_pcm_stream
> nvhdmi_pcm_digital_playback_2ch = {
>        .channels_min = 2,
>        .channels_max = 2,
>        .nid = Nv_Master_Convert_nid,
> +#if 0
>        .rates = SNDRV_PCM_RATE_48000,
>        .maxbps = 16,
>        .formats = SNDRV_PCM_FMTBIT_S16_LE,
> +#endif
>        .ops = {
>                .open = nvhdmi_dig_playback_pcm_open,
>                .close = nvhdmi_dig_playback_pcm_close_2ch,
>

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Wed, 3 Jun 2009 11:34:00 +0200,
Alan Alan wrote:
> 
> Hello Takashi !
> 
> What should this patch do ?

The patch removes the restriction patch_nvhdmi.c sets.  Instead,
the driver tries to detect the supported format, rate, etc from the
codec widget information.

> Is it a cure for the 96, 192khz or 24bit sample playback problem ?

Possibly.

> If so, will it resample or natievly pass the stream to the amplifier ?

It just enables the native support, possibly.

And, don't forget to give back the alsa-info.sh output (run with
--no-upload option).


Takashi

> 
> Thank you
> 
> Alan
> 
> 2009/6/3 Takashi Iwai <tiwai@suse.de>
> 
>     At Wed, 3 Jun 2009 10:00:24 +0200,
>     Alan Alan wrote:
>     >
>     > Hello Wei !
>     >
>     > Thanks for replying.
>     > I tested it with speaker-test and xbmc babylon. Both had problems with
>     > channel mapping.
>     > I used this command:
>     >
>     > speaker-test -Dhdmi -c6
>     > or
>     > speaker-test -Dhdmi -c8
>     >
>     > alternatively you could try
>     >
>     > speaker-test -Dhw:0,3 -c6
>     >
>     >
>     > I had to rund Xorg first and have the command from an xterm, becasue
>     from a
>     > normal terminal it would not work.
>     > I think that is normal as a limitation of hdmi.
>     >
>     > What about the support for 24bits 192khz sound ?
>     >
>     > speaker-test -Dhdmi -c6  -r192000 does not work.
>    
>     How about the patch below?
>    
>     Also, could you give alsa-info.sh output?
> 
>     Takashi
>    
>     ---
>     diff --git a/sound/pci/hda/patch_nvhdmi.c b/sound/pci/hda/patch_nvhdmi.c
>     index f5792e2..48443de 100644
>     --- a/sound/pci/hda/patch_nvhdmi.c
>     +++ b/sound/pci/hda/patch_nvhdmi.c
>     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
>     nvhdmi_pcm_digital_playback_8ch = {
>            .channels_min = 2,
>            .channels_max = 8,
>            .nid = Nv_Master_Convert_nid,
>     +#if 0
>            .rates = SNDRV_PCM_RATE_48000,
>            .maxbps = 16,
>            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     +#endif
>            .ops = {
>                    .open = nvhdmi_dig_playback_pcm_open,
>                    .close = nvhdmi_dig_playback_pcm_close_8ch,
>     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
>     nvhdmi_pcm_digital_playback_2ch = {
>            .channels_min = 2,
>            .channels_max = 2,
>            .nid = Nv_Master_Convert_nid,
>     +#if 0
>            .rates = SNDRV_PCM_RATE_48000,
>            .maxbps = 16,
>            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     +#endif
>            .ops = {
>                    .open = nvhdmi_dig_playback_pcm_open,
>                    .close = nvhdmi_dig_playback_pcm_close_2ch,
> 
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Alan Alan]

Hello Takashi !

I tried to apply the patch that removes restrictions to 16bits 48khz format.
Well now we have the following:
16bit 48khz                                     works
32bit 48khz                                     works
32bit 44khz, 96khz, 192khz              don't work
24bit 44khz, 48khz, 96khz, 192khz   don't work
16bit 44khz, 96khz, 192khz              don't work

I only get
Sample format not available for playback:
Setting of hwparams failed:

I enclosed to pastebin my alsainfo script output.

http://pastebin.com/m77dcfb3f

Thanks for the help !

2009/6/3 Takashi Iwai <tiwai@suse.de>

> At Wed, 3 Jun 2009 11:34:00 +0200,
> Alan Alan wrote:
> >
> > Hello Takashi !
> >
> > What should this patch do ?
>
> The patch removes the restriction patch_nvhdmi.c sets.  Instead,
> the driver tries to detect the supported format, rate, etc from the
> codec widget information.
>
> > Is it a cure for the 96, 192khz or 24bit sample playback problem ?
>
> Possibly.
>
> > If so, will it resample or natievly pass the stream to the amplifier ?
>
> It just enables the native support, possibly.
>
> And, don't forget to give back the alsa-info.sh output (run with
> --no-upload option).
>
>
> Takashi
>
> >
> > Thank you
> >
> > Alan
> >
> > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >
> >     At Wed, 3 Jun 2009 10:00:24 +0200,
> >     Alan Alan wrote:
> >     >
> >     > Hello Wei !
> >     >
> >     > Thanks for replying.
> >     > I tested it with speaker-test and xbmc babylon. Both had problems
> with
> >     > channel mapping.
> >     > I used this command:
> >     >
> >     > speaker-test -Dhdmi -c6
> >     > or
> >     > speaker-test -Dhdmi -c8
> >     >
> >     > alternatively you could try
> >     >
> >     > speaker-test -Dhw:0,3 -c6
> >     >
> >     >
> >     > I had to rund Xorg first and have the command from an xterm,
> becasue
> >     from a
> >     > normal terminal it would not work.
> >     > I think that is normal as a limitation of hdmi.
> >     >
> >     > What about the support for 24bits 192khz sound ?
> >     >
> >     > speaker-test -Dhdmi -c6  -r192000 does not work.
> >
> >     How about the patch below?
> >
> >     Also, could you give alsa-info.sh output?
> >
> >     Takashi
> >
> >     ---
> >     diff --git a/sound/pci/hda/patch_nvhdmi.c
> b/sound/pci/hda/patch_nvhdmi.c
> >     index f5792e2..48443de 100644
> >     --- a/sound/pci/hda/patch_nvhdmi.c
> >     +++ b/sound/pci/hda/patch_nvhdmi.c
> >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
> >     nvhdmi_pcm_digital_playback_8ch = {
> >            .channels_min = 2,
> >            .channels_max = 8,
> >            .nid = Nv_Master_Convert_nid,
> >     +#if 0
> >            .rates = SNDRV_PCM_RATE_48000,
> >            .maxbps = 16,
> >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     +#endif
> >            .ops = {
> >                    .open = nvhdmi_dig_playback_pcm_open,
> >                    .close = nvhdmi_dig_playback_pcm_close_8ch,
> >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
> >     nvhdmi_pcm_digital_playback_2ch = {
> >            .channels_min = 2,
> >            .channels_max = 2,
> >            .nid = Nv_Master_Convert_nid,
> >     +#if 0
> >            .rates = SNDRV_PCM_RATE_48000,
> >            .maxbps = 16,
> >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     +#endif
> >            .ops = {
> >                    .open = nvhdmi_dig_playback_pcm_open,
> >                    .close = nvhdmi_dig_playback_pcm_close_2ch,
> >
> >
>

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Thu, 4 Jun 2009 01:28:24 +0200,
Alan Alan wrote:
> 
> Hello Takashi !
> 
> I tried to apply the patch that removes restrictions to 16bits 48khz format.
> Well now we have the following:
> 16bit 48khz                                     works
> 32bit 48khz                                     works
> 32bit 44khz, 96khz, 192khz              don't work
> 24bit 44khz, 48khz, 96khz, 192khz   don't work
> 16bit 44khz, 96khz, 192khz              don't work
> 
> I only get
> Sample format not available for playback:
> Setting of hwparams failed:

As you can find in alsa-info output, it seems that the codec supports
only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
(hardware) bug or not.


Takashi

> 
> I enclosed to pastebin my alsainfo script output.
> 
> http://pastebin.com/m77dcfb3f
> 
> Thanks for the help !
> 
> 2009/6/3 Takashi Iwai <tiwai@suse.de>
> 
>     At Wed, 3 Jun 2009 11:34:00 +0200,
>     Alan Alan wrote:
>     >
>     > Hello Takashi !
>     >
>     > What should this patch do ?
>    
>     The patch removes the restriction patch_nvhdmi.c sets.  Instead,
>     the driver tries to detect the supported format, rate, etc from the
>     codec widget information.
>    
>     > Is it a cure for the 96, 192khz or 24bit sample playback problem ?
>    
>     Possibly.
>    
>     > If so, will it resample or natievly pass the stream to the amplifier ?
>    
>     It just enables the native support, possibly.
>    
>     And, don't forget to give back the alsa-info.sh output (run with
>     --no-upload option).
> 
>     Takashi
>    
>     >
>     > Thank you
>     >
>     > Alan
>     >
>     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >
>     >     At Wed, 3 Jun 2009 10:00:24 +0200,
>     >     Alan Alan wrote:
>     >     >
>     >     > Hello Wei !
>     >     >
>     >     > Thanks for replying.
>     >     > I tested it with speaker-test and xbmc babylon. Both had problems
>     with
>     >     > channel mapping.
>     >     > I used this command:
>     >     >
>     >     > speaker-test -Dhdmi -c6
>     >     > or
>     >     > speaker-test -Dhdmi -c8
>     >     >
>     >     > alternatively you could try
>     >     >
>     >     > speaker-test -Dhw:0,3 -c6
>     >     >
>     >     >
>     >     > I had to rund Xorg first and have the command from an xterm,
>     becasue
>     >     from a
>     >     > normal terminal it would not work.
>     >     > I think that is normal as a limitation of hdmi.
>     >     >
>     >     > What about the support for 24bits 192khz sound ?
>     >     >
>     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
>     >
>     >     How about the patch below?
>     >
>     >     Also, could you give alsa-info.sh output?
>     >
>     >     Takashi
>     >
>     >     ---
>     >     diff --git a/sound/pci/hda/patch_nvhdmi.c b/sound/pci/hda/
>     patch_nvhdmi.c
>     >     index f5792e2..48443de 100644
>     >     --- a/sound/pci/hda/patch_nvhdmi.c
>     >     +++ b/sound/pci/hda/patch_nvhdmi.c
>     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
>     >     nvhdmi_pcm_digital_playback_8ch = {
>     >            .channels_min = 2,
>     >            .channels_max = 8,
>     >            .nid = Nv_Master_Convert_nid,
>     >     +#if 0
>     >            .rates = SNDRV_PCM_RATE_48000,
>     >            .maxbps = 16,
>     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     +#endif
>     >            .ops = {
>     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >                    .close = nvhdmi_dig_playback_pcm_close_8ch,
>     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
>     >     nvhdmi_pcm_digital_playback_2ch = {
>     >            .channels_min = 2,
>     >            .channels_max = 2,
>     >            .nid = Nv_Master_Convert_nid,
>     >     +#if 0
>     >            .rates = SNDRV_PCM_RATE_48000,
>     >            .maxbps = 16,
>     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     +#endif
>     >            .ops = {
>     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >                    .close = nvhdmi_dig_playback_pcm_close_2ch,
>     >
>     >
> 
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Thu, 4 Jun 2009 08:18:23 +0200,
Alan Alan wrote:
> 
> 2009/6/4 Takashi Iwai <tiwai@suse.de>
> 
>     At Thu, 4 Jun 2009 01:28:24 +0200,
>     Alan Alan wrote:
>     >
>     > Hello Takashi !
>     >
>     > I tried to apply the patch that removes restrictions to 16bits 48khz
>     format.
>     > Well now we have the following:
>     > 16bit 48khz                                     works
>     > 32bit 48khz                                     works
>     > 32bit 44khz, 96khz, 192khz              don't work
>     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
>     > 16bit 44khz, 96khz, 192khz              don't work
>     >
>     > I only get
>     > Sample format not available for playback:
>     > Setting of hwparams failed:
>    
>     As you can find in alsa-info output, it seems that the codec supports
>     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
>     (hardware) bug or not.
> 
> The codec surely supports up to 192khz as it works on Windows.
> 
> I think this line is completely wrong as here it says:

It is not "wrong".  These are values that the codec reports, so it's
more or less correct as the h/w information.

>   rates [0xc0]: 48000 88200

Wei, is it right?  Or is anything missing here?


>     bits [0xf]: 8 16 20 24
> 
> 8, 20, 24bits are supported, but they all give an error message.

24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
24bit in 3 bytes format isn't supported on HD-audio.

> 32 bits are not supposed to be supported but it works with 48 and 88.2khz.
> 
> How can we change the software side to accept 96 or 192khz spample formats and
> 24 bits ?

As long as the codec reports so, it's no "software" problem.

> I think it is not hardware limited and we just need to change the allowed
> formats to be played.

If it's just a hardware problem that reports wrong values, you can
override the information, simply replacing the bits my patch disabled
to the right values.


Takashi

>    
>     >
>     > I enclosed to pastebin my alsainfo script output.
>     >
>     > http://pastebin.com/m77dcfb3f
>     >
>     > Thanks for the help !
>     >
>     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >
>     >     At Wed, 3 Jun 2009 11:34:00 +0200,
>     >     Alan Alan wrote:
>     >     >
>     >     > Hello Takashi !
>     >     >
>     >     > What should this patch do ?
>     >
>     >     The patch removes the restriction patch_nvhdmi.c sets.  Instead,
>     >     the driver tries to detect the supported format, rate, etc from the
>     >     codec widget information.
>     >
>     >     > Is it a cure for the 96, 192khz or 24bit sample playback problem ?
>     >
>     >     Possibly.
>     >
>     >     > If so, will it resample or natievly pass the stream to the
>     amplifier ?
>     >
>     >     It just enables the native support, possibly.
>     >
>     >     And, don't forget to give back the alsa-info.sh output (run with
>     >     --no-upload option).
>     >
>     >     Takashi
>     >
>     >     >
>     >     > Thank you
>     >     >
>     >     > Alan
>     >     >
>     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >     >
>     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
>     >     >     Alan Alan wrote:
>     >     >     >
>     >     >     > Hello Wei !
>     >     >     >
>     >     >     > Thanks for replying.
>     >     >     > I tested it with speaker-test and xbmc babylon. Both had
>     problems
>     >     with
>     >     >     > channel mapping.
>     >     >     > I used this command:
>     >     >     >
>     >     >     > speaker-test -Dhdmi -c6
>     >     >     > or
>     >     >     > speaker-test -Dhdmi -c8
>     >     >     >
>     >     >     > alternatively you could try
>     >     >     >
>     >     >     > speaker-test -Dhw:0,3 -c6
>     >     >     >
>     >     >     >
>     >     >     > I had to rund Xorg first and have the command from an xterm,
>     >     becasue
>     >     >     from a
>     >     >     > normal terminal it would not work.
>     >     >     > I think that is normal as a limitation of hdmi.
>     >     >     >
>     >     >     > What about the support for 24bits 192khz sound ?
>     >     >     >
>     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
>     >     >
>     >     >     How about the patch below?
>     >     >
>     >     >     Also, could you give alsa-info.sh output?
>     >     >
>     >     >     Takashi
>     >     >
>     >     >     ---
>     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c b/sound/pci/hda/
>     >     patch_nvhdmi.c
>     >     >     index f5792e2..48443de 100644
>     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
>     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
>     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
>     >     >     nvhdmi_pcm_digital_playback_8ch = {
>     >     >            .channels_min = 2,
>     >     >            .channels_max = 8,
>     >     >            .nid = Nv_Master_Convert_nid,
>     >     >     +#if 0
>     >     >            .rates = SNDRV_PCM_RATE_48000,
>     >     >            .maxbps = 16,
>     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     >     +#endif
>     >     >            .ops = {
>     >     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >     >                    .close = nvhdmi_dig_playback_pcm_close_8ch,
>     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
>     >     >     nvhdmi_pcm_digital_playback_2ch = {
>     >     >            .channels_min = 2,
>     >     >            .channels_max = 2,
>     >     >            .nid = Nv_Master_Convert_nid,
>     >     >     +#if 0
>     >     >            .rates = SNDRV_PCM_RATE_48000,
>     >     >            .maxbps = 16,
>     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     >     +#endif
>     >     >            .ops = {
>     >     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >     >                    .close = nvhdmi_dig_playback_pcm_close_2ch,
>     >     >
>     >     >
>     >
>     >
> 
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wei Ni]

About rates [0xc0]: 48000 88200, what chipset do you use? Or could
you tell me the codec id, I will check it with our hardware team.

Thanks
Wei.


-----Original Message-----
From: Takashi Iwai [mailto:tiwai@suse.de] 
Sent: Thursday, June 04, 2009 2:34 PM
To: Alan Alan
Cc: Wei Ni; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
Subject: Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio

At Thu, 4 Jun 2009 08:18:23 +0200,
Alan Alan wrote:
> 
> 2009/6/4 Takashi Iwai <tiwai@suse.de>
> 
>     At Thu, 4 Jun 2009 01:28:24 +0200,
>     Alan Alan wrote:
>     >
>     > Hello Takashi !
>     >
>     > I tried to apply the patch that removes restrictions to 16bits
48khz
>     format.
>     > Well now we have the following:
>     > 16bit 48khz                                     works
>     > 32bit 48khz                                     works
>     > 32bit 44khz, 96khz, 192khz              don't work
>     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
>     > 16bit 44khz, 96khz, 192khz              don't work
>     >
>     > I only get
>     > Sample format not available for playback:
>     > Setting of hwparams failed:
>    
>     As you can find in alsa-info output, it seems that the codec
supports
>     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
>     (hardware) bug or not.
> 
> The codec surely supports up to 192khz as it works on Windows.
> 
> I think this line is completely wrong as here it says:

It is not "wrong".  These are values that the codec reports, so it's
more or less correct as the h/w information.

>   rates [0xc0]: 48000 88200

Wei, is it right?  Or is anything missing here?


>     bits [0xf]: 8 16 20 24
> 
> 8, 20, 24bits are supported, but they all give an error message.

24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
24bit in 3 bytes format isn't supported on HD-audio.

> 32 bits are not supposed to be supported but it works with 48 and
88.2khz.
> 
> How can we change the software side to accept 96 or 192khz spample
formats and
> 24 bits ?

As long as the codec reports so, it's no "software" problem.

> I think it is not hardware limited and we just need to change the
allowed
> formats to be played.

If it's just a hardware problem that reports wrong values, you can
override the information, simply replacing the bits my patch disabled
to the right values.


Takashi

>    
>     >
>     > I enclosed to pastebin my alsainfo script output.
>     >
>     > http://pastebin.com/m77dcfb3f
>     >
>     > Thanks for the help !
>     >
>     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >
>     >     At Wed, 3 Jun 2009 11:34:00 +0200,
>     >     Alan Alan wrote:
>     >     >
>     >     > Hello Takashi !
>     >     >
>     >     > What should this patch do ?
>     >
>     >     The patch removes the restriction patch_nvhdmi.c sets.
Instead,
>     >     the driver tries to detect the supported format, rate, etc
from the
>     >     codec widget information.
>     >
>     >     > Is it a cure for the 96, 192khz or 24bit sample playback
problem ?
>     >
>     >     Possibly.
>     >
>     >     > If so, will it resample or natievly pass the stream to the
>     amplifier ?
>     >
>     >     It just enables the native support, possibly.
>     >
>     >     And, don't forget to give back the alsa-info.sh output (run
with
>     >     --no-upload option).
>     >
>     >     Takashi
>     >
>     >     >
>     >     > Thank you
>     >     >
>     >     > Alan
>     >     >
>     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >     >
>     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
>     >     >     Alan Alan wrote:
>     >     >     >
>     >     >     > Hello Wei !
>     >     >     >
>     >     >     > Thanks for replying.
>     >     >     > I tested it with speaker-test and xbmc babylon. Both
had
>     problems
>     >     with
>     >     >     > channel mapping.
>     >     >     > I used this command:
>     >     >     >
>     >     >     > speaker-test -Dhdmi -c6
>     >     >     > or
>     >     >     > speaker-test -Dhdmi -c8
>     >     >     >
>     >     >     > alternatively you could try
>     >     >     >
>     >     >     > speaker-test -Dhw:0,3 -c6
>     >     >     >
>     >     >     >
>     >     >     > I had to rund Xorg first and have the command from
an xterm,
>     >     becasue
>     >     >     from a
>     >     >     > normal terminal it would not work.
>     >     >     > I think that is normal as a limitation of hdmi.
>     >     >     >
>     >     >     > What about the support for 24bits 192khz sound ?
>     >     >     >
>     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
>     >     >
>     >     >     How about the patch below?
>     >     >
>     >     >     Also, could you give alsa-info.sh output?
>     >     >
>     >     >     Takashi
>     >     >
>     >     >     ---
>     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c
b/sound/pci/hda/
>     >     patch_nvhdmi.c
>     >     >     index f5792e2..48443de 100644
>     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
>     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
>     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
>     >     >     nvhdmi_pcm_digital_playback_8ch = {
>     >     >            .channels_min = 2,
>     >     >            .channels_max = 8,
>     >     >            .nid = Nv_Master_Convert_nid,
>     >     >     +#if 0
>     >     >            .rates = SNDRV_PCM_RATE_48000,
>     >     >            .maxbps = 16,
>     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     >     +#endif
>     >     >            .ops = {
>     >     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >     >                    .close =
nvhdmi_dig_playback_pcm_close_8ch,
>     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
>     >     >     nvhdmi_pcm_digital_playback_2ch = {
>     >     >            .channels_min = 2,
>     >     >            .channels_max = 2,
>     >     >            .nid = Nv_Master_Convert_nid,
>     >     >     +#if 0
>     >     >            .rates = SNDRV_PCM_RATE_48000,
>     >     >            .maxbps = 16,
>     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     >     +#endif
>     >     >            .ops = {
>     >     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >     >                    .close =
nvhdmi_dig_playback_pcm_close_2ch,
>     >     >
>     >     >
>     >
>     >
> 
> 
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Thu, 4 Jun 2009 14:39:16 +0800,
Wei Ni wrote:
> 
> About rates [0xc0]: 48000 88200, what chipset do you use? Or could
> you tell me the codec id, I will check it with our hardware team.

You can find the alsa-info.sh output of his machine below:
    http://pastebin.com/m77dcfb3f


Takashi


> 
> Thanks
> Wei.
> 
> 
> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai@suse.de] 
> Sent: Thursday, June 04, 2009 2:34 PM
> To: Alan Alan
> Cc: Wei Ni; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
> Subject: Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio
> 
> At Thu, 4 Jun 2009 08:18:23 +0200,
> Alan Alan wrote:
> > 
> > 2009/6/4 Takashi Iwai <tiwai@suse.de>
> > 
> >     At Thu, 4 Jun 2009 01:28:24 +0200,
> >     Alan Alan wrote:
> >     >
> >     > Hello Takashi !
> >     >
> >     > I tried to apply the patch that removes restrictions to 16bits
> 48khz
> >     format.
> >     > Well now we have the following:
> >     > 16bit 48khz                                     works
> >     > 32bit 48khz                                     works
> >     > 32bit 44khz, 96khz, 192khz              don't work
> >     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
> >     > 16bit 44khz, 96khz, 192khz              don't work
> >     >
> >     > I only get
> >     > Sample format not available for playback:
> >     > Setting of hwparams failed:
> >    
> >     As you can find in alsa-info output, it seems that the codec
> supports
> >     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
> >     (hardware) bug or not.
> > 
> > The codec surely supports up to 192khz as it works on Windows.
> > 
> > I think this line is completely wrong as here it says:
> 
> It is not "wrong".  These are values that the codec reports, so it's
> more or less correct as the h/w information.
> 
> >   rates [0xc0]: 48000 88200
> 
> Wei, is it right?  Or is anything missing here?
> 
> 
> >     bits [0xf]: 8 16 20 24
> > 
> > 8, 20, 24bits are supported, but they all give an error message.
> 
> 24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
> 24bit in 3 bytes format isn't supported on HD-audio.
> 
> > 32 bits are not supposed to be supported but it works with 48 and
> 88.2khz.
> > 
> > How can we change the software side to accept 96 or 192khz spample
> formats and
> > 24 bits ?
> 
> As long as the codec reports so, it's no "software" problem.
> 
> > I think it is not hardware limited and we just need to change the
> allowed
> > formats to be played.
> 
> If it's just a hardware problem that reports wrong values, you can
> override the information, simply replacing the bits my patch disabled
> to the right values.
> 
> 
> Takashi
> 
> >    
> >     >
> >     > I enclosed to pastebin my alsainfo script output.
> >     >
> >     > http://pastebin.com/m77dcfb3f
> >     >
> >     > Thanks for the help !
> >     >
> >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >     >
> >     >     At Wed, 3 Jun 2009 11:34:00 +0200,
> >     >     Alan Alan wrote:
> >     >     >
> >     >     > Hello Takashi !
> >     >     >
> >     >     > What should this patch do ?
> >     >
> >     >     The patch removes the restriction patch_nvhdmi.c sets.
> Instead,
> >     >     the driver tries to detect the supported format, rate, etc
> from the
> >     >     codec widget information.
> >     >
> >     >     > Is it a cure for the 96, 192khz or 24bit sample playback
> problem ?
> >     >
> >     >     Possibly.
> >     >
> >     >     > If so, will it resample or natievly pass the stream to the
> >     amplifier ?
> >     >
> >     >     It just enables the native support, possibly.
> >     >
> >     >     And, don't forget to give back the alsa-info.sh output (run
> with
> >     >     --no-upload option).
> >     >
> >     >     Takashi
> >     >
> >     >     >
> >     >     > Thank you
> >     >     >
> >     >     > Alan
> >     >     >
> >     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >     >     >
> >     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
> >     >     >     Alan Alan wrote:
> >     >     >     >
> >     >     >     > Hello Wei !
> >     >     >     >
> >     >     >     > Thanks for replying.
> >     >     >     > I tested it with speaker-test and xbmc babylon. Both
> had
> >     problems
> >     >     with
> >     >     >     > channel mapping.
> >     >     >     > I used this command:
> >     >     >     >
> >     >     >     > speaker-test -Dhdmi -c6
> >     >     >     > or
> >     >     >     > speaker-test -Dhdmi -c8
> >     >     >     >
> >     >     >     > alternatively you could try
> >     >     >     >
> >     >     >     > speaker-test -Dhw:0,3 -c6
> >     >     >     >
> >     >     >     >
> >     >     >     > I had to rund Xorg first and have the command from
> an xterm,
> >     >     becasue
> >     >     >     from a
> >     >     >     > normal terminal it would not work.
> >     >     >     > I think that is normal as a limitation of hdmi.
> >     >     >     >
> >     >     >     > What about the support for 24bits 192khz sound ?
> >     >     >     >
> >     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
> >     >     >
> >     >     >     How about the patch below?
> >     >     >
> >     >     >     Also, could you give alsa-info.sh output?
> >     >     >
> >     >     >     Takashi
> >     >     >
> >     >     >     ---
> >     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c
> b/sound/pci/hda/
> >     >     patch_nvhdmi.c
> >     >     >     index f5792e2..48443de 100644
> >     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
> >     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
> >     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
> >     >     >     nvhdmi_pcm_digital_playback_8ch = {
> >     >     >            .channels_min = 2,
> >     >     >            .channels_max = 8,
> >     >     >            .nid = Nv_Master_Convert_nid,
> >     >     >     +#if 0
> >     >     >            .rates = SNDRV_PCM_RATE_48000,
> >     >     >            .maxbps = 16,
> >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     >     >     +#endif
> >     >     >            .ops = {
> >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> >     >     >                    .close =
> nvhdmi_dig_playback_pcm_close_8ch,
> >     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
> >     >     >     nvhdmi_pcm_digital_playback_2ch = {
> >     >     >            .channels_min = 2,
> >     >     >            .channels_max = 2,
> >     >     >            .nid = Nv_Master_Convert_nid,
> >     >     >     +#if 0
> >     >     >            .rates = SNDRV_PCM_RATE_48000,
> >     >     >            .maxbps = 16,
> >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     >     >     +#endif
> >     >     >            .ops = {
> >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> >     >     >                    .close =
> nvhdmi_dig_playback_pcm_close_2ch,
> >     >     >
> >     >     >
> >     >
> >     >
> > 
> > 
> -----------------------------------------------------------------------------------
> This email message is for the sole use of the intended recipient(s) and may contain
> confidential information.  Any unauthorized review, use, disclosure or distribution
> is prohibited.  If you are not the intended recipient, please contact the sender by
> reply email and destroy all copies of the original message.
> -----------------------------------------------------------------------------------
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Alan Alan]

[-- Attachment #1: Type: text/plain, Size: 7167 bytes --]

I made a test on Windows. And it seems that my codec MCP7A does support up
to 8channels 192khz, 24bits no problem.

One important thing i noticed:
 If i plug the hdmi to the lcd TV, the windows control panel only shows up
to 2ch, 48khz. When i plug it into the Onkyo-TXSR605 av receiver and reboot,
only than it shows up the HD options.

I enclosed a screenshout about it. When i press test button, the audio
sounds in the receiver even with 192/24

Strange thing here that on win, it does not even show the 88.2khz number.
And sample rates depend on what device is plugged, but on alsa i always get
the same output (48, 88.2khz) whatever device i plug on.



2009/6/4 Takashi Iwai <tiwai@suse.de>

> At Thu, 4 Jun 2009 08:18:23 +0200,
> Alan Alan wrote:
> >
> > 2009/6/4 Takashi Iwai <tiwai@suse.de>
> >
> >     At Thu, 4 Jun 2009 01:28:24 +0200,
> >     Alan Alan wrote:
> >     >
> >     > Hello Takashi !
> >     >
> >     > I tried to apply the patch that removes restrictions to 16bits
> 48khz
> >     format.
> >     > Well now we have the following:
> >     > 16bit 48khz                                     works
> >     > 32bit 48khz                                     works
> >     > 32bit 44khz, 96khz, 192khz              don't work
> >     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
> >     > 16bit 44khz, 96khz, 192khz              don't work
> >     >
> >     > I only get
> >     > Sample format not available for playback:
> >     > Setting of hwparams failed:
> >
> >     As you can find in alsa-info output, it seems that the codec supports
> >     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
> >     (hardware) bug or not.
> >
> > The codec surely supports up to 192khz as it works on Windows.
> >
> > I think this line is completely wrong as here it says:
>
> It is not "wrong".  These are values that the codec reports, so it's
> more or less correct as the h/w information.
>
> >   rates [0xc0]: 48000 88200
>
> Wei, is it right?  Or is anything missing here?
>
>
> >     bits [0xf]: 8 16 20 24
> >
> > 8, 20, 24bits are supported, but they all give an error message.
>
> 24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
> 24bit in 3 bytes format isn't supported on HD-audio.
>
> > 32 bits are not supposed to be supported but it works with 48 and
> 88.2khz.
> >
> > How can we change the software side to accept 96 or 192khz spample
> formats and
> > 24 bits ?
>
> As long as the codec reports so, it's no "software" problem.
>
> > I think it is not hardware limited and we just need to change the allowed
> > formats to be played.
>
> If it's just a hardware problem that reports wrong values, you can
> override the information, simply replacing the bits my patch disabled
> to the right values.
>
>
> Takashi
>
> >
> >     >
> >     > I enclosed to pastebin my alsainfo script output.
> >     >
> >     > http://pastebin.com/m77dcfb3f
> >     >
> >     > Thanks for the help !
> >     >
> >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >     >
> >     >     At Wed, 3 Jun 2009 11:34:00 +0200,
> >     >     Alan Alan wrote:
> >     >     >
> >     >     > Hello Takashi !
> >     >     >
> >     >     > What should this patch do ?
> >     >
> >     >     The patch removes the restriction patch_nvhdmi.c sets.
>  Instead,
> >     >     the driver tries to detect the supported format, rate, etc from
> the
> >     >     codec widget information.
> >     >
> >     >     > Is it a cure for the 96, 192khz or 24bit sample playback
> problem ?
> >     >
> >     >     Possibly.
> >     >
> >     >     > If so, will it resample or natievly pass the stream to the
> >     amplifier ?
> >     >
> >     >     It just enables the native support, possibly.
> >     >
> >     >     And, don't forget to give back the alsa-info.sh output (run
> with
> >     >     --no-upload option).
> >     >
> >     >     Takashi
> >     >
> >     >     >
> >     >     > Thank you
> >     >     >
> >     >     > Alan
> >     >     >
> >     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >     >     >
> >     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
> >     >     >     Alan Alan wrote:
> >     >     >     >
> >     >     >     > Hello Wei !
> >     >     >     >
> >     >     >     > Thanks for replying.
> >     >     >     > I tested it with speaker-test and xbmc babylon. Both
> had
> >     problems
> >     >     with
> >     >     >     > channel mapping.
> >     >     >     > I used this command:
> >     >     >     >
> >     >     >     > speaker-test -Dhdmi -c6
> >     >     >     > or
> >     >     >     > speaker-test -Dhdmi -c8
> >     >     >     >
> >     >     >     > alternatively you could try
> >     >     >     >
> >     >     >     > speaker-test -Dhw:0,3 -c6
> >     >     >     >
> >     >     >     >
> >     >     >     > I had to rund Xorg first and have the command from an
> xterm,
> >     >     becasue
> >     >     >     from a
> >     >     >     > normal terminal it would not work.
> >     >     >     > I think that is normal as a limitation of hdmi.
> >     >     >     >
> >     >     >     > What about the support for 24bits 192khz sound ?
> >     >     >     >
> >     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
> >     >     >
> >     >     >     How about the patch below?
> >     >     >
> >     >     >     Also, could you give alsa-info.sh output?
> >     >     >
> >     >     >     Takashi
> >     >     >
> >     >     >     ---
> >     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c
> b/sound/pci/hda/
> >     >     patch_nvhdmi.c
> >     >     >     index f5792e2..48443de 100644
> >     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
> >     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
> >     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
> >     >     >     nvhdmi_pcm_digital_playback_8ch = {
> >     >     >            .channels_min = 2,
> >     >     >            .channels_max = 8,
> >     >     >            .nid = Nv_Master_Convert_nid,
> >     >     >     +#if 0
> >     >     >            .rates = SNDRV_PCM_RATE_48000,
> >     >     >            .maxbps = 16,
> >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     >     >     +#endif
> >     >     >            .ops = {
> >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> >     >     >                    .close =
> nvhdmi_dig_playback_pcm_close_8ch,
> >     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
> >     >     >     nvhdmi_pcm_digital_playback_2ch = {
> >     >     >            .channels_min = 2,
> >     >     >            .channels_max = 2,
> >     >     >            .nid = Nv_Master_Convert_nid,
> >     >     >     +#if 0
> >     >     >            .rates = SNDRV_PCM_RATE_48000,
> >     >     >            .maxbps = 16,
> >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     >     >     +#endif
> >     >     >            .ops = {
> >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> >     >     >                    .close =
> nvhdmi_dig_playback_pcm_close_2ch,
> >     >     >
> >     >     >
> >     >
> >     >
> >
> >
>

[-- Attachment #2: NV-hdmi.jpg --]
[-- Type: image/jpeg, Size: 144879 bytes --]

[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wei Ni]

Some HDTVs generally only support 32kHz-48kHz, and most (newer)
receivers could be able to support 192kHz.

In windows, the display driver will indicates which formats are
supported in the CEA registers. 

But the linux display driver will not do it.

 

Could you please add 

                .rates = SNDRV_PCM_RATE_192000,

                .maxbps = 24,

And then try to play 192khz/24bit

 

I also will try it.

 

Thanks

Wei.

 

From: Alan Alan [mailto:alanwww1@gmail.com] 
Sent: Friday, June 05, 2009 7:21 AM
To: Takashi Iwai
Cc: Wei Ni; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
Subject: Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio

 

I made a test on Windows. And it seems that my codec MCP7A does support
up to 8channels 192khz, 24bits no problem.

One important thing i noticed:
 If i plug the hdmi to the lcd TV, the windows control panel only shows
up to 2ch, 48khz. When i plug it into the Onkyo-TXSR605 av receiver and
reboot, only than it shows up the HD options.

I enclosed a screenshout about it. When i press test button, the audio
sounds in the receiver even with 192/24

Strange thing here that on win, it does not even show the 88.2khz
number. And sample rates depend on what device is plugged, but on alsa i
always get the same output (48, 88.2khz) whatever device i plug on.




2009/6/4 Takashi Iwai <tiwai@suse.de>

At Thu, 4 Jun 2009 08:18:23 +0200,

Alan Alan wrote:
>
> 2009/6/4 Takashi Iwai <tiwai@suse.de>
>
>     At Thu, 4 Jun 2009 01:28:24 +0200,
>     Alan Alan wrote:
>     >
>     > Hello Takashi !
>     >
>     > I tried to apply the patch that removes restrictions to 16bits
48khz
>     format.
>     > Well now we have the following:
>     > 16bit 48khz                                     works
>     > 32bit 48khz                                     works
>     > 32bit 44khz, 96khz, 192khz              don't work
>     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
>     > 16bit 44khz, 96khz, 192khz              don't work
>     >
>     > I only get
>     > Sample format not available for playback:
>     > Setting of hwparams failed:
>
>     As you can find in alsa-info output, it seems that the codec
supports
>     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
>     (hardware) bug or not.
>
> The codec surely supports up to 192khz as it works on Windows.
>
> I think this line is completely wrong as here it says:

It is not "wrong".  These are values that the codec reports, so it's
more or less correct as the h/w information.

>   rates [0xc0]: 48000 88200

Wei, is it right?  Or is anything missing here?



>     bits [0xf]: 8 16 20 24
>
> 8, 20, 24bits are supported, but they all give an error message.

24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
24bit in 3 bytes format isn't supported on HD-audio.


> 32 bits are not supposed to be supported but it works with 48 and
88.2khz.
>
> How can we change the software side to accept 96 or 192khz spample
formats and
> 24 bits ?

As long as the codec reports so, it's no "software" problem.


> I think it is not hardware limited and we just need to change the
allowed
> formats to be played.

If it's just a hardware problem that reports wrong values, you can
override the information, simply replacing the bits my patch disabled
to the right values.



Takashi

>
>     >
>     > I enclosed to pastebin my alsainfo script output.
>     >
>     > http://pastebin.com/m77dcfb3f
>     >
>     > Thanks for the help !
>     >
>     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >
>     >     At Wed, 3 Jun 2009 11:34:00 +0200,
>     >     Alan Alan wrote:
>     >     >
>     >     > Hello Takashi !
>     >     >
>     >     > What should this patch do ?
>     >
>     >     The patch removes the restriction patch_nvhdmi.c sets.
Instead,
>     >     the driver tries to detect the supported format, rate, etc
from the
>     >     codec widget information.
>     >
>     >     > Is it a cure for the 96, 192khz or 24bit sample playback
problem ?
>     >
>     >     Possibly.
>     >
>     >     > If so, will it resample or natievly pass the stream to the
>     amplifier ?
>     >
>     >     It just enables the native support, possibly.
>     >
>     >     And, don't forget to give back the alsa-info.sh output (run
with
>     >     --no-upload option).
>     >
>     >     Takashi
>     >
>     >     >
>     >     > Thank you
>     >     >
>     >     > Alan
>     >     >
>     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
>     >     >
>     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
>     >     >     Alan Alan wrote:
>     >     >     >
>     >     >     > Hello Wei !
>     >     >     >
>     >     >     > Thanks for replying.
>     >     >     > I tested it with speaker-test and xbmc babylon. Both
had
>     problems
>     >     with
>     >     >     > channel mapping.
>     >     >     > I used this command:
>     >     >     >
>     >     >     > speaker-test -Dhdmi -c6
>     >     >     > or
>     >     >     > speaker-test -Dhdmi -c8
>     >     >     >
>     >     >     > alternatively you could try
>     >     >     >
>     >     >     > speaker-test -Dhw:0,3 -c6
>     >     >     >
>     >     >     >
>     >     >     > I had to rund Xorg first and have the command from
an xterm,
>     >     becasue
>     >     >     from a
>     >     >     > normal terminal it would not work.
>     >     >     > I think that is normal as a limitation of hdmi.
>     >     >     >
>     >     >     > What about the support for 24bits 192khz sound ?
>     >     >     >
>     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
>     >     >
>     >     >     How about the patch below?
>     >     >
>     >     >     Also, could you give alsa-info.sh output?
>     >     >
>     >     >     Takashi
>     >     >
>     >     >     ---
>     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c
b/sound/pci/hda/
>     >     patch_nvhdmi.c
>     >     >     index f5792e2..48443de 100644
>     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
>     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
>     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
>     >     >     nvhdmi_pcm_digital_playback_8ch = {
>     >     >            .channels_min = 2,
>     >     >            .channels_max = 8,
>     >     >            .nid = Nv_Master_Convert_nid,
>     >     >     +#if 0
>     >     >            .rates = SNDRV_PCM_RATE_48000,
>     >     >            .maxbps = 16,
>     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     >     +#endif
>     >     >            .ops = {
>     >     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >     >                    .close =
nvhdmi_dig_playback_pcm_close_8ch,
>     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
>     >     >     nvhdmi_pcm_digital_playback_2ch = {
>     >     >            .channels_min = 2,
>     >     >            .channels_max = 2,
>     >     >            .nid = Nv_Master_Convert_nid,
>     >     >     +#if 0
>     >     >            .rates = SNDRV_PCM_RATE_48000,
>     >     >            .maxbps = 16,
>     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
>     >     >     +#endif
>     >     >            .ops = {
>     >     >                    .open = nvhdmi_dig_playback_pcm_open,
>     >     >                    .close =
nvhdmi_dig_playback_pcm_close_2ch,
>     >     >
>     >     >
>     >
>     >
>
>

 


-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

Re: ALSA patch Nvidia 8 channel hdmi audio

[Alan Alan]

Hi Wei !

I made the tests changig the values in the sound/pci/hda/patch_nvhdmi.c file
manually.
To summarize the results i have perfect sound up to 192khz/32bits(S32_LE
format). But only one configuration at a time.

How can we set multiple formats here, or let the system pass all formats to
the av receiver ?

Here are the detailed results i tested.

In the source code
        .rates = SNDRV_PCM_RATE_A
        .maxbps = B,
        .formats = SNDRV_PCM_FMTBIT_SC_LE,

case.

1. A=192000, B=24, C=16,      perfect sound, with speaker-test -Dhdmi -c6
-r192000 -FS16_LE, no other setting working
2. A= 192000, B=24, C=32,     perfect sound, with speaker-test -Dhdmi -c6
-r192000 -FS32_LE, no other setting working
3. A= 96000, B=24, C=32,       perfect sound, with speaker-test -Dhdmi -c6
-r96000 -FS32_LE, no other setting working
4. A=192000, B=24, C=24,      strange high pitched sound from all speakers,
with speaker-test -Dhdmi -c6 -r192000 -FS24_LE, no other settings working
5. A=  96000, B=24, C=24,      strange high pitched sound from all speakers,
with speaker-test -Dhdmi -c6 -r96000 -FS24_LE, no other settings working

If you want i will test more. But i think it can be seen that S24 formats
are not working, but all S16, S32 formats working up to 192khz.

Regards

Alan


2009/6/5 Wei Ni <wni@nvidia.com>

>  Some HDTVs generally only support 32kHz-48kHz, and most (newer) receivers
> could be able to support 192kHz.
>
> In windows, the display driver will indicates which formats are supported
> in the CEA registers.
>
> But the linux display driver will not do it.
>
>
>
> Could you please add
>
>                 .rates = SNDRV_PCM_RATE_192000,
>
>                 .maxbps = 24,
>
> And then try to play 192khz/24bit
>
>
>
> I also will try it.
>
>
>
> Thanks
>
> Wei.
>
>
>
> *From:* Alan Alan [mailto:alanwww1@gmail.com]
> *Sent:* Friday, June 05, 2009 7:21 AM
> *To:* Takashi Iwai
> *Cc:* Wei Ni; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
> *Subject:* Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio
>
>
>
> I made a test on Windows. And it seems that my codec MCP7A does support up
> to 8channels 192khz, 24bits no problem.
>
> One important thing i noticed:
>  If i plug the hdmi to the lcd TV, the windows control panel only shows up
> to 2ch, 48khz. When i plug it into the Onkyo-TXSR605 av receiver and reboot,
> only than it shows up the HD options.
>
> I enclosed a screenshout about it. When i press test button, the audio
> sounds in the receiver even with 192/24
>
> Strange thing here that on win, it does not even show the 88.2khz number.
> And sample rates depend on what device is plugged, but on alsa i always get
> the same output (48, 88.2khz) whatever device i plug on.
>
>
>  2009/6/4 Takashi Iwai <tiwai@suse.de>
>
> At Thu, 4 Jun 2009 08:18:23 +0200,
>
> Alan Alan wrote:
> >
> > 2009/6/4 Takashi Iwai <tiwai@suse.de>
> >
> >     At Thu, 4 Jun 2009 01:28:24 +0200,
> >     Alan Alan wrote:
> >     >
> >     > Hello Takashi !
> >     >
> >     > I tried to apply the patch that removes restrictions to 16bits
> 48khz
> >     format.
> >     > Well now we have the following:
> >     > 16bit 48khz                                     works
> >     > 32bit 48khz                                     works
> >     > 32bit 44khz, 96khz, 192khz              don't work
> >     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
> >     > 16bit 44khz, 96khz, 192khz              don't work
> >     >
> >     > I only get
> >     > Sample format not available for playback:
> >     > Setting of hwparams failed:
> >
> >     As you can find in alsa-info output, it seems that the codec supports
> >     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
> >     (hardware) bug or not.
> >
> > The codec surely supports up to 192khz as it works on Windows.
> >
> > I think this line is completely wrong as here it says:
>
> It is not "wrong".  These are values that the codec reports, so it's
> more or less correct as the h/w information.
>
> >   rates [0xc0]: 48000 88200
>
> Wei, is it right?  Or is anything missing here?
>
>
>
> >     bits [0xf]: 8 16 20 24
> >
> > 8, 20, 24bits are supported, but they all give an error message.
>
> 24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
> 24bit in 3 bytes format isn't supported on HD-audio.
>
>
> > 32 bits are not supposed to be supported but it works with 48 and
> 88.2khz.
> >
> > How can we change the software side to accept 96 or 192khz spample
> formats and
> > 24 bits ?
>
> As long as the codec reports so, it's no "software" problem.
>
>
> > I think it is not hardware limited and we just need to change the allowed
> > formats to be played.
>
> If it's just a hardware problem that reports wrong values, you can
> override the information, simply replacing the bits my patch disabled
> to the right values.
>
>
>
> Takashi
>
> >
> >     >
> >     > I enclosed to pastebin my alsainfo script output.
> >     >
> >     > http://pastebin.com/m77dcfb3f
> >     >
> >     > Thanks for the help !
> >     >
> >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >     >
> >     >     At Wed, 3 Jun 2009 11:34:00 +0200,
> >     >     Alan Alan wrote:
> >     >     >
> >     >     > Hello Takashi !
> >     >     >
> >     >     > What should this patch do ?
> >     >
> >     >     The patch removes the restriction patch_nvhdmi.c sets.
>  Instead,
> >     >     the driver tries to detect the supported format, rate, etc from
> the
> >     >     codec widget information.
> >     >
> >     >     > Is it a cure for the 96, 192khz or 24bit sample playback
> problem ?
> >     >
> >     >     Possibly.
> >     >
> >     >     > If so, will it resample or natievly pass the stream to the
> >     amplifier ?
> >     >
> >     >     It just enables the native support, possibly.
> >     >
> >     >     And, don't forget to give back the alsa-info.sh output (run
> with
> >     >     --no-upload option).
> >     >
> >     >     Takashi
> >     >
> >     >     >
> >     >     > Thank you
> >     >     >
> >     >     > Alan
> >     >     >
> >     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> >     >     >
> >     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
> >     >     >     Alan Alan wrote:
> >     >     >     >
> >     >     >     > Hello Wei !
> >     >     >     >
> >     >     >     > Thanks for replying.
> >     >     >     > I tested it with speaker-test and xbmc babylon. Both
> had
> >     problems
> >     >     with
> >     >     >     > channel mapping.
> >     >     >     > I used this command:
> >     >     >     >
> >     >     >     > speaker-test -Dhdmi -c6
> >     >     >     > or
> >     >     >     > speaker-test -Dhdmi -c8
> >     >     >     >
> >     >     >     > alternatively you could try
> >     >     >     >
> >     >     >     > speaker-test -Dhw:0,3 -c6
> >     >     >     >
> >     >     >     >
> >     >     >     > I had to rund Xorg first and have the command from an
> xterm,
> >     >     becasue
> >     >     >     from a
> >     >     >     > normal terminal it would not work.
> >     >     >     > I think that is normal as a limitation of hdmi.
> >     >     >     >
> >     >     >     > What about the support for 24bits 192khz sound ?
> >     >     >     >
> >     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
> >     >     >
> >     >     >     How about the patch below?
> >     >     >
> >     >     >     Also, could you give alsa-info.sh output?
> >     >     >
> >     >     >     Takashi
> >     >     >
> >     >     >     ---
> >     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c
> b/sound/pci/hda/
> >     >     patch_nvhdmi.c
> >     >     >     index f5792e2..48443de 100644
> >     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
> >     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
> >     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
> >     >     >     nvhdmi_pcm_digital_playback_8ch = {
> >     >     >            .channels_min = 2,
> >     >     >            .channels_max = 8,
> >     >     >            .nid = Nv_Master_Convert_nid,
> >     >     >     +#if 0
> >     >     >            .rates = SNDRV_PCM_RATE_48000,
> >     >     >            .maxbps = 16,
> >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     >     >     +#endif
> >     >     >            .ops = {
> >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> >     >     >                    .close =
> nvhdmi_dig_playback_pcm_close_8ch,
> >     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
> >     >     >     nvhdmi_pcm_digital_playback_2ch = {
> >     >     >            .channels_min = 2,
> >     >     >            .channels_max = 2,
> >     >     >            .nid = Nv_Master_Convert_nid,
> >     >     >     +#if 0
> >     >     >            .rates = SNDRV_PCM_RATE_48000,
> >     >     >            .maxbps = 16,
> >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> >     >     >     +#endif
> >     >     >            .ops = {
> >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> >     >     >                    .close =
> nvhdmi_dig_playback_pcm_close_2ch,
> >     >     >
> >     >     >
> >     >
> >     >
> >
> >
>
>
>  ------------------------------
>  This email message is for the sole use of the intended recipient(s) and
> may contain confidential information.  Any unauthorized review, use,
> disclosure or distribution is prohibited.  If you are not the intended
> recipient, please contact the sender by reply email and destroy all copies
> of the original message.
>  ------------------------------
>
>

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Sun, 7 Jun 2009 23:43:04 +0200,
Alan Alan wrote:
> 
> Hi Wei !
> 
> I made the tests changig the values in the sound/pci/hda/patch_nvhdmi.c file
> manually.
> To summarize the results i have perfect sound up to 192khz/32bits(S32_LE
> format). But only one configuration at a time.
> 
> How can we set multiple formats here, or let the system pass all formats to
> the av receiver ?
> 
> Here are the detailed results i tested.
> 
> In the source code
>         .rates = SNDRV_PCM_RATE_A
>         .maxbps = B,
>         .formats = SNDRV_PCM_FMTBIT_SC_LE,

Change like

	.rates = SNDRV_PCM_RATE_A | SNDRV_PCM_RATE_B | ...,
	.maxbps = 24,
	.formats = SNDRV_PCM_FMTBIT_A | SNDRV_PCM_FMTBIT_B | ...,

> 1. A=192000, B=24, C=16,      perfect sound, with speaker-test -Dhdmi -c6
> -r192000 -FS16_LE, no other setting working
> 2. A= 192000, B=24, C=32,     perfect sound, with speaker-test -Dhdmi -c6
> -r192000 -FS32_LE, no other setting working
> 3. A= 96000, B=24, C=32,       perfect sound, with speaker-test -Dhdmi -c6 
> -r96000 -FS32_LE, no other setting working
> 4. A=192000, B=24, C=24,      strange high pitched sound from all speakers,
> with speaker-test -Dhdmi -c6 -r192000 -FS24_LE, no other settings working
> 5. A=  96000, B=24, C=24,      strange high pitched sound from all speakers,
> with speaker-test -Dhdmi -c6 -r96000 -FS24_LE, no other settings working

S24_LE is no format for HD-audio.  It's 24bit packed in lower 3 bytes
of 4 byte frame.  For 24bit format, use S32_LE.


Takashi

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Fri, 5 Jun 2009 01:20:52 +0200,
Alan Alan wrote:
> 
> I made a test on Windows. And it seems that my codec MCP7A does support up
> to 8channels 192khz, 24bits no problem.
> 
> One important thing i noticed:
>  If i plug the hdmi to the lcd TV, the windows control panel only shows up
> to 2ch, 48khz. When i plug it into the Onkyo-TXSR605 av receiver and reboot,
> only than it shows up the HD options.
> 
> I enclosed a screenshout about it. When i press test button, the audio
> sounds in the receiver even with 192/24
> 
> Strange thing here that on win, it does not even show the 88.2khz number.
> And sample rates depend on what device is plugged, but on alsa i always get
> the same output (48, 88.2khz) whatever device i plug on.

As Wei pointed, currently the video driver doesn't send the correct
CEA register setup, thus the supported sample rates aren't updated.
(HDMI is a thing that is always coupled with the video.  That's why it
 works only when you start X.)

Maybe the easiest workaround is to give all possible values statically
independent on the plugging state.  Try to write other bits in
addition to 48k and 16bit format.

BTW, what happens if you plug HDMI before boot and loading the driver?


Takashi

> 
> 
> 
> 2009/6/4 Takashi Iwai <tiwai@suse.de>
> 
> > At Thu, 4 Jun 2009 08:18:23 +0200,
> > Alan Alan wrote:
> > >
> > > 2009/6/4 Takashi Iwai <tiwai@suse.de>
> > >
> > >     At Thu, 4 Jun 2009 01:28:24 +0200,
> > >     Alan Alan wrote:
> > >     >
> > >     > Hello Takashi !
> > >     >
> > >     > I tried to apply the patch that removes restrictions to 16bits
> > 48khz
> > >     format.
> > >     > Well now we have the following:
> > >     > 16bit 48khz                                     works
> > >     > 32bit 48khz                                     works
> > >     > 32bit 44khz, 96khz, 192khz              don't work
> > >     > 24bit 44khz, 48khz, 96khz, 192khz   don't work
> > >     > 16bit 44khz, 96khz, 192khz              don't work
> > >     >
> > >     > I only get
> > >     > Sample format not available for playback:
> > >     > Setting of hwparams failed:
> > >
> > >     As you can find in alsa-info output, it seems that the codec supports
> > >     only 48kHz and 88.2kHz, strangely.  I don't know whether it's a
> > >     (hardware) bug or not.
> > >
> > > The codec surely supports up to 192khz as it works on Windows.
> > >
> > > I think this line is completely wrong as here it says:
> >
> > It is not "wrong".  These are values that the codec reports, so it's
> > more or less correct as the h/w information.
> >
> > >   rates [0xc0]: 48000 88200
> >
> > Wei, is it right?  Or is anything missing here?
> >
> >
> > >     bits [0xf]: 8 16 20 24
> > >
> > > 8, 20, 24bits are supported, but they all give an error message.
> >
> > 24bit here means 24bit in 4 bytes, thus it's essentially 32bit format.
> > 24bit in 3 bytes format isn't supported on HD-audio.
> >
> > > 32 bits are not supposed to be supported but it works with 48 and
> > 88.2khz.
> > >
> > > How can we change the software side to accept 96 or 192khz spample
> > formats and
> > > 24 bits ?
> >
> > As long as the codec reports so, it's no "software" problem.
> >
> > > I think it is not hardware limited and we just need to change the allowed
> > > formats to be played.
> >
> > If it's just a hardware problem that reports wrong values, you can
> > override the information, simply replacing the bits my patch disabled
> > to the right values.
> >
> >
> > Takashi
> >
> > >
> > >     >
> > >     > I enclosed to pastebin my alsainfo script output.
> > >     >
> > >     > http://pastebin.com/m77dcfb3f
> > >     >
> > >     > Thanks for the help !
> > >     >
> > >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> > >     >
> > >     >     At Wed, 3 Jun 2009 11:34:00 +0200,
> > >     >     Alan Alan wrote:
> > >     >     >
> > >     >     > Hello Takashi !
> > >     >     >
> > >     >     > What should this patch do ?
> > >     >
> > >     >     The patch removes the restriction patch_nvhdmi.c sets.
> >  Instead,
> > >     >     the driver tries to detect the supported format, rate, etc from
> > the
> > >     >     codec widget information.
> > >     >
> > >     >     > Is it a cure for the 96, 192khz or 24bit sample playback
> > problem ?
> > >     >
> > >     >     Possibly.
> > >     >
> > >     >     > If so, will it resample or natievly pass the stream to the
> > >     amplifier ?
> > >     >
> > >     >     It just enables the native support, possibly.
> > >     >
> > >     >     And, don't forget to give back the alsa-info.sh output (run
> > with
> > >     >     --no-upload option).
> > >     >
> > >     >     Takashi
> > >     >
> > >     >     >
> > >     >     > Thank you
> > >     >     >
> > >     >     > Alan
> > >     >     >
> > >     >     > 2009/6/3 Takashi Iwai <tiwai@suse.de>
> > >     >     >
> > >     >     >     At Wed, 3 Jun 2009 10:00:24 +0200,
> > >     >     >     Alan Alan wrote:
> > >     >     >     >
> > >     >     >     > Hello Wei !
> > >     >     >     >
> > >     >     >     > Thanks for replying.
> > >     >     >     > I tested it with speaker-test and xbmc babylon. Both
> > had
> > >     problems
> > >     >     with
> > >     >     >     > channel mapping.
> > >     >     >     > I used this command:
> > >     >     >     >
> > >     >     >     > speaker-test -Dhdmi -c6
> > >     >     >     > or
> > >     >     >     > speaker-test -Dhdmi -c8
> > >     >     >     >
> > >     >     >     > alternatively you could try
> > >     >     >     >
> > >     >     >     > speaker-test -Dhw:0,3 -c6
> > >     >     >     >
> > >     >     >     >
> > >     >     >     > I had to rund Xorg first and have the command from an
> > xterm,
> > >     >     becasue
> > >     >     >     from a
> > >     >     >     > normal terminal it would not work.
> > >     >     >     > I think that is normal as a limitation of hdmi.
> > >     >     >     >
> > >     >     >     > What about the support for 24bits 192khz sound ?
> > >     >     >     >
> > >     >     >     > speaker-test -Dhdmi -c6  -r192000 does not work.
> > >     >     >
> > >     >     >     How about the patch below?
> > >     >     >
> > >     >     >     Also, could you give alsa-info.sh output?
> > >     >     >
> > >     >     >     Takashi
> > >     >     >
> > >     >     >     ---
> > >     >     >     diff --git a/sound/pci/hda/patch_nvhdmi.c
> > b/sound/pci/hda/
> > >     >     patch_nvhdmi.c
> > >     >     >     index f5792e2..48443de 100644
> > >     >     >     --- a/sound/pci/hda/patch_nvhdmi.c
> > >     >     >     +++ b/sound/pci/hda/patch_nvhdmi.c
> > >     >     >     @@ -258,9 +258,11 @@ static struct hda_pcm_stream
> > >     >     >     nvhdmi_pcm_digital_playback_8ch = {
> > >     >     >            .channels_min = 2,
> > >     >     >            .channels_max = 8,
> > >     >     >            .nid = Nv_Master_Convert_nid,
> > >     >     >     +#if 0
> > >     >     >            .rates = SNDRV_PCM_RATE_48000,
> > >     >     >            .maxbps = 16,
> > >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> > >     >     >     +#endif
> > >     >     >            .ops = {
> > >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> > >     >     >                    .close =
> > nvhdmi_dig_playback_pcm_close_8ch,
> > >     >     >     @@ -273,9 +275,11 @@ static struct hda_pcm_stream
> > >     >     >     nvhdmi_pcm_digital_playback_2ch = {
> > >     >     >            .channels_min = 2,
> > >     >     >            .channels_max = 2,
> > >     >     >            .nid = Nv_Master_Convert_nid,
> > >     >     >     +#if 0
> > >     >     >            .rates = SNDRV_PCM_RATE_48000,
> > >     >     >            .maxbps = 16,
> > >     >     >            .formats = SNDRV_PCM_FMTBIT_S16_LE,
> > >     >     >     +#endif
> > >     >     >            .ops = {
> > >     >     >                    .open = nvhdmi_dig_playback_pcm_open,
> > >     >     >                    .close =
> > nvhdmi_dig_playback_pcm_close_2ch,
> > >     >     >
> > >     >     >
> > >     >
> > >     >
> > >
> > >
> >

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wei Ni]

Hi, Alan

I reproduced this problems with speaker-test. The channels are wrong
mapped.

It's strange that if use mplayer to run, it looks ok.

I'm looking this problems, and try to find the root cause.

 

Thanks

 

From: Alan Alan [mailto:alanwww1@gmail.com] 
Sent: Wednesday, June 03, 2009 4:00 PM
To: Wei Ni
Cc: Marek Vasut; alsa-devel@alsa-project.org; Peer Chen
Subject: Re: ALSA patch Nvidia 8 channel hdmi audio

 

Hello Wei !

Thanks for replying.
I tested it with speaker-test and xbmc babylon. Both had problems with
channel mapping.
I used this command:

speaker-test -Dhdmi -c6
or
speaker-test -Dhdmi -c8

alternatively you could try 

speaker-test -Dhw:0,3 -c6


I had to rund Xorg first and have the command from an xterm, becasue
from a normal terminal it would not work.
I think that is normal as a limitation of hdmi.

What about the support for 24bits 192khz sound ?

speaker-test -Dhdmi -c6  -r192000 does not work.

Thanks for working on this.





2009/6/3 Wei Ni <wni@nvidia.com>

Hi, Marek and Alan
I'm a Nvidia engineer and develop this patch file.

I used mplayer to test. The channels are mapped good.
Running mplayer -ao alsa:device=hw=0.3 6ch.wav

I tried to use speaker-test to test, but I can't get the hdmi device
name.
I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
analog jack.
Which device name should I use?

I use aplay -l, it show
**** List of PLAYBACK Hardware Devices ****
card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
 Subdevices: 1/1
 Subdevice #0: subdevice #0
card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200 Digital]
 Subdevices: 1/1
 Subdevice #0: subdevice #0
card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
 Subdevices: 1/1
 Subdevice #0: subdevice #0

Thanks
Wei.


-----Original Message-----
From: Marek Vasut [mailto:marek.vasut@gmail.com]
Sent: Wednesday, June 03, 2009 8:01 AM
To: Alan Alan
Cc: alsa-devel@alsa-project.org; Wei Ni
Subject: Re: ALSA patch Nvidia 8 channel hdmi audio

On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> Thank's for the info.
> How can i contact him ?

He and alsa-devel mailing list are both in CC already, please preserve
them
there or fix your mailer.
>
>
> 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
>
> > I'm not the author of the patch, CCing alsa-devel and the original
> > author.
> >
> > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > Hi Marek !
> > >
> > > I just read your emailing with nvidia devs about alsa supporting 8
> >
> > channel
> >
> > > lpcm over hdmi.
> > >
> > > I checked it out an i have to tell it is working awsome, except
for a
> > > few bugs:
> > >
> > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6
gives
> > > the following result (first where it should sound)
> > > FL - FL
> > > C - RL
> > > FR - FR
> > > RR - LFE
> > > RL - C
> > >
> > > The other thing i realized that playback is limited to 16 bits 48
khz.
> > > I think the minimum which is supported by nvidia hardware is
24bits 96
> > > (or 192) khz.
> > > Why not ?
> > >
> > > Could you contact the devs as i don't know how to reach them.
> > >
> > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with
the
> > > oatched nvidia.c file. Anyway the amplifier shows multichannel
LPCM and
> > > that's great news.




------------------------------------------------------------------------
-----------
This email message is for the sole use of the intended recipient(s) and
may contain
confidential information.  Any unauthorized review, use, disclosure or
distribution
is prohibited.  If you are not the intended recipient, please contact
the sender by
reply email and destroy all copies of the original message.
------------------------------------------------------------------------
-----------

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Wed, 3 Jun 2009 18:25:09 +0800,
Wei Ni wrote:
> 
> Hi, Alan
> 
> I reproduced this problems with speaker-test. The channels are wrong
> mapped.
> 
> It's strange that if use mplayer to run, it looks ok.
> 
> I'm looking this problems, and try to find the root cause.

Actually, from historical reasons, the standard ALSA channel mapping
for 5.1 is
    FL/FR/RL/RR/C/LFE
while Windows sets FL/FR/C/LFE/RL/RR.  If you play a WAV file assuming
the latter channel assignment, it will be played like that.

Does the nvidia codec support channel-slot assignment (verb 0x734)?
If yes, we can assign the channels freely to fix this...


thanks,

Takashi


> 
>  
> 
> Thanks
> 
>  
> 
> From: Alan Alan [mailto:alanwww1@gmail.com] 
> Sent: Wednesday, June 03, 2009 4:00 PM
> To: Wei Ni
> Cc: Marek Vasut; alsa-devel@alsa-project.org; Peer Chen
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> 
>  
> 
> Hello Wei !
> 
> Thanks for replying.
> I tested it with speaker-test and xbmc babylon. Both had problems with
> channel mapping.
> I used this command:
> 
> speaker-test -Dhdmi -c6
> or
> speaker-test -Dhdmi -c8
> 
> alternatively you could try 
> 
> speaker-test -Dhw:0,3 -c6
> 
> 
> I had to rund Xorg first and have the command from an xterm, becasue
> from a normal terminal it would not work.
> I think that is normal as a limitation of hdmi.
> 
> What about the support for 24bits 192khz sound ?
> 
> speaker-test -Dhdmi -c6  -r192000 does not work.
> 
> Thanks for working on this.
> 
> 
> 
> 
> 
> 2009/6/3 Wei Ni <wni@nvidia.com>
> 
> Hi, Marek and Alan
> I'm a Nvidia engineer and develop this patch file.
> 
> I used mplayer to test. The channels are mapped good.
> Running mplayer -ao alsa:device=hw=0.3 6ch.wav
> 
> I tried to use speaker-test to test, but I can't get the hdmi device
> name.
> I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
> analog jack.
> Which device name should I use?
> 
> I use aplay -l, it show
> **** List of PLAYBACK Hardware Devices ****
> card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200 Digital]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> 
> Thanks
> Wei.
> 
> 
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut@gmail.com]
> Sent: Wednesday, June 03, 2009 8:01 AM
> To: Alan Alan
> Cc: alsa-devel@alsa-project.org; Wei Ni
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> 
> On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > Thank's for the info.
> > How can i contact him ?
> 
> He and alsa-devel mailing list are both in CC already, please preserve
> them
> there or fix your mailer.
> >
> >
> > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> >
> > > I'm not the author of the patch, CCing alsa-devel and the original
> > > author.
> > >
> > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > Hi Marek !
> > > >
> > > > I just read your emailing with nvidia devs about alsa supporting 8
> > >
> > > channel
> > >
> > > > lpcm over hdmi.
> > > >
> > > > I checked it out an i have to tell it is working awsome, except
> for a
> > > > few bugs:
> > > >
> > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6
> gives
> > > > the following result (first where it should sound)
> > > > FL - FL
> > > > C - RL
> > > > FR - FR
> > > > RR - LFE
> > > > RL - C
> > > >
> > > > The other thing i realized that playback is limited to 16 bits 48
> khz.
> > > > I think the minimum which is supported by nvidia hardware is
> 24bits 96
> > > > (or 192) khz.
> > > > Why not ?
> > > >
> > > > Could you contact the devs as i don't know how to reach them.
> > > >
> > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with
> the
> > > > oatched nvidia.c file. Anyway the amplifier shows multichannel
> LPCM and
> > > > that's great news.
> 
> 
> 
> 
> ------------------------------------------------------------------------
> -----------
> This email message is for the sole use of the intended recipient(s) and
> may contain
> confidential information.  Any unauthorized review, use, disclosure or
> distribution
> is prohibited.  If you are not the intended recipient, please contact
> the sender by
> reply email and destroy all copies of the original message.
> ------------------------------------------------------------------------
> -----------
> 
>  
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wei Ni]

Yes, my test wav file is from Windows.

I had used different player to test with my test wav file.
The mplayer has no mapping problem, but other players map channels
wrong.
As you said, it seems that is caused by the difference of the ALSA and
Windows.

Unfortunately, our nvidia codec doesn't support channel-slot assignment
yet.
How could we fix this problems, do you have any suggestions?

Thanks
Wei. 

-----Original Message-----
From: Takashi Iwai [mailto:tiwai@suse.de] 
Sent: Wednesday, June 03, 2009 7:02 PM
To: Wei Ni
Cc: Alan Alan; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
Subject: Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio

At Wed, 3 Jun 2009 18:25:09 +0800,
Wei Ni wrote:
> 
> Hi, Alan
> 
> I reproduced this problems with speaker-test. The channels are wrong
> mapped.
> 
> It's strange that if use mplayer to run, it looks ok.
> 
> I'm looking this problems, and try to find the root cause.

Actually, from historical reasons, the standard ALSA channel mapping
for 5.1 is
    FL/FR/RL/RR/C/LFE
while Windows sets FL/FR/C/LFE/RL/RR.  If you play a WAV file assuming
the latter channel assignment, it will be played like that.

Does the nvidia codec support channel-slot assignment (verb 0x734)?
If yes, we can assign the channels freely to fix this...


thanks,

Takashi


> 
>  
> 
> Thanks
> 
>  
> 
> From: Alan Alan [mailto:alanwww1@gmail.com] 
> Sent: Wednesday, June 03, 2009 4:00 PM
> To: Wei Ni
> Cc: Marek Vasut; alsa-devel@alsa-project.org; Peer Chen
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> 
>  
> 
> Hello Wei !
> 
> Thanks for replying.
> I tested it with speaker-test and xbmc babylon. Both had problems with
> channel mapping.
> I used this command:
> 
> speaker-test -Dhdmi -c6
> or
> speaker-test -Dhdmi -c8
> 
> alternatively you could try 
> 
> speaker-test -Dhw:0,3 -c6
> 
> 
> I had to rund Xorg first and have the command from an xterm, becasue
> from a normal terminal it would not work.
> I think that is normal as a limitation of hdmi.
> 
> What about the support for 24bits 192khz sound ?
> 
> speaker-test -Dhdmi -c6  -r192000 does not work.
> 
> Thanks for working on this.
> 
> 
> 
> 
> 
> 2009/6/3 Wei Ni <wni@nvidia.com>
> 
> Hi, Marek and Alan
> I'm a Nvidia engineer and develop this patch file.
> 
> I used mplayer to test. The channels are mapped good.
> Running mplayer -ao alsa:device=hw=0.3 6ch.wav
> 
> I tried to use speaker-test to test, but I can't get the hdmi device
> name.
> I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
> analog jack.
> Which device name should I use?
> 
> I use aplay -l, it show
> **** List of PLAYBACK Hardware Devices ****
> card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200
Digital]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> 
> Thanks
> Wei.
> 
> 
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut@gmail.com]
> Sent: Wednesday, June 03, 2009 8:01 AM
> To: Alan Alan
> Cc: alsa-devel@alsa-project.org; Wei Ni
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> 
> On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > Thank's for the info.
> > How can i contact him ?
> 
> He and alsa-devel mailing list are both in CC already, please preserve
> them
> there or fix your mailer.
> >
> >
> > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> >
> > > I'm not the author of the patch, CCing alsa-devel and the original
> > > author.
> > >
> > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > Hi Marek !
> > > >
> > > > I just read your emailing with nvidia devs about alsa supporting
8
> > >
> > > channel
> > >
> > > > lpcm over hdmi.
> > > >
> > > > I checked it out an i have to tell it is working awsome, except
> for a
> > > > few bugs:
> > > >
> > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6
> gives
> > > > the following result (first where it should sound)
> > > > FL - FL
> > > > C - RL
> > > > FR - FR
> > > > RR - LFE
> > > > RL - C
> > > >
> > > > The other thing i realized that playback is limited to 16 bits
48
> khz.
> > > > I think the minimum which is supported by nvidia hardware is
> 24bits 96
> > > > (or 192) khz.
> > > > Why not ?
> > > >
> > > > Could you contact the devs as i don't know how to reach them.
> > > >
> > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa
with
> the
> > > > oatched nvidia.c file. Anyway the amplifier shows multichannel
> LPCM and
> > > > that's great news.
> 
> 
> 
> 
>
------------------------------------------------------------------------
> -----------
> This email message is for the sole use of the intended recipient(s)
and
> may contain
> confidential information.  Any unauthorized review, use, disclosure or
> distribution
> is prohibited.  If you are not the intended recipient, please contact
> the sender by
> reply email and destroy all copies of the original message.
>
------------------------------------------------------------------------
> -----------
> 
>  
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Thu, 4 Jun 2009 14:05:52 +0800,
Wei Ni wrote:
> 
> Yes, my test wav file is from Windows.
> 
> I had used different player to test with my test wav file.
> The mplayer has no mapping problem, but other players map channels
> wrong.
> As you said, it seems that is caused by the difference of the ALSA and
> Windows.

I guess mplayer corrects the channel mapping automatically.

> Unfortunately, our nvidia codec doesn't support channel-slot assignment
> yet.

OK.

> How could we fix this problems, do you have any suggestions?

It's a bit difficult issue right now.  Usually we could "fix" the
channel mapping in alsa-lib configuration.  But, HD-audio has currently
a single alsa-lib configuration, and others may use the right channel
mapping.

I don't remember how is the case of Intel HDMI...  Can anyone check?
If it also suffers from the same problem, the channel conversion rule
can be put in hdmi configuration statically as a temporary workaround.

We need a proper API to inform / change the channel mapping, anyway.
Then it'll be better adjustable...


thanks,

Takashi


> Thanks
> Wei. 
> 
> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai@suse.de] 
> Sent: Wednesday, June 03, 2009 7:02 PM
> To: Wei Ni
> Cc: Alan Alan; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
> Subject: Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio
> 
> At Wed, 3 Jun 2009 18:25:09 +0800,
> Wei Ni wrote:
> > 
> > Hi, Alan
> > 
> > I reproduced this problems with speaker-test. The channels are wrong
> > mapped.
> > 
> > It's strange that if use mplayer to run, it looks ok.
> > 
> > I'm looking this problems, and try to find the root cause.
> 
> Actually, from historical reasons, the standard ALSA channel mapping
> for 5.1 is
>     FL/FR/RL/RR/C/LFE
> while Windows sets FL/FR/C/LFE/RL/RR.  If you play a WAV file assuming
> the latter channel assignment, it will be played like that.
> 
> Does the nvidia codec support channel-slot assignment (verb 0x734)?
> If yes, we can assign the channels freely to fix this...
> 
> 
> thanks,
> 
> Takashi
> 
> 
> > 
> >  
> > 
> > Thanks
> > 
> >  
> > 
> > From: Alan Alan [mailto:alanwww1@gmail.com] 
> > Sent: Wednesday, June 03, 2009 4:00 PM
> > To: Wei Ni
> > Cc: Marek Vasut; alsa-devel@alsa-project.org; Peer Chen
> > Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> > 
> >  
> > 
> > Hello Wei !
> > 
> > Thanks for replying.
> > I tested it with speaker-test and xbmc babylon. Both had problems with
> > channel mapping.
> > I used this command:
> > 
> > speaker-test -Dhdmi -c6
> > or
> > speaker-test -Dhdmi -c8
> > 
> > alternatively you could try 
> > 
> > speaker-test -Dhw:0,3 -c6
> > 
> > 
> > I had to rund Xorg first and have the command from an xterm, becasue
> > from a normal terminal it would not work.
> > I think that is normal as a limitation of hdmi.
> > 
> > What about the support for 24bits 192khz sound ?
> > 
> > speaker-test -Dhdmi -c6  -r192000 does not work.
> > 
> > Thanks for working on this.
> > 
> > 
> > 
> > 
> > 
> > 2009/6/3 Wei Ni <wni@nvidia.com>
> > 
> > Hi, Marek and Alan
> > I'm a Nvidia engineer and develop this patch file.
> > 
> > I used mplayer to test. The channels are mapped good.
> > Running mplayer -ao alsa:device=hw=0.3 6ch.wav
> > 
> > I tried to use speaker-test to test, but I can't get the hdmi device
> > name.
> > I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
> > analog jack.
> > Which device name should I use?
> > 
> > I use aplay -l, it show
> > **** List of PLAYBACK Hardware Devices ****
> > card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
> >  Subdevices: 1/1
> >  Subdevice #0: subdevice #0
> > card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200
> Digital]
> >  Subdevices: 1/1
> >  Subdevice #0: subdevice #0
> > card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
> >  Subdevices: 1/1
> >  Subdevice #0: subdevice #0
> > 
> > Thanks
> > Wei.
> > 
> > 
> > -----Original Message-----
> > From: Marek Vasut [mailto:marek.vasut@gmail.com]
> > Sent: Wednesday, June 03, 2009 8:01 AM
> > To: Alan Alan
> > Cc: alsa-devel@alsa-project.org; Wei Ni
> > Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> > 
> > On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > > Thank's for the info.
> > > How can i contact him ?
> > 
> > He and alsa-devel mailing list are both in CC already, please preserve
> > them
> > there or fix your mailer.
> > >
> > >
> > > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> > >
> > > > I'm not the author of the patch, CCing alsa-devel and the original
> > > > author.
> > > >
> > > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > > Hi Marek !
> > > > >
> > > > > I just read your emailing with nvidia devs about alsa supporting
> 8
> > > >
> > > > channel
> > > >
> > > > > lpcm over hdmi.
> > > > >
> > > > > I checked it out an i have to tell it is working awsome, except
> > for a
> > > > > few bugs:
> > > > >
> > > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6
> > gives
> > > > > the following result (first where it should sound)
> > > > > FL - FL
> > > > > C - RL
> > > > > FR - FR
> > > > > RR - LFE
> > > > > RL - C
> > > > >
> > > > > The other thing i realized that playback is limited to 16 bits
> 48
> > khz.
> > > > > I think the minimum which is supported by nvidia hardware is
> > 24bits 96
> > > > > (or 192) khz.
> > > > > Why not ?
> > > > >
> > > > > Could you contact the devs as i don't know how to reach them.
> > > > >
> > > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa
> with
> > the
> > > > > oatched nvidia.c file. Anyway the amplifier shows multichannel
> > LPCM and
> > > > > that's great news.
> > 
> > 
> > 
> > 
> >
> ------------------------------------------------------------------------
> > -----------
> > This email message is for the sole use of the intended recipient(s)
> and
> > may contain
> > confidential information.  Any unauthorized review, use, disclosure or
> > distribution
> > is prohibited.  If you are not the intended recipient, please contact
> > the sender by
> > reply email and destroy all copies of the original message.
> >
> ------------------------------------------------------------------------
> > -----------
> > 
> >  
> > 
> > _______________________________________________
> > Alsa-devel mailing list
> > Alsa-devel@alsa-project.org
> > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> > 
> 

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wu Fengguang]

On Thu, Jun 04, 2009 at 08:29:25AM +0200, Takashi Iwai wrote:
> At Thu, 4 Jun 2009 14:05:52 +0800,
> Wei Ni wrote:
> > 
> > Yes, my test wav file is from Windows.
> > 
> > I had used different player to test with my test wav file.
> > The mplayer has no mapping problem, but other players map channels
> > wrong.
> > As you said, it seems that is caused by the difference of the ALSA and
> > Windows.
> 
> I guess mplayer corrects the channel mapping automatically.
> 
> > Unfortunately, our nvidia codec doesn't support channel-slot assignment
> > yet.
> 
> OK.
> 
> > How could we fix this problems, do you have any suggestions?
> 
> It's a bit difficult issue right now.  Usually we could "fix" the
> channel mapping in alsa-lib configuration.  But, HD-audio has currently
> a single alsa-lib configuration, and others may use the right channel
> mapping.
> 
> I don't remember how is the case of Intel HDMI...  Can anyone check?

The codecs I tested - G35/G45/GM45 - does not support the channel-slot
assignment. I guess future hardwares may support that.

Thanks,
Fengguang

> If it also suffers from the same problem, the channel conversion rule
> can be put in hdmi configuration statically as a temporary workaround.
> 
> We need a proper API to inform / change the channel mapping, anyway.
> Then it'll be better adjustable...
>
> > -----Original Message-----
> > From: Takashi Iwai [mailto:tiwai@suse.de] 
> > Sent: Wednesday, June 03, 2009 7:02 PM
> > To: Wei Ni
> > Cc: Alan Alan; Peer Chen; Marek Vasut; alsa-devel@alsa-project.org
> > Subject: Re: [alsa-devel] ALSA patch Nvidia 8 channel hdmi audio
> > 
> > At Wed, 3 Jun 2009 18:25:09 +0800,
> > Wei Ni wrote:
> > > 
> > > Hi, Alan
> > > 
> > > I reproduced this problems with speaker-test. The channels are wrong
> > > mapped.
> > > 
> > > It's strange that if use mplayer to run, it looks ok.
> > > 
> > > I'm looking this problems, and try to find the root cause.
> > 
> > Actually, from historical reasons, the standard ALSA channel mapping
> > for 5.1 is
> >     FL/FR/RL/RR/C/LFE
> > while Windows sets FL/FR/C/LFE/RL/RR.  If you play a WAV file assuming
> > the latter channel assignment, it will be played like that.
> > 
> > Does the nvidia codec support channel-slot assignment (verb 0x734)?
> > If yes, we can assign the channels freely to fix this...
> > 
> > 
> > thanks,
> > 
> > Takashi
> > 
> > 
> > > 
> > >  
> > > 
> > > Thanks
> > > 
> > >  
> > > 
> > > From: Alan Alan [mailto:alanwww1@gmail.com] 
> > > Sent: Wednesday, June 03, 2009 4:00 PM
> > > To: Wei Ni
> > > Cc: Marek Vasut; alsa-devel@alsa-project.org; Peer Chen
> > > Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> > > 
> > >  
> > > 
> > > Hello Wei !
> > > 
> > > Thanks for replying.
> > > I tested it with speaker-test and xbmc babylon. Both had problems with
> > > channel mapping.
> > > I used this command:
> > > 
> > > speaker-test -Dhdmi -c6
> > > or
> > > speaker-test -Dhdmi -c8
> > > 
> > > alternatively you could try 
> > > 
> > > speaker-test -Dhw:0,3 -c6
> > > 
> > > 
> > > I had to rund Xorg first and have the command from an xterm, becasue
> > > from a normal terminal it would not work.
> > > I think that is normal as a limitation of hdmi.
> > > 
> > > What about the support for 24bits 192khz sound ?
> > > 
> > > speaker-test -Dhdmi -c6  -r192000 does not work.
> > > 
> > > Thanks for working on this.
> > > 
> > > 
> > > 
> > > 
> > > 
> > > 2009/6/3 Wei Ni <wni@nvidia.com>
> > > 
> > > Hi, Marek and Alan
> > > I'm a Nvidia engineer and develop this patch file.
> > > 
> > > I used mplayer to test. The channels are mapped good.
> > > Running mplayer -ao alsa:device=hw=0.3 6ch.wav
> > > 
> > > I tried to use speaker-test to test, but I can't get the hdmi device
> > > name.
> > > I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
> > > analog jack.
> > > Which device name should I use?
> > > 
> > > I use aplay -l, it show
> > > **** List of PLAYBACK Hardware Devices ****
> > > card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
> > >  Subdevices: 1/1
> > >  Subdevice #0: subdevice #0
> > > card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200
> > Digital]
> > >  Subdevices: 1/1
> > >  Subdevice #0: subdevice #0
> > > card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
> > >  Subdevices: 1/1
> > >  Subdevice #0: subdevice #0
> > > 
> > > Thanks
> > > Wei.
> > > 
> > > 
> > > -----Original Message-----
> > > From: Marek Vasut [mailto:marek.vasut@gmail.com]
> > > Sent: Wednesday, June 03, 2009 8:01 AM
> > > To: Alan Alan
> > > Cc: alsa-devel@alsa-project.org; Wei Ni
> > > Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
> > > 
> > > On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > > > Thank's for the info.
> > > > How can i contact him ?
> > > 
> > > He and alsa-devel mailing list are both in CC already, please preserve
> > > them
> > > there or fix your mailer.
> > > >
> > > >
> > > > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> > > >
> > > > > I'm not the author of the patch, CCing alsa-devel and the original
> > > > > author.
> > > > >
> > > > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > > > Hi Marek !
> > > > > >
> > > > > > I just read your emailing with nvidia devs about alsa supporting
> > 8
> > > > >
> > > > > channel
> > > > >
> > > > > > lpcm over hdmi.
> > > > > >
> > > > > > I checked it out an i have to tell it is working awsome, except
> > > for a
> > > > > > few bugs:
> > > > > >
> > > > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6
> > > gives
> > > > > > the following result (first where it should sound)
> > > > > > FL - FL
> > > > > > C - RL
> > > > > > FR - FR
> > > > > > RR - LFE
> > > > > > RL - C
> > > > > >
> > > > > > The other thing i realized that playback is limited to 16 bits
> > 48
> > > khz.
> > > > > > I think the minimum which is supported by nvidia hardware is
> > > 24bits 96
> > > > > > (or 192) khz.
> > > > > > Why not ?
> > > > > >
> > > > > > Could you contact the devs as i don't know how to reach them.
> > > > > >
> > > > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa
> > with
> > > the
> > > > > > oatched nvidia.c file. Anyway the amplifier shows multichannel
> > > LPCM and
> > > > > > that's great news.
> > > 
> > > 
> > > 
> > > 
> > >
> > ------------------------------------------------------------------------
> > > -----------
> > > This email message is for the sole use of the intended recipient(s)
> > and
> > > may contain
> > > confidential information.  Any unauthorized review, use, disclosure or
> > > distribution
> > > is prohibited.  If you are not the intended recipient, please contact
> > > the sender by
> > > reply email and destroy all copies of the original message.
> > >
> > ------------------------------------------------------------------------
> > > -----------
> > > 
> > >  
> > > 
> > > _______________________________________________
> > > Alsa-devel mailing list
> > > Alsa-devel@alsa-project.org
> > > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> > > 
> > 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: ALSA patch Nvidia 8 channel hdmi audio

[Takashi Iwai]

At Fri, 5 Jun 2009 12:28:29 +0800,
Wu Fengguang wrote:
> 
> On Thu, Jun 04, 2009 at 08:29:25AM +0200, Takashi Iwai wrote:
> > At Thu, 4 Jun 2009 14:05:52 +0800,
> > Wei Ni wrote:
> > > 
> > > Yes, my test wav file is from Windows.
> > > 
> > > I had used different player to test with my test wav file.
> > > The mplayer has no mapping problem, but other players map channels
> > > wrong.
> > > As you said, it seems that is caused by the difference of the ALSA and
> > > Windows.
> > 
> > I guess mplayer corrects the channel mapping automatically.
> > 
> > > Unfortunately, our nvidia codec doesn't support channel-slot assignment
> > > yet.
> > 
> > OK.
> > 
> > > How could we fix this problems, do you have any suggestions?
> > 
> > It's a bit difficult issue right now.  Usually we could "fix" the
> > channel mapping in alsa-lib configuration.  But, HD-audio has currently
> > a single alsa-lib configuration, and others may use the right channel
> > mapping.
> > 
> > I don't remember how is the case of Intel HDMI...  Can anyone check?
> 
> The codecs I tested - G35/G45/GM45 - does not support the channel-slot
> assignment. I guess future hardwares may support that.

OK, and the channel assignment is also FL/FR/C/LFE/RL/RR, right?


thanks,

Takashi

Re: ALSA patch Nvidia 8 channel hdmi audio

[Wu Fengguang]

On Fri, Jun 05, 2009 at 01:40:58PM +0800, Takashi Iwai wrote:
> At Fri, 5 Jun 2009 12:28:29 +0800,
> Wu Fengguang wrote:
> > 
> > On Thu, Jun 04, 2009 at 08:29:25AM +0200, Takashi Iwai wrote:
> > > At Thu, 4 Jun 2009 14:05:52 +0800,
> > > Wei Ni wrote:
> > > > 
> > > > Yes, my test wav file is from Windows.
> > > > 
> > > > I had used different player to test with my test wav file.
> > > > The mplayer has no mapping problem, but other players map channels
> > > > wrong.
> > > > As you said, it seems that is caused by the difference of the ALSA and
> > > > Windows.
> > > 
> > > I guess mplayer corrects the channel mapping automatically.
> > > 
> > > > Unfortunately, our nvidia codec doesn't support channel-slot assignment
> > > > yet.
> > > 
> > > OK.
> > > 
> > > > How could we fix this problems, do you have any suggestions?
> > > 
> > > It's a bit difficult issue right now.  Usually we could "fix" the
> > > channel mapping in alsa-lib configuration.  But, HD-audio has currently
> > > a single alsa-lib configuration, and others may use the right channel
> > > mapping.
> > > 
> > > I don't remember how is the case of Intel HDMI...  Can anyone check?
> > 
> > The codecs I tested - G35/G45/GM45 - does not support the channel-slot
> > assignment. I guess future hardwares may support that.
> 
> OK, and the channel assignment is also FL/FR/C/LFE/RL/RR, right?

It's FL/FR/LFE/FC/RL/RR for 5.1, but it can be more complicated. HDMI supports
a number of speaker configurations - which will be reported via ELD by HDMI
sinks.  And we have a big mapping table name channel_allocations in
patch_intelhdmi.c.

The 6 channel configurations listed in the table are:

                                 /* 5.1 */
{ .ca_index = 0x0b,  .speakers = {   0,    0,  RR,  RL,  FC,  LFE,  FR,  FL } },
{ .ca_index = 0x0c,  .speakers = {   0,   RC,  RR,  RL,   0,    0,  FR,  FL } },
{ .ca_index = 0x0d,  .speakers = {   0,   RC,  RR,  RL,   0,  LFE,  FR,  FL } },
{ .ca_index = 0x0e,  .speakers = {   0,   RC,  RR,  RL,  FC,    0,  FR,  FL } },


Thanks,
Fengguang

Re: ALSA patch Nvidia 8 channel hdmi audio

[Alan Alan]

Hello Wei !

If you used mplayer with this command,

"mplayer -ao alsa:device=hw=0.3 6ch.wav"

than i think it might be that the test fiile is faulty. Why not try this
test file with a dts stream saying the channles after each other:

http://rapidshare.com/files/240315685/xbmc_dts_test.mkv

I also tried xbmc media center which "treats" the hdmi device as a
multichannel analog device and there i found the same mapping problem.

Thanks


2009/6/3 Wei Ni <wni@nvidia.com>

>  Hi, Alan
>
> I reproduced this problems with speaker-test. The channels are wrong
> mapped.
>
> It’s strange that if use mplayer to run, it looks ok.
>
> I’m looking this problems, and try to find the root cause.
>
>
>
> Thanks
>
>
>
> *From:* Alan Alan [mailto:alanwww1@gmail.com]
> *Sent:* Wednesday, June 03, 2009 4:00 PM
> *To:* Wei Ni
> *Cc:* Marek Vasut; alsa-devel@alsa-project.org; Peer Chen
>
> *Subject:* Re: ALSA patch Nvidia 8 channel hdmi audio
>
>
>
> Hello Wei !
>
> Thanks for replying.
> I tested it with speaker-test and xbmc babylon. Both had problems with
> channel mapping.
> I used this command:
>
> speaker-test -Dhdmi -c6
> or
> speaker-test -Dhdmi -c8
>
> alternatively you could try
>
> speaker-test -Dhw:0,3 -c6
>
>
> I had to rund Xorg first and have the command from an xterm, becasue from a
> normal terminal it would not work.
> I think that is normal as a limitation of hdmi.
>
> What about the support for 24bits 192khz sound ?
>
> speaker-test -Dhdmi -c6  -r192000 does not work.
>
> Thanks for working on this.
>
>
>
>  2009/6/3 Wei Ni <wni@nvidia.com>
>
> Hi, Marek and Alan
> I'm a Nvidia engineer and develop this patch file.
>
> I used mplayer to test. The channels are mapped good.
> Running mplayer -ao alsa:device=hw=0.3 6ch.wav
>
> I tried to use speaker-test to test, but I can't get the hdmi device name.
> I used speaker-test -Dsurround51:NVidia -c6, the sound output from the
> analog jack.
> Which device name should I use?
>
> I use aplay -l, it show
> **** List of PLAYBACK Hardware Devices ****
> card 0: NVidia [HDA NVidia], device 0: ALC1200 Analog [ALC1200 Analog]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 1: ALC1200 Digital [ALC1200 Digital]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
> card 0: NVidia [HDA NVidia], device 3: NVIDIA HDMI [NVIDIA HDMI]
>  Subdevices: 1/1
>  Subdevice #0: subdevice #0
>
> Thanks
> Wei.
>
>
> -----Original Message-----
> From: Marek Vasut [mailto:marek.vasut@gmail.com]
> Sent: Wednesday, June 03, 2009 8:01 AM
> To: Alan Alan
> Cc: alsa-devel@alsa-project.org; Wei Ni
> Subject: Re: ALSA patch Nvidia 8 channel hdmi audio
>
> On Wednesday 03 of June 2009 00:30:04 Alan Alan wrote:
> > Thank's for the info.
> > How can i contact him ?
>
> He and alsa-devel mailing list are both in CC already, please preserve them
> there or fix your mailer.
> >
> >
> > 2009/6/3 Marek Vasut <marek.vasut@gmail.com>
> >
> > > I'm not the author of the patch, CCing alsa-devel and the original
> > > author.
> > >
> > > On Tuesday 02 of June 2009 23:58:26 Alan Alan wrote:
> > > > Hi Marek !
> > > >
> > > > I just read your emailing with nvidia devs about alsa supporting 8
> > >
> > > channel
> > >
> > > > lpcm over hdmi.
> > > >
> > > > I checked it out an i have to tell it is working awsome, except for a
> > > > few bugs:
> > > >
> > > > The channels are wrong mapped. Running a speakertest -Dhdmi -c6 gives
> > > > the following result (first where it should sound)
> > > > FL - FL
> > > > C - RL
> > > > FR - FR
> > > > RR - LFE
> > > > RL - C
> > > >
> > > > The other thing i realized that playback is limited to 16 bits 48
> khz.
> > > > I think the minimum which is supported by nvidia hardware is 24bits
> 96
> > > > (or 192) khz.
> > > > Why not ?
> > > >
> > > > Could you contact the devs as i don't know how to reach them.
> > > >
> > > > ps.: Mybe my build methot was not 100% ok ? I used 1.020 Alsa with
> the
> > > > oatched nvidia.c file. Anyway the amplifier shows multichannel LPCM
> and
> > > > that's great news.
>
>
>
> -----------------------------------------------------------------------------------
> This email message is for the sole use of the intended recipient(s) and may
> contain
> confidential information.  Any unauthorized review, use, disclosure or
> distribution
> is prohibited.  If you are not the intended recipient, please contact the
> sender by
> reply email and destroy all copies of the original message.
>
> -----------------------------------------------------------------------------------
>
>
>

Re: ALSA patch Nvidia 8 channel hdmi audio

[Pau Rodriguez-Estivill]

[-- Attachment #1: Type: text/plain, Size: 727 bytes --]

I attach a patch to support 24bits and 32kHz, 44.1kHz, 48kHz, 96kHz,
192kHz, all rates that appears in the screenshot submitted by Alan
Alan.

I created also an asound.conf to correct the speakers positions.
Since current driver only support 2, 4, 6 and 8 channels I specified
it in the asound.conf but I'm not sure if this slave.channels should
be fixed.
The main problem is that by using asound.conf I didn't found any way
to route to surround51 (or surround50 or surround40, ...) depending of
the number of channels offered by the sink player.
And when I play directly to hdmi, the AMP reports automatically the
correct number of speakers involved.

Of course it would be nice to have this routes hardcoded in the driver...

[-- Attachment #2: nvhdmi-192khz.patch --]
[-- Type: text/x-diff, Size: 912 bytes --]

--- alsa-kernel/pci/hda/patch_nvhdmi.c.orig	2009-06-27 13:38:01.493407279 +0200
+++ alsa-kernel/pci/hda/patch_nvhdmi.c	2009-07-01 02:34:00.969260248 +0200
@@ -258,9 +258,9 @@
 	.channels_min = 2,
 	.channels_max = 8,
 	.nid = Nv_Master_Convert_nid,
-	.rates = SNDRV_PCM_RATE_48000,
-	.maxbps = 16,
-	.formats = SNDRV_PCM_FMTBIT_S16_LE,
+	.rates = SNDRV_PCM_RATE_32000 | SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 | SNDRV_PCM_RATE_96000 | SNDRV_PCM_RATE_192000,
+	.maxbps = 24,
+	.formats = SNDRV_PCM_FMTBIT_S16_LE | SNDRV_PCM_FMTBIT_S32_LE,
 	.ops = {
 		.open = nvhdmi_dig_playback_pcm_open,
 		.close = nvhdmi_dig_playback_pcm_close_8ch,
@@ -273,7 +273,7 @@
 	.channels_min = 2,
 	.channels_max = 2,
 	.nid = Nv_Master_Convert_nid,
-	.rates = SNDRV_PCM_RATE_48000,
+	.rates = SNDRV_PCM_RATE_32000 | SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000,
 	.maxbps = 16,
 	.formats = SNDRV_PCM_FMTBIT_S16_LE,
 	.ops = {

[-- Attachment #3: asound.conf --]
[-- Type: application/octet-stream, Size: 1307 bytes --]

# Linux has channel order
# FL, FR, RL, RR, C, LFE, SL, SR
# HDMI has this one
# FL, FR, LFE, FC, RL, RR, RLC, RRC

pcm.!front {
  type route
  slave.pcm hdmi
  slave.channels 2
  ttable {
    0.0= 1
    1.1= 1
  }
}
pcm.!surround21 {
  type route
  slave.pcm hdmi
  slave.channels 4
  ttable {
    0.0= 1
    1.1= 1
    2.2= 1
  }
}
pcm.!surround30 {
  type route
  slave.pcm hdmi
  slave.channels 4
  ttable {
    0.0= 1
    1.1= 1
    2.3= 1
  }
}
pcm.!surround31 {
  type route
  slave.pcm hdmi
  slave.channels 4
  ttable {
    0.0= 1
    1.1= 1
    2.3= 1
    3.2= 1
  }
}
pcm.!surround40 {
  type route
  slave.pcm hdmi
  slave.channels 6
  ttable {
    0.0= 1
    1.1= 1
    2.4= 1
    3.5= 1
  }
}
pcm.!surround41 {
  type route
  slave.pcm hdmi
  slave.channels 6
  ttable {
    0.0= 1
    1.1= 1
    2.4= 1
    3.5= 1
    4.2= 1
  }
}
pcm.!surround50 {
  type route
  slave.pcm hdmi
  slave.channels 6
  ttable {
    0.0= 1
    1.1= 1
    2.4= 1
    3.5= 1
    4.3= 1
  }
}
pcm.!surround51 {
  type route
  slave.pcm hdmi
  slave.channels 6
  ttable {
    0.0= 1
    1.1= 1
    2.4= 1
    3.5= 1
    4.3= 1
    5.2= 1
  }
}
pcm.!surround71 {
  type route
  slave.pcm hdmi
  slave.channels 8
  ttable {
    0.0= 1
    1.1= 1
    2.4= 1
    3.5= 1
    4.3= 1
    5.2= 1
    6.6= 1
    7.7= 1
  }
}

[-- Attachment #4: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel