From: Gianluca Gennari <gennarone@gmail.com>
To: Mauro Carvalho Chehab <mchehab@redhat.com>,
Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH] [media] dvb-core: preserve the delivery system at cache clear
Date: Fri, 13 Jan 2012 17:04:38 +0100 [thread overview]
Message-ID: <4F105616.20001@gmail.com> (raw)
In-Reply-To: <1326462636-8869-1-git-send-email-mchehab@redhat.com>
Il 13/01/2012 14:50, Mauro Carvalho Chehab ha scritto:
> The changeset 240ab508aa is incomplete, as the first thing that
> happens at cache clear is to do a memset with 0 to the cache.
>
> So, the delivery system needs to be explicitly preserved there.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
>
> ---
>
> If Kaffeine doesn't call FE_SET_PROPERTY for non-DVB-S2, this should
> fix the current issue.
>
> drivers/media/dvb/dvb-core/dvb_frontend.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c
> index 2ad7faf..f5fa7aa 100644
> --- a/drivers/media/dvb/dvb-core/dvb_frontend.c
> +++ b/drivers/media/dvb/dvb-core/dvb_frontend.c
> @@ -904,8 +904,11 @@ static int dvb_frontend_clear_cache(struct dvb_frontend *fe)
> {
> struct dtv_frontend_properties *c = &fe->dtv_property_cache;
> int i;
> + u32 delsys;
>
> + delsys = c->delivery_system;
> memset(c, 0, sizeof(struct dtv_frontend_properties));
> + c->delivery_system = delsys;
>
> c->state = DTV_CLEAR;
>
Hi Mauro,
I applied this new patch on top of the current media_build tree and I
can confirm that the issue with Kaffeine is solved.
All of my DVB-T sticks works fine again.
Best regards,
Gianluca
next prev parent reply other threads:[~2012-01-13 16:10 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-10 13:28 Possible regression in 3.2 kernel with PCTV Nanostick T2 (em28xx, cxd2820r and tda18271) Jim Darby
2012-01-10 13:54 ` Steven Toth
2012-01-10 23:22 ` Jim Darby
2012-01-11 0:01 ` Steven Toth
2012-01-11 0:34 ` Andy Walls
2012-01-11 1:05 ` Antti Palosaari
2012-01-11 11:30 ` Jim Darby
2012-01-11 19:19 ` Jim Darby
2012-01-12 16:22 ` Gianluca Gennari
2012-01-12 16:35 ` Jim Darby
2012-01-12 17:13 ` Simon Jones
2012-01-12 17:34 ` Gianluca Gennari
2012-01-13 11:21 ` Mauro Carvalho Chehab
2012-01-13 11:45 ` Gianluca Gennari
2012-01-13 13:50 ` [PATCH] [media] dvb-core: preserve the delivery system at cache clear Mauro Carvalho Chehab
2012-01-13 16:04 ` Gianluca Gennari [this message]
2012-01-14 0:00 ` Jim Darby
2012-01-14 14:51 ` Mauro Carvalho Chehab
2012-01-13 13:09 ` Possible regression in 3.2 kernel with PCTV Nanostick T2 (em28xx, cxd2820r and tda18271) Jim Darby
2012-01-13 14:24 ` Gianluca Gennari
2012-01-12 14:29 ` Simon Jones
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F105616.20001@gmail.com \
--to=gennarone@gmail.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.