All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Sakari Ailus <sakari.ailus@iki.fi>, linux-media@vger.kernel.org
Subject: Re: [GIT PULL FOR v3.5] Implement V4L2_CID_PIXEL_RATE in various drivers
Date: Tue, 15 May 2012 01:11:47 +0200	[thread overview]
Message-ID: <1654469.oil1fXEATY@avalon> (raw)
In-Reply-To: <4FB12C74.8000306@redhat.com>

Hi Mauro,

On Monday 14 May 2012 13:01:56 Mauro Carvalho Chehab wrote:
> Em 14-05-2012 12:56, Sakari Ailus escreveu:
> > Hi all,
> > 
> > Here are a few patches that implement V4L2_CID_PIXEL_RATE in a couple of
> > drivers. The control is soon required by some CSI-2 receivers to configure
> > the hardware, such as the OMAP 3 ISP one.
> 
> Before spreading this everywhere, let me understand a little bit better
> about V4L2_CID_PIXEL_RATE and the other ioctl's that handle the image rate,
> as changing one affects the other.

Implementing V4L2_CID_PIXEL_RATE is required for sensor drivers used with the 
OMAP3 ISP. The ISP driver needs to know the pixel rate on the ISP sink to 
configure the hardware. This was previously done with a platform callback, the 
ISP now queries the sensor directly using the V4L2_CID_PIXEL_RATE control.

As you've pulled the ISP change for v3.5, I'd like these 3 patches to get in 
v3.5 as well to avoid regressions.

> > ---
> > 
> > The following changes since commit 
e89fca923f32de26b69bf4cd604f7b960b161551:
> >   [media] gspca - ov534: Add Hue control (2012-05-14 09:48:00 -0300)
> > 
> > are available in the git repository at:
> >   ssh://linuxtv.org/git/sailus/media_tree.git media-for-3.5
> > 
> > Laurent Pinchart (3):
> >       mt9t001: Implement V4L2_CID_PIXEL_RATE control
> >       mt9p031: Implement V4L2_CID_PIXEL_RATE control
> >       mt9m032: Implement V4L2_CID_PIXEL_RATE control
> >  
> >  drivers/media/video/mt9m032.c |   13 +++++++++++--
> >  drivers/media/video/mt9p031.c |    5 ++++-
> >  drivers/media/video/mt9t001.c |   13 +++++++++++--
> >  include/media/mt9t001.h       |    1 +
> >  4 files changed, 27 insertions(+), 5 deletions(-)

-- 
Regards,

Laurent Pinchart


      reply	other threads:[~2012-05-14 23:11 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-14 15:56 [GIT PULL FOR v3.5] Implement V4L2_CID_PIXEL_RATE in various drivers Sakari Ailus
2012-05-14 16:01 ` Mauro Carvalho Chehab
2012-05-14 23:11   ` Laurent Pinchart [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=1654469.oil1fXEATY@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=sakari.ailus@iki.fi \
    /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.