lkml.org 
[lkml]   [2022]   [Apr]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.17 1104/1126] media: ov6650: Add try support to selection API operations
    Date
    From: Janusz Krzysztofik <jmkrzyszt@gmail.com>

    commit c74052646496ffe0bc606152e6b9653137020cbf upstream.

    Try requests are now only supported by format processing pad operations
    implemented by the driver. The driver selection API operations
    currently respond to them with -EINVAL. While that is correct, it
    constraints video device drivers to not use subdevice cropping at all
    while processing user requested active frame size, otherwise their set
    try format results might differ from active. As a consequence, we
    can't fix set format pad operation as not to touch crop rectangle since
    that would affect users not being able to set arbitrary frame sizes.
    Moreover, without a working set try selection support we are not able
    to use pad config crop rectangle as a reference while processing set
    try format requests.

    Implement missing try selection support. Moreover, as it will be now
    possible to maintain the pad config crop rectangle via selection API,
    start using it instead of the active one as a reference while
    processing set try format requests.

    is_unscaled_ok() helper, now also called from set selection operation,
    has been just moved up in the source file to avoid a prototype, with no
    functional changes.

    [Sakari Ailus: Rebase on subdev state patches]

    Fixes: 717fd5b4907a ("[media] v4l2: replace try_mbus_fmt by set_fmt")
    Signed-off-by: Janusz Krzysztofik <jmkrzyszt@gmail.com>
    Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
    Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/media/i2c/ov6650.c | 54 +++++++++++++++++++++++++++++++++++----------
    1 file changed, 43 insertions(+), 11 deletions(-)

    --- a/drivers/media/i2c/ov6650.c
    +++ b/drivers/media/i2c/ov6650.c
    @@ -472,9 +472,16 @@ static int ov6650_get_selection(struct v
    {
    struct i2c_client *client = v4l2_get_subdevdata(sd);
    struct ov6650 *priv = to_ov6650(client);
    + struct v4l2_rect *rect;

    - if (sel->which != V4L2_SUBDEV_FORMAT_ACTIVE)
    - return -EINVAL;
    + if (sel->which == V4L2_SUBDEV_FORMAT_TRY) {
    + /* pre-select try crop rectangle */
    + rect = &sd_state->pads->try_crop;
    +
    + } else {
    + /* pre-select active crop rectangle */
    + rect = &priv->rect;
    + }

    switch (sel->target) {
    case V4L2_SEL_TGT_CROP_BOUNDS:
    @@ -483,14 +490,22 @@ static int ov6650_get_selection(struct v
    sel->r.width = W_CIF;
    sel->r.height = H_CIF;
    return 0;
    +
    case V4L2_SEL_TGT_CROP:
    - sel->r = priv->rect;
    + /* use selected crop rectangle */
    + sel->r = *rect;
    return 0;
    +
    default:
    return -EINVAL;
    }
    }

    +static bool is_unscaled_ok(int width, int height, struct v4l2_rect *rect)
    +{
    + return width > rect->width >> 1 || height > rect->height >> 1;
    +}
    +
    static void ov6650_bind_align_crop_rectangle(struct v4l2_rect *rect)
    {
    v4l_bound_align_image(&rect->width, 2, W_CIF, 1,
    @@ -510,12 +525,30 @@ static int ov6650_set_selection(struct v
    struct ov6650 *priv = to_ov6650(client);
    int ret;

    - if (sel->which != V4L2_SUBDEV_FORMAT_ACTIVE ||
    - sel->target != V4L2_SEL_TGT_CROP)
    + if (sel->target != V4L2_SEL_TGT_CROP)
    return -EINVAL;

    ov6650_bind_align_crop_rectangle(&sel->r);

    + if (sel->which == V4L2_SUBDEV_FORMAT_TRY) {
    + struct v4l2_rect *crop = &sd_state->pads->try_crop;
    + struct v4l2_mbus_framefmt *mf = &sd_state->pads->try_fmt;
    + /* detect current pad config scaling factor */
    + bool half_scale = !is_unscaled_ok(mf->width, mf->height, crop);
    +
    + /* store new crop rectangle */
    + *crop = sel->r;
    +
    + /* adjust frame size */
    + mf->width = crop->width >> half_scale;
    + mf->height = crop->height >> half_scale;
    +
    + return 0;
    + }
    +
    + /* V4L2_SUBDEV_FORMAT_ACTIVE */
    +
    + /* apply new crop rectangle */
    ret = ov6650_reg_write(client, REG_HSTRT, sel->r.left >> 1);
    if (!ret) {
    priv->rect.width += priv->rect.left - sel->r.left;
    @@ -567,11 +600,6 @@ static int ov6650_get_fmt(struct v4l2_su
    return 0;
    }

    -static bool is_unscaled_ok(int width, int height, struct v4l2_rect *rect)
    -{
    - return width > rect->width >> 1 || height > rect->height >> 1;
    -}
    -
    #define to_clkrc(div) ((div) - 1)

    /* set the format we will capture in */
    @@ -692,7 +720,11 @@ static int ov6650_set_fmt(struct v4l2_su
    break;
    }

    - *crop = priv->rect;
    + if (format->which == V4L2_SUBDEV_FORMAT_TRY)
    + *crop = sd_state->pads->try_crop;
    + else
    + *crop = priv->rect;
    +
    half_scale = !is_unscaled_ok(mf->width, mf->height, crop);

    /* adjust new crop rectangle position against its current center */

    \
     
     \ /
      Last update: 2022-09-17 16:20    [W:3.390 / U:0.044 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site