On 11/07/2014 02:57 PM, Sebastian Reimers wrote: >> b) Both FS and HS audio devices should still function as before, with >> no regression regarding CPU load while streaming. > > > The patch destroys audio quality (pops, clicks and alsa underruns). > Tested with a USB 2.0 (Focusrite) device. Great. I guess appling patch #1 one does not cause any harm, it is just #2 that breaks things. Is it better or worse than without "usb: musb: cppi41: tweak hrtimer values" ? Can you measure how long it takes on average to complete / how often you enter the re-try loop? If I understand the Daniel's commit message then #1 retry should be after approx 23us max (the 3us from packet size + 20 slack worst case). And since he tweaked the restart timer from 50 to 20 I would assume it is used once. > Test case is a simple audio loop: > > $ arecord | aplay > > Jackd generates cppi41_irq errors. Can you paste one of those? Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
