V4L/DVB (10462): radio-mr800: increase version and add comments
authorAlexey Klimov <klimov.linux@gmail.com>
Thu, 5 Feb 2009 11:58:31 +0000 (08:58 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 30 Mar 2009 15:42:40 +0000 (12:42 -0300)
Increase driver version to 0.10, remove old and add new useful comments.

Signed-off-by: Alexey Klimov <klimov.linux@gmail.com>
Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/radio/radio-mr800.c

index 56f1fac6d28d05f694faf531197f68fdc8ce4482..23c3c539157b9243a1bc37f43e8c51f3de9fef70 100644 (file)
@@ -22,7 +22,7 @@
  */
 
 /*
- * Big thanks to authors of dsbr100.c and radio-si470x.c
+ * Big thanks to authors and contributors of dsbr100.c and radio-si470x.c
  *
  * When work was looked pretty good, i discover this:
  * http://av-usbradio.sourceforge.net/index.php
  * Latest release of theirs project was in 2005.
  * Probably, this driver could be improved trough using their
  * achievements (specifications given).
- * So, we have smth to begin with.
+ * Also, Faidon Liambotis <paravoid@debian.org> wrote nice driver for this radio
+ * in 2007. He allowed to use his driver to improve current mr800 radio driver.
+ * http://kerneltrap.org/mailarchive/linux-usb-devel/2007/10/11/342492
  *
- * History:
  * Version 0.01:       First working version.
  *                     It's required to blacklist AverMedia USB Radio
  *                     in usbhid/hid-quirks.c
+ * Version 0.10:       A lot of cleanups and fixes: unpluging the device,
+ *                     few mutex locks were added, codinstyle issues, etc.
+ *                     Added stereo support. Thanks to
+ *                     Douglas Schilling Landgraf <dougsland@gmail.com> and
+ *                     David Ellingsworth <david@identd.dyndns.org>
+ *                     for discussion, help and support.
  *
  * Many things to do:
  *     - Correct power managment of device (suspend & resume)
- *     - Make x86 independance (little-endian and big-endian stuff)
  *     - Add code for scanning and smooth tuning
- *     - Checked and add stereo&mono stuff
  *     - Add code for sensitivity value
  *     - Correct mistakes
  *     - In Japan another FREQ_MIN and FREQ_MAX
@@ -62,8 +67,8 @@
 /* driver and module definitions */
 #define DRIVER_AUTHOR "Alexey Klimov <klimov.linux@gmail.com>"
 #define DRIVER_DESC "AverMedia MR 800 USB FM radio driver"
-#define DRIVER_VERSION "0.01"
-#define RADIO_VERSION KERNEL_VERSION(0, 0, 1)
+#define DRIVER_VERSION "0.10"
+#define RADIO_VERSION KERNEL_VERSION(0, 1, 0)
 
 MODULE_AUTHOR(DRIVER_AUTHOR);
 MODULE_DESCRIPTION(DRIVER_DESC);
@@ -309,14 +314,11 @@ static int amradio_set_stereo(struct amradio_device *radio, char argument)
        return retval;
 }
 
-
-
-/* USB subsystem interface begins here */
-
-/* handle unplugging of the device, release data structures
-if nothing keeps us from doing it.  If something is still
-keeping us busy, the release callback of v4l will take care
-of releasing it. */
+/* Handle unplugging the device.
+ * We call video_unregister_device in any case.
+ * The last function called in this procedure is
+ * usb_amradio_device_release.
+ */
 static void usb_amradio_disconnect(struct usb_interface *intf)
 {
        struct amradio_device *radio = usb_get_intfdata(intf);