[media] V4L: Remove "_ACTIVE" from the selection target name definitions

This patch drops the _ACTIVE part from the selection target names
as a prerequisite to unify the selection target names across the subdev
and regular video node API.

The meaning of V4L2_SEL_TGT_*_ACTIVE and V4L2_SUBDEV_SEL_TGT_*_ACTUAL
selection targets is logically the same. Different names add to confusion
where both APIs are used in a single driver or an application. For some
system configurations different names may lead to interoperability issues.

For backwards compatibility V4L2_SEL_TGT_CROP_ACTIVE and
V4L2_SEL_TGT_COMPOSE_ACTIVE are defined as aliases to V4L2_SEL_TGT_CROP
and V4L2_SEL_TGT_COMPOSE. These aliases will be removed after deprecation
period, according to Documentation/feature-removal-schedule.txt.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

1 files changed, 7 insertions, 7 deletions

diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-fimc/fimc-capture.c
index 62ce5399c4cf..a3cd78d33913 100644
--- a/drivers/media/video/s5p-fimc/fimc-capture.c
+++ b/drivers/media/video/s5p-fimc/fimc-capture.c
@@ -655,7 +655,7 @@ static void fimc_capture_try_selection(struct fimc_ctx *ctx,
                 r->left   = r->top = 0;
                 return;
         }
-        if (target == V4L2_SEL_TGT_COMPOSE_ACTIVE) {
+        if (target == V4L2_SEL_TGT_COMPOSE) {
                 if (ctx->rotation != 90 && ctx->rotation != 270)
                         align_h = 1;
                 max_sc_h = min(SCALER_MAX_HRATIO, 1 << (ffs(sink->width) - 3));
@@ -682,7 +682,7 @@ static void fimc_capture_try_selection(struct fimc_ctx *ctx,
                       rotate ? sink->f_height : sink->f_width);
         max_h = min_t(u32, FIMC_CAMIF_MAX_HEIGHT, sink->f_height);
 
-        if (target == V4L2_SEL_TGT_COMPOSE_ACTIVE) {
+        if (target == V4L2_SEL_TGT_COMPOSE) {
                 min_w = min_t(u32, max_w, sink->f_width / max_sc_h);
                 min_h = min_t(u32, max_h, sink->f_height / max_sc_v);
                 if (rotate) {
@@ -1147,9 +1147,9 @@ static int fimc_cap_g_selection(struct file *file, void *fh,
                 s->r.height = f->o_height;
                 return 0;
 
-        case V4L2_SEL_TGT_COMPOSE_ACTIVE:
+        case V4L2_SEL_TGT_COMPOSE:
                 f = &ctx->d_frame;
-        case V4L2_SEL_TGT_CROP_ACTIVE:
+        case V4L2_SEL_TGT_CROP:
                 s->r.left = f->offs_h;
                 s->r.top = f->offs_v;
                 s->r.width = f->width;
@@ -1185,9 +1185,9 @@ static int fimc_cap_s_selection(struct file *file, void *fh,
         if (s->type != V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE)
                 return -EINVAL;
 
-        if (s->target == V4L2_SEL_TGT_COMPOSE_ACTIVE)
+        if (s->target == V4L2_SEL_TGT_COMPOSE)
                 f = &ctx->d_frame;
-        else if (s->target == V4L2_SEL_TGT_CROP_ACTIVE)
+        else if (s->target == V4L2_SEL_TGT_CROP)
                 f = &ctx->s_frame;
         else
                 return -EINVAL;
@@ -1483,7 +1483,7 @@ static int fimc_subdev_set_selection(struct v4l2_subdev *sd,
                 return -EINVAL;
 
         mutex_lock(&fimc->lock);
-        fimc_capture_try_selection(ctx, r, V4L2_SEL_TGT_CROP_ACTIVE);
+        fimc_capture_try_selection(ctx, r, V4L2_SEL_TGT_CROP);
 
         switch (sel->target) {
         case V4L2_SUBDEV_SEL_TGT_COMPOSE_BOUNDS: