Hi Ezequiel, On Tue, Jun 12, 2018 at 07:48:27AM -0300, Ezequiel Garcia wrote:
> @@ -1013,10 +1016,10 @@ static int vim2m_probe(struct platform_device *pdev) > vfd->lock = &dev->dev_mutex; > vfd->v4l2_dev = &dev->v4l2_dev; > > - ret = video_register_device(vfd, VFL_TYPE_GRABBER, 0); > + ret = video_register_device(vfd, VFL_TYPE_MEM2MEM, 0); Shouldn't the original type be used when building without CONFIG_MEDIA_CONTROLLER? > @@ -1050,6 +1076,11 @@ static int vim2m_remove(struct platform_device *pdev) > struct vim2m_dev *dev = platform_get_drvdata(pdev); > > v4l2_info(&dev->v4l2_dev, "Removing " MEM2MEM_NAME); > + > +#ifdef CONFIG_MEDIA_CONTROLLER Gut suggests that media_device_unregister() should be called here. Then again my experience in media/ is limited so I could be miles off ;-) HTH Emil
