[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-02 Thread Anssi Hannula
02.11.2013 03:01, Rafa? Mi?ecki kirjoitti: > 2013/10/29 Anssi Hannula : >> Fix the code to pick the PCM SAD with the highest number of channels, >> while merging the rate masks of PCM SADs with lower amount of channels >> into the additional stereo rate mask byte. > > Don't you think that we

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-02 Thread Rafał Miłecki
2013/11/2 Anssi Hannula : > SAD with channels=6,freqs=32..96kHz,bits=16..24 implies that those freqs > and bps are supported for all channel counts up to 6 (since it is "Max > Number of channels"). Therefore the specified rates are supported in > stereo mode as well and I believe should be

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-02 Thread Rafał Miłecki
2013/10/29 Anssi Hannula : > + if (sad->channels > max_channels) { > + value = MAX_CHANNELS(sad->channels) | > + DESCRIPTOR_BYTE_2(sad->byte2) > | > +

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-02 Thread Rafał Miłecki
2013/10/29 Anssi Hannula : > Fix the code to pick the PCM SAD with the highest number of channels, > while merging the rate masks of PCM SADs with lower amount of channels > into the additional stereo rate mask byte. Don't you think that we should use SUPPORTED_FREQUENCIES_STEREO for stereo

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-01 Thread Anssi Hannula
01.11.2013 01:38, Rafa? Mi?ecki kirjoitti: > 2013/10/29 Anssi Hannula : >> Because of this, only the 2-channel SAD may be used if it appears before >> the 8-channel SAD. Unless other SADs require otherwise, this may cause >> the ALSA HDA driver to allow stereo playback only. > > I can confirm

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-01 Thread Rafał Miłecki
2013/11/1 Rafa? Mi?ecki : > 2013/10/29 Anssi Hannula : >> Fix the code to pick the PCM SAD with the highest number of channels, >> while merging the rate masks of PCM SADs with lower amount of channels >> into the additional stereo rate mask byte. > > Does it mean that now instead of 2 real SADs:

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-11-01 Thread Rafał Miłecki
2013/10/29 Anssi Hannula : > Because of this, only the 2-channel SAD may be used if it appears before > the 8-channel SAD. Unless other SADs require otherwise, this may cause > the ALSA HDA driver to allow stereo playback only. I can confirm that the problem exists. My SADs (Onkyo TX-SR605):

[PATCH] drm/radeon/audio: fix missing multichannel PCM SAD in some cases

2013-10-29 Thread Anssi Hannula
The current code writing SADs to the audio registers seems to assume that there is at most a single SAD per audio format. However, that is not the case. Especially for PCM it is somewhat common for sinks to have two SADs, one for 8-channel and one for 2-channel audio, which may have different