On 6/20/22 11:44, Tomi Valkeinen wrote:
Hi,
Hello all,
On Sun, Jun 19, 2022 at 12:24:42AM +0200, Marek Vasut wrote:
Any local subdev state should be allocated and free'd using
__v4l2_subdev_state_alloc()/__v4l2_subdev_state_free(), which
takes care of calling .init_cfg() subdev op. Without this,
subdev internal state might be uninitialized by the time
any other subdev op is called.
Does this fix a bug you have?
Yes
Wasn't this broken even before the active
state, as init_cfg was not called?
Yes, this was always broken. I suspect nobody tested this mode of
operation before. In my case, I have this DCMI driver connected directly
to MT9P006 sensor.
In any case, I think we have to do something like this, as the source
subdev might depend on a valid subdev state.
Right.
[...]
