All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org, Jonathan Corbet <corbet@lwn.net>
Subject: Re: [RFC PATCH 2/5] v4l2-subdev: add (un)register internal ops
Date: Fri, 7 Jan 2011 14:37:16 +0100	[thread overview]
Message-ID: <201101071437.16632.laurent.pinchart@ideasonboard.com> (raw)
In-Reply-To: <2bce44c24896652e068d2c2a679e13c6bd820b65.1294402580.git.hverkuil@xs4all.nl>

Hi Hans,

Thanks for the patch.

On Friday 07 January 2011 13:47:32 Hans Verkuil wrote:
> Some subdevs need to call into the board code after they are registered
> and have a valid struct v4l2_device pointer. The s_config op was abused
> for this, but now that it is removed we need a cleaner way of solving this.
> 
> So this patch adds a struct with internal ops that the v4l2 core can call.
> 
> Currently only two ops exist: register and unregister. Subdevs can
> implement these to call the board code and pass it the v4l2_device
> pointer, which the board code can then use to get access to the struct
> that embeds the v4l2_device.
> 
> It is expected that in the future open and close ops will also be added.
> 
> Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

-- 
Regards,

Laurent Pinchart

  reply	other threads:[~2011-01-07 13:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-07 12:47 [RFC PATCH 0/5] Use control framework in cafe_ccic and s_config removal Hans Verkuil
2011-01-07 12:47 ` [RFC PATCH 1/5] v4l2-subdev: remove core.s_config and v4l2_i2c_new_subdev_cfg() Hans Verkuil
2011-01-07 12:47   ` [RFC PATCH 2/5] v4l2-subdev: add (un)register internal ops Hans Verkuil
2011-01-07 13:37     ` Laurent Pinchart [this message]
2011-01-07 12:47   ` [RFC PATCH 3/5] v4l2-ctrls: v4l2_ctrl_handler_setup must set has_new to 1 Hans Verkuil
2011-01-07 13:25     ` Laurent Pinchart
2011-01-07 21:22       ` Hans Verkuil
2011-01-07 12:47   ` [RFC PATCH 4/5] ov7670: use the control framework Hans Verkuil
2011-01-07 12:47   ` [RFC PATCH 5/5] cafe_ccic: implement " Hans Verkuil
2011-01-07 13:53   ` [RFC PATCH 1/5] v4l2-subdev: remove core.s_config and v4l2_i2c_new_subdev_cfg() Laurent Pinchart
2011-01-07 21:21     ` Hans Verkuil
2011-01-07 21:28       ` Laurent Pinchart
2011-01-07 22:29   ` Sylwester Nawrocki

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=201101071437.16632.laurent.pinchart@ideasonboard.com \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=corbet@lwn.net \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /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.