On Tue, Mar 05, 2013 at 03:44:04PM -0800, chris hermansen wrote: > On Tue, Mar 5, 2013 at 3:03 PM, chris hermansen <clhermansen@xxxxxxxxx> wrote: > > On Mon, Mar 4, 2013 at 6:44 PM, chris hermansen <clhermansen@xxxxxxxxx> wrote: > >> On Mon, Mar 4, 2013 at 2:44 PM, Torstein Hegge <hegge@xxxxxxxxxxx> wrote: > >>> On Mon, Mar 04, 2013 at 11:30:03AM -0800, chris hermansen wrote: > >>> > I would be happy to test this on my Schiit Bifrost as well. I'm currently > >>> > using the TOSLINK connection but it would be easy peasy to hook up a laptop > >>> > via the USB. > >>> > >>> If you could test the patch i just posted to alsa-devel [1], that would > >>> be great. > >>> > >>> [1] http://article.gmane.org/gmane.linux.alsa.devel/106002 > >> > >> I will do my best to test it tonight (GMT-8). > > > > Well, last night came and went and time was short. > > > > Anyway, this morning I booted up my old Toshiba testbed currently > > running Lubuntu 13.04 and followed these instructions to build a new > > kernel: > > > > https://wiki.ubuntu.com/Kernel/BuildYourOwnKernel > > > > (by the way if anyone is reading this and thinking "I want to try > > too", establish the build environment BEFORE trying to get the > > kernel source). > > > > 13.04 is running 3.8.0 kernel at this point in time. > > > > I inserted Torstein's code into line 713 in pcm.c and built the > > kernel. > > > > I tried "sudo aplay -vD plughw:CARD=Interface,DEV=0" with my > > 44.1/16 wav file followed by my 96/24 wav file and... > > > > IT WORKS!!!! Great, thanks for testing. > > I know this isn't quite "testing the patch" itself, but I wasn't > > comfortable with the different versions. Should I do more? You might find it easier to keep track of the changes by saving the full patch email to a file and applying it with $ git am <filename> Then you'll get the commit message in your log as well. > Answering my own question, I decided to take a closer look at your > patch noted above (106002), which appears to patch sound/usb/clock.c, > which seems completely different than what you presented earlier, > which was the change to sound/usb/pcm.c, which is what I tested... > > Feeling a bit lost here... did I miss a part of the conversation? You might have missed the discussion of the first patch, http://thread.gmane.org/gmane.linux.alsa.devel/105941/focus=105971 The important bit is the same between v1 and v2, usb_set_interface() after setting sample frequency. v2 just tries to be a bit smarter, by just doing the reset on sample rate changes. To add to the confusion, I sent v3 of the patch as well. This is a minor change over v2, but adds support for more cards based on C-Media chips, like Asus Xonar Essence One. Torstein ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev _______________________________________________ Alsa-user mailing list Alsa-user@xxxxxxxxxxxxxxxxxxxxx https://lists.sourceforge.net/lists/listinfo/alsa-user
