All of lore.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 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.