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 <j...@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
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
