Skip to contentSkip to conversation

radio-mr800: cleanup of usb_amradio_open/close

PATCH In review Alexey Klimov 1 participant 1 message Aug 8, 2009 Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

PATCH 2/6 radio-mr800: cleanup of usb_amradio_open/close

Patch removes functions that shouldn't be in usb_amradio_open/close:
amradio_set_mute(), amradio_set_stereo(), amradio_setfreq().

--

linux/drivers/media/radio/radio-mr800.c +0 -27
diff -r 34b4e5c9d5c2 linux/drivers/media/radio/radio-mr800.c
@@ -538,29 +538,10 @@
 static int usb_amradio_open(struct file *file)
 {
 	struct amradio_device *radio = video_get_drvdata(video_devdata(file));
-	int retval;
 
 	radio->users = 1;
 	radio->muted = 1;
 
-	retval = amradio_set_mute(radio, AMRADIO_START);
-	if (retval < 0) {
-		amradio_dev_warn(&radio->videodev->dev,
-			"radio did not start up properly\n");
-		radio->users = 0;
-		return -EIO;
-	}
-
-	retval = amradio_set_stereo(radio, WANT_STEREO);
-	if (retval < 0)
-		amradio_dev_warn(&radio->videodev->dev,
-			"set stereo failed\n");
-
-	retval = amradio_setfreq(radio, radio->curfreq);
-	if (retval < 0)
-		amradio_dev_warn(&radio->videodev->dev,
-			"set frequency failed\n");
-
 	return 0;
 }
 
@@ -568,7 +549,6 @@
 static int usb_amradio_close(struct file *file)
 {
 	struct amradio_device *radio = video_get_drvdata(video_devdata(file));
-	int retval;
 
 	if (!radio)
 		return -ENODEV;
@@ -577,13 +557,6 @@
 	radio->users = 0;
 	mutex_unlock(&radio->lock);
 
-	if (!radio->removed) {
-		retval = amradio_set_mute(radio, AMRADIO_STOP);
-		if (retval < 0)
-			amradio_dev_warn(&radio->videodev->dev,
-				"amradio_stop failed\n");
-	}
-
 	return 0;
 }
 
Signature · 1 line
Best regards, Klimov Alexey