Hi Damien, Thanks for the test report. I have now successfully recorded with both the Mbox 2 and Mbox 2 Mini, however I had one occurrence where the Mbox 2 would not initialize even when I repeatedly unplugged and plugged it back. I have added your "magic" boot sequence back and since then I haven't seen this problem. I have been using that version ever since, so I cannot tell if simply unloading and reloading the driver would have fixed it. In any case I will re-submit the patch with the magic boot sequence for inclusion. Cheers, Vik On 9 July 2017 at 03:14, Damien Zammit <damien@xxxxxxxxxxxx> wrote: > Hi all, > > I have tested Viktor's patch on "Mbox 2" and it works very well. > Looks like my 'magic' boot sequence was not needed. > Rate switching works between 44.1 and 48k. > Good job. > > Damien > > On 03/05/17 20:16, Takashi Iwai 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 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
