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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).