| Message ID | 20180910152154.14291-1-ezequiel@collabora.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/2] vicodec: Drop unneeded symbol dependency | expand |
On 09/10/2018 05:21 PM, Ezequiel Garcia wrote: > The vicodec doesn't use the Subdev API, so drop the dependency. > > Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com> > --- > drivers/media/platform/vicodec/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/vicodec/Kconfig b/drivers/media/platform/vicodec/Kconfig > index 2503bcb1529f..ad13329e3461 100644 > --- a/drivers/media/platform/vicodec/Kconfig > +++ b/drivers/media/platform/vicodec/Kconfig > @@ -1,6 +1,6 @@ > config VIDEO_VICODEC > tristate "Virtual Codec Driver" > - depends on VIDEO_DEV && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API But it definitely needs the MEDIA_CONTROLLER. That's what it should depend on. Regards, Hans > + depends on VIDEO_DEV && VIDEO_V4L2 > select VIDEOBUF2_VMALLOC > select V4L2_MEM2MEM_DEV > default n >
On Mon, 2018-09-10 at 17:23 +0200, Hans Verkuil wrote: > On 09/10/2018 05:21 PM, Ezequiel Garcia wrote: > > The vicodec doesn't use the Subdev API, so drop the dependency. > > > > Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com> > > --- > > drivers/media/platform/vicodec/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/media/platform/vicodec/Kconfig b/drivers/media/platform/vicodec/Kconfig > > index 2503bcb1529f..ad13329e3461 100644 > > --- a/drivers/media/platform/vicodec/Kconfig > > +++ b/drivers/media/platform/vicodec/Kconfig > > @@ -1,6 +1,6 @@ > > config VIDEO_VICODEC > > tristate "Virtual Codec Driver" > > - depends on VIDEO_DEV && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API > > But it definitely needs the MEDIA_CONTROLLER. That's what it should depend on. > Does it really? The code have proper ifdefs.
On 09/10/2018 05:44 PM, Ezequiel Garcia wrote: > On Mon, 2018-09-10 at 17:23 +0200, Hans Verkuil wrote: >> On 09/10/2018 05:21 PM, Ezequiel Garcia wrote: >>> The vicodec doesn't use the Subdev API, so drop the dependency. >>> >>> Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com> >>> --- >>> drivers/media/platform/vicodec/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/media/platform/vicodec/Kconfig b/drivers/media/platform/vicodec/Kconfig >>> index 2503bcb1529f..ad13329e3461 100644 >>> --- a/drivers/media/platform/vicodec/Kconfig >>> +++ b/drivers/media/platform/vicodec/Kconfig >>> @@ -1,6 +1,6 @@ >>> config VIDEO_VICODEC >>> tristate "Virtual Codec Driver" >>> - depends on VIDEO_DEV && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API >> >> But it definitely needs the MEDIA_CONTROLLER. That's what it should depend on. >> > > Does it really? The code have proper ifdefs. > You are right, it is not needed now, but will be when stateless codec support is added to vicodec in the future. Regards, Hans
diff --git a/drivers/media/platform/vicodec/Kconfig b/drivers/media/platform/vicodec/Kconfig index 2503bcb1529f..ad13329e3461 100644 --- a/drivers/media/platform/vicodec/Kconfig +++ b/drivers/media/platform/vicodec/Kconfig @@ -1,6 +1,6 @@ config VIDEO_VICODEC tristate "Virtual Codec Driver" - depends on VIDEO_DEV && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API + depends on VIDEO_DEV && VIDEO_V4L2 select VIDEOBUF2_VMALLOC select V4L2_MEM2MEM_DEV default n
The vicodec doesn't use the Subdev API, so drop the dependency. Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com> --- drivers/media/platform/vicodec/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)