Hi again,
I have bought an Mbox 2 and tested playback at both 44.1 and 48kHz using Jackd, and 44.1kHz using PulseAudio. It appeared to work without problems (it played without any dropouts and at the correct speed / pitch). I haven't tested recording or done more rigorous testing on the Mbox yet as it only arrived today (I did test recording with the Mini though).
Have you got a proper list of things to test before merging the patch or should I try to make it up myself?
Cheers, Vik
On 6 May 2017 at 09:03, Viktor Radnai viktor.radnai@gmail.com wrote:
Hi Takashi,
Thanks, I have also reached out to Damien and he just got back to me. Unfortunately he no longer has the device and is quite busy with other things.
I am going to buy an Mbox 2 and test this myself. You can expect results within two weeks, hopefully sooner.
In the meantime if anyone on the list has this device I would really appreciate if you could test it.
Cheers, Vik
On 3 May 2017 at 11:16, Takashi Iwai tiwai@suse.de wrote:
On Fri, 28 Apr 2017 22:36:09 +0200, Viktor Radnai wrote:
Hi there,
I have recently bought a Digidesign Mbox 2 Mini and found the lack of support for 44100Hz sample rate in the driver limiting (the sample rate
was
hard-set to 48000Hz by the original author even though the device
supports
it, because he couldn't get the frame rate switching to work reliably).
I had a go at hacking the driver and came up with the following patch
which
appears to work on the MBox 2 Mini: https://pastebin.com/kzzTDj9T
I have built and tested this with Linux 4.8.15 without any realtime patches. I am planning to pick up a "full size" Mbox 2 as well, in the meantime if anyone on the list owns one I would appreciate if you could test this and let me know the result. There is a possibility that it
will
not work without the "magic" init sequence that I have removed because
it
does not look like it's needed on the Mini. Although I would be a bit surprised because the two devices are apparently identical to the point that the Mini allows the use of it's physically nonexistent features.
Once someone could confirm that this works, I would like to get it
merged.
This is my first attempt at submitting a patch for any Linux device
driver
so please let me know if I need to do anything else :)
At least it'd be better to the original author in the loop. Damien, could you take a look if you still have the device?
thanks,
Takashi
-- My other sig is hilarious