All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anssi Hannula <anssi.hannula@iki.fi>
To: LANGLOIS Olivier PIS -EXT <olivier.pis.langlois@transport.alstom.com>
Cc: "Takashi Iwai" <tiwai@suse.de>,
	"Olivier Langlois" <olivier@trillion01.com>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	"Rafał Miłecki" <zajec5@gmail.com>,
	"Peter Frühberger" <fritsch@xbmc.org>
Subject: Re: [RFC/RFT PATCH] ALSA: hda - hdmi: Add ATI/AMD multi-channel audio support
Date: Thu, 26 Sep 2013 01:21:36 +0300	[thread overview]
Message-ID: <524361F0.80009@iki.fi> (raw)
In-Reply-To: <89F8E4011FD7234ABFE48744E300CBC206B5C9FD@041-DB3MPN1-091.041d.mgd.msft.net>

25.09.2013 18:17, LANGLOIS Olivier PIS -EXT kirjoitti:
> Anssi,
> 
>>
>> Great to hear that, thanks. So Revision Id 3+ works :)
>>
>> I forgot to ask for it explicitely, but I guess you checked that channels
>> seemed to mapped correctly (at least for each speaker you actually have)?
> 
> Absolutely. I haven't been explicit about it but I would have mention it if something was wrong in that regard. I have 8 speakers.
> 
> I have used speaker-test to test stereo, 5.1 and 7.1 mapping and the mapping is perfect.
> 
> One small detail that I have noticed is that there seem to be a small delay as I add more and more channels. With 8 channels, there might be up to 1 sec between the time speaker-test prints a channel name and the sound from previous chan stops and I start to hear noise from the new one.

This happens with other hw as well, I think speaker-test simply does not
account for PCM playback latency when displaying the channel names.

> I have concluded that this may come from my receiver that seem to reinitialize itself during each channel switch but I'm not totally sure.
>>
>> Also, could you paste the whole /proc/asound/card0/codec#0, for the
>> record?
> 
> I'll provide that info to you when I'm back home later. For now, from what I remember, as it is the first thing that I have checked after having reloaded the driver with the patch, there is nothing printed there that reflect the new multi channel capability of the codec. but maybe it is me that does not know what to look for.

That is expected, I'm just generally interested in the contents (pin
connections etc).

> Also, if you want me to test functionality only available with my hardware revision, just prepare me small test cases. I'll be happy to run them. No chance that I run on 'advanced' features on my own as my ALSA usage is very simple.

Nothing more special to test on your hw at this time, I think :)
I will get back to you if something comes up.

-- 
Anssi Hannula

  reply	other threads:[~2013-09-25 22:21 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-22 17:50 [RFC/RFT PATCH] ALSA: hda - hdmi: Add ATI/AMD multi-channel audio support Anssi Hannula
2013-09-22 21:14 ` Anssi Hannula
2013-09-25  4:22 ` Olivier Langlois
2013-09-25 14:10   ` Anssi Hannula
2013-09-25 15:17     ` LANGLOIS Olivier PIS -EXT
2013-09-25 22:21       ` Anssi Hannula [this message]
2013-09-26  5:54         ` Olivier Langlois
2013-09-26  9:59 ` Takashi Iwai
2013-09-27  9:27 ` Rafał Miłecki
2013-09-27 10:19 ` Rafał Miłecki
2013-09-27 13:10   ` Anssi Hannula

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=524361F0.80009@iki.fi \
    --to=anssi.hannula@iki.fi \
    --cc=alsa-devel@alsa-project.org \
    --cc=fritsch@xbmc.org \
    --cc=olivier.pis.langlois@transport.alstom.com \
    --cc=olivier@trillion01.com \
    --cc=tiwai@suse.de \
    --cc=zajec5@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.