V4L/DVB (13062): radio-mr800: simplify error paths in usb probe callback
[safe/jmp/linux-2.6] / drivers / media / radio / radio-mr800.c
1 /*
2  * A driver for the AverMedia MR 800 USB FM radio. This device plugs
3  * into both the USB and an analog audio input, so this thing
4  * only deals with initialization and frequency setting, the
5  * audio data has to be handled by a sound driver.
6  *
7  * Copyright (c) 2008 Alexey Klimov <klimov.linux@gmail.com>
8  *
9  * This program is free software; you can redistribute it and/or modify
10  * it under the terms of the GNU General Public License as published by
11  * the Free Software Foundation; either version 2 of the License, or
12  * (at your option) any later version.
13  *
14  * This program is distributed in the hope that it will be useful,
15  * but WITHOUT ANY WARRANTY; without even the implied warranty of
16  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
17  * GNU General Public License for more details.
18  *
19  * You should have received a copy of the GNU General Public License
20  * along with this program; if not, write to the Free Software
21  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
22  */
23
24 /*
25  * Big thanks to authors and contributors of dsbr100.c and radio-si470x.c
26  *
27  * When work was looked pretty good, i discover this:
28  * http://av-usbradio.sourceforge.net/index.php
29  * http://sourceforge.net/projects/av-usbradio/
30  * Latest release of theirs project was in 2005.
31  * Probably, this driver could be improved trough using their
32  * achievements (specifications given).
33  * Also, Faidon Liambotis <paravoid@debian.org> wrote nice driver for this radio
34  * in 2007. He allowed to use his driver to improve current mr800 radio driver.
35  * http://kerneltrap.org/mailarchive/linux-usb-devel/2007/10/11/342492
36  *
37  * Version 0.01:        First working version.
38  *                      It's required to blacklist AverMedia USB Radio
39  *                      in usbhid/hid-quirks.c
40  * Version 0.10:        A lot of cleanups and fixes: unpluging the device,
41  *                      few mutex locks were added, codinstyle issues, etc.
42  *                      Added stereo support. Thanks to
43  *                      Douglas Schilling Landgraf <dougsland@gmail.com> and
44  *                      David Ellingsworth <david@identd.dyndns.org>
45  *                      for discussion, help and support.
46  * Version 0.11:        Converted to v4l2_device.
47  *
48  * Many things to do:
49  *      - Correct power management of device (suspend & resume)
50  *      - Add code for scanning and smooth tuning
51  *      - Add code for sensitivity value
52  *      - Correct mistakes
53  *      - In Japan another FREQ_MIN and FREQ_MAX
54  */
55
56 /* kernel includes */
57 #include <linux/kernel.h>
58 #include <linux/module.h>
59 #include <linux/init.h>
60 #include <linux/slab.h>
61 #include <linux/smp_lock.h>
62 #include <linux/input.h>
63 #include <linux/videodev2.h>
64 #include <media/v4l2-device.h>
65 #include <media/v4l2-ioctl.h>
66 #include <linux/usb.h>
67 #include <linux/version.h>      /* for KERNEL_VERSION MACRO */
68 #include <linux/mutex.h>
69
70 /* driver and module definitions */
71 #define DRIVER_AUTHOR "Alexey Klimov <klimov.linux@gmail.com>"
72 #define DRIVER_DESC "AverMedia MR 800 USB FM radio driver"
73 #define DRIVER_VERSION "0.11"
74 #define RADIO_VERSION KERNEL_VERSION(0, 1, 1)
75
76 MODULE_AUTHOR(DRIVER_AUTHOR);
77 MODULE_DESCRIPTION(DRIVER_DESC);
78 MODULE_LICENSE("GPL");
79
80 #define USB_AMRADIO_VENDOR 0x07ca
81 #define USB_AMRADIO_PRODUCT 0xb800
82
83 /* dev_warn macro with driver name */
84 #define MR800_DRIVER_NAME "radio-mr800"
85 #define amradio_dev_warn(dev, fmt, arg...)                              \
86                 dev_warn(dev, MR800_DRIVER_NAME " - " fmt, ##arg)
87
88 /* Probably USB_TIMEOUT should be modified in module parameter */
89 #define BUFFER_LENGTH 8
90 #define USB_TIMEOUT 500
91
92 /* Frequency limits in MHz -- these are European values.  For Japanese
93 devices, that would be 76 and 91.  */
94 #define FREQ_MIN  87.5
95 #define FREQ_MAX 108.0
96 #define FREQ_MUL 16000
97
98 /*
99  * Commands that device should understand
100  * List isnt full and will be updated with implementation of new functions
101  */
102 #define AMRADIO_SET_FREQ        0xa4
103 #define AMRADIO_SET_MUTE        0xab
104 #define AMRADIO_SET_MONO        0xae
105
106 /* Comfortable defines for amradio_set_mute */
107 #define AMRADIO_START           0x00
108 #define AMRADIO_STOP            0x01
109
110 /* Comfortable defines for amradio_set_stereo */
111 #define WANT_STEREO             0x00
112 #define WANT_MONO               0x01
113
114 /* module parameter */
115 static int radio_nr = -1;
116 module_param(radio_nr, int, 0);
117 MODULE_PARM_DESC(radio_nr, "Radio Nr");
118
119 static int usb_amradio_probe(struct usb_interface *intf,
120                              const struct usb_device_id *id);
121 static void usb_amradio_disconnect(struct usb_interface *intf);
122 static int usb_amradio_open(struct file *file);
123 static int usb_amradio_close(struct file *file);
124 static int usb_amradio_suspend(struct usb_interface *intf,
125                                 pm_message_t message);
126 static int usb_amradio_resume(struct usb_interface *intf);
127
128 /* Data for one (physical) device */
129 struct amradio_device {
130         /* reference to USB and video device */
131         struct usb_device *usbdev;
132         struct video_device videodev;
133         struct v4l2_device v4l2_dev;
134
135         unsigned char *buffer;
136         struct mutex lock;      /* buffer locking */
137         int curfreq;
138         int stereo;
139         int users;
140         int removed;
141         int muted;
142 };
143
144 /* USB Device ID List */
145 static struct usb_device_id usb_amradio_device_table[] = {
146         {USB_DEVICE_AND_INTERFACE_INFO(USB_AMRADIO_VENDOR, USB_AMRADIO_PRODUCT,
147                                                         USB_CLASS_HID, 0, 0) },
148         { }                                             /* Terminating entry */
149 };
150
151 MODULE_DEVICE_TABLE(usb, usb_amradio_device_table);
152
153 /* USB subsystem interface */
154 static struct usb_driver usb_amradio_driver = {
155         .name                   = MR800_DRIVER_NAME,
156         .probe                  = usb_amradio_probe,
157         .disconnect             = usb_amradio_disconnect,
158         .suspend                = usb_amradio_suspend,
159         .resume                 = usb_amradio_resume,
160         .reset_resume           = usb_amradio_resume,
161         .id_table               = usb_amradio_device_table,
162         .supports_autosuspend   = 0,
163 };
164
165 /* switch on/off the radio. Send 8 bytes to device */
166 static int amradio_set_mute(struct amradio_device *radio, char argument)
167 {
168         int retval;
169         int size;
170
171         BUG_ON(!mutex_is_locked(&radio->lock));
172
173         radio->buffer[0] = 0x00;
174         radio->buffer[1] = 0x55;
175         radio->buffer[2] = 0xaa;
176         radio->buffer[3] = 0x00;
177         radio->buffer[4] = AMRADIO_SET_MUTE;
178         radio->buffer[5] = argument;
179         radio->buffer[6] = 0x00;
180         radio->buffer[7] = 0x00;
181
182         retval = usb_bulk_msg(radio->usbdev, usb_sndintpipe(radio->usbdev, 2),
183                 (void *) (radio->buffer), BUFFER_LENGTH, &size, USB_TIMEOUT);
184
185         if (retval < 0 || size != BUFFER_LENGTH)
186                 return retval;
187
188         radio->muted = argument;
189
190         return retval;
191 }
192
193 /* set a frequency, freq is defined by v4l's TUNER_LOW, i.e. 1/16th kHz */
194 static int amradio_setfreq(struct amradio_device *radio, int freq)
195 {
196         int retval;
197         int size;
198         unsigned short freq_send = 0x10 + (freq >> 3) / 25;
199
200         BUG_ON(!mutex_is_locked(&radio->lock));
201
202         radio->buffer[0] = 0x00;
203         radio->buffer[1] = 0x55;
204         radio->buffer[2] = 0xaa;
205         radio->buffer[3] = 0x03;
206         radio->buffer[4] = AMRADIO_SET_FREQ;
207         radio->buffer[5] = 0x00;
208         radio->buffer[6] = 0x00;
209         radio->buffer[7] = 0x08;
210
211         retval = usb_bulk_msg(radio->usbdev, usb_sndintpipe(radio->usbdev, 2),
212                 (void *) (radio->buffer), BUFFER_LENGTH, &size, USB_TIMEOUT);
213
214         if (retval < 0 || size != BUFFER_LENGTH)
215                 return retval;
216
217         /* frequency is calculated from freq_send and placed in first 2 bytes */
218         radio->buffer[0] = (freq_send >> 8) & 0xff;
219         radio->buffer[1] = freq_send & 0xff;
220         radio->buffer[2] = 0x01;
221         radio->buffer[3] = 0x00;
222         radio->buffer[4] = 0x00;
223         /* 5 and 6 bytes of buffer already = 0x00 */
224         radio->buffer[7] = 0x00;
225
226         retval = usb_bulk_msg(radio->usbdev, usb_sndintpipe(radio->usbdev, 2),
227                 (void *) (radio->buffer), BUFFER_LENGTH, &size, USB_TIMEOUT);
228
229         return retval;
230 }
231
232 static int amradio_set_stereo(struct amradio_device *radio, char argument)
233 {
234         int retval;
235         int size;
236
237         BUG_ON(!mutex_is_locked(&radio->lock));
238
239         radio->buffer[0] = 0x00;
240         radio->buffer[1] = 0x55;
241         radio->buffer[2] = 0xaa;
242         radio->buffer[3] = 0x00;
243         radio->buffer[4] = AMRADIO_SET_MONO;
244         radio->buffer[5] = argument;
245         radio->buffer[6] = 0x00;
246         radio->buffer[7] = 0x00;
247
248         retval = usb_bulk_msg(radio->usbdev, usb_sndintpipe(radio->usbdev, 2),
249                 (void *) (radio->buffer), BUFFER_LENGTH, &size, USB_TIMEOUT);
250
251         if (retval < 0 || size != BUFFER_LENGTH) {
252                 radio->stereo = -1;
253                 return retval;
254         }
255
256         radio->stereo = 1;
257
258         return retval;
259 }
260
261 /* Handle unplugging the device.
262  * We call video_unregister_device in any case.
263  * The last function called in this procedure is
264  * usb_amradio_device_release.
265  */
266 static void usb_amradio_disconnect(struct usb_interface *intf)
267 {
268         struct amradio_device *radio = usb_get_intfdata(intf);
269
270         mutex_lock(&radio->lock);
271         radio->removed = 1;
272         mutex_unlock(&radio->lock);
273
274         usb_set_intfdata(intf, NULL);
275         video_unregister_device(&radio->videodev);
276         v4l2_device_disconnect(&radio->v4l2_dev);
277 }
278
279 /* vidioc_querycap - query device capabilities */
280 static int vidioc_querycap(struct file *file, void *priv,
281                                         struct v4l2_capability *v)
282 {
283         struct amradio_device *radio = video_drvdata(file);
284
285         strlcpy(v->driver, "radio-mr800", sizeof(v->driver));
286         strlcpy(v->card, "AverMedia MR 800 USB FM Radio", sizeof(v->card));
287         usb_make_path(radio->usbdev, v->bus_info, sizeof(v->bus_info));
288         v->version = RADIO_VERSION;
289         v->capabilities = V4L2_CAP_TUNER;
290         return 0;
291 }
292
293 /* vidioc_g_tuner - get tuner attributes */
294 static int vidioc_g_tuner(struct file *file, void *priv,
295                                 struct v4l2_tuner *v)
296 {
297         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
298         int retval;
299
300         mutex_lock(&radio->lock);
301
302         /* safety check */
303         if (radio->removed) {
304                 retval = -EIO;
305                 goto unlock;
306         }
307
308         if (v->index > 0) {
309                 retval = -EINVAL;
310                 goto unlock;
311         }
312
313 /* TODO: Add function which look is signal stereo or not
314  *      amradio_getstat(radio);
315  */
316
317 /* we call amradio_set_stereo to set radio->stereo
318  * Honestly, amradio_getstat should cover this in future and
319  * amradio_set_stereo shouldn't be here
320  */
321         retval = amradio_set_stereo(radio, WANT_STEREO);
322         if (retval < 0)
323                 amradio_dev_warn(&radio->videodev.dev,
324                         "set stereo failed\n");
325
326         strcpy(v->name, "FM");
327         v->type = V4L2_TUNER_RADIO;
328         v->rangelow = FREQ_MIN * FREQ_MUL;
329         v->rangehigh = FREQ_MAX * FREQ_MUL;
330         v->rxsubchans = V4L2_TUNER_SUB_MONO | V4L2_TUNER_SUB_STEREO;
331         v->capability = V4L2_TUNER_CAP_LOW;
332         if (radio->stereo)
333                 v->audmode = V4L2_TUNER_MODE_STEREO;
334         else
335                 v->audmode = V4L2_TUNER_MODE_MONO;
336         v->signal = 0xffff;     /* Can't get the signal strength, sad.. */
337         v->afc = 0; /* Don't know what is this */
338
339 unlock:
340         mutex_unlock(&radio->lock);
341         return retval;
342 }
343
344 /* vidioc_s_tuner - set tuner attributes */
345 static int vidioc_s_tuner(struct file *file, void *priv,
346                                 struct v4l2_tuner *v)
347 {
348         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
349         int retval;
350
351         mutex_lock(&radio->lock);
352
353         /* safety check */
354         if (radio->removed) {
355                 retval = -EIO;
356                 goto unlock;
357         }
358
359         if (v->index > 0) {
360                 retval = -EINVAL;
361                 goto unlock;
362         }
363
364         /* mono/stereo selector */
365         switch (v->audmode) {
366         case V4L2_TUNER_MODE_MONO:
367                 retval = amradio_set_stereo(radio, WANT_MONO);
368                 if (retval < 0)
369                         amradio_dev_warn(&radio->videodev.dev,
370                                 "set mono failed\n");
371                 break;
372         case V4L2_TUNER_MODE_STEREO:
373                 retval = amradio_set_stereo(radio, WANT_STEREO);
374                 if (retval < 0)
375                         amradio_dev_warn(&radio->videodev.dev,
376                                 "set stereo failed\n");
377                 break;
378         default:
379                 retval = -EINVAL;
380         }
381
382 unlock:
383         mutex_unlock(&radio->lock);
384         return retval;
385 }
386
387 /* vidioc_s_frequency - set tuner radio frequency */
388 static int vidioc_s_frequency(struct file *file, void *priv,
389                                 struct v4l2_frequency *f)
390 {
391         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
392         int retval;
393
394         mutex_lock(&radio->lock);
395
396         /* safety check */
397         if (radio->removed) {
398                 retval = -EIO;
399                 goto unlock;
400         }
401
402         radio->curfreq = f->frequency;
403
404         retval = amradio_setfreq(radio, radio->curfreq);
405         if (retval < 0)
406                 amradio_dev_warn(&radio->videodev.dev,
407                         "set frequency failed\n");
408
409 unlock:
410         mutex_unlock(&radio->lock);
411         return retval;
412 }
413
414 /* vidioc_g_frequency - get tuner radio frequency */
415 static int vidioc_g_frequency(struct file *file, void *priv,
416                                 struct v4l2_frequency *f)
417 {
418         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
419         int retval = 0;
420
421         mutex_lock(&radio->lock);
422
423         /* safety check */
424         if (radio->removed) {
425                 retval = -EIO;
426                 goto unlock;
427         }
428
429         f->type = V4L2_TUNER_RADIO;
430         f->frequency = radio->curfreq;
431
432 unlock:
433         mutex_unlock(&radio->lock);
434         return retval;
435 }
436
437 /* vidioc_queryctrl - enumerate control items */
438 static int vidioc_queryctrl(struct file *file, void *priv,
439                                 struct v4l2_queryctrl *qc)
440 {
441         switch (qc->id) {
442         case V4L2_CID_AUDIO_MUTE:
443                 return v4l2_ctrl_query_fill(qc, 0, 1, 1, 1);
444         }
445
446         return -EINVAL;
447 }
448
449 /* vidioc_g_ctrl - get the value of a control */
450 static int vidioc_g_ctrl(struct file *file, void *priv,
451                                 struct v4l2_control *ctrl)
452 {
453         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
454         int retval = -EINVAL;
455
456         mutex_lock(&radio->lock);
457
458         /* safety check */
459         if (radio->removed) {
460                 retval = -EIO;
461                 goto unlock;
462         }
463
464         switch (ctrl->id) {
465         case V4L2_CID_AUDIO_MUTE:
466                 ctrl->value = radio->muted;
467                 retval = 0;
468                 break;
469         }
470
471 unlock:
472         mutex_unlock(&radio->lock);
473         return retval;
474 }
475
476 /* vidioc_s_ctrl - set the value of a control */
477 static int vidioc_s_ctrl(struct file *file, void *priv,
478                                 struct v4l2_control *ctrl)
479 {
480         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
481         int retval = -EINVAL;
482
483         mutex_lock(&radio->lock);
484
485         /* safety check */
486         if (radio->removed) {
487                 retval = -EIO;
488                 goto unlock;
489         }
490
491         switch (ctrl->id) {
492         case V4L2_CID_AUDIO_MUTE:
493                 if (ctrl->value) {
494                         retval = amradio_set_mute(radio, AMRADIO_STOP);
495                         if (retval < 0) {
496                                 amradio_dev_warn(&radio->videodev.dev,
497                                         "amradio_stop failed\n");
498                         }
499                 } else {
500                         retval = amradio_set_mute(radio, AMRADIO_START);
501                         if (retval < 0) {
502                                 amradio_dev_warn(&radio->videodev.dev,
503                                         "amradio_start failed\n");
504                         }
505                 }
506                 break;
507         }
508
509 unlock:
510         mutex_unlock(&radio->lock);
511         return retval;
512 }
513
514 /* vidioc_g_audio - get audio attributes */
515 static int vidioc_g_audio(struct file *file, void *priv,
516                                 struct v4l2_audio *a)
517 {
518         if (a->index > 1)
519                 return -EINVAL;
520
521         strcpy(a->name, "Radio");
522         a->capability = V4L2_AUDCAP_STEREO;
523         return 0;
524 }
525
526 /* vidioc_s_audio - set audio attributes  */
527 static int vidioc_s_audio(struct file *file, void *priv,
528                                         struct v4l2_audio *a)
529 {
530         if (a->index != 0)
531                 return -EINVAL;
532         return 0;
533 }
534
535 /* vidioc_g_input - get input */
536 static int vidioc_g_input(struct file *filp, void *priv, unsigned int *i)
537 {
538         *i = 0;
539         return 0;
540 }
541
542 /* vidioc_s_input - set input */
543 static int vidioc_s_input(struct file *filp, void *priv, unsigned int i)
544 {
545         if (i != 0)
546                 return -EINVAL;
547         return 0;
548 }
549
550 /* open device - amradio_start() and amradio_setfreq() */
551 static int usb_amradio_open(struct file *file)
552 {
553         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
554         int retval = 0;
555
556         mutex_lock(&radio->lock);
557
558         if (radio->removed) {
559                 retval = -EIO;
560                 goto unlock;
561         }
562
563         radio->users = 1;
564         radio->muted = 1;
565
566         retval = amradio_set_mute(radio, AMRADIO_START);
567         if (retval < 0) {
568                 amradio_dev_warn(&radio->videodev.dev,
569                         "radio did not start up properly\n");
570                 radio->users = 0;
571                 goto unlock;
572         }
573
574         retval = amradio_set_stereo(radio, WANT_STEREO);
575         if (retval < 0)
576                 amradio_dev_warn(&radio->videodev.dev,
577                         "set stereo failed\n");
578
579         retval = amradio_setfreq(radio, radio->curfreq);
580         if (retval < 0)
581                 amradio_dev_warn(&radio->videodev.dev,
582                         "set frequency failed\n");
583
584 unlock:
585         mutex_unlock(&radio->lock);
586         return retval;
587 }
588
589 /*close device */
590 static int usb_amradio_close(struct file *file)
591 {
592         struct amradio_device *radio = video_get_drvdata(video_devdata(file));
593         int retval = 0;
594
595         mutex_lock(&radio->lock);
596
597         if (radio->removed) {
598                 retval = -EIO;
599                 goto unlock;
600         }
601
602         radio->users = 0;
603
604         if (!radio->removed) {
605                 retval = amradio_set_mute(radio, AMRADIO_STOP);
606                 if (retval < 0)
607                         amradio_dev_warn(&radio->videodev.dev,
608                                 "amradio_stop failed\n");
609         }
610
611 unlock:
612         mutex_unlock(&radio->lock);
613         return retval;
614 }
615
616 /* Suspend device - stop device. Need to be checked and fixed */
617 static int usb_amradio_suspend(struct usb_interface *intf, pm_message_t message)
618 {
619         struct amradio_device *radio = usb_get_intfdata(intf);
620         int retval;
621
622         mutex_lock(&radio->lock);
623
624         retval = amradio_set_mute(radio, AMRADIO_STOP);
625         if (retval < 0)
626                 dev_warn(&intf->dev, "amradio_stop failed\n");
627
628         dev_info(&intf->dev, "going into suspend..\n");
629
630         mutex_unlock(&radio->lock);
631         return 0;
632 }
633
634 /* Resume device - start device. Need to be checked and fixed */
635 static int usb_amradio_resume(struct usb_interface *intf)
636 {
637         struct amradio_device *radio = usb_get_intfdata(intf);
638         int retval;
639
640         mutex_lock(&radio->lock);
641
642         retval = amradio_set_mute(radio, AMRADIO_START);
643         if (retval < 0)
644                 dev_warn(&intf->dev, "amradio_start failed\n");
645
646         dev_info(&intf->dev, "coming out of suspend..\n");
647
648         mutex_unlock(&radio->lock);
649         return 0;
650 }
651
652 /* File system interface */
653 static const struct v4l2_file_operations usb_amradio_fops = {
654         .owner          = THIS_MODULE,
655         .open           = usb_amradio_open,
656         .release        = usb_amradio_close,
657         .ioctl          = video_ioctl2,
658 };
659
660 static const struct v4l2_ioctl_ops usb_amradio_ioctl_ops = {
661         .vidioc_querycap    = vidioc_querycap,
662         .vidioc_g_tuner     = vidioc_g_tuner,
663         .vidioc_s_tuner     = vidioc_s_tuner,
664         .vidioc_g_frequency = vidioc_g_frequency,
665         .vidioc_s_frequency = vidioc_s_frequency,
666         .vidioc_queryctrl   = vidioc_queryctrl,
667         .vidioc_g_ctrl      = vidioc_g_ctrl,
668         .vidioc_s_ctrl      = vidioc_s_ctrl,
669         .vidioc_g_audio     = vidioc_g_audio,
670         .vidioc_s_audio     = vidioc_s_audio,
671         .vidioc_g_input     = vidioc_g_input,
672         .vidioc_s_input     = vidioc_s_input,
673 };
674
675 static void usb_amradio_video_device_release(struct video_device *videodev)
676 {
677         struct amradio_device *radio = video_get_drvdata(videodev);
678
679         v4l2_device_unregister(&radio->v4l2_dev);
680
681         /* free rest memory */
682         kfree(radio->buffer);
683         kfree(radio);
684 }
685
686 /* check if the device is present and register with v4l and usb if it is */
687 static int usb_amradio_probe(struct usb_interface *intf,
688                                 const struct usb_device_id *id)
689 {
690         struct amradio_device *radio;
691         struct v4l2_device *v4l2_dev;
692         int retval = 0;
693
694         radio = kzalloc(sizeof(struct amradio_device), GFP_KERNEL);
695
696         if (!radio) {
697                 dev_err(&intf->dev, "kmalloc for amradio_device failed\n");
698                 retval = -ENOMEM;
699                 goto err;
700         }
701
702         radio->buffer = kmalloc(BUFFER_LENGTH, GFP_KERNEL);
703
704         if (!radio->buffer) {
705                 dev_err(&intf->dev, "kmalloc for radio->buffer failed\n");
706                 retval = -ENOMEM;
707                 goto err_nobuf;
708         }
709
710         v4l2_dev = &radio->v4l2_dev;
711         retval = v4l2_device_register(&intf->dev, v4l2_dev);
712         if (retval < 0) {
713                 dev_err(&intf->dev, "couldn't register v4l2_device\n");
714                 goto err_v4l2;
715         }
716
717         strlcpy(radio->videodev.name, v4l2_dev->name,
718                 sizeof(radio->videodev.name));
719         radio->videodev.v4l2_dev = v4l2_dev;
720         radio->videodev.fops = &usb_amradio_fops;
721         radio->videodev.ioctl_ops = &usb_amradio_ioctl_ops;
722         radio->videodev.release = usb_amradio_video_device_release;
723
724         radio->removed = 0;
725         radio->users = 0;
726         radio->usbdev = interface_to_usbdev(intf);
727         radio->curfreq = 95.16 * FREQ_MUL;
728         radio->stereo = -1;
729
730         mutex_init(&radio->lock);
731
732         video_set_drvdata(&radio->videodev, radio);
733
734         retval = video_register_device(&radio->videodev, VFL_TYPE_RADIO,
735                                         radio_nr);
736         if (retval < 0) {
737                 dev_err(&intf->dev, "could not register video device\n");
738                 goto err_vdev;
739         }
740
741         usb_set_intfdata(intf, radio);
742         return 0;
743
744 err_vdev:
745         v4l2_device_unregister(v4l2_dev);
746 err_v4l2:
747         kfree(radio->buffer);
748 err_nobuf:
749         kfree(radio);
750 err:
751         return retval;
752 }
753
754 static int __init amradio_init(void)
755 {
756         int retval = usb_register(&usb_amradio_driver);
757
758         pr_info(KBUILD_MODNAME
759                 ": version " DRIVER_VERSION " " DRIVER_DESC "\n");
760
761         if (retval)
762                 pr_err(KBUILD_MODNAME
763                         ": usb_register failed. Error number %d\n", retval);
764
765         return retval;
766 }
767
768 static void __exit amradio_exit(void)
769 {
770         usb_deregister(&usb_amradio_driver);
771 }
772
773 module_init(amradio_init);
774 module_exit(amradio_exit);
775