Hi Sakari,
Thank you for the patch.
On Fri, Sep 22, 2023 at 02:57:24PM +0300, Sakari Ailus wrote:
> Make use of sub-device active state. In most cases the effect on need for
> acquiring the mutex is non-existent as access to the driver's core data
> structure still needs to be serialised.
Will this change when you drop
const struct ccs_csi_data_format *csi_format;
const struct ccs_csi_data_format *internal_csi_format;
struct v4l2_rect pa_src, scaler_sink, src_src;
from struct ccs_sensor ?
> This still removes a lot of code as the code paths for active and try
> state are the same in many cases.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> ---
> drivers/media/i2c/ccs/ccs-core.c | 278 ++++++++++++-------------------
> drivers/media/i2c/ccs/ccs.h | 4 +-
> 2 files changed, 103 insertions(+), 179 deletions(-)
>
> diff --git a/drivers/media/i2c/ccs/ccs-core.c b/drivers/media/i2c/ccs/ccs-core.c
> index a4f593866ad6..7c53bbda5b8d 100644
> --- a/drivers/media/i2c/ccs/ccs-core.c
> +++ b/drivers/media/i2c/ccs/ccs-core.c
> @@ -508,9 +508,8 @@ static void __ccs_update_exposure_limits(struct ccs_sensor *sensor)
> struct v4l2_ctrl *ctrl = sensor->exposure;
> int max;
>
> - max = sensor->pixel_array->crop[CCS_PA_PAD_SRC].height
> - + sensor->vblank->val
> - - CCS_LIM(sensor, COARSE_INTEGRATION_TIME_MAX_MARGIN);
> + max = sensor->pa_src.height + sensor->vblank->val -
> + CCS_LIM(sensor, COARSE_INTEGRATION_TIME_MAX_MARGIN);
>
> __v4l2_ctrl_modify_range(ctrl, ctrl->minimum, max, ctrl->step, max);
> }
> @@ -728,15 +727,12 @@ static int ccs_set_ctrl(struct v4l2_ctrl *ctrl)
> break;
> case V4L2_CID_VBLANK:
> rval = ccs_write(sensor, FRAME_LENGTH_LINES,
> - sensor->pixel_array->crop[
> - CCS_PA_PAD_SRC].height
> - + ctrl->val);
> + sensor->pa_src.height + ctrl->val);
>
> break;
> case V4L2_CID_HBLANK:
> rval = ccs_write(sensor, LINE_LENGTH_PCK,
> - sensor->pixel_array->crop[CCS_PA_PAD_SRC].width
> - + ctrl->val);
> + sensor->pa_src.width + ctrl->val);
>
> break;
> case V4L2_CID_TEST_PATTERN:
> @@ -1214,15 +1210,13 @@ static void ccs_update_blanking(struct ccs_sensor *sensor)
>
> min = max_t(int,
> CCS_LIM(sensor, MIN_FRAME_BLANKING_LINES),
> - min_fll - sensor->pixel_array->crop[CCS_PA_PAD_SRC].height);
> - max = max_fll - sensor->pixel_array->crop[CCS_PA_PAD_SRC].height;
> + min_fll - sensor->pa_src.height);
> + max = max_fll - sensor->pa_src.height;
>
> __v4l2_ctrl_modify_range(vblank, min, max, vblank->step, min);
>
> - min = max_t(int,
> - min_llp - sensor->pixel_array->crop[CCS_PA_PAD_SRC].width,
> - min_lbp);
> - max = max_llp - sensor->pixel_array->crop[CCS_PA_PAD_SRC].width;
> + min = max_t(int, min_llp - sensor->pa_src.width, min_lbp);
> + max = max_llp - sensor->pa_src.width;
>
> __v4l2_ctrl_modify_range(hblank, min, max, hblank->step, min);
>
> @@ -1246,10 +1240,8 @@ static int ccs_pll_blanking_update(struct ccs_sensor *sensor)
>
> dev_dbg(&client->dev, "real timeperframe\t100/%d\n",
> sensor->pll.pixel_rate_pixel_array /
> - ((sensor->pixel_array->crop[CCS_PA_PAD_SRC].width
> - + sensor->hblank->val) *
> - (sensor->pixel_array->crop[CCS_PA_PAD_SRC].height
> - + sensor->vblank->val) / 100));
> + ((sensor->pa_src.width + sensor->hblank->val) *
> + (sensor->pa_src.height + sensor->vblank->val) / 100));
>
> return 0;
> }
> @@ -1756,28 +1748,22 @@ static int ccs_start_streaming(struct ccs_sensor *sensor)
> goto out;
>
> /* Analog crop start coordinates */
> - rval = ccs_write(sensor, X_ADDR_START,
> - sensor->pixel_array->crop[CCS_PA_PAD_SRC].left);
> + rval = ccs_write(sensor, X_ADDR_START, sensor->pa_src.left);
> if (rval < 0)
> goto out;
>
> - rval = ccs_write(sensor, Y_ADDR_START,
> - sensor->pixel_array->crop[CCS_PA_PAD_SRC].top);
> + rval = ccs_write(sensor, Y_ADDR_START, sensor->pa_src.top);
> if (rval < 0)
> goto out;
>
> /* Analog crop end coordinates */
> - rval = ccs_write(
> - sensor, X_ADDR_END,
> - sensor->pixel_array->crop[CCS_PA_PAD_SRC].left
> - + sensor->pixel_array->crop[CCS_PA_PAD_SRC].width - 1);
> + rval = ccs_write(sensor, X_ADDR_END,
> + sensor->pa_src.left + sensor->pa_src.width - 1);
> if (rval < 0)
> goto out;
>
> - rval = ccs_write(
> - sensor, Y_ADDR_END,
> - sensor->pixel_array->crop[CCS_PA_PAD_SRC].top
> - + sensor->pixel_array->crop[CCS_PA_PAD_SRC].height - 1);
> + rval = ccs_write(sensor, Y_ADDR_END,
> + sensor->pa_src.top + sensor->pa_src.height - 1);
> if (rval < 0)
> goto out;
>
> @@ -1789,27 +1775,23 @@ static int ccs_start_streaming(struct ccs_sensor *sensor)
> /* Digital crop */
> if (CCS_LIM(sensor, DIGITAL_CROP_CAPABILITY)
> == CCS_DIGITAL_CROP_CAPABILITY_INPUT_CROP) {
> - rval = ccs_write(
> - sensor, DIGITAL_CROP_X_OFFSET,
> - sensor->scaler->crop[CCS_PAD_SINK].left);
> + rval = ccs_write(sensor, DIGITAL_CROP_X_OFFSET,
> + sensor->scaler_sink.left);
> if (rval < 0)
> goto out;
>
> - rval = ccs_write(
> - sensor, DIGITAL_CROP_Y_OFFSET,
> - sensor->scaler->crop[CCS_PAD_SINK].top);
> + rval = ccs_write(sensor, DIGITAL_CROP_Y_OFFSET,
> + sensor->scaler_sink.top);
> if (rval < 0)
> goto out;
>
> - rval = ccs_write(
> - sensor, DIGITAL_CROP_IMAGE_WIDTH,
> - sensor->scaler->crop[CCS_PAD_SINK].width);
> + rval = ccs_write(sensor, DIGITAL_CROP_IMAGE_WIDTH,
> + sensor->scaler_sink.width);
> if (rval < 0)
> goto out;
>
> - rval = ccs_write(
> - sensor, DIGITAL_CROP_IMAGE_HEIGHT,
> - sensor->scaler->crop[CCS_PAD_SINK].height);
> + rval = ccs_write(sensor, DIGITAL_CROP_IMAGE_HEIGHT,
> + sensor->scaler_sink.height);
> if (rval < 0)
> goto out;
> }
> @@ -1827,12 +1809,10 @@ static int ccs_start_streaming(struct ccs_sensor *sensor)
> }
>
> /* Output size from sensor */
> - rval = ccs_write(sensor, X_OUTPUT_SIZE,
> - sensor->src->crop[CCS_PAD_SRC].width);
> + rval = ccs_write(sensor, X_OUTPUT_SIZE, sensor->src_src.width);
> if (rval < 0)
> goto out;
> - rval = ccs_write(sensor, Y_OUTPUT_SIZE,
> - sensor->src->crop[CCS_PAD_SRC].height);
> + rval = ccs_write(sensor, Y_OUTPUT_SIZE, sensor->src_src.height);
> if (rval < 0)
> goto out;
>
> @@ -2053,24 +2033,8 @@ static int __ccs_get_format(struct v4l2_subdev *subdev,
> struct v4l2_subdev_state *sd_state,
> struct v4l2_subdev_format *fmt)
> {
> - struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> -
> - if (fmt->which == V4L2_SUBDEV_FORMAT_TRY) {
> - fmt->format = *v4l2_subdev_get_try_format(subdev, sd_state,
> - fmt->pad);
> - } else {
> - struct v4l2_rect *r;
> -
> - if (fmt->pad == ssd->source_pad)
> - r = &ssd->crop[ssd->source_pad];
> - else
> - r = &ssd->sink_fmt;
> -
> - fmt->format.code = __ccs_get_mbus_code(subdev, fmt->pad);
> - fmt->format.width = r->width;
> - fmt->format.height = r->height;
> - fmt->format.field = V4L2_FIELD_NONE;
> - }
> + fmt->format = *v4l2_subdev_get_pad_format(subdev, sd_state, fmt->pad);
> + fmt->format.code = __ccs_get_mbus_code(subdev, fmt->pad);
>
> return 0;
> }
> @@ -2092,28 +2056,18 @@ static int ccs_get_format(struct v4l2_subdev *subdev,
> static void ccs_get_crop_compose(struct v4l2_subdev *subdev,
> struct v4l2_subdev_state *sd_state,
> struct v4l2_rect **crops,
> - struct v4l2_rect **comps, int which)
> + struct v4l2_rect **comps)
> {
> struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> unsigned int i;
>
> - if (which == V4L2_SUBDEV_FORMAT_ACTIVE) {
> - if (crops)
> - for (i = 0; i < subdev->entity.num_pads; i++)
> - crops[i] = &ssd->crop[i];
> - if (comps)
> - *comps = &ssd->compose;
> - } else {
> - if (crops) {
> - for (i = 0; i < subdev->entity.num_pads; i++)
> - crops[i] = v4l2_subdev_get_try_crop(subdev,
> - sd_state,
> - i);
> - }
> - if (comps)
> - *comps = v4l2_subdev_get_try_compose(subdev, sd_state,
> - CCS_PAD_SINK);
> - }
> + if (crops)
> + for (i = 0; i < subdev->entity.num_pads; i++)
> + crops[i] =
> + v4l2_subdev_get_pad_crop(subdev, sd_state, i);
> + if (comps)
> + *comps = v4l2_subdev_get_pad_compose(subdev, sd_state,
> + ssd->sink_pad);
> }
>
> /* Changes require propagation only on sink pad. */
> @@ -2125,7 +2079,7 @@ static void ccs_propagate(struct v4l2_subdev *subdev,
> struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> struct v4l2_rect *comp, *crops[CCS_PADS];
>
> - ccs_get_crop_compose(subdev, sd_state, crops, &comp, which);
> + ccs_get_crop_compose(subdev, sd_state, crops, &comp);
>
> switch (target) {
> case V4L2_SEL_TGT_CROP:
> @@ -2136,6 +2090,7 @@ static void ccs_propagate(struct v4l2_subdev *subdev,
> sensor->scale_m = CCS_LIM(sensor, SCALER_N_MIN);
> sensor->scaling_mode =
> CCS_SCALING_MODE_NO_SCALING;
> + sensor->scaler_sink = *comp;
> } else if (ssd == sensor->binner) {
> sensor->binning_horizontal = 1;
> sensor->binning_vertical = 1;
> @@ -2144,6 +2099,8 @@ static void ccs_propagate(struct v4l2_subdev *subdev,
> fallthrough;
> case V4L2_SEL_TGT_COMPOSE:
> *crops[CCS_PAD_SRC] = *comp;
> + if (which == V4L2_SUBDEV_FORMAT_ACTIVE && ssd == sensor->src)
> + sensor->src_src = *crops[CCS_PAD_SRC];
> break;
> default:
> WARN_ON_ONCE(1);
> @@ -2252,14 +2209,12 @@ static int ccs_set_format(struct v4l2_subdev *subdev,
> CCS_LIM(sensor, MIN_Y_OUTPUT_SIZE),
> CCS_LIM(sensor, MAX_Y_OUTPUT_SIZE));
>
> - ccs_get_crop_compose(subdev, sd_state, crops, NULL, fmt->which);
> + ccs_get_crop_compose(subdev, sd_state, crops, NULL);
>
> crops[ssd->sink_pad]->left = 0;
> crops[ssd->sink_pad]->top = 0;
> crops[ssd->sink_pad]->width = fmt->format.width;
> crops[ssd->sink_pad]->height = fmt->format.height;
> - if (fmt->which == V4L2_SUBDEV_FORMAT_ACTIVE)
> - ssd->sink_fmt = *crops[ssd->sink_pad];
> ccs_propagate(subdev, sd_state, fmt->which, V4L2_SEL_TGT_CROP);
>
> mutex_unlock(&sensor->mutex);
> @@ -2482,7 +2437,7 @@ static int ccs_set_compose(struct v4l2_subdev *subdev,
> struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> struct v4l2_rect *comp, *crops[CCS_PADS];
>
> - ccs_get_crop_compose(subdev, sd_state, crops, &comp, sel->which);
> + ccs_get_crop_compose(subdev, sd_state, crops, &comp);
>
> sel->r.top = 0;
> sel->r.left = 0;
> @@ -2501,8 +2456,8 @@ static int ccs_set_compose(struct v4l2_subdev *subdev,
> return 0;
> }
>
> -static int __ccs_sel_supported(struct v4l2_subdev *subdev,
> - struct v4l2_subdev_selection *sel)
> +static int ccs_sel_supported(struct v4l2_subdev *subdev,
> + struct v4l2_subdev_selection *sel)
> {
> struct ccs_sensor *sensor = to_ccs_sensor(subdev);
> struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> @@ -2545,33 +2500,18 @@ static int ccs_set_crop(struct v4l2_subdev *subdev,
> {
> struct ccs_sensor *sensor = to_ccs_sensor(subdev);
> struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> - struct v4l2_rect *src_size, *crops[CCS_PADS];
> - struct v4l2_rect _r;
> + struct v4l2_rect src_size = { 0 }, *crops[CCS_PADS], *comp;
>
> - ccs_get_crop_compose(subdev, sd_state, crops, NULL, sel->which);
> + ccs_get_crop_compose(subdev, sd_state, crops, &comp);
>
> - if (sel->which == V4L2_SUBDEV_FORMAT_ACTIVE) {
> - if (sel->pad == ssd->sink_pad)
> - src_size = &ssd->sink_fmt;
> - else
> - src_size = &ssd->compose;
> + if (sel->pad == ssd->sink_pad) {
> + struct v4l2_mbus_framefmt *mfmt =
> + v4l2_subdev_get_pad_format(subdev, sd_state, sel->pad);
> +
> + src_size.width = mfmt->width;
> + src_size.height = mfmt->height;
> } else {
> - if (sel->pad == ssd->sink_pad) {
> - _r.left = 0;
> - _r.top = 0;
> - _r.width = v4l2_subdev_get_try_format(subdev,
> - sd_state,
> - sel->pad)
> - ->width;
> - _r.height = v4l2_subdev_get_try_format(subdev,
> - sd_state,
> - sel->pad)
> - ->height;
> - src_size = &_r;
> - } else {
> - src_size = v4l2_subdev_get_try_compose(
> - subdev, sd_state, ssd->sink_pad);
> - }
> + src_size = *comp;
> }
>
> if (ssd == sensor->src && sel->pad == CCS_PAD_SRC) {
> @@ -2579,16 +2519,19 @@ static int ccs_set_crop(struct v4l2_subdev *subdev,
> sel->r.top = 0;
> }
>
> - sel->r.width = min(sel->r.width, src_size->width);
> - sel->r.height = min(sel->r.height, src_size->height);
> + sel->r.width = min(sel->r.width, src_size.width);
> + sel->r.height = min(sel->r.height, src_size.height);
>
> - sel->r.left = min_t(int, sel->r.left, src_size->width - sel->r.width);
> - sel->r.top = min_t(int, sel->r.top, src_size->height - sel->r.height);
> + sel->r.left = min_t(int, sel->r.left, src_size.width - sel->r.width);
> + sel->r.top = min_t(int, sel->r.top, src_size.height - sel->r.height);
>
> *crops[sel->pad] = sel->r;
The crops variable stores all crop rectangles for all pads, and here you
end up using the crop rectangle on sel->pad only. I understand where
ccs_get_crop_compose() comes from, with the need to manually handle the
active and try state, but now that this has gone, I would replace the
call to ccs_get_crop_compose() above with a call to
v4l2_subdev_get_pad_crop(). ccs_set_format() can be cleaned up
similarly, and actually ccs_set_compose() too. I think dropping
ccs_get_crop_compose() would now be cleaner.
>
> if (ssd != sensor->pixel_array && sel->pad == CCS_PAD_SINK)
> ccs_propagate(subdev, sd_state, sel->which, V4L2_SEL_TGT_CROP);
> + else if (sel->which == V4L2_SUBDEV_FORMAT_ACTIVE &&
> + ssd == sensor->pixel_array)
> + sensor->pa_src = sel->r;
pa_src is assigned here only, and accessed in different places in the
driver. I think you can already drop the field, and access the active
state in the places where pa_src is used. The same seems to be true for
scaler_sink and src_src actually.
If you want to make those modifications on top I'm not opposed to it,
but they should land sooner than later, and I don't really see a reason
for not doing it now. I may be missing something of course.
>
> return 0;
> }
> @@ -2601,44 +2544,36 @@ static void ccs_get_native_size(struct ccs_subdev *ssd, struct v4l2_rect *r)
> r->height = CCS_LIM(ssd->sensor, Y_ADDR_MAX) + 1;
> }
>
> -static int __ccs_get_selection(struct v4l2_subdev *subdev,
> - struct v4l2_subdev_state *sd_state,
> - struct v4l2_subdev_selection *sel)
> +static int ccs_get_selection(struct v4l2_subdev *subdev,
> + struct v4l2_subdev_state *sd_state,
> + struct v4l2_subdev_selection *sel)
> {
> struct ccs_sensor *sensor = to_ccs_sensor(subdev);
> struct ccs_subdev *ssd = to_ccs_subdev(subdev);
> struct v4l2_rect *comp, *crops[CCS_PADS];
> - struct v4l2_rect sink_fmt;
> int ret;
>
> - ret = __ccs_sel_supported(subdev, sel);
> + ret = ccs_sel_supported(subdev, sel);
> if (ret)
> return ret;
>
> - ccs_get_crop_compose(subdev, sd_state, crops, &comp, sel->which);
> -
> - if (sel->which == V4L2_SUBDEV_FORMAT_ACTIVE) {
> - sink_fmt = ssd->sink_fmt;
> - } else {
> - struct v4l2_mbus_framefmt *fmt =
> - v4l2_subdev_get_try_format(subdev, sd_state,
> - ssd->sink_pad);
> -
> - sink_fmt.left = 0;
> - sink_fmt.top = 0;
> - sink_fmt.width = fmt->width;
> - sink_fmt.height = fmt->height;
> - }
> + ccs_get_crop_compose(subdev, sd_state, crops, &comp);
>
> switch (sel->target) {
> case V4L2_SEL_TGT_CROP_BOUNDS:
> case V4L2_SEL_TGT_NATIVE_SIZE:
> - if (ssd == sensor->pixel_array)
> + if (ssd == sensor->pixel_array) {
> ccs_get_native_size(ssd, &sel->r);
> - else if (sel->pad == ssd->sink_pad)
> - sel->r = sink_fmt;
> - else
> + } else if (sel->pad == ssd->sink_pad) {
> + struct v4l2_mbus_framefmt *sink_fmt =
> + v4l2_subdev_get_pad_format(subdev, sd_state,
> + ssd->sink_pad);
> + sel->r.top = sel->r.left = 0;
> + sel->r.width = sink_fmt->width;
> + sel->r.height = sink_fmt->height;
> + } else {
> sel->r = *comp;
> + }
> break;
> case V4L2_SEL_TGT_CROP:
> case V4L2_SEL_TGT_COMPOSE_BOUNDS:
> @@ -2652,20 +2587,6 @@ static int __ccs_get_selection(struct v4l2_subdev *subdev,
> return 0;
> }
>
> -static int ccs_get_selection(struct v4l2_subdev *subdev,
> - struct v4l2_subdev_state *sd_state,
> - struct v4l2_subdev_selection *sel)
> -{
> - struct ccs_sensor *sensor = to_ccs_sensor(subdev);
> - int rval;
> -
> - mutex_lock(&sensor->mutex);
> - rval = __ccs_get_selection(subdev, sd_state, sel);
> - mutex_unlock(&sensor->mutex);
> -
> - return rval;
> -}
> -
> static int ccs_set_selection(struct v4l2_subdev *subdev,
> struct v4l2_subdev_state *sd_state,
> struct v4l2_subdev_selection *sel)
> @@ -2673,7 +2594,7 @@ static int ccs_set_selection(struct v4l2_subdev *subdev,
> struct ccs_sensor *sensor = to_ccs_sensor(subdev);
> int ret;
>
> - ret = __ccs_sel_supported(subdev, sel);
> + ret = ccs_sel_supported(subdev, sel);
> if (ret)
> return ret;
>
> @@ -2964,10 +2885,14 @@ static int ccs_register_subdev(struct ccs_sensor *sensor,
> return rval;
> }
>
> + rval = v4l2_subdev_init_finalize(&ssd->sd);
> + if (rval)
> + goto out_media_entity_cleanup;
> +
> rval = v4l2_device_register_subdev(sensor->src->sd.v4l2_dev, &ssd->sd);
> if (rval) {
> dev_err(&client->dev, "v4l2_device_register_subdev failed\n");
> - goto out_media_entity_cleanup;
> + goto out_v4l2_subdev_cleanup;
> }
>
> rval = media_create_pad_link(&ssd->sd.entity, source_pad,
> @@ -2983,6 +2908,9 @@ static int ccs_register_subdev(struct ccs_sensor *sensor,
> out_v4l2_device_unregister_subdev:
> v4l2_device_unregister_subdev(&ssd->sd);
>
> +out_v4l2_subdev_cleanup:
> + v4l2_subdev_cleanup(&ssd->sd);
> +
> out_media_entity_cleanup:
> media_entity_cleanup(&ssd->sd.entity);
>
> @@ -3059,16 +2987,9 @@ static void ccs_create_subdev(struct ccs_sensor *sensor,
>
> v4l2_i2c_subdev_set_name(&ssd->sd, client, sensor->minfo.name, name);
>
> - ccs_get_native_size(ssd, &ssd->sink_fmt);
> -
> - ssd->compose.width = ssd->sink_fmt.width;
> - ssd->compose.height = ssd->sink_fmt.height;
> - ssd->crop[ssd->source_pad] = ssd->compose;
> ssd->pads[ssd->source_pad].flags = MEDIA_PAD_FL_SOURCE;
> - if (ssd != sensor->pixel_array) {
> - ssd->crop[ssd->sink_pad] = ssd->compose;
> + if (ssd != sensor->pixel_array)
> ssd->pads[ssd->sink_pad].flags = MEDIA_PAD_FL_SINK;
> - }
>
> ssd->sd.entity.ops = &ccs_entity_ops;
>
> @@ -3090,24 +3011,24 @@ static int ccs_init_cfg(struct v4l2_subdev *sd,
> mutex_lock(&sensor->mutex);
>
> for (i = 0; i < ssd->npads; i++) {
> - struct v4l2_mbus_framefmt *try_fmt =
> - v4l2_subdev_get_try_format(sd, sd_state, i);
> - struct v4l2_rect *try_crop =
> - v4l2_subdev_get_try_crop(sd, sd_state, i);
> - struct v4l2_rect *try_comp;
> + struct v4l2_mbus_framefmt *fmt =
> + v4l2_subdev_get_pad_format(sd, sd_state, i);
> + struct v4l2_rect *crop =
> + v4l2_subdev_get_pad_crop(sd, sd_state, i);
> + struct v4l2_rect *comp;
>
> - ccs_get_native_size(ssd, try_crop);
> + ccs_get_native_size(ssd, crop);
>
> - try_fmt->width = try_crop->width;
> - try_fmt->height = try_crop->height;
> - try_fmt->code = sensor->internal_csi_format->code;
> - try_fmt->field = V4L2_FIELD_NONE;
> + fmt->width = crop->width;
> + fmt->height = crop->height;
> + fmt->code = sensor->internal_csi_format->code;
> + fmt->field = V4L2_FIELD_NONE;
>
> if (ssd == sensor->pixel_array)
> continue;
>
> - try_comp = v4l2_subdev_get_try_compose(sd, sd_state, i);
> - *try_comp = *try_crop;
> + comp = v4l2_subdev_get_pad_compose(sd, sd_state, i);
> + *comp = *crop;
> }
>
> mutex_unlock(&sensor->mutex);
> @@ -3632,6 +3553,10 @@ static int ccs_probe(struct i2c_client *client)
> if (rval < 0)
> goto out_media_entity_cleanup;
>
> + rval = v4l2_subdev_init_finalize(&sensor->src->sd);
> + if (rval)
> + goto out_media_entity_cleanup;
> +
> rval = ccs_write_msr_regs(sensor);
> if (rval)
> goto out_media_entity_cleanup;
> @@ -3691,6 +3616,7 @@ static void ccs_remove(struct i2c_client *client)
>
> for (i = 0; i < sensor->ssds_used; i++) {
> v4l2_device_unregister_subdev(&sensor->ssds[i].sd);
> + v4l2_subdev_cleanup(subdev);
> media_entity_cleanup(&sensor->ssds[i].sd.entity);
> }
> ccs_cleanup(sensor);
> diff --git a/drivers/media/i2c/ccs/ccs.h b/drivers/media/i2c/ccs/ccs.h
> index a94c796cea48..9c3587b2fbe7 100644
> --- a/drivers/media/i2c/ccs/ccs.h
> +++ b/drivers/media/i2c/ccs/ccs.h
> @@ -182,9 +182,6 @@ struct ccs_binning_subtype {
> struct ccs_subdev {
> struct v4l2_subdev sd;
> struct media_pad pads[CCS_PADS];
> - struct v4l2_rect sink_fmt;
> - struct v4l2_rect crop[CCS_PADS];
> - struct v4l2_rect compose; /* compose on sink */
> unsigned short sink_pad;
> unsigned short source_pad;
> int npads;
> @@ -220,6 +217,7 @@ struct ccs_sensor {
> u32 mbus_frame_fmts;
> const struct ccs_csi_data_format *csi_format;
> const struct ccs_csi_data_format *internal_csi_format;
> + struct v4l2_rect pa_src, scaler_sink, src_src;
> u32 default_mbus_frame_fmts;
> int default_pixel_order;
> struct ccs_data_container sdata, mdata;
--
Regards,
Laurent Pinchart
