Re: [PATCH 2/2] au0828: Cancel stream-restart operation if frontend is disconnected

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux