Re: Status of CM6631 USB

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Torstein, list:

On Tue, Mar 5, 2013 at 11:18 PM, Torstein Hegge <hegge@xxxxxxxxxxx> wrote:
> 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.

My pleasure!

>
>> > 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.

Thanks for the pointer.

>
>> 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

Yes probably.  Perhaps it's the dev as opposed to user list?

>
> 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.

So I have now tried the v2 patch.

It seems to work mostly.  I can't create a problem with aplay using a
device of plughw:CARD=Interface,DEV=0 switching between 44.1/16 and
96/24 tracks.

Using gmusicbrowser with alsa and a device of
plughw:CARD=Interface,DEV=0 once in awhile I seem to move to a 96/24
that plays in the old bad noisy/static-y fashion, but I can't seem to
re-create the circumstances reliably.  If it happens, it seems to be
when I use the fast forward to skip to the next song.  It has not
happened more than once in a given gmusicbrowser session, of which I
have probably tried 10 now, and it has only happened three times.

I don't know what the problem might be.  It could be some weird thing
in gmusicbrowser, too.

>
> 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.

I haven't tried that one.

Please let me know if you would like me to try some more experimenting.

And thanks again for trying to fix this!

-- 
Chris Hermansen · clhermansen "at" gmail "dot" com

C'est ma façon de parler.

------------------------------------------------------------------------------
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



[Index of Archives]     [ALSA Devel]     [Linux Audio Users]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]

  Powered by Linux