media: i2c: og01a1b: Replace .open with .init_state internal ops

Instead of wiping the camera sensor subdevice initial state on every
open() syscall it would be better to set the initial state just once.

Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
This commit is contained in:
Vladimir Zapolskiy 2026-02-26 15:37:37 +02:00 committed by Mauro Carvalho Chehab
parent a62d99ed89
commit d935219d19

View File

@ -746,10 +746,21 @@ static int og01a1b_enum_frame_size(struct v4l2_subdev *sd,
return 0;
}
static int og01a1b_open(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh)
static int og01a1b_init_state(struct v4l2_subdev *sd,
struct v4l2_subdev_state *state)
{
og01a1b_update_pad_format(&supported_modes[0],
v4l2_subdev_state_get_format(fh->state, 0));
struct og01a1b *og01a1b = to_og01a1b(sd);
struct v4l2_subdev_format fmt = {
.which = V4L2_SUBDEV_FORMAT_TRY,
.pad = 0,
.format = {
.width = og01a1b->cur_mode->width,
.height = og01a1b->cur_mode->height,
.code = MEDIA_BUS_FMT_Y10_1X10,
},
};
og01a1b_set_format(sd, state, &fmt);
return 0;
}
@ -777,7 +788,7 @@ static const struct media_entity_operations og01a1b_subdev_entity_ops = {
};
static const struct v4l2_subdev_internal_ops og01a1b_internal_ops = {
.open = og01a1b_open,
.init_state = og01a1b_init_state,
};
static int og01a1b_identify_module(struct og01a1b *og01a1b)