From: Sakari Ailus <sakari.ailus@iki.fi>
To: Tomasz Stanislawski <t.stanislaws@samsung.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
linux-media@vger.kernel.org, dacohen@gmail.com, snjw23@gmail.com,
andriy.shevchenko@linux.intel.com, tuukkat76@gmail.com,
k.debski@samsung.com, riverful@gmail.com, hverkuil@xs4all.nl,
teturtia@gmail.com
Subject: Re: [PATCH v4 04/34] v4l: VIDIOC_SUBDEV_S_SELECTION and VIDIOC_SUBDEV_G_SELECTION IOCTLs
Date: Tue, 06 Mar 2012 21:55:52 +0200 [thread overview]
Message-ID: <4F566BC8.9090400@iki.fi> (raw)
In-Reply-To: <4F563E02.7010406@samsung.com>
Hi Tomasz,
Tomasz Stanislawski wrote:
> On 03/06/2012 05:27 PM, Laurent Pinchart wrote:
>> On Tuesday 06 March 2012 17:50:36 Sakari Ailus wrote:
>>> On Mon, Mar 05, 2012 at 11:59:22AM +0100, Laurent Pinchart wrote:
>>>> On Friday 02 March 2012 19:30:12 Sakari Ailus wrote:
>>>>> Add support for VIDIOC_SUBDEV_S_SELECTION and VIDIOC_SUBDEV_G_SELECTION
>>>>> IOCTLs. They replace functionality provided by VIDIOC_SUBDEV_S_CROP and
>>>>> VIDIOC_SUBDEV_G_CROP IOCTLs and also add new functionality (composing).
>>>>>
>>>>> VIDIOC_SUBDEV_G_CROP and VIDIOC_SUBDEV_S_CROP continue to be supported.
>>>>>
>>>>> Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
>>>>
>>>> Except for the ACTIVE name,
>>>>
>>>> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>>>>
>>>> Maybe we could discuss this on IRC with Tomasz ?
>>>
>>> Tomasz wasn't online when I checked.
>>>
>>> How about "CURRENT"?
>>
>> Sounds good to me. Let's see if Tomasz will be online tomorrow ;-)
>>
> Hi,
>
> What do you think about ACTIVE -> ACTUAL?
> Similar word with the same number of letters :)
Sounds good to me; perhaps even better than CURRENT. I'm fine with
either with perhaps a small inclination towards "actual".
Laurent?
Cheers,
--
Sakari Ailus
sakari.ailus@iki.fi
next prev parent reply other threads:[~2012-03-06 19:56 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-02 17:32 [PATCH v4 0/34] V4L2 subdev and sensor control changes, SMIA++ driver and N9 camera board code Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 01/34] v4l: Introduce integer menu controls Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 02/34] v4l: Document " Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 03/34] vivi: Add an integer menu test control Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 04/34] v4l: VIDIOC_SUBDEV_S_SELECTION and VIDIOC_SUBDEV_G_SELECTION IOCTLs Sakari Ailus
2012-03-05 10:59 ` Laurent Pinchart
2012-03-06 15:50 ` Sakari Ailus
2012-03-06 16:27 ` Laurent Pinchart
2012-03-06 16:40 ` Tomasz Stanislawski
2012-03-06 19:55 ` Sakari Ailus [this message]
2012-03-06 19:57 ` Laurent Pinchart
2012-03-06 19:58 ` Sakari Ailus
2012-03-08 16:10 ` Tomasz Stanislawski
2012-03-02 17:30 ` [PATCH v4 05/34] v4l: vdev_to_v4l2_subdev() should have return type "struct v4l2_subdev *" Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 06/34] v4l: Check pad number in get try pointer functions Sakari Ailus
2012-03-05 11:01 ` Laurent Pinchart
2012-03-02 17:30 ` [PATCH v4 07/34] v4l: Support s_crop and g_crop through s/g_selection Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 08/34] v4l: Add subdev selections documentation: svg and dia files Sakari Ailus
2012-03-05 11:07 ` Laurent Pinchart
2012-03-06 15:09 ` Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 09/34] v4l: Add subdev selections documentation Sakari Ailus
2012-03-05 11:47 ` Laurent Pinchart
2012-03-06 12:08 ` Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 10/34] v4l: Mark VIDIOC_SUBDEV_G_CROP and VIDIOC_SUBDEV_S_CROP obsolete Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 11/34] v4l: Image source control class Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 12/34] v4l: Image processing " Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 13/34] v4l: Document raw bayer 4CC codes Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 14/34] v4l: Add DPCM compressed formats Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 15/34] media: Add link_validate() op to check links to the sink pad Sakari Ailus
2012-03-05 11:10 ` Laurent Pinchart
2012-03-02 17:30 ` [PATCH v4 16/34] media: Collect entities that are part of the pipeline before link validation Sakari Ailus
2012-03-05 11:13 ` Laurent Pinchart
2012-03-06 12:09 ` Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 17/34] v4l: Improve sub-device documentation for pad ops Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 18/34] v4l: Implement v4l2_subdev_link_validate() Sakari Ailus
2012-03-05 11:16 ` Laurent Pinchart
2012-03-02 17:30 ` [PATCH v4 19/34] v4l: Allow changing control handler lock Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 20/34] omap3isp: Support additional in-memory compressed bayer formats Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 21/34] omap3isp: Move definitions required by board code under include/media Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 22/34] omap3: add definition for CONTROL_CAMERA_PHY_CTRL Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 23/34] omap3isp: Move setting constaints above media_entity_pipeline_start Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 24/34] omap3isp: Assume media_entity_pipeline_start may fail Sakari Ailus
2012-03-05 11:26 ` Laurent Pinchart
2012-03-06 10:22 ` Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 25/34] omap3isp: Add lane configuration to platform data Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 26/34] omap3isp: Add information on external subdev to struct isp_pipeline Sakari Ailus
2012-03-05 11:28 ` Laurent Pinchart
2012-03-02 17:30 ` [PATCH v4 27/34] omap3isp: Introduce isp_video_check_external_subdevs() Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 28/34] omap3isp: Use external rate instead of vpcfg Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 29/34] omap3isp: Default link validation for ccp2, csi2, preview and resizer Sakari Ailus
2012-03-05 11:31 ` Laurent Pinchart
2012-03-02 17:30 ` [PATCH v4 30/34] omap3isp: Move CCDC link validation to ccdc_link_validate() Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 31/34] omap3isp: Configure CSI-2 phy based on platform data Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 32/34] omap3isp: Add resizer data rate configuration to resizer_link_validate Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 33/34] smiapp: Add driver Sakari Ailus
2012-03-02 17:30 ` [PATCH v4 34/34] rm680: Add camera init Sakari Ailus
2012-03-05 11:36 ` Laurent Pinchart
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=4F566BC8.9090400@iki.fi \
--to=sakari.ailus@iki.fi \
--cc=andriy.shevchenko@linux.intel.com \
--cc=dacohen@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=k.debski@samsung.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=riverful@gmail.com \
--cc=snjw23@gmail.com \
--cc=t.stanislaws@samsung.com \
--cc=teturtia@gmail.com \
--cc=tuukkat76@gmail.com \
/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.