Skip to contentSkip to conversation

Make ASUS Xonar U3 S/PDIF channel accessible

PATCH In review Denis Washington 2 participants 5 messages Dec 11, 2012 – Dec 13, 2012

lore
Share & tools
lore
#0 lore raw

cover Make ASUS Xonar U3 S/PDIF channel accessible

Hello,

This is my first ALSA patch ever (in fact, also my first kernel patch and my first e-mail to this mailing list); as such, I'm sorry when I did something wrong and/or didn't follow the procedure properly. I hope you can forgive me.

I recently bought an ASUS Xonar U3 USB audio card. It general it works well with Linux, but unfortunately, it's PCM output channel is unusable for headphones because it allows virtually no volume control (maximum loudness is already reach at a volume level of 2 or 3). The card does also have a digital (S/PDIF) output channel, which however isn't switched on by the Linux USB audio driver:

http://www.mail-archive.com/[email protected]/msg28970.html

When I boot into Windows (where the ASUS driver seemingly enables the channel) and then reboot into Linux, I can use this channel and control volume as expected. It's just this turning on of S/PDIF which is missing, as suggested by the alsa-user thread linked to above.

The patchset that follows enables S/PDIF on the Xonar U3 by using the (seemingly compatible) mixer quirks code as for the U1, just as done and tested by the initiator of the alsa-user thread, together with a very minor cleanup (renaming internal xonar_u1_* identifiers to just xonar_u_*). The change seems to work well (tested with the 3.7.0 Ubuntu kernel as found in today' daily version).

Regard,
Denis Washington

#1 lore raw

At Tue, 11 Dec 2012 11:37:26 +0100,

Quoted 22 lines · Denis Washington

Hello,

This is my first ALSA patch ever (in fact, also my first kernel patch and my first e-mail to this mailing list); as such, I'm sorry when I did something wrong and/or didn't follow the procedure properly. I hope you can forgive me.

I recently bought an ASUS Xonar U3 USB audio card. It general it works well with Linux, but unfortunately, it's PCM output channel is unusable for headphones because it allows virtually no volume control (maximum loudness is already reach at a volume level of 2 or 3). The card does also have a digital (S/PDIF) output channel, which however isn't switched on by the Linux USB audio driver:

http://www.mail-archive.com/[email protected]/msg28970.html

When I boot into Windows (where the ASUS driver seemingly enables the channel) and then reboot into Linux, I can use this channel and control volume as expected. It's just this turning on of S/PDIF which is missing, as suggested by the alsa-user thread linked to above.

Quoted 6 lines

The patchset that follows enables S/PDIF on the Xonar U3 by using the (seemingly compatible) mixer quirks code as for the U1, just as done and tested by the initiator of the alsa-user thread, together with a very minor cleanup (renaming internal xonar_u1_* identifiers to just xonar_u_*). The change seems to work well (tested with the 3.7.0 Ubuntu kernel as found in today' daily version).

Thanks for patches.

I think the rename of xonar_u1_* isn't needed. What we need is rather giving more comments there, that it's for both U1 and U3.

In anyway, could you resubmit with your sign-off?
Put a line like

in the tail of the change log text.

See Documentation/SubmittingPatches the section 12 "Sign your work" for details.

Takashi

#2 lore raw

Am 11.12.2012 12:41, schrieb Takashi Iwai:

Quoted 31 lines

At Tue, 11 Dec 2012 11:37:26 +0100,

Quoted 22 lines · Denis Washington

Hello,

This is my first ALSA patch ever (in fact, also my first kernel patch and my first e-mail to this mailing list); as such, I'm sorry when I did something wrong and/or didn't follow the procedure properly. I hope you can forgive me.

I recently bought an ASUS Xonar U3 USB audio card. It general it works well with Linux, but unfortunately, it's PCM output channel is unusable for headphones because it allows virtually no volume control (maximum loudness is already reach at a volume level of 2 or 3). The card does also have a digital (S/PDIF) output channel, which however isn't switched on by the Linux USB audio driver:

http://www.mail-archive.com/[email protected]/msg28970.html

When I boot into Windows (where the ASUS driver seemingly enables the channel) and then reboot into Linux, I can use this channel and control volume as expected. It's just this turning on of S/PDIF which is missing, as suggested by the alsa-user thread linked to above.

Quoted 6 lines

The patchset that follows enables S/PDIF on the Xonar U3 by using the (seemingly compatible) mixer quirks code as for the U1, just as done and tested by the initiator of the alsa-user thread, together with a very minor cleanup (renaming internal xonar_u1_* identifiers to just xonar_u_*). The change seems to work well (tested with the 3.7.0 Ubuntu kernel as found in today' daily version).

Quoted 4 lines

Thanks for patches.

I think the rename of xonar_u1_* isn't needed. What we need is rather giving more comments there, that it's for both U1 and U3.

Yes, that would be useful. I don't know if I'm qualified to write a useful comment for this, though.

Quoted 4 lines

In anyway, could you resubmit with your sign-off?
Put a line like

in the tail of the change log text.

OK, no problem.

Regards,
Denis

#3 lore raw

Am 11.12.2012 19:24, schrieb Denis Washington:

Quoted 6 lines
Quoted 4 lines

In anyway, could you resubmit with your sign-off?
Put a line like

in the tail of the change log text.

OK, no problem.

Done. Anything else important missing until the patch can be considered?

Regards,
Denis Washington

#4 lore raw

At Thu, 13 Dec 2012 07:40:18 +0100,

Quoted 10 lines · Denis Washington

Am 11.12.2012 19:24, schrieb Denis Washington:

Quoted 6 lines
Quoted 4 lines

In anyway, could you resubmit with your sign-off?
Put a line like

in the tail of the change log text.

OK, no problem.

Done. Anything else important missing until the patch can be considered?

No, it's fine. I just forgot to post I merged the first patch, sorry. The second one was skipped as mentioned.

Thanks!

Takashi