]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
usb: gadget: f_midi: enable use of the index parameter
authorAndrzej Pietrasiewicz <andrzej.p@samsung.com>
Thu, 16 Oct 2014 11:33:25 +0000 (13:33 +0200)
committerFelipe Balbi <balbi@ti.com>
Wed, 5 Nov 2014 19:36:02 +0000 (13:36 -0600)
The soundcard index to use for the ALSA device creation is passed as a
parameter to f_midi_bind_config(), but is assigned to midi->index only
after the call to f_midi_register_card(midi). So no matter what is passed
to f_midi_bind_config(), the actual index for snd_card_new() is always 0.
This probably works ok if at the moment of f_midi's bind there are no
other snd_cards, but if there are, it is not possible to bind f_midi.

This patch moves the assignment to a place before the call to
f_midi_register_card(midi).

Signed-off-by: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/function/f_midi.c

index 807b31c0edc31b547b079637cac1bf26a06fe6c8..bf329578c95713c589bd1e763b66c6976e0c2c50 100644 (file)
@@ -954,6 +954,7 @@ int __init f_midi_bind_config(struct usb_configuration *c,
        /* set up ALSA midi devices */
        midi->in_ports = in_ports;
        midi->out_ports = out_ports;
+       midi->index = index;
        status = f_midi_register_card(midi);
        if (status < 0)
                goto setup_fail;
@@ -966,7 +967,6 @@ int __init f_midi_bind_config(struct usb_configuration *c,
        midi->func.disable     = f_midi_disable;
 
        midi->id = kstrdup(id, GFP_KERNEL);
-       midi->index = index;
        midi->buflen = buflen;
        midi->qlen = qlen;