Re: em28xx breaks after hibernate

[Johannes Stezenbach <js@linuxtv.org>]

On Sun, Sep 28, 2014 at 08:12:11AM -0300, Mauro Carvalho Chehab wrote:
> Em Sun, 28 Sep 2014 12:55:40 +0200
> Johannes Stezenbach <js@linuxtv.org> escreveu:
> 
> > I tried again both with and without the patch.  The issue above
> > odesn't reproduce, but after hibernate it fails to tune
> > (while it works after suspend-to-ram).  Restarting dvbv5-zap
> > does not fix it.  All I get is:
> > 
> > [  500.299216] drxk: Error -22 on dvbt_sc_command
> > [  500.301012] drxk: Error -22 on set_dvbt
> > [  500.301967] drxk: Error -22 on start
> 
> Just to be 100% sure if I understood well: you're having exactly
> the same behavior with and without my patch, right?

Yes, no observable difference in my tests.

> I'll see if I can work on another patch for you today. If not,
> I won't be able to touch on it until the end of the week, as I'm
> traveling next week.

no need to hurry

(BTW, I still think you should make sure the hang-on-resume fix,
revert of b89193e0b06f, goes into 3.17, but it's your call.)

> > On rmmod it Oopsed:
...
> Please try this change:
> 
> [media] em28xx: remove firmware before releasing xc5000 priv state
> 
> hybrid_tuner_release_state() can free the priv state, so we need to
> release the firmware before calling it.
> 
> Signed-off-by: Mauro Carvalho Chehab
> 
> diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c
> index e44c8aba6074..803a0e63d47e 100644
> --- a/drivers/media/tuners/xc5000.c
> +++ b/drivers/media/tuners/xc5000.c
> @@ -1333,9 +1333,9 @@ static int xc5000_release(struct dvb_frontend *fe)
>  
>  	if (priv) {
>  		cancel_delayed_work(&priv->timer_sleep);
> -		hybrid_tuner_release_state(priv);
>  		if (priv->firmware)
>  			release_firmware(priv->firmware);
> +		hybrid_tuner_release_state(priv);
>  	}
>  
>  	mutex_unlock(&xc5000_list_mutex);
> 

Works.  And after module reload, dvbv5-zap can work again.


Thanks,
Johannes