[alsa-devel] Patch for Digidesign Mbox 2 (set sample rate)

[Viktor Radnai]

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



-- 
My other sig is hilarious