On 11/06/2021 08:49, Tomi Valkeinen wrote:
Patch "media: v4l2-subdev: add subdev-wide state struct" describes the semantic patch used to convert drivers to the new subdev-wide state. For some reason the semantic patch didn't catch a single case of: #if IS_ENABLED(CONFIG_VIDEO_V4L2_SUBDEV_API) this was patched #else this was not patched #endif Manually fix this one case. Note that this patch is meant to be squashed into the main patch. Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx> --- drivers/media/i2c/mt9v111.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/mt9v111.c b/drivers/media/i2c/mt9v111.c index b438f8c077d5..2dc4a0f24ce8 100644 --- a/drivers/media/i2c/mt9v111.c +++ b/drivers/media/i2c/mt9v111.c @@ -800,7 +800,7 @@ static struct v4l2_mbus_framefmt *__mt9v111_get_pad_format( #if IS_ENABLED(CONFIG_VIDEO_V4L2_SUBDEV_API) return v4l2_subdev_get_try_format(&mt9v111->sd, sd_state, pad); #else - return &cfg->try_fmt; + return &sd_state->pads->try_fmt; #endif case V4L2_SUBDEV_FORMAT_ACTIVE: return &mt9v111->fmt;
I missed adding: Reported-by: kernel test robot <lkp@xxxxxxxxx> Tomi
