Commit dd0c8abf authored by Thierry Reding's avatar Thierry Reding Committed by Mauro Carvalho Chehab

[media] tm6000: Execute lightweight reset on close

When the last user closes the device, perform a lightweight reset of the
device to bring it into a well-known state.

Note that this is not always enough with the TM6010, which sometimes
needs a hard reset to get into a working state again.
Signed-off-by: default avatarThierry Reding <thierry.reding@avionic-design.de>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 88e834a8
...@@ -599,6 +599,49 @@ int tm6000_init(struct tm6000_core *dev) ...@@ -599,6 +599,49 @@ int tm6000_init(struct tm6000_core *dev)
return rc; return rc;
} }
int tm6000_reset(struct tm6000_core *dev)
{
int pipe;
int err;
msleep(500);
err = usb_set_interface(dev->udev, dev->isoc_in.bInterfaceNumber, 0);
if (err < 0) {
tm6000_err("failed to select interface %d, alt. setting 0\n",
dev->isoc_in.bInterfaceNumber);
return err;
}
err = usb_reset_configuration(dev->udev);
if (err < 0) {
tm6000_err("failed to reset configuration\n");
return err;
}
msleep(5);
err = usb_set_interface(dev->udev, dev->isoc_in.bInterfaceNumber, 2);
if (err < 0) {
tm6000_err("failed to select interface %d, alt. setting 2\n",
dev->isoc_in.bInterfaceNumber);
return err;
}
msleep(5);
pipe = usb_rcvintpipe(dev->udev,
dev->int_in.endp->desc.bEndpointAddress & USB_ENDPOINT_NUMBER_MASK);
err = usb_clear_halt(dev->udev, pipe);
if (err < 0) {
tm6000_err("usb_clear_halt failed: %d\n", err);
return err;
}
return 0;
}
int tm6000_set_audio_bitrate(struct tm6000_core *dev, int bitrate) int tm6000_set_audio_bitrate(struct tm6000_core *dev, int bitrate)
{ {
int val = 0; int val = 0;
......
...@@ -1603,9 +1603,16 @@ static int tm6000_release(struct file *file) ...@@ -1603,9 +1603,16 @@ static int tm6000_release(struct file *file)
dev->users--; dev->users--;
res_free(dev, fh); res_free(dev, fh);
if (!dev->users) { if (!dev->users) {
int err;
tm6000_uninit_isoc(dev); tm6000_uninit_isoc(dev);
videobuf_mmap_free(&fh->vb_vidq); videobuf_mmap_free(&fh->vb_vidq);
err = tm6000_reset(dev);
if (err < 0)
dev_err(&vdev->dev, "reset failed: %d\n", err);
} }
kfree(fh); kfree(fh);
......
...@@ -311,6 +311,7 @@ int tm6000_set_reg_mask(struct tm6000_core *dev, u8 req, u16 value, ...@@ -311,6 +311,7 @@ int tm6000_set_reg_mask(struct tm6000_core *dev, u8 req, u16 value,
u16 index, u16 mask); u16 index, u16 mask);
int tm6000_i2c_reset(struct tm6000_core *dev, u16 tsleep); int tm6000_i2c_reset(struct tm6000_core *dev, u16 tsleep);
int tm6000_init(struct tm6000_core *dev); int tm6000_init(struct tm6000_core *dev);
int tm6000_reset(struct tm6000_core *dev);
int tm6000_init_analog_mode(struct tm6000_core *dev); int tm6000_init_analog_mode(struct tm6000_core *dev);
int tm6000_init_digital_mode(struct tm6000_core *dev); int tm6000_init_digital_mode(struct tm6000_core *dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment