Message ID | 20180420170327.31569-3-jorge.sanjuan@codethink.co.uk (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Fri, 20 Apr 2018 19:03:25 +0200, Jorge Sanjuan wrote: > > From: Michael Drake <michael.drake@codethink.co.uk> > > The channel mapping is defined by bChRelationship, not bChPurpose. > > Signed-off-by: Michael Drake <michael.drake@codethink.co.uk> The change looks OK, but your sign-off is missing. If you submit a patch, please give always your signed-off-by tag, no matter whether it's your original patch or not. Also, the Fixes tag would be helpful for such a correction. thanks, Takashi
On Fri, Apr 20, 2018 at 8:03 PM, Jorge Sanjuan <jorge.sanjuan@codethink.co.uk> wrote: > From: Michael Drake <michael.drake@codethink.co.uk> > > The channel mapping is defined by bChRelationship, not bChPurpose. > > Signed-off-by: Michael Drake <michael.drake@codethink.co.uk> > --- > sound/usb/stream.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/usb/stream.c b/sound/usb/stream.c > index 6a8f5843334e..956be9f7c72a 100644 > --- a/sound/usb/stream.c > +++ b/sound/usb/stream.c > @@ -349,7 +349,7 @@ snd_pcm_chmap_elem *convert_chmap_v3(struct uac3_cluster_header_descriptor > * TODO: this conversion is not complete, update it > * after adding UAC3 values to asound.h > */ > - switch (is->bChPurpose) { > + switch (is->bChRelationship) { Good catch! Somehow I overlooked this, so in my case of Generic Audio it is always mono. Reviewed-by: Ruslan Bilovol <ruslan.bilovol@gmail.com> > case UAC3_CH_MONO: > map = SNDRV_CHMAP_MONO; > break; > -- > 2.11.0 > > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@alsa-project.org > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
diff --git a/sound/usb/stream.c b/sound/usb/stream.c index 6a8f5843334e..956be9f7c72a 100644 --- a/sound/usb/stream.c +++ b/sound/usb/stream.c @@ -349,7 +349,7 @@ snd_pcm_chmap_elem *convert_chmap_v3(struct uac3_cluster_header_descriptor * TODO: this conversion is not complete, update it * after adding UAC3 values to asound.h */ - switch (is->bChPurpose) { + switch (is->bChRelationship) { case UAC3_CH_MONO: map = SNDRV_CHMAP_MONO; break;