[alsa-devel] [PATCH 0/2] 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/alsa-user@lists.sourceforge.net/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
At Tue, 11 Dec 2012 11:37:26 +0100, Denis Washington wrote:
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/alsa-user@lists.sourceforge.net/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).
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 Signed-of-by: Your Name foo@bar.com in the tail of the change log text.
See Documentation/SubmittingPatches the section 12 "Sign your work" for details.
Takashi
Am 11.12.2012 12:41, schrieb Takashi Iwai:
At Tue, 11 Dec 2012 11:37:26 +0100, Denis Washington wrote:
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/alsa-user@lists.sourceforge.net/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).
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.
In anyway, could you resubmit with your sign-off? Put a line like Signed-of-by: Your Name foo@bar.com in the tail of the change log text.
OK, no problem.
Regards, Denis
Am 11.12.2012 19:24, schrieb Denis Washington:
In anyway, could you resubmit with your sign-off? Put a line like Signed-of-by: Your Name foo@bar.com 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
At Thu, 13 Dec 2012 07:40:18 +0100, Denis Washington wrote:
Am 11.12.2012 19:24, schrieb Denis Washington:
In anyway, could you resubmit with your sign-off? Put a line like Signed-of-by: Your Name foo@bar.com 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
participants (2)
-
Denis Washington -
Takashi Iwai