public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Sakari Ailus <sakari.ailus@iki.fi>
Cc: linux-media@vger.kernel.org, laurent.pinchart@ideasonboard.com,
	hverkuil@xs4all.nl
Subject: Re: [GIT PULL FOR 3.1] Bitmask controls, flash API and adp1653 driver
Date: Thu, 30 Jun 2011 22:27:10 -0300	[thread overview]
Message-ID: <4E0D226E.5010809@redhat.com> (raw)
In-Reply-To: <20110610092703.GH7830@valkosipuli.localdomain>

Em 10-06-2011 06:27, Sakari Ailus escreveu:
> Hi Mauro,
> 
> This pull request adds the bitmask controls, flash API and the adp1653
> driver. What has changed since the patches is:
> 
> - Adp1653 flash faults control is volatile. Fix this.
> - Flash interface marked as experimental.
> - Moved the DocBook documentation to a new location.
> - The target version is 3.1, not 2.6.41.
> 
> The following changes since commit 75125b9d44456e0cf2d1fbb72ae33c13415299d1:
> 
>   [media] DocBook: Don't be noisy at make cleanmediadocs (2011-06-09 16:40:58 -0300)
> 
> are available in the git repository at:
>   ssh://linuxtv.org/git/sailus/media_tree.git media-for-3.1
> 
> Hans Verkuil (3):
>       v4l2-ctrls: add new bitmask control type.
>       vivi: add bitmask test control.
>       DocBook: document V4L2_CTRL_TYPE_BITMASK.

I'm sure I've already mentioned, but I think it was at the Hans pull request:
the specs don't mention what endiannes is needed for the bitmask controls: 
machine endianess, little endian or big endian.  IMO, we should stick with either
LE or BE.

> 
> Sakari Ailus (3):
>       v4l: Add a class and a set of controls for flash devices.
>       v4l: Add flash control documentation
>       adp1653: Add driver for LED flash controller
> 
>  Documentation/DocBook/media/v4l/compat.xml         |   11 +
>  Documentation/DocBook/media/v4l/controls.xml       |  283 ++++++++++++
>  Documentation/DocBook/media/v4l/v4l2.xml           |    9 +-
>  .../DocBook/media/v4l/vidioc-g-ext-ctrls.xml       |    7 +
>  .../DocBook/media/v4l/vidioc-queryctrl.xml         |   12 +-
>  drivers/media/video/Kconfig                        |    9 +
>  drivers/media/video/Makefile                       |    1 +
>  drivers/media/video/adp1653.c                      |  485 ++++++++++++++++++++
>  drivers/media/video/v4l2-common.c                  |    3 +
>  drivers/media/video/v4l2-ctrls.c                   |   62 +++-
>  drivers/media/video/vivi.c                         |   18 +-
>  include/linux/videodev2.h                          |   37 ++
>  include/media/adp1653.h                            |  126 +++++
>  13 files changed, 1058 insertions(+), 5 deletions(-)
>  create mode 100644 drivers/media/video/adp1653.c
>  create mode 100644 include/media/adp1653.h
> 


  reply	other threads:[~2011-07-01  1:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-10  9:27 [GIT PULL FOR 3.1] Bitmask controls, flash API and adp1653 driver Sakari Ailus
2011-07-01  1:27 ` Mauro Carvalho Chehab [this message]
2011-07-01  7:57   ` Hans Verkuil
2011-07-01  8:20     ` Sakari Ailus
2011-07-01 11:15     ` Sakari Ailus
2011-07-01 11:42       ` Mauro Carvalho Chehab
2011-07-02 18:03         ` Sakari Ailus
2011-07-02 18:29           ` Mauro Carvalho Chehab
  -- strict thread matches above, loose matches on Subject: below --
2011-07-05 11:20 Sakari Ailus
2011-07-06 12:38 Sakari Ailus

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=4E0D226E.5010809@redhat.com \
    --to=mchehab@redhat.com \
    --cc=hverkuil@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox