* [GIT PULL FOR 3.5] V4L2 and V4L2 subdev selection target and flag changes
@ 2012-06-15 18:36 Sakari Ailus
2012-06-25 16:51 ` [PATCH] [media] Schedule the selections API compatibility definitions for removal Sylwester Nawrocki
0 siblings, 1 reply; 5+ messages in thread
From: Sakari Ailus @ 2012-06-15 18:36 UTC (permalink / raw)
To: linux-media
Cc: Tomasz Stanislawski, Laurent Pinchart, Hans Verkuil, Sylwester Nawrocki
Hi Mauro,
This patchset contains changes to unify the selection targets and flags
across the V4L2 and V4L2 subdev interfaces, as well as a fix to
conflicting information in the documentation.
Since the patchset (v4) I've addressed Laurent's comments:
- Fixed badly formatted xref references
- Spelling corrections and references to wrong defines
- Always tell KEEP_CONFIG will not change any other processing steps
than the one it applies to
The following changes since commit 5472d3f17845c4398c6a510b46855820920c2181:
[media] mt9m032: Implement V4L2_CID_PIXEL_RATE control (2012-05-24
09:27:24 -0300)
are available in the git repository at:
ssh://linuxtv.org/git/sailus/media_tree.git media-for-3.5-unify-targets
Sakari Ailus (6):
v4l: Remove "_ACTUAL" from subdev selection API target definition
names
v4l: Unify selection targets across V4L2 and V4L2 subdev interfaces
v4l: Common documentation for selection targets
v4l: Unify selection flags
v4l: Unify selection flags documentation
v4l: Correct conflicting V4L2 subdev selection API documentation
Sylwester Nawrocki (1):
V4L: Remove "_ACTIVE" from the selection target name definitions
Documentation/DocBook/media/v4l/compat.xml | 9 +-
Documentation/DocBook/media/v4l/dev-subdev.xml | 36 ++--
Documentation/DocBook/media/v4l/selection-api.xml | 34 ++--
.../DocBook/media/v4l/selections-common.xml | 164
++++++++++++++++++++
Documentation/DocBook/media/v4l/v4l2.xml | 5 +
.../DocBook/media/v4l/vidioc-g-selection.xml | 86 ++---------
.../media/v4l/vidioc-subdev-g-selection.xml | 79 +---------
drivers/media/video/omap3isp/ispccdc.c | 8 +-
drivers/media/video/omap3isp/isppreview.c | 8 +-
drivers/media/video/omap3isp/ispresizer.c | 6 +-
drivers/media/video/s5p-fimc/fimc-capture.c | 32 ++--
drivers/media/video/s5p-fimc/fimc-lite.c | 15 +-
drivers/media/video/s5p-jpeg/jpeg-core.c | 4 +-
drivers/media/video/s5p-tv/mixer_video.c | 8 +-
drivers/media/video/smiapp/smiapp-core.c | 40 +++---
drivers/media/video/v4l2-ioctl.c | 8 +-
drivers/media/video/v4l2-subdev.c | 4 +-
include/linux/v4l2-common.h | 58 +++++++
include/linux/v4l2-subdev.h | 20 +--
include/linux/videodev2.h | 27 +---
20 files changed, 359 insertions(+), 292 deletions(-)
create mode 100644 Documentation/DocBook/media/v4l/selections-common.xml
create mode 100644 include/linux/v4l2-common.h
Kind regards,
--
Sakari Ailus
sakari.ailus@iki.fi
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] [media] Schedule the selections API compatibility definitions for removal
2012-06-15 18:36 [GIT PULL FOR 3.5] V4L2 and V4L2 subdev selection target and flag changes Sakari Ailus
@ 2012-06-25 16:51 ` Sylwester Nawrocki
2012-06-25 16:59 ` Sylwester Nawrocki
2012-06-25 19:14 ` [PATCH v2] [media] Feature removal: V4L2 selections API target and flag definitions Sylwester Nawrocki
0 siblings, 2 replies; 5+ messages in thread
From: Sylwester Nawrocki @ 2012-06-25 16:51 UTC (permalink / raw)
To: sakari.ailus, mchehab; +Cc: linux-media, Sylwester Nawrocki
Signed-off-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
---
Documentation/feature-removal-schedule.txt | 15 +++++++++++++++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index 09701af..ef9f942 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -558,3 +558,18 @@ Why: The V4L2_CID_VCENTER, V4L2_CID_HCENTER controls have been deprecated
There are newer controls (V4L2_CID_PAN*, V4L2_CID_TILT*) that provide
similar functionality.
Who: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
+
+----------------------------
+
+What: Remove the backward compatibility V4L2 selections target and selections
+ flags definitions
+When: 3.8
+Why: The regular V4L2 selections and the subdev selection API originally
+ defined distinct names for the target rectangles and flags - V4L2_SEL_*
+ and V4L2_SUBDEV_SEL_*. Although, it turned out that the meaning of these
+ target rectangles is virtually identical and the APIs were consolidated
+ to use single set of names - V4L2_SEL_*. This consolidation didn't
+ change the ABI in any way. Alias definitions were created for the
+ original ones to avoid any instabilities in the user space interface.
+ After few cycles these comptibility definitions will be removed.
+Who: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
--
1.7.4.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] [media] Schedule the selections API compatibility definitions for removal
2012-06-25 16:51 ` [PATCH] [media] Schedule the selections API compatibility definitions for removal Sylwester Nawrocki
@ 2012-06-25 16:59 ` Sylwester Nawrocki
2012-06-25 19:14 ` [PATCH v2] [media] Feature removal: V4L2 selections API target and flag definitions Sylwester Nawrocki
1 sibling, 0 replies; 5+ messages in thread
From: Sylwester Nawrocki @ 2012-06-25 16:59 UTC (permalink / raw)
To: sakari.ailus; +Cc: Sylwester Nawrocki, mchehab, linux-media
On 06/25/2012 06:51 PM, Sylwester Nawrocki wrote:
> Signed-off-by: Sylwester Nawrocki<sylvester.nawrocki@gmail.com>
> ---
Hi Sakari,
Let me add an explanation that was supposed to be originally included
in that patch..
Here is the patch for Documentation/feature-removal-schedule.txt that
is mentioned in the description of the first patch in this series.
This change set looks good to me, except there seem to be missing
compatibility definitions for the selections flags. I presume we
are going to need those since the original subdev selections API
will exist in v3.5 kernels, and this change set is going to be
applied only to v3.6.
So something like that might be needed:
#define V4L2_SUBDEV_SEL_FLAG_SIZE_GE V4L2_SEL_FLAG_SIZE_GE
#define V4L2_SUBDEV_SEL_FLAG_SIZE_LE V4L2_SEL_FLAG_SIZE_LE
#define V4L2_SUBDEV_SEL_FLAG_KEEP_CONFIG V4L2_SEL_FLAG_KEEP_CONFIG
After adding that this series seems good for merging to me.
Thanks and regards,
Sylwester
> Documentation/feature-removal-schedule.txt | 15 +++++++++++++++
> 1 files changed, 15 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
> index 09701af..ef9f942 100644
> --- a/Documentation/feature-removal-schedule.txt
> +++ b/Documentation/feature-removal-schedule.txt
> @@ -558,3 +558,18 @@ Why: The V4L2_CID_VCENTER, V4L2_CID_HCENTER controls have been deprecated
> There are newer controls (V4L2_CID_PAN*, V4L2_CID_TILT*) that provide
> similar functionality.
> Who: Sylwester Nawrocki<sylvester.nawrocki@gmail.com>
> +
> +----------------------------
> +
> +What: Remove the backward compatibility V4L2 selections target and selections
> + flags definitions
> +When: 3.8
> +Why: The regular V4L2 selections and the subdev selection API originally
> + defined distinct names for the target rectangles and flags - V4L2_SEL_*
> + and V4L2_SUBDEV_SEL_*. Although, it turned out that the meaning of these
> + target rectangles is virtually identical and the APIs were consolidated
> + to use single set of names - V4L2_SEL_*. This consolidation didn't
> + change the ABI in any way. Alias definitions were created for the
> + original ones to avoid any instabilities in the user space interface.
> + After few cycles these comptibility definitions will be removed.
> +Who: Sylwester Nawrocki<sylvester.nawrocki@gmail.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2] [media] Feature removal: V4L2 selections API target and flag definitions
2012-06-25 16:51 ` [PATCH] [media] Schedule the selections API compatibility definitions for removal Sylwester Nawrocki
2012-06-25 16:59 ` Sylwester Nawrocki
@ 2012-06-25 19:14 ` Sylwester Nawrocki
2012-06-30 16:46 ` Sakari Ailus
1 sibling, 1 reply; 5+ messages in thread
From: Sylwester Nawrocki @ 2012-06-25 19:14 UTC (permalink / raw)
To: sakari.ailus, mchehab; +Cc: linux-media, Sylwester Nawrocki
Signed-off-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
---
Added more precise description of what is being removed.
---
Documentation/feature-removal-schedule.txt | 18 ++++++++++++++++++
1 files changed, 18 insertions(+), 0 deletions(-)
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index 09701af..b998030 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -558,3 +558,21 @@ Why: The V4L2_CID_VCENTER, V4L2_CID_HCENTER controls have been deprecated
There are newer controls (V4L2_CID_PAN*, V4L2_CID_TILT*) that provide
similar functionality.
Who: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
+
+----------------------------
+
+What: V4L2 selections API target rectangle and flags unification, the
+ following definitions will be removed: V4L2_SEL_TGT_CROP_ACTIVE,
+ V4L2_SEL_TGT_COMPOSE_ACTIVE, V4L2_SUBDEV_SEL_*, V4L2_SUBDEV_SEL_FLAG_*
+ in favor of common V4L2_SEL_TGT_* and V4L2_SEL_FLAG_* definitions.
+ For more details see include/linux/v4l2-common.h.
+When: 3.8
+Why: The regular V4L2 selections and the subdev selection API originally
+ defined distinct names for the target rectangles and flags - V4L2_SEL_*
+ and V4L2_SUBDEV_SEL_*. Although, it turned out that the meaning of these
+ target rectangles is virtually identical and the APIs were consolidated
+ to use single set of names - V4L2_SEL_*. This didn't involve any ABI
+ changes. Alias definitions were created for the original ones to avoid
+ any instabilities in the user space interface. After few cycles these
+ backward compatibility definitions will be removed.
+Who: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
--
1.7.4.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] [media] Feature removal: V4L2 selections API target and flag definitions
2012-06-25 19:14 ` [PATCH v2] [media] Feature removal: V4L2 selections API target and flag definitions Sylwester Nawrocki
@ 2012-06-30 16:46 ` Sakari Ailus
0 siblings, 0 replies; 5+ messages in thread
From: Sakari Ailus @ 2012-06-30 16:46 UTC (permalink / raw)
To: Sylwester Nawrocki; +Cc: mchehab, linux-media
On Mon, Jun 25, 2012 at 09:14:41PM +0200, Sylwester Nawrocki wrote:
> Signed-off-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
> ---
> Added more precise description of what is being removed.
> ---
> Documentation/feature-removal-schedule.txt | 18 ++++++++++++++++++
> 1 files changed, 18 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
> index 09701af..b998030 100644
> --- a/Documentation/feature-removal-schedule.txt
> +++ b/Documentation/feature-removal-schedule.txt
> @@ -558,3 +558,21 @@ Why: The V4L2_CID_VCENTER, V4L2_CID_HCENTER controls have been deprecated
> There are newer controls (V4L2_CID_PAN*, V4L2_CID_TILT*) that provide
> similar functionality.
> Who: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
> +
> +----------------------------
> +
> +What: V4L2 selections API target rectangle and flags unification, the
> + following definitions will be removed: V4L2_SEL_TGT_CROP_ACTIVE,
> + V4L2_SEL_TGT_COMPOSE_ACTIVE, V4L2_SUBDEV_SEL_*, V4L2_SUBDEV_SEL_FLAG_*
> + in favor of common V4L2_SEL_TGT_* and V4L2_SEL_FLAG_* definitions.
> + For more details see include/linux/v4l2-common.h.
> +When: 3.8
> +Why: The regular V4L2 selections and the subdev selection API originally
> + defined distinct names for the target rectangles and flags - V4L2_SEL_*
> + and V4L2_SUBDEV_SEL_*. Although, it turned out that the meaning of these
> + target rectangles is virtually identical and the APIs were consolidated
> + to use single set of names - V4L2_SEL_*. This didn't involve any ABI
> + changes. Alias definitions were created for the original ones to avoid
> + any instabilities in the user space interface. After few cycles these
> + backward compatibility definitions will be removed.
> +Who: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
Thanks, Sylwester!
I've added this to my patchset.
Cheers,
--
Sakari Ailus
e-mail: sakari.ailus@iki.fi jabber/XMPP/Gmail: sailus@retiisi.org.uk
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-06-30 16:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-15 18:36 [GIT PULL FOR 3.5] V4L2 and V4L2 subdev selection target and flag changes Sakari Ailus
2012-06-25 16:51 ` [PATCH] [media] Schedule the selections API compatibility definitions for removal Sylwester Nawrocki
2012-06-25 16:59 ` Sylwester Nawrocki
2012-06-25 19:14 ` [PATCH v2] [media] Feature removal: V4L2 selections API target and flag definitions Sylwester Nawrocki
2012-06-30 16:46 ` Sakari Ailus
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.