Hi Changbing, Em Thu, 08 May 2014 14:28:23 +0800 Changbing Xiong <cb.xiong@xxxxxxxxxxx> escreveu: > If the tuner is already disconnected, It is meaningless to go on doing the > stream-restart operation, It is better to cancel this operation. > > Signed-off-by: Changbing Xiong <cb.xiong@xxxxxxxxxxx> > --- > drivers/media/usb/au0828/au0828-dvb.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/media/usb/au0828/au0828-dvb.c b/drivers/media/usb/au0828/au0828-dvb.c > index 878f66f..6995309 100755 > --- a/drivers/media/usb/au0828/au0828-dvb.c > +++ b/drivers/media/usb/au0828/au0828-dvb.c > @@ -422,6 +422,8 @@ void au0828_dvb_unregister(struct au0828_dev *dev) > dvb_unregister_frontend(dvb->frontend); > dvb_frontend_detach(dvb->frontend); > dvb_unregister_adapter(&dvb->adapter); > + > + cancel_work_sync(&dev->restart_streaming); I'm wandering that, if you move the cancel_work_sync() to be called earlier in this function (before the dvb_unregister calls), then maybe we can avoid the first patch. > } > > /* All the DVB attach calls go here, this function get's modified > -- > 1.7.9.5 > -- Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html