All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Andy Walls <awalls@radix.net>,
	Douglas Landgraf <dougsland@gmail.com>,
	Alan Cox <alan@redhat.com>,
	video4linux-list@redhat.com, linux-kernel@vger.kernel.org,
	Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Re: [PATCH] video4linux: Push down the BKL
Date: Mon, 26 May 2008 18:14:06 -0300	[thread overview]
Message-ID: <20080526181406.40f9e082@gaivota> (raw)
In-Reply-To: <200805261846.35758.hverkuil@xs4all.nl>


> > Could you send this patch to the ML for people to review and for Andy
> > to port it to cx18?
> 
> Unless there is an objection, I would prefer to take Douglas' patch and 
> merge it into the v4l-dvb ivtv driver myself. There were several things 
> in the patch I didn't like, but I need to 'work' with it a bit to see 
> how/if it can be done better.

I didn't analyze his patch in detail. However, it is a starting point
for you to work with. I'm sure you can improve it if needed.

> I can work on it tonight and tomorrow. Hopefully it is finished by then. 
> I can move the BKL down at the same time for ivtv. It is unlikely that 
> I will have time to do cx18 as well as I'm abroad from Wednesday until 
> Monday, but I think Andy can do that easily based on the ivtv changes.

This strategy seems OK to me.


Cheers,
Mauro

WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: video4linux-list@redhat.com, linux-kernel@vger.kernel.org,
	Alan Cox <alan@redhat.com>, Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Re: [PATCH] video4linux: Push down the BKL
Date: Mon, 26 May 2008 18:14:06 -0300	[thread overview]
Message-ID: <20080526181406.40f9e082@gaivota> (raw)
In-Reply-To: <200805261846.35758.hverkuil@xs4all.nl>


> > Could you send this patch to the ML for people to review and for Andy
> > to port it to cx18?
> 
> Unless there is an objection, I would prefer to take Douglas' patch and 
> merge it into the v4l-dvb ivtv driver myself. There were several things 
> in the patch I didn't like, but I need to 'work' with it a bit to see 
> how/if it can be done better.

I didn't analyze his patch in detail. However, it is a starting point
for you to work with. I'm sure you can improve it if needed.

> I can work on it tonight and tomorrow. Hopefully it is finished by then. 
> I can move the BKL down at the same time for ivtv. It is unlikely that 
> I will have time to do cx18 as well as I'm abroad from Wednesday until 
> Monday, but I think Andy can do that easily based on the ivtv changes.

This strategy seems OK to me.


Cheers,
Mauro

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2008-05-26 21:14 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-22 21:37 [PATCH] video4linux: Push down the BKL Alan Cox
2008-05-22 21:37 ` Alan Cox
2008-05-23  2:08 ` Andy Walls
2008-05-23  2:08   ` Andy Walls
2008-05-23  6:16   ` Hans Verkuil
2008-05-23  6:16     ` Hans Verkuil
2008-05-23  6:28     ` Hans Verkuil
2008-05-23  6:28       ` Hans Verkuil
2008-05-26 16:39     ` Mauro Carvalho Chehab
2008-05-26 16:39       ` Mauro Carvalho Chehab
2008-05-23  9:09   ` Alan Cox
2008-05-23  9:09     ` Alan Cox
2008-05-26 16:34     ` Mauro Carvalho Chehab
2008-05-26 16:34       ` Mauro Carvalho Chehab
2008-05-26 16:46       ` Hans Verkuil
2008-05-26 16:46         ` Hans Verkuil
2008-05-26 21:14         ` Mauro Carvalho Chehab [this message]
2008-05-26 21:14           ` Mauro Carvalho Chehab
2008-06-01  2:34         ` [PATCH] cx18: convert driver to video_ioctl2() (Re: [PATCH] video4linux: Push down the BKL) Andy Walls
2008-06-01 10:15           ` Hans Verkuil
2008-06-01 19:01             ` Andy Walls
2008-06-03 21:20               ` Mauro Carvalho Chehab
2008-06-03 22:08                 ` [ivtv-devel] " Alan Cox
2008-06-04  0:44                   ` Andy Walls
2008-06-04 10:02                     ` Alan Cox
2008-05-23 13:56 ` [PATCH] video4linux: Push down the BKL Jonathan Corbet
2008-05-23 15:39   ` Alan Cox
2008-05-23 15:39     ` Alan Cox
2008-05-23 16:09     ` Jonathan Corbet
2008-05-23 18:58       ` Alan Cox
2008-05-23 18:58         ` Alan Cox
2008-05-23 19:05 ` Hans Verkuil
2008-05-23 19:05   ` Hans Verkuil
2008-05-25 23:46 ` Mike Isely
2008-05-25 23:46   ` Mike Isely
2008-05-26 16:59 ` Mauro Carvalho Chehab
2008-05-26 16:59   ` Mauro Carvalho Chehab
2008-05-26 20:23   ` Alan Cox
2008-05-26 20:23     ` Alan Cox
2008-05-26 21:10     ` Mauro Carvalho Chehab
2008-05-26 21:10       ` Mauro Carvalho Chehab
2008-05-26 22:01       ` Alan Cox
2008-05-26 22:01         ` Alan Cox
2008-05-27 13:10         ` Mauro Carvalho Chehab
2008-05-27 13:10           ` Mauro Carvalho Chehab
2008-05-27 15:41           ` Jonathan Corbet
2008-05-27 16:31             ` Mauro Carvalho Chehab
2008-05-27 16:31               ` Mauro Carvalho Chehab
2008-05-27 16:37               ` Jonathan Corbet
2008-05-27 18:59                 ` Mauro Carvalho Chehab
2008-05-27 18:59                   ` Mauro Carvalho Chehab
2008-05-27 19:26                   ` Devin Heitmueller
2008-05-27 19:26                     ` Devin Heitmueller
2008-05-27 21:00                     ` Mauro Carvalho Chehab
2008-05-27 21:00                       ` Mauro Carvalho Chehab
2008-05-27 21:22                       ` Devin Heitmueller
2008-05-27 21:22                         ` Devin Heitmueller
2008-05-27 23:48                       ` Andy Walls
2008-05-27 23:48                         ` Andy Walls
2008-05-28  0:46                         ` Devin Heitmueller
2008-05-28  0:46                           ` Devin Heitmueller
2008-05-28  2:37                           ` Andy Walls
2008-05-28  2:37                             ` Andy Walls
2008-05-28  2:47                             ` Devin Heitmueller
2008-05-28  2:47                               ` Devin Heitmueller
2008-05-28 23:30                               ` Andy Walls
2008-05-28  8:34                             ` Alan Cox
2008-05-28  8:34                               ` Alan Cox
2008-05-28  6:13                           ` Hans Verkuil
2008-05-28  6:13                             ` Hans Verkuil
2008-05-27 19:50                   ` Arjan van de Ven
2008-05-27 20:24                     ` Mauro Carvalho Chehab
2008-05-27 20:24                       ` Mauro Carvalho Chehab
2008-05-27 18:14               ` Alan Cox
2008-05-27 18:14                 ` Alan Cox

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=20080526181406.40f9e082@gaivota \
    --to=mchehab@infradead.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=alan@redhat.com \
    --cc=awalls@radix.net \
    --cc=dougsland@gmail.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=video4linux-list@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.