From: Andy Walls <awalls@md.metrocast.net>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Hans Verkuil <hverkuil@xs4all.nl>, linux-media@vger.kernel.org
Subject: Re: [GIT PATCHES FOR 2.6.39] fix cx18 regression
Date: Wed, 02 Feb 2011 18:50:12 -0500 [thread overview]
Message-ID: <1296690612.2402.4.camel@localhost> (raw)
In-Reply-To: <4D46C36A.1040407@redhat.com>
On Mon, 2011-01-31 at 12:12 -0200, Mauro Carvalho Chehab wrote:
> Em 26-01-2011 05:23, Hans Verkuil escreveu:
> > Mauro, please get this upstream asap since this fix needs to go into 2.6.38
> > as well.
> >
> > Regards,
> >
> > Hans
> >
> > The following changes since commit e5fb95675639f064ca40df7ad319f1c380443999:
> > Hans Verkuil (1):
> > [media] vivi: fix compiler warning
> >
> > are available in the git repository at:
> >
> > ssh://linuxtv.org/git/hverkuil/media_tree.git cx18-fix
> >
> > Hans Verkuil (1):
> > cx18: fix kernel oops when setting MPEG control before capturing.
> >
> > drivers/media/video/cx18/cx18-driver.c | 1 +
> > 1 files changed, 1 insertions(+), 0 deletions(-)
> >
>
> I tried to apply it against 2.6.38-rc2, but it failed:
>
> mutex_init(&cx->serialize_lock);
> mutex_init(&cx->gpio_lock);
> mutex_init(&cx->epu2apu_mb_lock);
> mutex_init(&cx->epu2cpu_mb_lock);
>
> ret = cx18_create_in_workq(cx);
> <<<<<<<
> =======
> cx->cxhdl.capabilities = CX2341X_CAP_HAS_TS | CX2341X_CAP_HAS_SLICED_VBI;
> cx->cxhdl.ops = &cx18_cxhdl_ops;
> cx->cxhdl.func = cx18_api_func;
> cx->cxhdl.priv = &cx->streams[CX18_ENC_STREAM_TYPE_MPG];
> ret = cx2341x_handler_init(&cx->cxhdl, 50);
> >>>>>>>
> if (ret)
> return ret;
>
> Perhaps this change requires some patch delayed for .39?
The bug was authored on 31 Dec 2010, but not comitted until 23 Jan 2011:
http://git.linuxtv.org/hverkuil/media_tree.git?a=commit;h=82f205b2f2a1deb1ab700a601ef48a4db4ca4f4e
Kernel 2.6.38-rc2 appears to have a date one day prior: 22 Jan 2011:
http://git.linuxtv.org/hverkuil/media_tree.git?a=commit;h=1bae4ce27c9c90344f23c65ea6966c50ffeae2f5
So the bug will be in whatever version comes out after 2.6.38-rc2
Regards,
Andy
> Cheers,
> Mauro
next prev parent reply other threads:[~2011-02-02 23:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-26 7:23 [GIT PATCHES FOR 2.6.39] fix cx18 regression Hans Verkuil
2011-01-31 14:12 ` Mauro Carvalho Chehab
2011-02-02 23:50 ` Andy Walls [this message]
2011-02-03 7:09 ` Hans Verkuil
2011-02-05 12:13 ` Hans Verkuil
-- strict thread matches above, loose matches on Subject: below --
2011-02-03 12:23 Andy Walls
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=1296690612.2402.4.camel@localhost \
--to=awalls@md.metrocast.net \
--cc=hverkuil@xs4all.nl \
--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.