mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: viro@zeniv.linux.org.uk, linux-media@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	Jonathan Corbet <corbet@lwn.net>,
	Mauro Carvalho Chehab <mchehab@redhat.com>
Subject: Re: [RFCv4 PATCH 0/6]: add poll_requested_events() function
Date: Wed, 5 Oct 2011 14:01:56 -0700	[thread overview]
Message-ID: <20111005140156.70ed0329.akpm@linux-foundation.org> (raw)
In-Reply-To: <201110050947.09488.hverkuil@xs4all.nl>

On Wed, 5 Oct 2011 09:47:09 +0200
Hans Verkuil <hverkuil@xs4all.nl> wrote:

> On Thursday 29 September 2011 09:44:06 Hans Verkuil wrote:
> > This is the fourth version of this patch series, incorporating the comments
> > from Andrew Morton: I've split up the multiple-assignment line and added a
> > comment explaining the purpose of the new function in poll.h.
> > 
> > It's also rebased to the current staging/for_v3.2 branch of the linux-media
> > tree.
> > 
> > There are no other changes compared to the RFCv3 patches.
> > 
> > I'd very much like to get an Acked-by (or additional comments) from Al or
> > Andrew! This patch series really should go into v3.2 which is getting
> > close.
> > 
> > Normally I would have posted this v4 3 weeks ago, but due to Real Life
> > interference in the past few weeks I was unable to. But I'm back, and this
> > is currently the highest priority for me.
> 
> This is becoming annoying. Andrew, Al, can one of you please Ack this patch or 
> review it? We *really* need this enhancement for our v4l drivers. I've been 
> asking for an ack (or review) for ages and for the most part I got radio 
> silence. Jon Corbet has already reviewed the code in early July (!), so I 
> don't see why this is taking so long.
> 
> Mauro needs an ack from one of you before he can merge it.
> 

It looks OK to me, but obviously it would be better if Al were to go
through it also.  If he doesn't then there isn't a lot we can do about
it - I suggest that you proceed with the plan to merge it into -rc1.

      reply	other threads:[~2011-10-05 21:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-29  7:44 Hans Verkuil
2011-09-29  7:44 ` [RFCv4 PATCH 1/6] poll: " Hans Verkuil
2011-09-29  7:44   ` [RFCv4 PATCH 2/6] ivtv: only start streaming in poll() if polling for input Hans Verkuil
2011-10-02 12:13     ` Andy Walls
2011-09-29  7:44   ` [RFCv4 PATCH 3/6] videobuf2: only start streaming in poll() if so requested by the poll mask Hans Verkuil
2011-09-29  7:44   ` [RFCv4 PATCH 4/6] videobuf: " Hans Verkuil
2011-09-29  7:44   ` [RFCv4 PATCH 5/6] videobuf2-core: also test for pending events Hans Verkuil
2011-09-29  7:44   ` [RFCv4 PATCH 6/6] vivi: let vb2_poll handle events Hans Verkuil
2011-10-05  7:47 ` [RFCv4 PATCH 0/6]: add poll_requested_events() function Hans Verkuil
2011-10-05 21:01   ` Andrew Morton [this message]

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=20111005140156.70ed0329.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=corbet@lwn.net \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=viro@zeniv.linux.org.uk \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome